Skip to content
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

Stream download aws #997

Merged
merged 15 commits into from Jan 31, 2024
Merged

Stream download aws #997

merged 15 commits into from Jan 31, 2024

Conversation

jlahovnik
Copy link
Collaborator

implements direct streaming of assets for aws download plugin

@jlahovnik jlahovnik self-assigned this Jan 19, 2024
Copy link
Contributor

github-actions bot commented Jan 19, 2024

Test Results

    4 files  ± 0      4 suites  ±0   4m 37s ⏱️ +23s
  455 tests + 3    452 ✅ + 3   3 💤 ±0  0 ❌ ±0 
1 820 runs  +12  1 740 ✅ +10  80 💤 +2  0 ❌ ±0 

Results for commit 6f56a4f. ± Comparison against base commit dda681a.

This pull request removes 1 and adds 4 tests. Note that renamed tests count towards both.
eodag.types.__init__ ‑ eodag.types.model_fields_to_annotated_tuple
eodag.types.__init__ ‑ eodag.types.model_fields_to_annotated
tests.units.test_http_server.RequestTestCase ‑ test_timeout_error
tests.units.test_search_plugins.TestSearchPluginQueryStringSearch ‑ test_plugins_search_querystringseach_timeout
tests.units.test_stac_utils.TestStacUtils ‑ test_search_products_fail

♻️ This comment has been updated with latest results.

Copy link
Contributor

github-actions bot commented Jan 19, 2024

Code Coverage (Ubuntu)

File Coverage
All files 81%

Minimum allowed coverage is 70%

Generated by 🐒 cobertura-action against 6f56a4f

Copy link
Contributor

github-actions bot commented Jan 19, 2024

Code Coverage (Windows)

File Coverage
All files 75%

Minimum allowed coverage is 70%

Generated by 🐒 cobertura-action against 6f56a4f

eodag/plugins/download/base.py Outdated Show resolved Hide resolved
eodag/plugins/download/base.py Outdated Show resolved Hide resolved
eodag/plugins/download/aws.py Outdated Show resolved Hide resolved
eodag/plugins/download/aws.py Outdated Show resolved Hide resolved
eodag/plugins/download/aws.py Outdated Show resolved Hide resolved
eodag/plugins/download/aws.py Show resolved Hide resolved
eodag/plugins/download/aws.py Outdated Show resolved Hide resolved
@sbrunato sbrunato marked this pull request as draft January 23, 2024 08:50
eodag/plugins/download/aws.py Outdated Show resolved Hide resolved
@jlahovnik jlahovnik marked this pull request as ready for review January 23, 2024 15:18
eodag/plugins/download/base.py Outdated Show resolved Hide resolved
eodag/plugins/download/base.py Outdated Show resolved Hide resolved
eodag/plugins/download/base.py Outdated Show resolved Hide resolved
eodag/plugins/download/base.py Outdated Show resolved Hide resolved
@sbrunato sbrunato marked this pull request as draft January 25, 2024 14:19
@jlahovnik jlahovnik marked this pull request as ready for review January 26, 2024 07:37
eodag/api/product/_assets.py Show resolved Hide resolved
eodag/api/product/_assets.py Outdated Show resolved Hide resolved
eodag/api/product/_assets.py Show resolved Hide resolved
@sbrunato sbrunato marked this pull request as draft January 29, 2024 09:48
@jlahovnik jlahovnik marked this pull request as ready for review January 29, 2024 10:35
@jlahovnik
Copy link
Collaborator Author

typing problems should be fixed

@sbrunato sbrunato marked this pull request as draft January 29, 2024 18:22
@sbrunato sbrunato marked this pull request as ready for review January 31, 2024 15:22
@sbrunato sbrunato merged commit 72e7a48 into develop Jan 31, 2024
12 checks passed
@sbrunato sbrunato deleted the stream-download-aws branch January 31, 2024 15:24
@sbrunato sbrunato added this to the 2.12.0 milestone Feb 12, 2024
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.

None yet

3 participants