Open-EO / openeo-python-client

Python client API for OpenEO
https://open-eo.github.io/openeo-python-client/
Apache License 2.0
156 stars 42 forks source link

Smarter geometry loading from `DataCube.aggregate_spatial` #581

Open soxofaan opened 5 months ago

soxofaan commented 5 months ago

DataCube.aggregate_spatial(geometries) and DataCube.filter_spatial(geometries) currently allows geometries to be a string or pathlib.Path, but then it assumes this is a backend-side path and "loads" this with non-standard read_vector.

It would be better (more user friendly, and more standard) to be a bit smarter here:

Also

soxofaan commented 5 months ago

(this was triggered from https://discuss.eodc.eu/t/aggregate-spatial/766)

jdries commented 14 hours ago

This looks like low hanging fruit that can solve some issues. The client can also be smart about injecting a 'load_geojson' process, if that is supported by the backend. That way, users automatically get the 'preferred' behaviour and avoid deprecation warnings from the backend.