Skip to content

Conversation

@jsignell
Copy link
Member

@jsignell jsignell commented Feb 9, 2021

This should go in after the distributed PR dask/distributed#4495

@jakirkham
Copy link
Member

Are we just waiting on CI here? If so given that macOS builds are still queued and others have already passed, should we go ahead here?

cc @jrbourbeau (in case you also have thoughts on these questions)

Copy link
Member

@jrbourbeau jrbourbeau left a comment

Choose a reason for hiding this comment

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

Yep, this LGTM. Thanks for your work on this @jsignell!

@jrbourbeau jrbourbeau merged commit 707af9d into dask:master Mar 8, 2021
@jakirkham
Copy link
Member

Thanks Julia and James! 😄

@jsignell
Copy link
Member Author

jsignell commented Mar 9, 2021

YAY!! Thanks to you as well @jrbourbeau :)

@jsignell jsignell deleted the master-main branch March 9, 2021 00:03
dcherian added a commit to dcherian/dask that referenced this pull request Mar 18, 2021
* upstream/main:
  Change default branch from master to main (dask#7198)
  Add Xarray to CI software environment (dask#7338)
  Update repartition argument name in error text (dask#7336)
  Run upstream tests based on commit message (dask#7329)
  Use pytest.register_assert_rewrite on util modules (dask#7278)
  add example on using specific chunk sizes in from_array() (dask#7330)
  Move numpy skip into test (dask#7247)
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.

3 participants