-
Notifications
You must be signed in to change notification settings - Fork 612
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
Concatenate pbf tiles and TileJSONs #6
Comments
It seems to be done by Mapnik usually (see klokantech/tileserver-mapnik#13), but we may prefer an implementation without Mapnik dependency here. |
The speed of the PBF JavaScript-native library https://github.com/mapbox/pbf has improved recently - if we decide to parse and reencode the PBFs for overzooming (scale + shift) at some point. The https://github.com/mapbox/vector-tile-js is using it already. In case we decide to go for mapnik C/C++ implementation, here is a node-js wrapper example: It would be practical to compare the speed of the two implementations, once finished. |
I would suggest not to implement this feature, as this feature is not only unnecessary but hard to properly implemented. One problem is how do you solve vector layer id conflict? What if I composite too much pbf s and exceed 500K limits? Simple codes are easy to test and maintenance. Omit the unnecessary feature and keep the code clean, I suggest. |
We need to be able to process following requests:
/vector/{mbtiles_name},{mbtiles_name}/{z}/{x}/{y}.pbf
pbf
s need to be concatenated (we already do this for fonts)pbf
s from different zoom levels/vector/a,b/8/19/21.pbf
whenb
has maxzoom6
a/8/19/21.pbf
and part ofb/6/4/5.pbf
(the tile needs to be unpacked, scaled, repacked)/vector/{mbtiles_name},{mbtiles_name}.json
name
,maxzoom
,bbox
,center
, ...The text was updated successfully, but these errors were encountered: