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 dependencies [WIP] #130

Merged
merged 6 commits into from
Jan 16, 2025
Merged

Update dependencies [WIP] #130

merged 6 commits into from
Jan 16, 2025

Conversation

gaelforget
Copy link
Contributor

@gaelforget gaelforget commented Jan 15, 2025

  1. with these updates the tests suite all seem to work except for one aspect
  • the resetting of longitude to -180 + 180 no longer works on the fly
  • I edited lon by hand in the sresa1b_ncar_ccsm3-example.nc file as a quick workaround
  1. not sure how much this helps but I figured you may want to know

@danlooo
Copy link
Owner

danlooo commented Jan 16, 2025

Thanks for your PR :) Seems that YAXArrays 0.6 but not 0.5 raises ERROR: LoadError: lon must be within [-180, 180]. This is because YAXArrays now uses type lon instead of Dim{:lon}.

@danlooo danlooo merged commit f662e11 into danlooo:main Jan 16, 2025
2 checks passed
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.

2 participants