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

Complain about an unreasonably small buildings for types indicating bigger buildings #6573

Closed
matkoniecz opened this issue Jun 24, 2019 · 1 comment
Labels
wontfix-low-impact Doesn't improve the user experience enough to spend time on

Comments

@matkoniecz
Copy link
Contributor

matkoniecz commented Jun 24, 2019

Some 3D mappers instead of using building:part use building areas.

For example see https://www.openstreetmap.org/way/302589861/history https://www.openstreetmap.org/way/302589873/history

"hospital" with 4 or 8 square meters is always wrong and it would be useful to have an explicit warning.

Note that some building like kiosks may be extremely small so it makes only for cases where building tag was explicitly specified and area is below say 10 square meters or area is smaller than say 0.5 square meter.

JOSM issue: https://josm.openstreetmap.de/ticket/17846#ticket

@quincylvania
Copy link
Collaborator

While I agree with the basic ideas of this warning, it seems too nitpicky for iD and low impact overall. The specific case you pointed to seems uncommon and can be fixed manually. Future cases could be mitigated by better 3D mapping instruction.

This is the kind of issue that would be better suited for after #6483, when people can argue over tagging minutia outside the scope of core iD functionality.

@quincylvania quincylvania added the wontfix-low-impact Doesn't improve the user experience enough to spend time on label Jun 24, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
wontfix-low-impact Doesn't improve the user experience enough to spend time on
Projects
None yet
Development

No branches or pull requests

2 participants