-
Notifications
You must be signed in to change notification settings - Fork 146
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
No cldfra2d diagnostics saved in RRTMGP #957
Comments
@mzhangw |
@dustinswales it appears there are already a lot of changes in PR #956. It might be a good idea to merge this to a different PR. It can be included in a PR @Qingfu-Liu is creating to update cloud condensates in cloud cover and radaition calculations. |
@yangfanglin That would be excellent. |
@yangfanglin @Qingfu-Liu @mzhangw |
It is to add a counterpart of the RRTMG cldfra2d in GP. It looks good to me.
… On Oct 7, 2022, at 11:25 AM, dustinswales ***@***.***> wrote:
@yangfanglin <https://github.com/yangfanglin> @Qingfu-Liu <https://github.com/Qingfu-Liu> @mzhangw <https://github.com/mzhangw>
I added a fix to this issue in ufs-community/ccpp-phsyics PR #9 <ufs-community#9>
—
Reply to this email directly, view it on GitHub <#957 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AG7TW2VSNRS6IGBUJOBGWBTWCBMHFANCNFSM6AAAAAAQAVB5T4>.
You are receiving this because you were mentioned.
|
Fixed in ufs-community#9 |
There is no cldfra2d diagnostics in RRTMGP as it does in RRTMG. It leads to an incorrect ZERO value of total cloud fraction when RRTMGP is used in SCM.
The text was updated successfully, but these errors were encountered: