-
Notifications
You must be signed in to change notification settings - Fork 37
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
fixed stratosphere warm bias and code optimization for MERRA2 #79
Conversation
…ade radiation_aerosols.f ccpp compliant
…Provided as ccpp interstitials.
… Provided as ccpp interstitials.
…Provided as ccpp interstitials.
…vided as ccpp interstitials.
… as ccpp interstitials.
Update CODEOWNERS
UFS/dev PR#6
UFS-dev PR#42
UFS-dev PR#54
UFS-dev PR#49
UFS-dev PR#57
UFS-dev PR#56
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.
These changes look OK to me from a CCPP point of view.
@@ -875,17 +875,12 @@ subroutine cmp_reff_Thompson(nLev, nCol, i_cldliq, i_cldice, i_cldsnow, i_cldice | |||
qi_mp(iCol,iLay) = tracer(iCol,iLay,i_cldice) / (1.-q_lay(iCol,iLay)) | |||
qs_mp(iCol,iLay) = tracer(iCol,iLay,i_cldsnow) / (1.-q_lay(iCol,iLay)) | |||
ni_mp(iCol,iLay) = tracer(iCol,iLay,i_cldice_nc) / (1.-q_lay(iCol,iLay)) | |||
if (ltaerosol) then | |||
if (ltaerosol .or. mraerosol) then |
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.
This is fine, but probably not needed.
The RRTMGP-ThompsonMP coupling is in a state of disarray and needs to be updated with many changes that have occurred in RRTMG, but were not ported to RRTMGP. (It's on my list to do this at somepoint...)
sync with ufs/dev
resolved issue ufs-community/ufs-weather-model#1786