-
Notifications
You must be signed in to change notification settings - Fork 21
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
Skip tests that require kerchunk #259
base: main
Are you sure you want to change the base?
Conversation
@@ -0,0 +1,60 @@ | |||
name: min-deps |
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.
There was probably a better way to do this than just making an entire extra workflow file but my github-actions-fu is weak
"xarray>=2024.06.0", | ||
"kerchunk>=0.2.5", | ||
"h5netcdf", | ||
"numpy>=2.0.0", | ||
"ujson", | ||
"packaging", | ||
"universal-pathlib", | ||
"numcodecs", |
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.
The minimum set of dependencies should look like this, and you could probably even make universal-pathlib
optional for reading from non-remote files...
All I've done here is skip all of those that currently require kerchunk, and add a new CI job to check that the others pass without kerchunk installed. I'm hoping that we can merge this PR then incrementally re-write all of the tests (apart from those in
test_readers/test_kerchunk.py
&test_writers/test_kerchunk.py
) to run and test the same functionality overall without requiring kerchunk.