fix(providers): Updates mapping for some Sentinel and CLMS datasets on wekeo_main#2116
Open
dalpasso wants to merge 18 commits into
Open
fix(providers): Updates mapping for some Sentinel and CLMS datasets on wekeo_main#2116dalpasso wants to merge 18 commits into
wekeo_main#2116dalpasso wants to merge 18 commits into
Conversation
Contributor
Collaborator
Code Coverage (Ubuntu)DetailsDiff against developResults for commit: 734d87e Minimum allowed coverage is ♻️ This comment has been updated with latest results |
Collaborator
Code Coverage (Windows)DetailsDiff against developResults for commit: 734d87e Minimum allowed coverage is ♻️ This comment has been updated with latest results |
64b850b to
6bef918
Compare
jlahovnik
requested changes
Apr 13, 2026
bf61db7 to
068461b
Compare
jlahovnik
approved these changes
Apr 13, 2026
jlahovnik
reviewed
Apr 14, 2026
sbrunato
requested changes
Apr 22, 2026
Collaborator
There was a problem hiding this comment.
use STAC-formatted parameters with mapped values, or if not available add a prefix (e.g. wekeo_main:foo:
- online -> order:status (adapt values to STAC format)
- polarisation -> sar:polarizations (adapt values to STAC format)
- format -> wekeo_main:format
d6ad634 to
db67387
Compare
5fa854d to
33f68aa
Compare
734d87e to
ebbddd6
Compare
Contributor
Code Coverage (Ubuntu)DetailsDiff against developResults for commit: d822dd5 Minimum allowed coverage is ♻️ This comment has been updated with latest results |
Contributor
Code Coverage (Windows)DetailsDiff against developResults for commit: d822dd5 Minimum allowed coverage is ♻️ This comment has been updated with latest results |
a0caef3 to
401da67
Compare
Contributor
Benchmark comparisonCompared baseline: |
37ef22b to
d822dd5
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
metadata_mappingupdated according to https://help.wekeo.eu/en/articles/13791456-updates-to-hda-api-query-parameters-for-some-sentinel-and-clms-datasetsList of collections of the provider
wekeo_mainaffected by this PR (grouped by dataset ID):Fixed also a typo and added search_by id_mapping for Sentinel collections.
Test script: