Skip to content

Conversation

@ValentinaHutter
Copy link

No description provided.

@ValentinaHutter
Copy link
Author

Sorry for the delay! We have been working on a number of updates for sen2like with telespazio and wanted to finish the updates before publishing the udp. The biggest change, that also affected this PR, is that openEO's sen2like can now also process SENTINEL2_L2A, so I updated the example to start from this collection.

Other than that, I would appreciate your reviews!

@jdries jdries linked an issue Nov 19, 2025 that may be closed by this pull request
Copy link
Contributor

@jdries jdries left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice, thanks for submitting the PR!
We normally have a convenient 'preview' feature, allowing to inspect how it looks like in the real catalog, but the github action seems to fail.
So I anyway went ahead for a review, made some suggestions.

"process_id": "sen2like",
"namespace": "https://raw.githubusercontent.com/ESA-APEx/apex_algorithms/refs/heads/sen2like/algorithm_catalog/eodc/sen2like/openeo_udp/sen2like.json",
"arguments": {
"bbox": {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the 'bbox' parameter would be 'spatial_extent' according to current udp definition.
If I however try to run this scenario, it gives "No Sentinel2 tiles available for provided request!"
Test job id: d5c7336a-fe72-4867-9762-c4fad4c10025

Suggested change
"bbox": {
"spatial_extent": {
"east": 16.414,
"north": 48.008,
"south": 47.962,
"west": 16.342
},
"temporal_extent": [
"2023-05-01T00:00:00Z",
"2023-05-30T00:00:00Z"
]

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I now updated the parameter name. Sorry, I initially tested it on the dev backend, where it ran through successfully, but I only deployed the latest changes to production today.

@ValentinaHutter
Copy link
Author

Nice, thanks for submitting the PR! We normally have a convenient 'preview' feature, allowing to inspect how it looks like in the real catalog, but the github action seems to fail. So I anyway went ahead for a review, made some suggestions.

Is this error related to the udp and is there anything I can try updating in the PR? I was trying to understand what is failing, but since it is in the Generate a token for gh cli step, I am not sure, if it could be fixed in the PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Sen2Like (EODC)

2 participants