-
Notifications
You must be signed in to change notification settings - Fork 814
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
update list of shops rendered as dot #2090
Comments
Also shop=street_vendor may be worth excluding. What is it supposed to be? Shop selling street vendors? Or a separate tagging scheme not categorizing by what is sold but how? I am not sure what makes less sense. It turns out to be used on large scale for two market places, certainly it should be filtered out. See http://www.openstreetmap.org/note/527860 and http://www.openstreetmap.org/note/527859 |
We have amenity = marketplace approved |
Case of fishmonger is weird - it is appearing in openstreetmap-carto/amenity-points.mss Line 676 in 6a7a9cc
It is also not excluded by https://github.com/gravitystorm/openstreetmap-carto/blob/master/scripts/shop_values.rb Curious. |
It was planed to render. We had discussed this value. |
sent from a phone
hasn't storage_units the same issues? Isn't candy a duplicate of confectionery? |
That is why I propose to raise threshold to 150 as it gives much better results:
|
2016-03-15 11:28 GMT+01:00 Mateusz Konieczny notifications@github.com:
I feel many of these have issues and have not been discussed on the tagging some issues that come to mind (from the added list):
based on these considerations, I'd rather add only these as shop values: and from the other list: on a general note I feel we should encourage more verbose tags, because |
Candy translates into English as sweets, confectionery is fancy cakes. |
2016-03-15 12:21 GMT+01:00 trigpoint notifications@github.com:
wikipedia has different opinions on this: Confectionery,[1] https://en.wikipedia.org/wiki/Confectionery_store A confectionery store (more commonly referred to as a sweet shop in the See also: http://www.oxforddictionaries.com/definition/english/confectionery Until one year ago, this was the photograph in the OSM wiki: http://wiki.openstreetmap.org/wiki/File:Sweet_Shop,_Chester.JPG (I just changed here: |
I was trying to clean the state of "dotted" shops, but still didn't finish it. I made a list of different issues on my Wiki and I started a discussion on a Tagging list, which was detailed and quite intensive for some time, but it was in the fall and I still didn't summarise it before returning to the community with some outcome. The whole thing is long overdue, but not abandoned - it's still on my radar. |
shop=glass seems to be duplicate with shop=glaziery. |
sent from a phone
at least after you have declared it a duplicate in the wiki ;-) |
Changes that I proposed have no clear support, there are some beneficial changes that now have PRs. So I am closing this issue. |
Without any new filtering at this moment it would result in following changes (based on https://gist.github.com/matkoniecz/1a33e35fcf32ac4cccc3 generated based on https://github.com/gravitystorm/openstreetmap-carto/blob/master/scripts/shop_values.rb and the current query):
Removed: "phone", "real_estate"
Added: "agrarian", "anime", "auto_parts", "bathroom", "cafe", "candy", "catalogue", "doors", "fishmonger", "glass", "haberdashery", "lamps", "marketplace", "nutrition_supplements", "party", "perfume", "plumber", "provisions", "storage_rental", "storage_units", "street_vendor", "supplements", "swimming_pool", "tableware", "watch", "water", "weapons", "wedding", "window_blind", "windows", "wood"
auto_parts seems to be a clear duplicate of car_parts, there are also other dubious entries.
But raising threshold to 150 gives much better results:
Removed: "builder", "building_materials", "camera", "car_service", "medical", "phone", "real_estate", "salon", "tanning"
Added: "anime", "fishmonger", "haberdashery", "lamps", "marketplace", "party", "storage_rental", "street_vendor", "water", "weapons", "window_blind", "windows"
Added entries make sense (and at least some are documented), removed are undocumented, some are in addition duplicates of documented more popular values.
I would only propose to not include shop=storage_rental as it is not a shop (see failed proposal at http://wiki.openstreetmap.org/wiki/Proposed_features/shop%3Dstorage_rental).
The text was updated successfully, but these errors were encountered: