Skip to content
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

Removing deprecated, rarely used shops #1900

Merged
merged 1 commit into from Oct 11, 2015

Conversation

kocio-pl
Copy link
Collaborator

@kocio-pl kocio-pl commented Oct 6, 2015

Removing rendering of shop values deprecated according to Wiki pages:

  • antique (not deprecated, it just lacks wiki page, but is less used version of antiques)
  • betting
  • delicatessen
  • dive
  • fish
  • gambling
  • insurance
  • organic
  • pharmacy

@kocio-pl kocio-pl force-pushed the deprecated-shops branch 2 times, most recently from 9a248de to 2d03fa7 Compare October 6, 2015 18:30
@matkoniecz
Copy link
Contributor

fish

I remember this one from https://lists.openstreetmap.org/pipermail/tagging/2014-July/018573.html (thread that resulted in https://josm.openstreetmap.de/ticket/10594 and noticeable decrease in how often some shop values are now appearing)

Somebody claimed that shop=fish is used for shops selling solely freshwater fish (and where seafood is not fitting).

@HolgerJeromin
Copy link
Contributor

As I understood the right shop is seafood or fishmonger with no clear solution. Especially with fresh water fish and native speakers.

Fish is no real alternative. So thanks for removing.

@matthijsmelissen
Copy link
Collaborator

+1

@pnorman
Copy link
Collaborator

pnorman commented Oct 6, 2015

👎 on shop=organic. I believe it's ~1k usage puts it well over some other shop values that are rendered with the generic icon.

@kocio-pl
Copy link
Collaborator Author

kocio-pl commented Oct 6, 2015

Have you checked shop=organic page?:

Deprecated - Use organic=* in combination with a general shop=* key instead

@pnorman
Copy link
Collaborator

pnorman commented Oct 6, 2015

Have you checked shop=organic page?:

Deprecated - Use organic=* in combination with a general shop=* key instead

Yes, I checked the wiki page.

@kocio-pl
Copy link
Collaborator Author

kocio-pl commented Oct 6, 2015

So why do you want to render clearly deprecated feature?

@matthijsmelissen
Copy link
Collaborator

It seems the tag was 'deprecated' by one user without discussion: http://wiki.openstreetmap.org/w/index.php?title=Tag:shop%3Dorganic&diff=735472&oldid=689739

I think the poor state of osm's wiki documentation and the lack of a clear process to decide on changes to the documentation, together with the lack of importance many mappers give to the documentation, is something that seriously holds back the project, but unfortunately it seems there's little we can do about that.

@kocio-pl
Copy link
Collaborator Author

kocio-pl commented Oct 6, 2015

OK, this is a legitimate reason to leave it unchanged. I just wanted to get rid of things which are clearly not wanted here, there is however a lot of other uncertain values and I want to discuss them on Tagging before taking any action.

BTW: I've just discovered what the shop_values.rb script is for. 😄 However in current form it's not usable for managing our shop list. It would be good if it could compare what we already use as shop values in project.yaml and show the difference.

@matkoniecz
Copy link
Contributor

However in current form it's not usable for managing our shop list.

Are you worrying about excluding shops rendered with special icons?

@jojo4u
Copy link

jojo4u commented Oct 7, 2015

The state of documentation is sometimes sad, that's true. But wiki affects mapping, which affects Taginfo. You could change the title from: "Removing deprecated shops" to "Removing deprecated, rarely used shops".

@kocio-pl kocio-pl changed the title Removing deprecated shops Removing deprecated, rarely used shops Oct 7, 2015
@kocio-pl
Copy link
Collaborator Author

kocio-pl commented Oct 7, 2015

@jojo4u I would remove every deprecated item, no matter how popular, it's just easier with less popular shops to remove them at hand and currently no popular shop type is deprecated. I would however try to make the transition smooth (deprecated usually means "this is the wrong schema, change it to something more appropriate"), so this is like those uncertain values I was talking about: more time is needed to resolve these problems, so they don't belong to this PR.

@matkoniecz My first problem is that just replacing the list would break diff output for project.yaml and I like to control the change, no matter how trivial it seems (#1800 proves me it can be the case). But, frankly saying, I also don't know if the shop values can be in both lists ("dotted" and "iconized" shops), but I guess they should be moved from dotted list to iconized list, but it means more spotting and manual (tedious and error prone) work. Am I right?

@pnorman
Copy link
Collaborator

pnorman commented Oct 7, 2015

So why do you want to render clearly deprecated feature?

For better or worse, there is no process to depreciate a tag within OSM. As a data consumer I do not feel compelled to stop rendering something based on what one particular wiki page states.

@jojo4u
Copy link

jojo4u commented Oct 7, 2015

For better or worse, there is no process to depreciate a tag within OSM.

There is. But it is often not successful because OSM is no democracy ;) A new, accepted proposal can deprecate old tags. But sometimes the switch takes time (building=entrance) and sometimes the new tag is rejected by many (water=*).

@kocio-pl
Copy link
Collaborator Author

kocio-pl commented Oct 7, 2015

Citing wiki documentation:

OpenStreetMap does not have "banned features", as anybody is allowed and encouraged to use any tag they think is useful.

However, from time to time the wiki community suggests a new feature instead of an older one, to improve tagging clarity or structure. If there is no strong reason to stick to the old tags in these cases, you might want to go along with the recommendation and use the new tags suggested here.

In my opinion the fact that there is no sure or hard way to do anything in OSM (not only deprecating), doesn't mean we can't add/change common interpretation of tags - for example to unify something or avoid mistakes. This style is documented as part of a mapper feedback loop, so it is better to stick to this common interpretation.

@kocio-pl
Copy link
Collaborator Author

kocio-pl commented Oct 7, 2015

there is however a lot of other uncertain values and I want to discuss them on Tagging before taking any action.

The discussion is started here and it looks like people are really interested in the subject.

@matthijsmelissen matthijsmelissen merged commit 034cb1b into gravitystorm:master Oct 11, 2015
@kocio-pl kocio-pl deleted the deprecated-shops branch October 11, 2015 21:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants