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

Pipeline should be visible #640

Closed
Cetusek opened this issue Jun 16, 2014 · 85 comments · Fixed by #4070
Closed

Pipeline should be visible #640

Cetusek opened this issue Jun 16, 2014 · 85 comments · Fixed by #4070

Comments

@Cetusek
Copy link

Cetusek commented Jun 16, 2014

Why are pipelines not rendered?
They are good landmarks.

@RobJN
Copy link

RobJN commented Jun 16, 2014

Above ground pipelines may be landmarks and therefore you could argue that they should be rendered. The underground pipelines shouldn't be rendered as they will confuse the map view.

For a map of pipeline in OSM see http://www.itoworld.com/map/220

@javbw
Copy link

javbw commented Jun 17, 2014

If the pipe is tagged overground/overhead/outdoor, why can't it be rendered at, lets say, Zoom level 17 and higher?
Zenrin's data in Japan shows pipes at high zoom levels (what Google switches to as you zoom in Japan).

http://goo.gl/maps/853rS

screen shot 2014-06-17 at 11 48 06 am

It's for some pretty big pipes.
screen shot 2014-06-17 at 11 50 01 am

@matkoniecz
Copy link
Contributor

I am sure that it is a good idea to display pipelines, except underground ones. Note that there are pipeline bridges.

@Cetusek
Copy link
Author

Cetusek commented Jun 17, 2014

Of course I ment overground pipelines. Is there any chance to make them visible?

@matthijsmelissen
Copy link
Collaborator

What's the tagging scheme to distinguish between underground/overground?

In the Sahara, pipelines are also important landmarks: http://www.theleader.info/media/images/articles/thumbnails/11784_320x0.jpg
They are in fact so visible that some of the used to be tagged in OSM as 'highway=primary' from early low-res satellite imagery!

@Klumbumbus
Copy link

What's the tagging scheme to distinguish between underground/overground?

http://wiki.openstreetmap.org/wiki/Key:location

@gravitystorm gravitystorm modified the milestones: New features, 3.x - Needs upgrade to mapnik or openstreetmap-carto.style Jun 17, 2014
@matkoniecz
Copy link
Contributor

So probably the best solution is to show only ones with location=overground

@Cetusek
Copy link
Author

Cetusek commented Jun 17, 2014

You are right. What should we do to achieve this goal?

@RobJN
Copy link

RobJN commented Jun 17, 2014

The best way would be to get involved with writing the code (the style-sheets on this github project). In github terminology that means forking the code to your personal space, updating the code, and supplying a "Pull Request" back to this github project. That's all explained on github's help (or just google it) as it's a github process not an OpenStreetMap process.

You may want to watch Andy's workshop at the recent SotM-EU conference.
https://www.youtube.com/watch?v=W7La0UXec3A

If you cannot find someone willing to help with this, you will simply have to wait until one of the other members on this github project pick it up. There's quite a back-log so patience will be required.

@Rovastar
Copy link
Contributor

Do we have location as a tag in the db, if not then then it will be trickier too.

I do think we could render these I see the benefit.
I suppose the next question is how do we want it to look. We have soooo many different colours and styles for the lines on the map. Any suggestions that do not clash?

@pnorman
Copy link
Collaborator

pnorman commented Jun 17, 2014

Do we have location as a tag in the db

Don't believe so, which puts this to 3.0+

@matkoniecz
Copy link
Contributor

Maybe dark gray line, wider than barriers? It is a special kind of barrier, so it should be OK to have it similar. With bridge casings for elevated ones (with bridge=*).

@kocio-pl
Copy link
Collaborator

Maybe dark gray line, wider than barriers? It is a special kind of barrier, so it should be OK to have it similar. With bridge casings for elevated ones (with bridge=*).

Sounds good for me.

I'm also thinking about segmented line, resembling pipe joinings (rough ASCII sketch):

|——||——||——||——||——|

@javbw
Copy link

javbw commented Dec 19, 2016 via email

@kocio-pl
Copy link
Collaborator

@kocio-pl
Copy link
Collaborator

kocio-pl commented May 1, 2018

I have finally found a way to do it:

screenshot-2018-5-2 openstreetmap carto kosmtik

The trick was to use image (LinePatternSymbolizer in Mapnik), the same as in the cliff rendering - above rendering is done simply with replacing the SVG file. The only problem was that segments had no space between them, despite the space on both ends. However adding invisible blocks (alpha=0) helped.

Now we can get back to design. What color should be used? What segment length (currently 13 px total, with endings x 1 px), endings size (5x1 px) and the space between segments (1+1=2 px)?

@Tomasz-W
Copy link

Tomasz-W commented May 2, 2018

Surely it's too strong at the moment. It also is too sililar to aerialway=* rendering, eg.
https://www.openstreetmap.org/#map=16/46.2161/6.7975

@kocio-pl
Copy link
Collaborator

kocio-pl commented May 2, 2018

The spaces are too small? What's your proposition?

All the lines are similar and it's just the convention. I believe the context will make it clear - aerialways consist of short straight lines and have stations on both ends, pipelines tend to be long and curved.

@turnsole80
Copy link

turnsole80 commented May 6, 2018

I like it. If you can get it to work, i'll carry the same ideas to carry over to #1999 , because right now i'm drawing a blank there.

Colour wise, i'd go with that generic grey you use for things like bunkers, etc

@eigenwillig
Copy link

