Skip to content

Fix two bugs in DataTree.update() - #9214

Merged
shoyer merged 2 commits into
pydata:mainfrom
shoyer:datatree-fix-update
Jul 8, 2024
Merged

Fix two bugs in DataTree.update()#9214
shoyer merged 2 commits into
pydata:mainfrom
shoyer:datatree-fix-update

Conversation

@shoyer

Copy link
Copy Markdown
Member
  1. Fix handling of coordinates on a Dataset argument (previously these were silently dropped).
  2. Do not copy inherited coordinates down to lower level nodes.
  • Tests added

1. Fix handling of coordinates on a Dataset argument (previously these
were silently dropped).
2. Do not copy inherited coordinates down to lower level nodes.
@TomNicholasTomNicholas added the topic-DataTree Related to the implementation of a DataTree class label Jul 8, 2024
@shoyer

Copy link
Copy Markdown
MemberAuthor

@TomNicholas can you please take a quick look here?

@TomNicholasTomNicholas left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good to me @shoyer !

@shoyer
shoyer merged commit 179c670 into pydata:mainJul 8, 2024
@shoyer
shoyer deleted the datatree-fix-update branch July 8, 2024 17:12
dcherian added a commit to dcherian/xarray that referenced this pull request Jul 11, 2024
* main:
exclude the bots from the release notes (pydata#9235)
switch the documentation to run with `numpy>=2` (pydata#9177)
`numpy` 2 compatibility in the iris code paths (pydata#9156)
`numpy` 2 compatibility in the `netcdf4` and `h5netcdf` backends (pydata#9136)
Fix time indexing regression in `convert_calendar` (pydata#9192)
Use duckarray assertions in test_coding_times (pydata#9226)
Use reshape and ravel from duck_array_ops in coding/times.py (pydata#9225)
Cleanup test_coding_times.py (pydata#9223)
Only use necessary dims when creating temporary dataarray (pydata#9206)
Fix two bugs in DataTree.update() (pydata#9214)
Use numpy 2.0-compat `np.complex64` dtype in test (pydata#9217)
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

topic-DataTreeRelated to the implementation of a DataTree class

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants

@shoyer@TomNicholas