search

Open-EO / openeo-processes

Interoperable processes for openEO's big Earth observation cloud processing.
https://processes.openeo.org
Apache License 2.0
48 stars 15 forks source link

merge_cubes: resample implicitly #402

Closed m-mohr closed 1 year ago

m-mohr commented 1 year ago

Process ID: merge_cubes

Describe the issue:

@jdries wrote at https://discuss.eodc.eu/t/terrascope-backend-upgrade/542:

merge_cubes and mask will now try to automatically resample input data cubes if required, reducing the need to always use resample_cube_spatial When catalogs contain 2 versions of the same product, we now detect this better and try to use the most recent version. This improves performance, but also reproducibility.

This is not ideal wrt reproducibility, I think. Different back-ends behave differently then. It's non breaking as it purely adds behavior for what before just threw an error.

Proposed solution:

We discussed this in the PSC and it seems reasonable here to be user-friendly and implicitly add the resampling to the merge_cubes process in general. It would be described just as an implicit call to resample_cube_spatial where the first cube is always the baseline.

soxofaan commented 1 year ago

This feature could also be considered for the mask process