-
Notifications
You must be signed in to change notification settings - Fork 240
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
pmtiles root/leaf index size issue ? #653
Comments
I think this is probably because tilemaker doesn't generate clustered pmtiles archives:
Because tilemaker's tile generation is multi-threaded, and some tiles may take much longer to generate than others (due to complex geometries), we can't guarantee that tiles will be output in any particular order. Therefore we don't get the efficiency gains that a clustered archive would give. For a clustered archive, you'd need to create an .mbtiles with tilemaker, then use go-pmtiles to turn that into a clustered .pmtiles. There's some discussion of this in the original PMTiles PR, #620, in particular:
|
Thanks Richard, I'll go that way, mbtiles + pmtiles conversion. One way to deal with unordered generation, is to add some inbetween queue. |
😁 Yes, you're right. I'm slightly anxious about a queue getting blocked on a tile with a really horrible multipolygon geometry (Saimaa or the US National Forests, that sort of thing) but there are possibilities for the future. |
I created a WIP branch of a new (Note: for small archives you will need the tilemaker for
So the savings are almost 80% total for a large country (~55 MB -> 11 MB), should also mean partial directory downloads are much smaller. Welcome to experiment on the PR for different areas. |
I've compared a full planet pmtiles file generated by pmtiles and one generated directly by tilemaker.
The nginx log show regular multi megabytes ranges requested on the timaker version, and nothing like that on the pmtile version.
I suspect the tile index not being hierarchical enough.
You can query both files here if you want to have a look :
The text was updated successfully, but these errors were encountered: