-
Notifications
You must be signed in to change notification settings - Fork 164
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
stop assuming that all shop=organic are supermarkets #884
Conversation
🍱 Preview the tagging presets of this pull request here: https://pr-884--ideditor-presets-preview.netlify.app/id/dist/#locale=en. |
Another alternative would probably be shop=convenience + organic=yes/only. Maybe iD can be enhanced to allow the user to select from two or more alternatives. This could also be used for other tags, e.g. for sport=diving, which could be scuba_diving or cliff_diving. |
Agree that the current tag upgrade is not general applicable. Other examples for shop types where
While this is true in a strict sense, it is IMO reasonable to assume that a user would not have chosen |
though (but with such interpretation migration to |
I always read the wiki that
(highlights by me)
Sure, but I would argue that such a tag change would only "hide" the underlying data quality problem we have with this tag: We still don't know which shop type this is and in general some manual research is needed to fix that. Basically, the tag deprecation / upgrade rules here are just not a good place for this type of data quality issue. |
while |
ok, good point. Let's change it to //edit: done in 0a8c721 |
to make it explicit that the shop type is actually unknown and picked up by further quality assurance tools
also, some may be
organic=yes
so even migrating toshop=yes
is not viable - this requires human review