-
Notifications
You must be signed in to change notification settings - Fork 35
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
MYNN-EDMF update #43
MYNN-EDMF update #43
Conversation
Joe, could you please add a link to the ppt about these updates you presented at the RRFS meeting ? Thx |
Joe, nice talk the other day. You said there was no noticeable differences with a winter case. I'm wondering if one would see a difference in the onset of snow at the surface when the snow is included in the mixing. Some years ago Brad Ferrier and I thought about mixing snow in the MYJ PBL. The NAM tended to have a delay in the onset of snow reaching the ground. Vertical cross sections showed it was dry in the lowest few hundred mbs, but RH was near 100% above that. We wondered if mixing snow would increase the RH sooner in the lowest levels. |
Fanglin, I have update the description to include links to some pdfs. Eric, I should clarify that I didn't see any differences in the overall cloud cover (this wasn't the ultimate fix for the "missing cloud fraction" problem) but there may be some other small impacts, like you mention. I'll definitely take a closer look. We will also be running more winter retros soon. Summer retros have been getting most of the CPU hrs. -joe |
Thanks, Joe! It will be interesting to see if there is a difference in snow onset for the winter cases. |
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 don't have any major concerns, just the mismatch of the working precision in the scheme, which should get resolved. I know it's an invasive step, so I'm fine with it being part of a follow up PR.
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.
Only minor comments from me. You definitely want to check if the #ifdef SINGLE_PREC
is still needed in order to not break the recent single precision changes by NRL and Sam.
@joeolson42 I didn't catch this earlier, but please update all of your PR branches to the top of their respective develop branches: ufs/dev for ccpp/physics, develop for fv3atm, and develop for ufs-weather-model. |
…e Dom and add Dustin in CMakeLists.txt authors
Remove MYNN SFC logic from noahmpdrv.F90 and fix CMakeLists.txt authors
MYNN-EDMF update including MYNN-related modules
Companion PRs:
NOAA-EMC/fv3atm#630
ufs-community/ufs-weather-model#1636