Dask mask array #2824
Merged
Dask mask array #2824
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Use the branch from dask PR#2301 in travis
…#2699) Integrate with dask masked array support (just code changes, no tests fixes)
Remove fill_value from cube cml
Purge use of old _lazy_array functionality.
Remove usages of replace method
Fix DataManager unit tests and MaskedConstant in as_lazy
Lazy masked array fill_value retrieval
* PP + merge test fixes * Review actions.
* Add skippers to all failing tests
Fix cube unit tests
* Fix concat tests, mostly reverted to v1.13
* Fixes for GRIB unit tests * Fix GRIB _DataProxy tests * Use .data instead of ._data
Run Travis tests on 8 processors
* upstream/master: matplotlib defaults for tests Set nosetests to run on 8 cores on travis
Feature branch merge: master into dask_mask_array
* Revert to latest normal dask version in Travis. * Bump python3 test version to 3.6, to avoid netcdf build problem.
* Handle fill value on netCDF save
* Fix lazy stats; enhance mdtol test. * Tidy lazy mdtol code; use dask.core.elemwise. * pep8 fixes. * Improved use of dask-mask code. * Simplified with latest dask. * Add lazy-stats tests for all 'axis' options. * Efficient points count in _build_dask_mdtol_function. * Simplify points-count calculation. * Simpler lazy stats calculations for mdtol>=1.0 * Remove+fix final skip_dask_mask in analysis tests.
* Add context to netcdf save warnings; rework warnings tester function. * Make warnings checker a generic IrisTest function. * Implement new approach in PP saving.
Dask maskarray merge master
Fix remaining skips in the dask mask branch.
Contributor
|
I had hoped it would only show the last commit after #2813... |
Remove core-dtype from Cube.xml() and references in result test files
Member
|
Why would it do that? You're merging the other way. None of these commits are in |
Member
|
Thanks @pp-mo & @djkirkham. Glad we don't have any complications with this merge - the |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Only #2823 is outstanding now from the "dask_mask project", and that doesn't actually need to happen before this.