eigenwillig commented Jun 20, 2018

@kocio-pl Could you please publish the code with LinePatternSymbolizer here?

@jidanni
Copy link

jidanni commented Dec 19, 2019

Better footing on this massive water pipeline / aqueduct / bridge tagged as above but not rendered.

@daniel-callejas-sevilla

You are welcome. I have a number of PRs open, so I hope someone else can continue to work on this one, but I look forward to reviewing it.

Where can I find the PR with your changes? I'd like to try it in local.

@jeisenbe
Copy link
Collaborator

jeisenbe commented Dec 22, 2019

https://github.com/jeisenbe/openstreetmap-carto/tree/pipeline - this is the branch that I was using to test possible renderings for man_made=pipeline, but I did not submit a PR yet.

@jeisenbe
Copy link
Collaborator

jeisenbe commented Jan 9, 2020

@daniel-callejas-sevilla were you interested in submitting an PR to add this feature? Do you need any help?

@Adamant36
Copy link
Contributor

@jeisenbe, would the PR be at a mergable point close to where your test branch has it or would it still need any major modification?

@jeisenbe
Copy link
Collaborator

jeisenbe commented Jan 10, 2020 via email

@Adamant36
Copy link
Contributor

A place in Poland with the pipelines running next to each other. It looks fine from z19-z17, but starts getting iffy after that.

z19
Poland 1 z19
z18
Poland 1 z18
z17
Poland 1 z17
z16
Poland 1 z16
z15
Poland 1 z15
z14
Poland 1 z14

@flacombe
Copy link

Easier for me to write than to do, it may be an option to make them disappear or even show like solid lines < z17

@Adamant36
Copy link
Contributor

I think making not display at anything more zoomed out then z17 is probably the answer. If we make them solid lines it would be hard to tell them from other things. They don't look good sooner then z17 by themselves anyway.

Single pipeline z16 (check out the last two parts on the left side. Doesn't look good due to the pipeline doing a little zigzag)

Poland single pipeline z16

Doesn't look to bad at z15 or z14 if they are straight.
z15
Stright pipelines containing water next to each other at z15
z14
Stright pipelines containing water next to each other at z14
Single pipeline over water at z18. I was looking for a water pipeline over water but couldn't find one.
Single pipeline z18 over water

@jeisenbe
Copy link
Collaborator

jeisenbe commented Jan 12, 2020 via email

@jidanni
Copy link

jidanni commented Jan 12, 2020

Note these are often thousands of kilometers long structures.
It would be very frustrating needing to zoom in and out hunting for these like a blind man.

@jeisenbe
Copy link
Collaborator

jeisenbe commented Jan 12, 2020 via email

@Adamant36
Copy link
Contributor

We show even minor power lines, streams, private driveways and private
footways at z16 or sooner, so it would be strange to limit pipelines
to z17.

Yeah, I agree.

@jidanni
Copy link

jidanni commented Jan 12, 2020

I can thankfully see this underground canal
https://www.openstreetmap.org/way/219361140#map=12/24.1175/120.9406
all the way down to zoom 12.

If you think of it from the point of view of oil companies, they would rather not have the public know about their pipelines.

But from the point of view of environmentalists, it is important to know about pipelines.

I recall opening up great (paper) atlases of the world in the library, and seeing the whole Russian infrastructure, Trans-Siberian pipeline, railroad, highway, all on the same big map. That's what made them great.

@jidanni
Copy link

jidanni commented Jan 12, 2020

If people are concerned about the Amazon forests, they would like to see where pipelines have got to, not only roads, at the same zoom levels. High voltage power lines too. One look at OSM and people will say "Now there's a map that puts the (vegan) meat on the table!" (Of course with a crowding factor added to keep them from getting too dense.)

@jidanni
Copy link

jidanni commented Jan 12, 2020

Goo*** etc. commercial, and especially government maps, have all kinds of pressures to keep various multinational corporations' stuff off the map. This is where OSM can outshine them.

@jidanni
Copy link

jidanni commented Jan 12, 2020 via email

@Adamant36
Copy link
Contributor

@jeisenbe, any suggestions since its your design and code?

@jeisenbe
Copy link
Collaborator

jeisenbe commented Jan 14, 2020 via email

@javbw
Copy link

javbw commented Jan 14, 2020 via email

@jeisenbe
Copy link
Collaborator

@Adamant36 - were you considering opening a PR for this?

@Adamant36
Copy link
Contributor

uuummm, yeah. But it doesn't look good at certain zoom levels and everyone thinks they should still be rendered at those levels. So, that needs to be worked out first.

@jeisenbe
Copy link
Collaborator

jeisenbe commented Feb 16, 2020 via email

@Adamant36
Copy link
Contributor

I don't know. I just thought it could have looked better. Although I don't have an exact, specific problem with it though. We really could just do the PR as is. Since it looks good otherwise and I don't think the problem on z16 (if there even is one) is worth holding it up.

@Adamant36
Copy link
Contributor

@jeisenbe, you want me to do the PR for it?

@jeisenbe
Copy link
Collaborator

jeisenbe commented Feb 24, 2020 via email

@pnorman
Copy link
Collaborator

pnorman commented Feb 24, 2020

If you don’t think it is ready yet, then you can add “[WIP]” for now, and
ask for help with the parts that you think need improving.

It's better to do it as a draft PR if you don't think its ready.

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

Successfully merging a pull request may close this issue.