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

Add rendering for man_made=bunker_silo #372

Closed
MrPlumbum opened this issue Mar 1, 2014 · 8 comments
Closed

Add rendering for man_made=bunker_silo #372

MrPlumbum opened this issue Mar 1, 2014 · 8 comments

Comments

@MrPlumbum
Copy link

Please create a rendering for the approved tag man_made=bunker_silo (Wiki documentation). For those mapped as polygon I would propose a rendering as a grey area with a highlighted (dark grey) frame.

Now there are several hundreds mapped, but I would approximate that the number will increase to several tens of thousands.

*Example 1
*Example 2

Thanks

@matthijsmelissen
Copy link
Collaborator

@daganzdaanda
Copy link

These are common in rural Bavaria, in farm yards and also just at some field's edge. I would like to see these with a similar colour to farmyard, a bit darker.

@matthijsmelissen matthijsmelissen added this to the New features milestone Aug 18, 2014
@matkoniecz
Copy link
Contributor

http://taginfo.openstreetmap.org/tags/man_made=bunker_silo - 1243 objects mapped

I see no good way to render these objects without making it highly confusing.

@matthijsmelissen matthijsmelissen changed the title rendering of man_made=bunker_silo Add rendering for man_made=bunker_silo Sep 24, 2014
@matkoniecz
Copy link
Contributor

Also, such objects are not highly important, I think that rendering them is not the best idea.

@pnorman pnorman closed this as completed May 17, 2015
@MrPlumbum
Copy link
Author

I like the rendering-style proposed by daganzdaanda: "with a similar colour to farmyard, a bit darker".

There should be a rendering because in the areas covered with a bunker_silo is nothing else.

@kocio-pl
Copy link
Collaborator

10 775 cases already and they do use some space, so I think it's time to rethink the issue.

@jeisenbe
Copy link
Collaborator

I looked into this. I can't think of a good rendering.

They are not buildings (no roof, only 3 walls), and they are not really similar to farmyards, so we should not use the same color as a building or farmyard.

They are rectangular, and building-sized, not like a whole farmyard area.

Most do not have any barrier=wall features added for the 3 sides with walls, though this could be used and is already rendered.

So, besides the walls, we would be rendering the area of the floor, which might be covered with agricultural products or sillage. I'm not sure that anything besides the walls is of interest for a general-purpose map.

Does anyone have other ideas about rendering this in a reasonable way, or can we close the issue?

@jeisenbe
Copy link
Collaborator

I have no ideas about how to render these in a good way, except to map and render the walls that surround them on 3 sides. They should usually be inside a landuse=farmyard or landuse=farmland area which we can also render. So I will close this for now, but if someone has a great idea about how to render these, it can be reopened.

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

No branches or pull requests

7 participants