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

`load_url` with GeoJSON: alignation with `load_geojson` #450

Open soxofaan opened 1 year ago

soxofaan commented 1 year ago

This is follow-up of multiple earlier discussions (e.g. #322 #415 #427 #346).

Current openeo-processes draft has:

The properties parameter of load_geojson is not that trivial (e.g. see #448), but it is quite an important feature.

How to get this feature in load_url+GeoJSON? I guess currently the only way is that the "GeoJSON" input format (GET /file_formats) defines a properties parameter that aligns with the properties parameter of load_geojson.

This has a couple of disadvantages for users, backends and documentation:

Possible alternative: add support in load_geojson for loading from URL in addition to inline GeoJSON. I would guess that the additional implementation work for a back-end implementer is very minimal to add this.

This has been discussed before, but I can not completely reconstruct if this idea was intentionally dropped or forgotten.

SerRichard commented 6 months ago

This would also be good for the openeo-processes-dask. Currently we're using a custom "load_vector_cube" process for loading the remote geometries and it would be better to replace it with the load_geojson process in the implementations ( for the obvious benefit of alignment ).

In order to realise this change we would currently need to support the additional process of load_url in the dask processes, but, as already mentioned the code to support loading from URLs direclty in the geojson implementation is minimal, and would be less work than implementing load_url and ensuring correct behaviour on our side.

Whenever the discussion is held, also a thumbs up from eodc for this!