cogeotiff / rio-tiler

User friendly Rasterio plugin to read raster datasets.
https://cogeotiff.github.io/rio-tiler/
BSD 3-Clause "New" or "Revised" License
512 stars 106 forks source link

Performance issue with `main.tile` mask creation #26

Closed vincentsarago closed 6 years ago

vincentsarago commented 6 years ago

There is a bug in rasterio (mentioned in https://github.com/mapbox/rasterio/issues/1191, https://github.com/mapbox/rasterio/issues/1266) WrappedVrt which might be linked in poor performance when reading mask withing a file.

For sentinel, landsat and CBERS we don't read the mask because we are creating one using a nodata value.

vincentsarago commented 6 years ago

fixed by https://github.com/mapbox/rio-tiler/pull/45