-
Notifications
You must be signed in to change notification settings - Fork 885
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Vue cinemas #3420
Comments
I think iD will not offer to upgrade names that are ambiguous. |
Ok. Would it be possible for iD to compare only the first part of the name (i.e. before the ~)? And then since there is a country code to separate the UK/IRL and Netherlands brands, only one brand will match, which could be offered as an upgrade? |
I think sometime in the future, the geometry will be what disambiguates the entries and all the tools that we use will have the ability to know whether a user is editing within that geographical region. iD won't need string disambiguators. But for right now - we use strings because that was the fastest way to get something working that isn't broken. What makes this problem tricky is:
A good example of this is "Price Chopper", which has 2 different entries (New York and Kansas City). https://nsi.guide/index.html?k=shop&v=supermarket#Price%20Chopper~(Kansas%20City) Another is "Giant" fuel stations - one is related to a Supermarket (and there are multiple ambiguous Giant supermarkets), another is a southwestern chain that is in the middle of a rebrand to "Marathon" or maybe "Speedway" (see 9e1668a) https://overpass-turbo.eu/s/OK0 |
Could the two Vue entries have the "~(IrelandAndUK)" and "~(Netherlands)" removed, as both have country code references? I tried placing a random node in the UK, and the Vue suggested was the UK one, and placing a node in the Netherlands resulted in the Netherlands Vue being suggested. |
@UKChris-osm: They still need unique keys in the json data which is why we add those. In theory iD (or any downstream software) could remove that bit of the string when shown to the user, but you'd need to make sure two disambiguated brands are exclusive to regions with no overlap before doing that - which is quite sophisticated to implement for such a small change. |
Ah I see, I wasn't sure as I saw both entries had the same "Vue" name & brand tag and assumed this meant the same initial value (amenity/cinema|Vue) could appear in multiple entries with different Country Codes. |
The work on #2714 (comment) unblocks us to being able to disambiguate the entries by their locationSet geometry instead of a string like "(IrelandAndUK)" . Closing this issue in favor of #3995, which is the same idea. |
Normally in iD when you have a plain POI - e.g. with amenity=cinema and name=Vue - iD would prompt you to upgrade the tag if it is present in the NSI.
For Vue Cinemas, if you click in the name field and press space iD offers the UK/Ireland and Netherlands versions.
Can this be fixed in the NSI so that Vue on its own in the UK is recognised as a brand needing to be upgraded? As it seems like the "(IrelandAndUK)" part of the name is causing a problem.
Let me know if this issue should be in the iD github instead.
The text was updated successfully, but these errors were encountered: