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

Barriers as building parts #192

Open
Sinuhe20 opened this issue Jul 5, 2022 · 2 comments
Open

Barriers as building parts #192

Sinuhe20 opened this issue Jul 5, 2022 · 2 comments

Comments

@Sinuhe20
Copy link

Sinuhe20 commented Jul 5, 2022

It looks like that min_height of barriers is not used, so you can't use them correctly as building parts. See the fence on the Victory Column in Berlin: https://www.openstreetmap.org/relation/4639168

F4Map, on the other hand, renders the fence at the right place: https://demo.f4map.com/#lat=52.5148244&lon=13.3501419&zoom=20

Or any other idea how to tag the fence correctly?

@tordanik
Copy link
Owner

tordanik commented Jul 6, 2022

While both min_height and barrier are supported, the combination of the two is not (yet?) supported. The reason is that there's no agreed-on standard for min_height on anything except building parts. I do see that it's something that might be useful, though, so while I'm currently unsure how to best handle this, I'm leaving this issue open as a reminder.

What OSM2World does already support is location=roof and/or level=* tags on various features – including barriers. This can often fill the same need. Theoretically, that would work for the Victory Column as well. (Just tested it, though, and it didn't work. Not sure why, might be another bug.)

@Sinuhe20
Copy link
Author

Sinuhe20 commented Jul 7, 2022

I looked into the code a bit and saw that way and node features like barriers or flagpoles are not really supported for building parts. It would be cool if you could just place them on a building part or roof outline (maybe with level=*) and the program would figure out the correct minimum height (like for windows).

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

No branches or pull requests

2 participants