-
Notifications
You must be signed in to change notification settings - Fork 40
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
Unpin dask #286
Unpin dask #286
Conversation
Codecov Report
@@ Coverage Diff @@
## master #286 +/- ##
==========================================
- Coverage 95.45% 94.85% -0.61%
==========================================
Files 20 24 +4
Lines 1914 2681 +767
==========================================
+ Hits 1827 2543 +716
- Misses 87 138 +51
Continue to review full report at Codecov.
|
Think it needs to be >= 2021.3.1. 2021.3 caused issues in xhistogram (xgcm/xhistogram#27). Could hang fire here until xhistogram puts out a new release (xgcm/xhistogram#40) or make that a separate PR. |
!= means every version but that one. Worked before that version and will work after 2021.3.0. Please commit your preferred solution to this PR. |
Thanks @aaronspring |
Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Closes #280
Type of change
Please delete options that are not relevant.
asv
to detect performance changes)How Has This Been Tested?
Please describe the tests that you ran to verify your changes. This could point to a cell in the updated notebooks. Or a snippet of code with accompanying figures here.
Checklist (while developing)
pytest
, if necessary.Pre-Merge Checklist (final steps)
References
Please add any references to manuscripts, textbooks, etc.