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

JEP udf callApplyMetadata should change band dimension #266

Closed JeroenVerstraelen closed 6 months ago

JeroenVerstraelen commented 7 months ago

Currently callApplyMetadata:

We want to support:

EmileSonneveld commented 7 months ago

Minimal example to reproduce: https://gist.github.com/JeroenVerstraelen/57847e544374c4eb172e205f8da32f02

EmileSonneveld commented 7 months ago

@JeroenVerstraelen in the example from here, the returned array is in dtype float64, I had to cast it to float32 to avoid an error. Do you think openEO should be dealing with it instead? ...transpose(*dims).astype('float32')

EmileSonneveld commented 6 months ago

In the UDF, a .name should be added [openeo.metadata.Band(f\"{x.name}_STD\", None, None) for x in metadata.bands] (relevant: https://github.com/Open-EO/openeo-api/issues/530 )

After this, the CRS seems to go missing.

JanssenBrm commented 6 months ago

Thanks guys, nice work! I got it working on dev using your fixes and sample code (https://github.com/Open-EO/FuseTS/blob/feat/123_mogpr_uncertainties/src/fusets/openeo/mogpr_udf.py#L44-L57).

Any idea on when this would be available on production? Looking forward!