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

Stop treating all tag values with OSM Wiki page as recommended #5460

Closed
matkoniecz opened this issue Nov 3, 2018 · 3 comments · Fixed by #5461
Closed

Stop treating all tag values with OSM Wiki page as recommended #5460

matkoniecz opened this issue Nov 3, 2018 · 3 comments · Fixed by #5461
Labels
new contributor opportunity Best for first-time contributors without prior experience. You won’t be assigned; PRs welcome!

Comments

@matkoniecz
Copy link
Contributor

matkoniecz commented Nov 3, 2018

tl;dr: OSM Wiki page describing highway=unsurfaced as an outdated tagging scheme is causing iD to propose this tag to mappers. This is undesirable.

highway unsurfaced in id


There are some tags that were used in past but nowadays are almost unused and should not be used ever again. Part of handling this situation is to create page on OSM wiki describing why tag is problematic and to mention possible alternatives.

It leads to one unfortunate side effect. As I undestand, in raw tag editing mode, iD proposes tags that are either with some minimal usage or have their own OSM wiki page.

https://github.com/openstreetmap/iD/blob/master/modules/services/taginfo.js#L93

So tags like

https://wiki.openstreetmap.org/wiki/Tag:noexit%3Dno
https://wiki.openstreetmap.org/wiki/Tag:natural%3Dland
https://wiki.openstreetmap.org/wiki/Tag:highway%3Dunsurfaced
https://wiki.openstreetmap.org/wiki/Tag:landuse%3Dwood
https://wiki.openstreetmap.org/wiki/Tag:amenity%3Deducation
https://wiki.openstreetmap.org/wiki/Tag:highway%3Dabandoned
https://wiki.openstreetmap.org/wiki/Tag:amenity%3Dproposed
https://wiki.openstreetmap.org/wiki/Tag:amenity%3Ddisused
https://wiki.openstreetmap.org/wiki/Tag:highway%3Dbyway
https://wiki.openstreetmap.org/wiki/Tag:amenity%3Dhotel
https://wiki.openstreetmap.org/wiki/Tag:natural%3Dmoor

and other will be proposed by iD for use even after their usage reaches 0.

Is there any situation where almost unused tag should be promoted just because it has its own OSM wiki page? I think that only completely failed proposals will fail to get >0.0% usage, and promoting them is not necessary.

I propose to stop considering "has wiki page" as sufficient reason to suggest tags and rely just on Taginfo usage.

It is related to #4508 that proposed much stronger and more controversial change.

@matkoniecz
Copy link
Contributor Author

matkoniecz commented Nov 3, 2018

I suspect that changing

return parseFloat(d.fraction) > 0.0 || d.in_wiki;

to

return parseFloat(d.fraction) > 0.0;

in https://github.com/openstreetmap/iD/blob/master/modules/services/taginfo.js#L93

is sufficient to avoid this problem without causing unwanted side effects.

@bhousel
Copy link
Member

bhousel commented Nov 3, 2018

Thanks for researching this @matkoniecz I agree. Whether or not a tag has a wiki page is not a useful signal. I'd happily accept a pull request with your proposed fix..

@bhousel bhousel added the new contributor opportunity Best for first-time contributors without prior experience. You won’t be assigned; PRs welcome! label Nov 3, 2018
@matkoniecz
Copy link
Contributor Author

PR opened at #5461

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new contributor opportunity Best for first-time contributors without prior experience. You won’t be assigned; PRs welcome!
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants