Open-EO / openeo-processes

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

filter_spatial vs filter_vector #460

Closed soxofaan closed 10 months ago

soxofaan commented 11 months ago

Not sure if there is still room to change a lot here, but while looking into vector cube issues I observed this:

I think at least we should have better cross-referencing (documentation-wise) between these processes (there is no documentation link between filter_spatial and filter_vector)

m-mohr commented 11 months ago

I considered it, but based on the relation parameter I found it akward.

You'd need to assume the Pixels are points at the pixel center, but I'm not sure whether you'd want to implement all the allowed relation types that you have for vectors also for rasters. What happens if you only support intersection for raster and all other relation types for vector? Confusing. filter_bbox was merged as it doesn't support the relation types and is much simpler in that regard.

But indeed, crosslinking between them would be good.