-
Notifications
You must be signed in to change notification settings - Fork 19
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
Make precipitation part of the working fluid #3536
base: main
Are you sure you want to change the base?
Conversation
0a1d8d0
to
6aedeac
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.
I only looked at changes to grid-mean now and they look good, thanks! Could you double check it still makes sense when we use TD.relative_humidity and TD.vol_vapor_mixing_ratio in the code? They should be mostly in radiation callback and diagnostics.
ifelse( | ||
p.precomputed.ᶜspecific.q_liq + p.precomputed.ᶜspecific.q_ice > 0, | ||
1, | ||
0, | ||
), |
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.
Do we have an has_condensate
for non-equilibrium thermo state?
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.
We do. However if would return true for a case where hypothetically q_liq=0
but q_rai!=0
. I wanted to avoid that problem, so I do it by hand with the specific precomputed values
q_rai, | ||
q_sno, |
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.
Just note that these could be negative.
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.
That should be fine. I'll double check just in case but all microphysics functions return zero if one of the tracers is negative. And all the limiters inside ClimaAtmos precipitation are written anticipating tracers can be negative too. No trust!
Water vapor related things should be fine since I'm not so sure about some other thermodynamics functions. Like, do we want out potential temperature to now also contain precipitation contribution? This kind of ties back to the buoyancy gradients too. |
6aedeac
to
86580b4
Compare
ffff256
to
8fe2274
Compare
No description provided.