-
Notifications
You must be signed in to change notification settings - Fork 48
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
fix(providers): S2_MSI_L1C search-by-id for earth_search #1053
fix(providers): S2_MSI_L1C search-by-id for earth_search #1053
Conversation
Code Coverage (Ubuntu)
Diff against develop
Results for commit: f8820d0 Minimum allowed coverage is ♻️ This comment has been updated with latest results |
Code Coverage (Windows)
Diff against develop
Results for commit: f8820d0 Minimum allowed coverage is ♻️ This comment has been updated with latest results |
c91aabc
to
ffbc054
Compare
9af69c6
to
f8820d0
Compare
- fix id mapping for product S2_MSI_L1C - add product S2_MSI_L2A
f8820d0
to
d487a07
Compare
Code Coverage (Ubuntu)
Diff against develop
Results for commit: c13be77 Minimum allowed coverage is ♻️ This comment has been updated with latest results |
Code Coverage (Windows)
Diff against develop
Results for commit: c13be77 Minimum allowed coverage is ♻️ This comment has been updated with latest results |
e00866e
to
c13be77
Compare
In the provider
earth_search
:id
mapping for productS2_MSI_L1C
Note that
S2_MSI_L2A_JP2
was not added for this provider because assets s3 path is incorrect for.jp2
files on https://earth-search.aws.element84.com/v1/collections/sentinel-2-c1-l2a, and products cannot be downloaded.For example,
tiles/11/X/NM/2020/8/1/0/B02.jp2
does not exist, and should be one of:tiles/11/X/NM/2020/8/1/0/R10m/B02.jp2
,tiles/11/X/NM/2020/8/1/0/R20m/B02.jp2
,tiles/11/X/NM/2020/8/1/0/R60m/B02.jp2