#152 add support for ResampledTiles

Open-EO / openeo-geotrellis-extensions

Java/Scala extensions for Geotrellis, for use with OpenEO GeoPySpark backend.

Apache License 2.0

5 stars 3 forks source link

#152 add support for ResampledTiles #168

Closed JeroenVerstraelen closed 1 year ago

JeroenVerstraelen commented 1 year ago

Added noResampleOnRead datacube parameter Added ResampledRasterSource Added ResampledTile Added testS2UpsampledTilesFromLayer

There is one small issue with CroppedTile.convert, this creates a new ArrayTile and destroys the underlying ResampledTile. We use this whenever we read from a BandCompositeRasterSource. To circumvent this issue I introduced the ResampledTile.cropAndConvert method.

jdries commented 1 year ago

looks generally fine, and safe to merge. Maybe we can merge these things into develop already, but two things to consider:

ResampleTile was added to tile_grid package, not sure if that is a logical place, maybe consider putting it somewhere under geotrellis-common?
Add unit test on the new tile and RasterSource? Maybe we can even copy basic tests from geotrellis itself?