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

Update pangeo-notebook metapackage version to 2024.09.01 #577

Merged
merged 4 commits into from
Sep 11, 2024

Conversation

pangeo-bot
Copy link
Collaborator

A new pangeo-metapackage version has been detected.
Updated environment.ymls to use 2024.09.01.

@pangeo-bot pangeo-bot requested a review from scottyhq September 2, 2024 00:16
Copy link
Contributor

github-actions bot commented Sep 2, 2024

Binder 👈 Try on Mybinder.org!

@pangeo-bot
Copy link
Collaborator Author

/condalock
Automatically locking new conda environment, building, and testing images...

wrapt==1.16.0
x264==1!164.3095
x265==3.5
xarray==2024.7.0
Copy link
Member

Choose a reason for hiding this comment

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

not sure why this downgrade happened...

Comment on lines 1117 to 1118
xarray: '>=0.18,<2024.3'
url: https://conda.anaconda.org/conda-forge/noarch/argopy-0.1.16-pyhd8ed1ab_0.conda
Copy link
Member

Choose a reason for hiding this comment

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

not sure why this downgrade happened...

Tracked it down to argopy pinning to xarray<2024.3 as mentioned in their changelog https://github.com/euroargodev/argopy/blob/2f2e2dc884cf5d3b39aa38339e1027260fc4c60d/docs/whats-new.rst?plain=1#L57. Seems to be something about pydata/xarray#8909

Copy link
Member

Choose a reason for hiding this comment

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

Hmmm, my brief understanding is there is an issue specifically for the case of writing out netCDF files with the scipy engine.

I prefer to favor supporting the most general packages like xarray when conflicts with more domain-specific packages (like argopy) come up. One positive outcome of these images is just motivating broad ecosystem compatibility and fixing certain upstream issues eventually! But short-term I think pinning the domain-specific package is best to not block other updates like this.

@scottyhq
Copy link
Member

/condalock

@scottyhq scottyhq merged commit e2ee472 into master Sep 11, 2024
5 checks passed
@scottyhq scottyhq deleted the upgrade-pangeo-metapackage-2024.09.01 branch September 11, 2024 10:09
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