search

Open-EO / openeo-python-driver

Common parts of a Python driver implementation for OpenEO
Apache License 2.0
11 stars 5 forks source link

Limited set of processes in property filter #145

Open m-mohr opened 1 year ago

m-mohr commented 1 year ago

I'm using lt in a property filter in load_collection, but get:

Failed to process synchronously on backend vito: OpenEoApiError("[400] PropertyConditionInvalid: Property filtering only supports ['eq', 'lte', 'gte'], not 'lt'. (ref: r-b2321fd4b96b49889b889322f5c12b36)")

While I can easily migrate to lte, it shouldn't be to difficult to also support lt, gt, neq etc, right?

soxofaan commented 1 year ago

error comes from here: https://github.com/Open-EO/openeo-python-driver/blob/b02761cd8f115d177f16a073945e2e2359f36a58/openeo_driver/filter_properties.py#L28-L31

also related to https://github.com/Open-EO/openeo-geotrellis-extensions/issues/39

As far as I currently understand all the moving parts here, adding support for lt or gt is not as trivial as it seems (because only the threshold is preserved while the actual comparison operator is stripped somewhere in the pipeline)