search

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 CBERS #24

Closed vincentsarago closed 6 years ago

vincentsarago commented 6 years ago

with rio-tiler==1.0a2 retrieving CBERS tiles is way slower and need more memory.

I wonder if reading a mask while we know it's not needed could be the reason. https://github.com/mapbox/rio-tiler/blob/6100616a6b4a56e7778cf616da4c909530cc6126/rio_tiler/utils.py#L165-L168

Note: For Landsat/Sentinel we don't need to read the masks because there is no internal nodata and need to create the mask manually.

vincentsarago commented 6 years ago

closed by #25