search

gravitystorm / openstreetmap-carto

A general-purpose OpenStreetMap mapnik style, in CartoCSS
Other
1.53k stars 817 forks source link

Add rendering of aerialway=pylon #3155

Open jragusa opened 6 years ago

jragusa commented 6 years ago

Expected behavior

I propose to render aerialway=pylon in a similar way than power=tower. Rendering should start at z=17. I will understood if you consider that's too noisy with the pattern of aerialways.

Actual behavior

Pylons are not displayed.

Links and screenshots illustrating the problem

aerialway=chair_lift (https://www.openstreetmap.org/way/288893232) aerialway_pylon_z17 aerialway=gondola (https://www.openstreetmap.org/way/174540569) aerialway_pylon2_z17

kocio-pl commented 6 years ago

I think at z19+ it would not be a problem, since it's just a detail, but still the existing (tagged) one. Highest zoom level is for micromapping people to have their work visible - after all why somebody zooms that strong such a big feature as aerialways?

jragusa commented 6 years ago

@kocio-pl what do you propose ? I can make a PR for z=19 or if you consider it's useless, I close this issue.

kocio-pl commented 6 years ago

Go on, please.

matkoniecz commented 6 years ago

Note that it may be tricky to display both pylon-like symbolic symbols at lower zoom and pylon locations at high zoom.

Maybe to avoid confusion it would be better to not introduce display of pylons.

jragusa commented 6 years ago

I see what do you mean. I checked in Western Alps and I didn't find any isolated aerialway=pylon. Mappers preferentially render aerialway lines than pylons.