search

Open-EO / openeo-geotrellis-extensions

Java/Scala extensions for Geotrellis, for use with OpenEO GeoPySpark backend.
Apache License 2.0
5 stars 4 forks source link

Dateline #310

Open EmileSonneveld opened 4 months ago

EmileSonneveld commented 4 months ago

The issue was an intersection calculation losing a lot of precision due to reprojecting and back. Now the intersection between the feature and the requested extent is calculated in the target extent crs.

This made that some tests now load a smaller piece of the feature, on some places {FloatConstantNoDataArrayTile@16026} ArrayTile(256,256,float32)became {PaddedTile@15829} PaddedTile(ArrayTile(140,122,float32),17,134,256,256)

EmileSonneveld commented 4 months ago

Added a test for feature in LatLng and target extent in UTM. Had to change the code too. This comment describes the change:

We convert both extents to a common CRS before taking the intersection. If one of the CRSes can cover the whole world (non-UTM), this will be used as common CRS. We give priority to use the target CRS as common one, because the intersection will be converted to it anyway

EmileSonneveld commented 3 months ago

There where already many cases where 2 different UTM extentds where used. Confirmed by logging