Updates to MYNN-EDMF with additional output for downdrafts#248
Conversation
|
Please submit your reviews asap! |
|
@joeolson42 @clark-evans Are we going to put this code into a dev? |
We should. OK to take the TEMPO hail mods out of dev2 for now? (Longer term, assuming these mods look good, I presume we'd want to put them into dev1 to inform the prog cloud scheme?) |
dustinswales
left a comment
There was a problem hiding this comment.
These changes look good to me. Thanks!
|
@joeolson42 Can you update your branch with develop? |
done |
AndersJensen-NOAA
left a comment
There was a problem hiding this comment.
Will approve but would like to see cloudiness changes in a dev.
|
@AndersJensen-NOAA I think I'm starting to understand the group lingo. |
Thanks for adding that label, @dustinswales. Running major developments in a dev (referring to MPAS_physics_dev1 and MPAS_physics_dev2, in case others were wondering) is a standard practice. The challenge arises when we have multiple developments needing real-time evaluation, such as we do now with the prog cloud scheme, other TEMPO updates, and now the v8.4 merge, this PR, and the prognostic aerosol emissions. |
clark-evans
left a comment
There was a problem hiding this comment.
Approved pending spacing changes. OK to merge after real-time testing.
In module_bl_mynnedmf.F90, I see several changes related to hurricane/higher wind speed tuning. As best as I can tell, at least for the new code in lines 1978-1980 (which are just one of the changes), it looks like they would primarily foster higher wind speeds. Is that correct? If so, I’d like to test this code in a Melissa retro before the 2026 HFIP.
|
@clark-evans Understood. |
I think that's the "Code review approved" workflow. I just disabled that. We enabled it at the code discussion meeting a couple of weeks ago as a test, to see if it would apply whenever an approval came in or only after all requested approvals had come in. Since it appears to be the former, I don't think we want it enabled. |
|
@joeolson42 could you update the repo for this PR to the top of noaa/develop? There are also a couple of nitpicky spacing change requests from me above. I think we should be able to merge this at tomorrow's code meeting if it's updated by then. |
|
@joeolson42 @clark-evans I think this one is ready. Let's merge this at our meeting in a few minutes. |
This PR provides a significant update to the downdraft component of the MYNN-EDMF with minor tweaks elsewhere. Addition 2d output field have been added to expedite tuning/debugging. The following list overviews the changes:
This updated code has been tested in both MPAS , WRF, and CI tests.
Mandatory Questions
Reviews