-
Notifications
You must be signed in to change notification settings - Fork 51
feat: earthdata providers #874
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
base: develop
Are you sure you want to change the base?
Conversation
Test Results 4 files ± 0 4 suites ±0 4m 28s ⏱️ +12s Results for commit 1fdf83f. ± Comparison against base commit e931ca1. This pull request removes 7 tests.♻️ This comment has been updated with latest results. |
|
Code Coverage (Ubuntu)
Minimum allowed coverage is Generated by 🐒 cobertura-action against ee8e041 |
|
Code Coverage (Windows)
Minimum allowed coverage is Generated by 🐒 cobertura-action against ee8e041 |
3726a21 to
43a5e89
Compare
Code Coverage (Ubuntu)Diff against developResults for commit: c0b7011 Minimum allowed coverage is ♻️ This comment has been updated with latest results |
Code Coverage (Windows)Diff against developResults for commit: 86c7a01 Minimum allowed coverage is ♻️ This comment has been updated with latest results |
45ed697 to
86c24ef
Compare
Code Coverage (Ubuntu)Diff against developResults for commit: 1fdf83f Minimum allowed coverage is ♻️ This comment has been updated with latest results |
Code Coverage (Windows)Diff against developResults for commit: 1fdf83f Minimum allowed coverage is ♻️ This comment has been updated with latest results |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
rebase needed
f1d3725 to
a0f0376
Compare
| download: !plugin | ||
| type: HTTPDownload | ||
| --- | ||
| !provider # MARK: earthdata_lars_asdc |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
collections for the 3 providers are paginated (100 max per request)
| download: !plugin | ||
| type: HTTPDownload | ||
| --- | ||
| !provider # MARK: earthdata_lars_asdc |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
tried to download item from ACTIVATE-FLEXPART_1 on earthdata_lars_asdc, and got a 11KB file containing authentication html page instead of a tgz
eodag/resources/providers.yml
Outdated
| priority: 0 | ||
| search: !plugin | ||
| type: StacSearch | ||
| timeout: 20 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| timeout: 20 | |
| timeout: 60 |
| productType: '{productType}' | ||
| download: !plugin | ||
| type: SessionHTTPDownload | ||
| auth: !plugin |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
downloaded files on OPERA_L3_DSWX-S1_V1_1.0 were empty, there might be an issue with the plugin
788a084 to
ce6022f
Compare
…ection + new providers for PODAAC, LARC_ASDC and OB_DAAC (#773) Co-authored-by: Anne-Sophie Tonneau (CELAD) <[email protected]>
afb4ab5 to
d51eb62
Compare
Follows #773
Fixes #755
https://www.earthdata.nasa.gov/