Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Integration with xarray datatree #511

Closed
andersy005 opened this issue Aug 23, 2022 · 0 comments · Fixed by #512
Closed

Integration with xarray datatree #511

andersy005 opened this issue Aug 23, 2022 · 0 comments · Fixed by #512
Assignees
Labels
enhancement Issues that are found to be a reasonable candidate feature additions

Comments

@andersy005
Copy link
Member

Is your feature request related to a problem? Please describe.

A while ago, we added support for an xcollection.Collection object which contains a mapping of datasets. This object behaves like a dictionary with additional functionality. However, with the more recent development of xarray-datatree --which will eventually be integrated into Xarray proper--, we should transition to using DataTree instead of Collection.

Describe the solution you'd like

  • Provide a cat.to_datatree() method which returns an xarray DataTree object
  • Remove the .to_collection() method

Describe alternatives you've considered

We currently provide a cat.to_collection() method which returns an xcollection.Collection object.

Additional context

Add any other context or screenshots about the feature request here.

Cc @mgrover1

@andersy005 andersy005 added this to the August 2022 Release milestone Aug 23, 2022
@mgrover1 mgrover1 self-assigned this Aug 24, 2022
@mgrover1 mgrover1 added the enhancement Issues that are found to be a reasonable candidate feature additions label Aug 24, 2022
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
enhancement Issues that are found to be a reasonable candidate feature additions
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants