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

Corrections and fixing double-moment Thompson cloud water #230

Merged
merged 1 commit into from
Nov 30, 2020

Conversation

gthompsnJCSDA
Copy link
Contributor

This update fixes some problems with the code for Thompson microphysics in UPP calculator of effective radius (mostly for CRTM radiance calculations). Most importantly 2-moment cloud water variable (number concentration) should now work as it does in WRF and other models. The graupel Y-intercept parameter is made much simpler in coordination with changes to the latest WRF version of the Thompson microphysics scheme. The code as it was had a serious bug due to the re-usage of the "rg" variable from an included file versus the use in the calculation of effective radius which was using a conflicting name.

@gthompsnJCSDA
Copy link
Contributor Author

The "Linked issues" portion on the right is not allowing me to LINK a corresponding issue, which is issue#231

@fossell
Copy link
Contributor

fossell commented Nov 25, 2020

@gthompsnJCSDA - Once tested and merge, I assume we will want to pull this into release/public-v2 as well for distribution in the SRW app?
@mkavulich - Please keep an eye on this and bring into public-v2 if needed.

@gthompsnJCSDA
Copy link
Contributor Author

@gthompsnJCSDA - Once tested and merge, I assume we will want to pull this into release/public-v2 as well for distribution in the SRW app?
@mkavulich - Please keep an eye on this and bring into public-v2 if needed.

Hi Kate - I don't even know what that means about the SRW app - foreign to me. But whatever release, public or not does not concern me, others can decide.

@fossell
Copy link
Contributor

fossell commented Nov 25, 2020

@gthompsnJCSDA - Once tested and merge, I assume we will want to pull this into release/public-v2 as well for distribution in the SRW app?
@mkavulich - Please keep an eye on this and bring into public-v2 if needed.

Hi Kate - I don't even know what that means about the SRW app - foreign to me. But whatever release, public or not does not concern me, others can decide.

@gthompsnJCSDA - Sorry about that, it's the UFS Short Range Weather Application. @mkavulich - Please check with the SRW team to see if this mp option is used or available for the release, if yes we'll need to take this bug fix once it's finalized in develop.

@jwolff-ncar
Copy link

Yes, @mkavulich, please make sure this is pulled into public-v2. Thanks!

@WenMeng-NOAA
Copy link
Collaborator

The UPP regression tests were conducted on WCOSS Dell, Cray and Hera. With this update, the baseline for hrrr and rap will be re-created.

@WenMeng-NOAA
Copy link
Collaborator

This PR is ready for merging.

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.

UPP, calculate radiative effective radius, Thompson 2-moment cloud water, also graupel calculation
5 participants