protomaps / basemaps

Basemap PMTiles generation and cartographic styles for OpenStreetMap data and more
https://maps.protomaps.com/
Other
378 stars 49 forks source link

Bus stops missing in POIs #163

Closed eikes closed 3 weeks ago

eikes commented 1 year ago

The tileset does not include bus stops. It would be nice to have them.

bdon commented 1 year ago

Thanks for the PR! @nvkelso it looks like bus_stop is defined in Tilezen for z16, which we don't do for tileset v3. So it makes sense for v3 to add this additively.

In general we should continue with additions that align with Tilezen, this one looks like it should have a minimal impact on overall tile size.

nvkelso commented 1 year ago

Looking at the Tilezen source code (ultimate source of truth versus docs), zoom 18 is more appropriate. Otherwise they really crowd the map!

bdon commented 1 year ago

Need to add a visualization for bus stops, blocked on #160

bdon commented 1 year ago

@eikes this is in the latest daily build:

https://protomaps.github.io/PMTiles/?url=https%3A%2F%2Fbuild.protomaps.com%2F20231111.pmtiles#map=18.33/37.765429/-122.419588

Screenshot 2023-11-12 at 11 56 31
eikes commented 1 year ago

Nice!

bdon commented 3 weeks ago

Bus stop icons are in styles 4.2: CleanShot 2024-10-23 at 15 08 32@2x