Skip to content

Conversation

@djkirkham
Copy link
Contributor

Closes #2713

@djkirkham djkirkham requested a review from pp-mo August 29, 2017 09:45
@djkirkham djkirkham closed this Oct 11, 2017
@djkirkham djkirkham reopened this Oct 11, 2017
@QuLogic QuLogic added this to the dask-mask milestone Oct 12, 2017
@djkirkham
Copy link
Contributor Author

@pp-mo @pelson This should be ready to merge now

@pp-mo pp-mo self-assigned this Oct 17, 2017
@pp-mo
Copy link
Member

pp-mo commented Oct 17, 2017

I'm good with this, once tests pass -- now testing locally as Travis is failing us so much atmo :-(

Copy link
Member

@pelson pelson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice. Thanks @djkirkham.

@pp-mo pp-mo merged commit 8a0c5f7 into SciTools:dask_mask_array Oct 17, 2017
@djkirkham djkirkham deleted the cml_fixes branch October 26, 2017 13:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants