-
Notifications
You must be signed in to change notification settings - Fork 79
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
Test suite without dask #422
base: grid_ufunc_refactor_project
Are you sure you want to change the base?
Test suite without dask #422
Conversation
Wait why didn't the CI run the code tests?? |
Umm that is weird...no idea why this wouldnt work... |
Do you envision this being part of 0.7.0? I would say this is probably more suited for the next round where we modify tests substantially? |
I think this is low-priority, and can wait until we have finalised the internals based on other criteria. |
Not sure this should be closed. Could be part of #466 |
The only reason this was closed is because when I tried deleting the |
Ok then ill reopen so we can keep track of it |
Removes dask from the "core dependencies" CI environment file, so that no we have a CI run that tests everything without dask installed.
I altered an existing run rather than added a new run because this seemed to better fit the intention of "just core dependencies".
I don't expect this to pass - but we can push to this branch to start fixing any bugs that this reveals.