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

Support building=manufacture with dedicated building description #523

Closed
matkoniecz opened this issue Jul 10, 2022 · 10 comments · Fixed by #524
Closed

Support building=manufacture with dedicated building description #523

matkoniecz opened this issue Jul 10, 2022 · 10 comments · Fixed by #524

Comments

@matkoniecz
Copy link
Contributor

matkoniecz commented Jul 10, 2022

See https://wiki.openstreetmap.org/wiki/Tag%3Abuilding%3Dmanufacture

s_Screenshot 2022-07-10 at 11-26-51 OpenStreetMap s_Screenshot 2022-07-10 at 11-26-27 OpenStreetMap

Found via streetcomplete/StreetComplete#4194

@peternewman
Copy link
Contributor

What is the distinction between industrial and manufacture? Even the wiki doesn't really clarify!

It's also missing from https://wiki.openstreetmap.org/wiki/Key:building

@tyrasd
Copy link
Member

tyrasd commented Jul 11, 2022

I'm hesitating slightly because the tag usage stagnates since 2015, as seen in this graph:

taghistory(2)

Maybe it should be included as a hidden (searchable=false) preset?

@matkoniecz
Copy link
Contributor Author

searchable=false also looks like a good idea to me

In many cases it was fueled by imports and distinction is tiny.

Hmm, is it possible to declare it as a synonym of building=industrial to avoid adding one more string to translate?

@tyrasd
Copy link
Member

tyrasd commented Jul 11, 2022

Hmm, is it possible to declare it as a synonym of building=industrial

Unfortunately, not really. The closest thing would be to add a deprecation rule building=manufacture to building=industrial. But at first glance, I think that would probably not be appropriate for this tag right now.

@matkoniecz
Copy link
Contributor Author

Then I added "searchable": false

No matter what is the source, I think that about 100k buildings deserves a label.

@matkoniecz
Copy link
Contributor Author

BTW, I am considering reviewing other buildings values for missing labels: would you be happy to get other similar PRs for valid building types with say >100 000 occurrences?

@tyrasd
Copy link
Member

tyrasd commented Jul 11, 2022

100k occurences sounds reasonable. 👍 I'd prefer if you could do a single pull request for all of the missing types, but if you prefer to do it individually, that's also ok.

@matkoniecz
Copy link
Contributor Author

matkoniecz commented Jul 11, 2022

Building scan found outbuilding and allotment_house (if anyone wants: feel free to make PR if you find good descriptions for this values)

@tyrasd Would you be interested in such listing for say shop amenity barrier leisure natural (with adjusted thresholds)?

I believe that JOSM has similar script that is listing popular tags (keys?) and they maintain list of popular tags which are deliberately not supported.

@tyrasd
Copy link
Member

tyrasd commented Jul 12, 2022

@tyrasd Would you be interested in such listing for say shop amenity barrier leisure natural (with adjusted thresholds)?

👍

I believe that JOSM has similar script that is listing popular tags (keys?) and they maintain list of popular tags which are deliberately not supported.

I'd be interested to learn more about how they do this

@matkoniecz
Copy link
Contributor Author

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants