Add precipitation surface fluxes for 1M microphysics #3084
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
trontrytel
force-pushed
the
aj/surface_precip_fluxes
branch
from
25f8b0e to
fb17c7a
Compare
szy21
approved these changes
Jun 14, 2024
Sure! Yes, it would be great to do the version update after this is merged in. So that I can also update the Coupler and run the simulations with land there |
trontrytel
force-pushed
the
aj/surface_precip_fluxes
branch
from
aaacbf7 to
a0e2640
Compare
trontrytel
force-pushed
the
aj/surface_precip_fluxes
branch
from
a0e2640 to
e1f741d
Compare
|
Just a heads up, I noticed that the 1M run in this week's build breaks after ~5 days: https://buildkite.com/clima/climaatmos-gpulongruns/builds/206#01901a45-511a-47d8-a869-666664bc3d9b. It seems we don't have a regression test for microphysics, but this seems to be the only PR that is related. Maybe double check the change in the tendency? |
szy21
added this to the O1.5.2e (atmos) Run AMIP simulation with 1M microphysics milestone
2 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR adds the surface precipitation fluxes for the 1M microphysics scheme. I'm not super sure if this will be correct with topography, but I tried and had support from @dennisYatunin
It also renames the 0M precipitation fluxes from
col_integrated_rain(and snow) tosurface_rain_flux(and snow). It is a breaking release because of this. I will follow up with a PR to the Coupler.I added surface precipitation plots whenever I saw surface evaporation plots in the CI.