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

Render man_made=reservoir_covered areas #2620

Open
stragu opened this issue Apr 26, 2017 · 23 comments
Open

Render man_made=reservoir_covered areas #2620

stragu opened this issue Apr 26, 2017 · 23 comments

Comments

@stragu
Copy link

stragu commented Apr 26, 2017

It would be great to have objects tagged with man_made=reservoir_covered being rendered. There is currently about 123000 ways tagged with it in the database.

Example object: http://www.openstreetmap.org/way/17740394
See a satellite image to see the object. It is a significant structure in the landscape.

How it is currently (not) rendered on the default renderer:
nothing much to see

Issues that might be linked to this:

@pnorman
Copy link
Collaborator

pnorman commented Apr 26, 2017

Any suggestions on how to render it?

@dieterdreist
Copy link

dieterdreist commented Apr 26, 2017 via email

@stragu
Copy link
Author

stragu commented Apr 26, 2017

Any suggestions on how to render it?

like a man_made=storage_tank ?

That sounds sensible. Both of those objects are hard to decide on how to render them according to their physical appearance because it varies so much. I would be happy to see it rendered in the same way as storage tanks (which currently is the same as generic buildings, right?).

@mboeringa
Copy link

A lot of these are covered by grass, in rare cases with some form of park above. So if anything alternative needs to be developed, that may be kept in mind.

Rendering it simply as buildings is probably not bad though...

@imagico
Copy link
Collaborator

imagico commented Apr 26, 2017

A bit of additional info here:

  • man_made=reservoir_covered has 5k uses on nodes, 15k on ways (of which 6k have a building tag).
  • man_made=storage_tank has 55k uses on nodes, 123k uses on ways (of which 82k have a building tag).
  • building=storage_tank has 18k uses half of which also feature man_made=storage_tank.

The example shown seems to be a building. I wonder if there are practical examples of covered reservoirs where neither building=yes nor location=underground apply.

@dieterdreist
Copy link

dieterdreist commented Apr 26, 2017 via email

@imagico
Copy link
Collaborator

imagico commented Apr 26, 2017

Note i was asking for practical real world examples - independent on what definition of building this is based on.

@mboeringa
Copy link

The example shown seems to be a building. I wonder if there are practical examples of covered reservoirs where neither building=yes nor location=underground apply.

Depends on what you call a building and underground. I think this example from the Netherlands is pretty typical for some:

https://upload.wikimedia.org/wikipedia/commons/thumb/8/83/De_Waterdruppel.jpg/600px-De_Waterdruppel.jpg

@imagico
Copy link
Collaborator

imagico commented Apr 27, 2017

That is like the sample image on the wiki - just in larger. To me that is location=underground, even if the ground is banked up rather than it being dug into the ground.

@mboeringa
Copy link

mboeringa commented Apr 27, 2017

And then there are the monstrous big ones like the Réservoir de Montsouris in Paris:

https://fr.wikipedia.org/wiki/R%C3%A9servoir_de_Montsouris

which is currently tagged as building:

http://www.openstreetmap.org/way/79804222

@imagico
Copy link
Collaborator

imagico commented Apr 27, 2017

Same situation. Incidentally this well illustrates that the only thing that makes features like that possibly not be considered a building is that it is located underground. Kind of proves the point i think that a covered reservoir will either be a building or be located underground.

This of course does not mean it should not be rendered, just it has some implications on how it should be rendered.

@dieterdreist
Copy link

dieterdreist commented Apr 27, 2017 via email

@hholzgra
Copy link

About half of the covered reservoirs in my area have a small operations building on to, so mapping the actual reservoir as a building would end up with a building on top of a building. This could of course be solved by using different layer=... values for the actual reservoir and the operations building, but somehow that still feels wrong ...

@dieterdreist
Copy link

dieterdreist commented Feb 18, 2018 via email

@matthijsmelissen
Copy link
Collaborator

See also #1167.

@jragusa
Copy link
Contributor

jragusa commented May 8, 2018

If we consider underground building as figured on the wiki page of man_made=reservoir_covered, JOSM shows these features with a blue dashed line without filling. This would be consistent with the suggestion of @Tomasz-W in #552 for underground buildings.

@jragusa
Copy link
Contributor

jragusa commented May 8, 2018

For example :
https://www.openstreetmap.org/way/69582545
z=19
reservoir_covered
z=18
reservoir_covered_z18

@kocio-pl
Copy link
Collaborator

kocio-pl commented May 8, 2018

This rendering resembles track or some other kind of highway more than building to me.

@Tomasz-W
Copy link

Tomasz-W commented May 8, 2018

I think this feature should be rendered just the same as other underground buildings (#552)

@pnorman
Copy link
Collaborator

pnorman commented May 28, 2018

We're using blue dashed lines for other features already.

@chazanov
Copy link

@jragsusa: May you please publish your changes to the style for the blue lines rendering?

@kocio-pl
Copy link
Collaborator

Maybe we should just render an icon+label, because cover can be anything - a building (so it should be tagged as a building) or some underground construction. No matter what we will do with rendering underground objects, it makes sense to show what is there.

Something like two waves with a roof is what might work for the icon shape.

@jragusa
Copy link
Contributor

jragusa commented Nov 26, 2018

My purpose is indeed to use the same rendering than underground buildings because we can deduce the shape of this feature since it's frequently semi-buried. A label is not useful because they do not have necessarily a name (at least in France).

@jeisenbe jeisenbe changed the title Render closed ways tagged with man_made=reservoir_covered Render man_made=reservoir_covered areas Nov 10, 2019
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