-
Notifications
You must be signed in to change notification settings - Fork 17
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
load_stac: stac_items #501
Comments
@jdries so, if I understand it correctly, you would like to directly pass the STAC items as json/text in the process graph instead of an URL? It could be a good idea! From what I understand, if we integrate it in
|
This can quickly become problematic. Many STAC Items don't have absolute URLs and then you can't load the data if the self url isn't set. Usually you can use the Item URL if no self url is given, but the URL is not available here as fallback. Also, the JSON size can explode quickly if people start to pass thousand of Items. Generally, I think I'd prefer a separate process if at all. |
it's indeed limited to cases where you use absolute url's and don't send thousands of items. In fact, our new load_stac sample somewhat illustrates it: |
The place that would allow users to do that is the openEO /files endpoints. That was the original intention that users could upload any related files such as GeoJSON, STAC, etc. there. Due to the lack of implementation we didn't push this through the processes either, but maybe we should to encourage it. The other thing with the STAC example you linked to: Creating a STAC Item for this purpose seems "overkill".
I assume you don't need the geometry and the projected bbox is enough, but not sure. Do we have an agreed consensus across providers what the STAC Items need to contain to be read (and maybe optional ones for more efficiency)? And then I'm wondering, why not just: |
Proposed Process ID: load_stac
Proposed Parameter Name: stac_items
Optional: yes, default: None
Context
load_stac is very popular for loading user defined data, but require the stac json to be available via http url.
In many cases, such a url is not available, and the user thus needs to rely on a 3rd party service (e.g. github) to upload the stac json.
I see also a use case for systems that require signed urls for data access, where the user first needs to sign urls using a secret key.
Description
stac_items, if provided, is an array of valid STAC Item object. The backend will load all assets in the provided items.
Data Type
array of objects
Additional changes
the other parameters would no longer need to be present if stac_items is provided directly.
Alternative option is of course to turn this into a separate 'load_stac_items' process, with a single parameter?
The text was updated successfully, but these errors were encountered: