Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Install xarray from main #106

Merged
merged 2 commits into from
May 10, 2024
Merged

Conversation

jsignell
Copy link
Contributor

Noticed while setting up my env. Now that #8872 has merged

@TomNicholas
Copy link
Member

Thanks! FYI we should be able to point to an actual xarray release very soon (hopefully later today) pydata/xarray#9018

@TomNicholas
Copy link
Member

We should also remove the note in the usage.md page in the docs about using a specific branch of xarray

https://virtualizarr.readthedocs.io/en/latest/usage.html#manual-concatenation-ordering

@@ -246,7 +242,7 @@ combined_vds['air'].data.manifest.dict()
The keyword arguments `coords='minimal', compat='override'` are currently necessary because the default behaviour of xarray will attempt to load coordinates in order to check their compatibility with one another. In future this [default will be changed](https://github.com/pydata/xarray/issues/8778), such that passing these two arguments explicitly will become unnecessary.
```

The general multi-dimensional version of this contatenation-by-order-supplied can be achieved using `xarray.combine_nested`.
The general multi-dimensional version of this concatenation-by-order-supplied can be achieved using `xarray.combine_nested`.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Seems like concatenation is an annoying word to spell 🤣

@TomNicholas TomNicholas merged commit c5c38fb into zarr-developers:main May 10, 2024
5 checks passed
@TomNicholas
Copy link
Member

Thanks @jsignell !

@jsignell jsignell deleted the xarray-main branch May 10, 2024 16:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants