Skip to content

Revert is_dask_collection; back to previous implementation#9062

Merged
jsignell merged 3 commits intodask:mainfrom
douglasdavis:revert-is-dask-collection-change
May 10, 2022
Merged

Revert is_dask_collection; back to previous implementation#9062
jsignell merged 3 commits intodask:mainfrom
douglasdavis:revert-is-dask-collection-change

Conversation

@douglasdavis
Copy link
Member

Copy link
Member

@jsignell jsignell left a comment

Choose a reason for hiding this comment

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

Thanks Doug!

@ian-r-rose
Copy link
Collaborator

Should this also update the protocol @douglasdavis?

@douglasdavis douglasdavis deleted the revert-is-dask-collection-change branch May 10, 2022 15:07
@douglasdavis
Copy link
Member Author

@ian-r-rose my first thought was hesitation, and this function would bridge us to a point of a better conclusion after some discussion, but if xarray folks are relying on it then yeah perhaps that’s a good idea.

erayaslan pushed a commit to erayaslan/dask that referenced this pull request May 12, 2022
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.

New Collections protocol breaks xarray objects

3 participants