-
Notifications
You must be signed in to change notification settings - Fork 818
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
Do not render building=proposed #4825
Comments
I am unsure if proposed highway should be part of osm database, but with proposed buildings I am pretty sure they should not be put in there. |
I agree that neither However, the reality is that these objects do exist in the OSM database. The example I showed is part of a shopping mall that is definitely going to be constructed but where construction hasn't started yet. A local mapper has gone through the effort to already map it. I share your opinion that this should not have been done before the start of construction. But I also do not want to destroy that mappers work (a mapper that does a lot of valuable mapping otherwise). I think in the rendering department we have to deal with what is in the database, not with what we would wish to be in the database. |
building=proposed has 2354 uses and clearly there is no consensus that this is valid tagging. Among mappers who do not care about the lack of verifiability the most broadly accepted way to map something like this is using lifecycle prefixes: https://wiki.openstreetmap.org/wiki/Lifecycle_prefix. See https://taginfo.openstreetmap.org/keys/proposed:building Closing as wontfix because adding code to the style to explicitly hide something that is tagged in a way that has no wide acceptance is not likely going to find support here. |
Thanks for the swift response. I understand. But if I may ask this question of understanding: Why does carto then have extra code to suppress rendering Perhaps you can understand that to an amateur like me this seems inconsistent. So perhaps you can enlighten me? |
It does not. The string |
I might have been a bit cryptic here in my replies so i will add a bit more detailed explanation of the background.
This is different for That means we implicitly render Now we could technically add |
Related: #4824
Expected behavior
building=proposed
is not rendered. (the same ashighway=proposed
is not rendered).Actual behavior
building=proposed
is rendered the same as regularbuilding=*
Screenshots with links illustrating the problem
See this building:
The text was updated successfully, but these errors were encountered: