-
Notifications
You must be signed in to change notification settings - Fork 33
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
Convection #414
Convection #414
Conversation
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, nothing to add really. I like the new approach and I find the code exceptionally readable 👏
Out of curiosity, how would we eventually go about verifying the correctness of this?
The last commit adds the calculation of the geopotential |
@white-alistair I also realised that the formula we use for saturation humidity is actually the Tetens equation |
Oh yeah @white-alistair that looks like some convection to me! T85 over 20 days. Looks good enough for me at this stage (it does qualitatively the right thing, is not absolutely off in terms of magnitude and is seemingly stable) precip_conv.mp4in comparison this is large-scale preicipitation precip_cond.mp4 |
That looks amazing, congrats @milankl! |
@p-hss SpeedyWeather has a not too-unrealistic closed hydrological cycle now! |
This enables the
diagnose_convection!
function, actually doingconvection!
is still commented.One issue we had was that Fortran SPEEDY uses humidity in g/kg, we switched to kg/kg but I believe that still has some issues in various places, e.g. I just found (was top, now bottom)
Value seems to be an approximation to (from wikipedia)
Cloud top (dark blue means way above the surface) indicates where convection is active
which is almost everywhere in the beginning and it also spreads across the domain within days. Not what I was hoping but I bet there's still tons of bugs everywhere. To be fixed!