-
Notifications
You must be signed in to change notification settings - Fork 5
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 #535
update dependencies #535
Conversation
056c9df
to
ebb1b43
Compare
ebb1b43
to
710f64f
Compare
49ffb79
to
5819886
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There's something that pins hdf5 to 1.12, can you please update that too?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thank you, @juliasloan25 ! I presume the plotting errors in the PR description above are resolved? If so, after addressing Gabriele's comments and checking that the AMIP runs produce comparable results to main (the finished slabplanet experiments look good), feel free to merge.
c79e232
to
90717b9
Compare
a8c61e8
to
add1cd5
Compare
The AMIP plots look good compared to main, so I'll merge! |
Purpose
update versions:
get_integrator
closes #521
supersedes #518
supersedes #489
supersedes #499
supersedes #531
supersedes #463
supersedes #523
Current status (as of 12/6 am)