forked from NCAR/ccpp-physics
-
Notifications
You must be signed in to change notification settings - Fork 11
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
Cleanup of radiation (clouds) - microphysics interaction (based on #33) #35
Merged
DomHeinzeller
merged 5 commits into
NOAA-GSL:gsd/develop
from
climbfuji:radcloud_cleanup
Jun 5, 2020
Merged
Cleanup of radiation (clouds) - microphysics interaction (based on #33) #35
DomHeinzeller
merged 5 commits into
NOAA-GSL:gsd/develop
from
climbfuji:radcloud_cleanup
Jun 5, 2020
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
@joeolson42 Joe, a couple questions:
|
What's the way forward? Merge as is or make changes? Results are looking good, all tests completed. I am ready to merge when you are. |
@joeolson42 <https://github.com/joeolson42> Joe, a couple questions:
1. Do you think that Xu-Randall cloud fraction should be computed only
from qc+qi, and not all hydrometeors?
In WRF, we replace Xu-Randall entirely with the MYNN clouds. Using more
hydrometeors ruins ceiling forecasts and doesn't have that big of an impact
on the SWDOWN.
1. In the WRF code we added MYNN sub-grid clouds to existing QC and QI:
qc(i,k,j)=qc(i,k,j) + QC_BL(i,k,j)*(MIN(1., MAX(0.,
(t(i,k,j)-254.)/15.)))
*CLDFRA_BL(i,k,j) qi(i,k,j)=qi(i,k,j) + QC_BL(i,k,j)*(1. - MIN(1.,
MAX(0., (t(i,k,j)-254.)/15.)))*CLDFRA_BL(i,k,j)
And in SGSCloud_RadPre we replace QC, QI with QC_BL and QI_BL:
qc(i,k) = qc_bl(i,k)*cldfra_bl(i,k)
Is it the intent?
This is the intent. We can always change it later if need be.
1. If GFDL MP runs together with MYNN PBL, you propose to use MYNN
cloud fraction instead of internal GFDL cloud fraction. Is it correct?
Yes, for lack of a better way.
Just merge the code please.
…
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#35 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ADLRR3UIWL3S3COXN5CNA53RVENUTANCNFSM4NTFRLPQ>
.
--
Joseph Olson
Meteorologist
Environmental Prediction Advancement Division
Global Systems Laboratory
NOAA-Earth System Research Labs
Boulder, Colorado
|
DomHeinzeller
approved these changes
Jun 5, 2020
christinaholtNOAA
pushed a commit
that referenced
this pull request
Mar 2, 2023
Bug fix for cloud effective radius for convective clouds (HR1)
zhanglikate
pushed a commit
to zhanglikate/ccpp-physics
that referenced
this pull request
Mar 1, 2024
Bug fix for cloud effective radius for convective clouds (HR1)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR contains:
physics/module_SGSCloud_RadPre.F90
Associated PRs:
#35
NOAA-GSL/fv3atm#33
NOAA-GSL/ufs-weather-model#26
For regression testing information, see below NOAA-GSL/ufs-weather-model#26.