Closed boothym closed 4 years ago
Ok, I did this, but I'm not sure how good it will be.
Internally this library maintains an index of key/value|name
matches. I ended up disabling the warnings about name collisions for the *=yes
values because this introduced hundreds of name collisions.
For example a feature like amenity/yes|Cosmo
could map to either:
"amenity/fuel|Cosmo"
or "amenity/restaurant|Cosmo"
We just don't know. iD will probably offer to upgrade it to something, but it might not be the right thing.
If this turns out to be a real problem, I can throw out all the duplicates after the name matching index is built. But then iD won't flag them as issues, which I suppose is the point of all this.
I'll close for now, but we'll keep an eye on this if it causes problems or if there is a better way.
Thanks - is there a way to test this before it's released to the masses?
I ended up disabling the warnings about name collisions for the *=yes values because this introduced hundreds of name collisions.
Is this just for shops, or amenities as well? Because I don't think amenity=yes is used in the same way or as many times, so could be restricted to just shops.
Thanks - is there a way to test this before it's released to the masses?
I think we would need to set up a preview site for the 2.x branch on iD.. This is something we should do, but haven't done it yet.
Is this just for shops, or amenities as well? Because I don't think amenity=yes is used in the same way or as many times, so could be restricted to just shops.
It's for any key that we track in this index.. But from what I can see of the warnings, it looks like all the tags do work this way.
I've pasted the details below.. You can see how any amenity=yes
, shop=yes
, etc can often map to multiple tags, and building=yes
can map to really anything.
Reposted from https://github.com/openstreetmap/iD/issues/7197
I came across a node tagged shop=yes and name="Topps tiles". I changed this to shop=tiles and then iD offered to upgrade the tags with brand data.
For shop=yes objects where the name matches* would it be possible for iD to offer to upgrade the tags? This would be useful since these shops can be missed when looking at an area, as they are not otherwise highlighted by iD.
*a case insensitive match would be good