-
Notifications
You must be signed in to change notification settings - Fork 1
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
updated to check a wave-fc file and check for valid kerchunk indices #5
updated to check a wave-fc file and check for valid kerchunk indices #5
Conversation
A couple test failures:
I think we changed this to
I think now we need the files to be present, since we actually read them (previously we just used the path structure). I'd prefer not to have large data files in the source repo. So we'll want to fetch these (probably from Azure Blob Storage) when we run the test. We can cache it locally so that we don't need to download it every time. Can you take care of that? |
Done. |
Looks like one test failure, from the updated base64 logic:
|
… ones for the item level
Before you submit a pull request, please fill in the following:
Related Issue(s):
N/A
Description:
Added in an update tests/test_stac.py that checked for valid kerchunk indices and used a wave-fc file for the checks. This file was meant to be a part of the previous PR.
PR checklist:
scripts/format
).scripts/lint
).scripts/test
).