Skip to content
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

Fraction of frozen precipitation with RUC LSM #337

Merged
merged 2 commits into from
Oct 10, 2019
Merged

Fraction of frozen precipitation with RUC LSM #337

merged 2 commits into from
Oct 10, 2019

Conversation

tanyasmirnova
Copy link
Collaborator

Use fraction of frozen precipitation computed in Thompson or GFDL microphysics to fill array SRFLAG. This change will affect only the combination of RUC LSM with Thompson or GFDL microphysics, and will not affect any physics combinations not using RUC LSM. The reason for this change is to exclude temperature-based treatment of convective precipitation from the computation of SRFLAG, as is implemented in the WRF-based RAP and HRRR.

… microphysics, directly without recomputing it with taking into account temperature-based treatment of convective precipitation. This change will affect only use of RUC LSM with GFDL or Thompson microphysics.
@climbfuji
Copy link
Collaborator

@climbfuji
Copy link
Collaborator

This PR contains #336

@climbfuji
Copy link
Collaborator

Great work @tanyasmirnova, thanks very much for finding and fixing the Thompson MP bug. This change is completely safe from my point of view, but I'll give other GMTB folks a few minutes to look at it, too. I assumed you compiled this combined code change and ran a few tests?

@tanyasmirnova
Copy link
Collaborator Author

tanyasmirnova commented Oct 10, 2019

Dom,
Thank you very much for taking a look at this from Osaka. Yes, I have compiled both changes and ran a test on C96 resolution. I also printed the values of SRFLAG in my tests, and they look reasonable. Of course, I ran only the GSD suite tests.

@climbfuji
Copy link
Collaborator

Dom,
Thank you very much for taking a look at this from Osaka. Yes, I have compiled booth changes and ran a test on C96 resolution. I also printed the values of SRFLAG in my tests, and they look reasonable. Of course, I ran only the GSD suite tests.

That's great, thanks. Since the code changes only affect the CCPP Thompson MP and RUC LSM, this is good enough for merging into gsd/develop.

@ligiabernardet
Copy link
Collaborator

Thanks for finding the bug in Thompson pcp. I confirm that the changes in GFS_MP_generic.F90 will only affect RUC. Therefore, there is no risk of it inadvertently affecting other suites. Looks good.

@climbfuji climbfuji merged commit 543f640 into NCAR:gsd/develop Oct 10, 2019
@climbfuji
Copy link
Collaborator

All, we need to merge this update for gsd/develop into gmtb/develop and bring it to the master branches before the UFS release code freeze!

hannahcbarnes pushed a commit to hannahcbarnes/ccpp-physics that referenced this pull request Aug 3, 2022
NCAR#356)

* update standard names in accordance with CCPPStandardNames PR#13 as of 2021/7/1
* Fix merge conflicts in ccpp/data/GFS_typedefs.meta

Co-authored-by: grantfirl <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants