Editor: existing object is updated instead of creation of a new one #2298
Labels
No labels
Accessibility
Accessibility
Address
Address
Android
Android
Android Auto
Android Auto
Android Automotive (AAOS)
Android Automotive (AAOS)
API
API
AppGallery
AppGallery
AppStore
AppStore
Battery and Performance
Battery and Performance
Blocker
Blocker
Bookmarks and Tracks
Bookmarks and Tracks
Borders
Borders
Bug
Bug
Build
Build
CarPlay
CarPlay
Classificator
Classificator
Community
Community
Core
Core
CrashReports
CrashReports
Cycling
Cycling
Desktop
Desktop
DevEx
DevEx
DevOps
DevOps
dev_sandbox
dev_sandbox
Directions
Directions
Documentation
Documentation
Downloader
Downloader
Drape
Drape
Driving
Driving
Duplicate
Duplicate
Editor
Editor
Elevation
Elevation
Enhancement
Enhancement
Epic
Epic
External Map Datasets
External Map Datasets
F-Droid
F-Droid
Fonts
Fonts
Frequently User Reported
Frequently User Reported
Fund
Fund
Generator
Generator
Good first issue
Good first issue
Google Play
Google Play
GPS
GPS
GSoC
GSoC
iCloud
iCloud
Icons
Icons
iOS
iOS
Legal
Legal
Linux Desktop
Linux Desktop
Linux packaging
Linux packaging
Linux Phone
Linux Phone
Mac OS
Mac OS
Map Data
Map Data
Metro
Metro
Navigation
Navigation
Need Feedback
Need Feedback
Night Mode
Night Mode
NLnet 2024-06-281
NLnet 2024-06-281
No Feature Parity
No Feature Parity
Opening Hours
Opening Hours
Outdoors
Outdoors
POI Info
POI Info
Privacy
Privacy
Public Transport
Public Transport
Raw Idea
Raw Idea
Refactoring
Refactoring
Regional
Regional
Regression
Regression
Releases
Releases
RoboTest
RoboTest
Route Planning
Route Planning
Routing
Routing
Ruler
Ruler
Search
Search
Security
Security
Styles
Styles
Tests
Tests
Track Recording
Track Recording
Translations
Translations
TTS
TTS
UI
UI
UX
UX
Walk Navigation
Walk Navigation
Watches
Watches
Web
Web
Wikipedia
Wikipedia
Windows
Windows
Won't fix
Won't fix
World Map
World Map
No milestone
No project
No assignees
1 participant
Due date
No due date set.
Dependencies
No dependencies set.
Reference: organicmaps/organicmaps-tmp#2298
Loading…
Add table
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Creating a POI at the same place where another POI is already located doesn't look right. Better behavior would be to disable this scenario or avoid it.
This happened to me twice by accident yesterday.
A bakery that went out of business was replaced by a restaurant. Because OM doesn't allow for change of main tag I wanted to create a new node right next to old POI with intention of deleting old POI later in JOSM. This resulted in name being overwritten but amenity=restaurant additionally added to shop=bakery. I didn't even intentionally put location exactly on existing POI but because the building has a very high POI density it was still close enough to cause updating the existing POI instead of creating new one.
https://www.openstreetmap.org/node/1437310148/history
The second time an existing POI was in wrong location (should have been on the other side of the building) so I had to create new POI very close to it. This again caused an unintended overwrite of existing POI.
https://www.openstreetmap.org/node/7127363186/history
I think if "add new place" is selected OM should always create a new POI even if node coordinates are almost identical with existing POI.
What about duplicate features? One of the popular cases is when several users add the same place from Organic Maps independently from each other.
This case is when POI is not available yet in Organic Maps but is in Openstreetmap I think? In this case 'merge' function in Organic Maps will not help as it is not aware about it.
The most popular case: several OM users try to add the same place. Until they update the map data from OSM, they won't see that the place was already added. More frequent map updates will improve the situation but not fix it completely. So the better approach is to make a smarter "merge" algorithm that knows about different types and can decide properly when a feature should be merged/updated or not. Why is it important? Because someone can input only the type and name, someone else can also add opening hours, etc.
Maybe merge function could be limited to instances where new and existing POI have identical name? This should pretty much eliminate accidental overwrite of still existing POIs.
What about identical types (also with a missing name, for example)? Or with a typo in the name (or minor difference - space, dash, ' or ")?