-
Notifications
You must be signed in to change notification settings - Fork 9
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 and add ocean mask #808
Conversation
@@ -400,9 +400,9 @@ if device_suffix == "cpu" | |||
titles = ["W", "σS", "T_sfc"] | |||
for (f, n) in zip(fields, titles) | |||
fig = Figure(size = (1000, 1000)) | |||
ax = Axis(fig[1, 1]) | |||
ax = Axis(fig[1, 1], title = n) |
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.
I had to make this change to get this to pass, Gabriele.
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.
Let me look into this
9a567f4
to
04eb4f0
Compare
@juliasloan25 the long_run/bucket.jl script runs here, so maybe we can merge this first, and then your PR? |
7544df8
to
0670b7d
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.
This looks good to me! The longrun plots still look good. We might want to make a note (in the commit or PR description) that the ocean mask is just used for plotting.
We'll also need to update to ClimaCore v0.14.18 after it's released later today, but I'm happy to do that as part of #675
Purpose
To-do
Future PR: move long_run/bucket.jl to experiments/standalone/Bucket/ in place of bucket_era5.jl
At the same time, use ClimaDiagnostics for all standalone bucket runs and compare diagnostics, not output state to CSV.
Replace plotting with ClimaAnalysis
Content