search

mapbox / rio-mbtiles

A plugin command for the Rasterio CLI that exports a raster dataset to an MBTiles 1.1 SQLite file
MIT License
125 stars 36 forks source link

Add control over tile size #44

Closed jdmcbr closed 5 years ago

jdmcbr commented 5 years ago

Closes #33 by adding user control over tile size and setting the default to 512x512. Apologies for the commit history being a mess, I shouldn't have been working on this last year at the same time as the tile skipping PR.

jdmcbr commented 5 years ago

@sgillies Cool, pushed that. I forgot in the PR to re-raise an issue I'd mentioned in the #33, which was that for a subset of the images I'm working with, the 512x512 option resulted in some tiles that were too large for mapbox (above the 500kb limit). I'm not sure if I'm just unusual, but 256x256 tiles might be a better default.

sgillies commented 5 years ago

Let's stick with 256 as the default then @jdmcbr .

sgillies commented 5 years ago

@jdmcbr thanks!