-
Notifications
You must be signed in to change notification settings - Fork 3
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
transition to using new bucket model version #117
Conversation
anomaly = true | ||
hs_sfc = false | ||
Y.bucket.T_sfc = map(coords) do coord | ||
Y.bucket.T = map(coords.subsurface) do coord |
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.
@LenkaNovak right now the anomaly is 0 - do we want to change this?
FWIW, longer runs seem to run into this now (at ~7 days) maxiter reached in saturation_adjustment: |
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.
Looks good. Only a small comment left in the code. Thanks!
@@ -47,11 +47,12 @@ function calculate_surface_fluxes_atmos_grid!(integrator, info_sfc) | |||
FT = eltype(integrator.u.c.ρ) | |||
thermo_params = CAP.thermodynamics_params(integrator.p.params) | |||
surface_flux_params = CAP.surface_fluxes_params(integrator.p.params) | |||
uₕ_int = Geometry.UVVector.(Spaces.level(Y.c.uₕ, 1)) |
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.
Doesn't this allocate more than previously?
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.
Ill revert it back
issue with atmos u bugs fixed but not well formatting and should run, psnow zero swap boundary space for horizontal space updated utils revert change cli typo cleanup review comment
ecee64b
to
30e5ef4
Compare
bors r+ |
PULL REQUEST
Purpose and Content
This PR updates the coupler driver file and other helper files so that the coupled runs work with the new version of the bucket model (with snow, and with a multi layer heat equation).
To do: followup on CliMA/ClimaCore.jl#943
Benefits and Risks
The benefit of this is that our AMIP model will be slightly more realistic.
Linked Issues
(Provide references to any link issues. Use closes #issuenum to automatically close an open issue)
PR Checklist