tilezen / vector-datasource

Tilezen vector tile service - OpenStreetMap data in several formats
https://www.nextzen.org/
Other
508 stars 119 forks source link

Remove `osm/` from default tile URL path #652

Closed nvkelso closed 8 years ago

nvkelso commented 8 years ago

We use several different data providers in our tiles, calling these osm/ is not accurate. Either we drop it or replace it with another name.

Proposed: url: https://vector.mapzen.com/all/{z}/{x}/{y}.topojson or url: https://vector.mapzen.com/newname/all/{z}/{x}/{y}.topojson

Existing: url: https://vector.mapzen.com/osm/all/{z}/{x}/{y}.topojson

Still support the old syntax, though (especially for the archived v0.7, v0.8, & etc tile sets).

zerebubuth commented 8 years ago

It's 99.83% OSM by size, which means calling it "osm" is accurate. However, because we label each data item with a source, it's superfluous to name the dataset as well. My first thought was to drop the prefix completely, but perhaps we should think about what the most appropriate way is to handle tilesets with default "extra" data baked in: Is it preferable to have additional "meta-layers" like all, or use a different prefix?

nvkelso commented 8 years ago

Redirect users asking for the osm prefix to the no-prefix version.

nvkelso commented 8 years ago

Or: https://tile.mapzen.com/vector/v1/all/{z}/{x}/{y}.topojson

nvkelso commented 8 years ago

https://tile.mapzen.com/raster/normal/v1/all/{z}/{x}/{y}.png

nvkelso commented 8 years ago

https://tile.mapzen.com/neat/v1/all/{z}/{x}/{y}.topojson

nvkelso commented 8 years ago

Drop raster, https://tile.mapzen.com/normal/v1/all/{z}/{x}/{y}.png

nvkelso commented 8 years ago

/cc @bcamper

bcamper commented 8 years ago

This is heading in a logical direction. Not sure all make sense for raster tiles (or at least normals/elevation, maybe it would for rendered rasters if we ever did that)?

nvkelso commented 8 years ago

Yep.

rmarianski commented 8 years ago

@nvkelso should the tilejson urls look like this?

https://tile.mapzen.com/vector/v1/tilejson/mapbox.json
nvkelso commented 8 years ago

Working for me.

nvkelso commented 8 years ago

As discussed in the tiles meeting today, we should add a user account part to the URL pattern:

General pattern:

Specific examples:

And we're still searching for a better name besides vector for the default basemap project.

nvkelso commented 8 years ago

We should track any additional work for final production deploy in https://github.com/mapzen/tile-tasks/issues/94.

nvkelso commented 8 years ago

Confirmed with product & documentation teams that we'll stick with vector as the project name for v1 launch. There isn't a compelling alternative & there's cost of getting people to adopt whatever the new thing might have been.

nvkelso commented 8 years ago

Nothing more to do here.