der-stefan / OpenTopoMap

A topographic map from OpenStreetMap and SRTM data
https://opentopomap.org
Other
461 stars 118 forks source link

wind generators and military areas too dominant in low zoom #151

Open hottrails opened 6 years ago

hottrails commented 6 years ago

Hello opentopos,

first let me congratulate to your project. It's the best osm style I have seen so far.

I just want to share my thoughts on two minor things:

  1. wind generators show up already in zoom level 12, which is very early compared to most other symboles. Especially in areas where there are a lot of them I find that distracting (see area south of paderborn, for example).

I would expect it to be handled like communication towers, which are displayed from zoom level 13.

  1. (smaller) military areas are shown beginning from zoom level 11 and are quite dominant (red & wide strokes). Maybe showing them only in higher zooms, choosing a less dominant style in level 11&12 or being even more selective on area size would be an improvement.

This screenshot gives examples:

image

Best regards, Thomas

der-stefan commented 6 years ago

Servus Thomas,

regarding your two issues:

hottrails commented 6 years ago

Hi Stefan,

comparing wind generators to telecommunication towers because of their visibility is a good point. I also thought of that, but was confused as a 200m height communication-tower in the respective area didn't show up until zoom 13 (OSM)

I looked at other communcation towers I know and found one other very visible missing: Stimmstamm (OSM) and two other, which are obviously only relais stations of mobile communication near Seybothenreuth (OSM) in the city of Warstein (OSM)

It's quite obvious that the problem here is the data quality. The documentation is very clear that a huge tower for transmitting radio applications should be tagged man_made=communications_tower and tower:type=communication for smaller ones. The examples above mix that up.

To deal with that perhaps it would be an option to also include the height attribute, e.g. including also [tower:type=communication] if their height exceeds lets say 100 m?

Regards, Thomas

der-stefan commented 6 years ago

Good idea, Thomas! I wanted to implement it right away, but noticed that the column height is missing in our database. So with the next import we can activate it. (Hopefully I don't forget it then.)

der-stefan commented 5 years ago

I'm not sure why your towers don't show up now. I thought the issue was fixed.