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

Dry run of STAC Catalog iteration #44

Merged
merged 23 commits into from
Dec 14, 2021
Merged

Dry run of STAC Catalog iteration #44

merged 23 commits into from
Dec 14, 2021

Conversation

banesullivan
Copy link
Contributor

@banesullivan banesullivan commented Dec 10, 2021

A dry run test of iterating over the STAC Catalogs. Both Sentinel and Landsat catalogs error after 10000 items, but the Landsat one gives an error message:

search_phase_execution_exception: [illegal_argument_exception] Reason: Result window is too large, from + size must be less than or equal to: [10000] but was [10010]. See the scroll api for a more efficient way to request large data sets. This limit can be set by changing the [index.max_result_window] index level setting.

The S3 static catalogs work fine.


This seems relevant: sat-utils/sat-api#225

@codecov-commenter
Copy link

codecov-commenter commented Dec 10, 2021

Codecov Report

Merging #44 (1bbe14e) into main (0c974ba) will decrease coverage by 0.21%.
The diff coverage is 84.61%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main      #44      +/-   ##
==========================================
- Coverage   74.24%   74.03%   -0.22%     
==========================================
  Files          34       34              
  Lines         695      697       +2     
==========================================
  Hits          516      516              
- Misses        179      181       +2     
Impacted Files Coverage Δ
rgd-watch-client/rgd_watch_client/plugin.py 95.45% <75.00%> (-4.55%) ⬇️
rgd-watch-client/rgd_watch_client/__init__.py 78.57% <83.33%> (-1.43%) ⬇️
watch/core/tests/test_client.py 100.00% <100.00%> (ø)
watch/core/tests/test_stac.py 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8296998...1bbe14e. Read the comment docs.

@banesullivan banesullivan changed the title Enable dry run test of STAC Catlog iteration Enable dry run test of STAC Catalog iteration Dec 10, 2021
@banesullivan
Copy link
Contributor Author

banesullivan commented Dec 10, 2021

@mcovalt, will you be able to look into this after finishing the deployment scripts, etc?

@banesullivan banesullivan changed the title Enable dry run test of STAC Catalog iteration Dry run of STAC Catalog iteration Dec 11, 2021
@banesullivan banesullivan marked this pull request as ready for review December 14, 2021 06:39
@banesullivan
Copy link
Contributor Author

This is working for everything but landsat/sentinel. Merging...

We can enable the LS/S2 in a follow up PR

@banesullivan banesullivan merged commit 9811c85 into main Dec 14, 2021
@banesullivan banesullivan deleted the test-stac-iter branch December 14, 2021 06:42
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.

2 participants