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

Feature/cloud rad #8

Closed

Conversation

Qingfu-Liu
Copy link
Collaborator

Add the convective cloud water (liquid water + ice water) in the calculations of Cloud Water Path and Ice Water Path for RRTMG radiation cloud properties. Only one file changed: radiation_clouds.f (code changed by Ruiyu Sun)
Qingfu

@dustinswales
Copy link
Collaborator

@Qingfu-Liu
Thanks for opening this. I switched it to a "draft" for the time being. I think you may be working with an older code base?

I will propagate these RRTMG changes to RRTMGP, submit a PR to this branch, then we can start the review process.
Sound good?
(I will work on the GP code this evening)

@ChunxiZhang-NOAA
Copy link

@Qingfu-Liu Thanks for opening this. I switched it to a "draft" for the time being. I think you may be working with an older code base?

I will propagate these RRTMG changes to RRTMGP, submit a PR to this branch, then we can start the review process. Sound good? (I will work on the GP code this evening)

@Qingfu-Liu I think your work is based on NCAR/ccpp-physics main branch. Please check out ufs-community/ccpp-physics ufs/dev branch and put your changes from there. Or you can hold there PR till ufs/dev branch is merged with the NCAR/ccpp-physics branch in about one week, then you sync your branch with ufs/dev branch.

@dustinswales
Copy link
Collaborator

@Qingfu-Liu @yangfanglin @JongilHan66 @RuiyuSun
In GFS_rrtmg_pre, before calling radiation_cloud_props, the Thompson effective particle sizes and number concentrations are updated using the cloud mixing-ratios. Since we are including the convective condensate in the cloud-fraction and water path calculations, should we add the convective condensate in here as well?

@RuiyuSun
Copy link
Collaborator

@dustinswales In the test completed, the radii for the convective water and path are assumed to have the same as what are calculated in the calc_effectRad. What is being tested now using separate convective cloud water path, radius, and convective cloud ice path and radius.

@dustinswales
Copy link
Collaborator

dustinswales commented Sep 28, 2022

@RuiyuSun My only concern is that the optics computed with the radiation scheme, which uses both the water paths and effective particle sizes, is now seeing water paths that has convective condensate added, while the calculated effective radii does not

@RuiyuSun
Copy link
Collaborator

@dustinswales This is crude first try. I hope to refine it and use separate convective cloud water path and radius at least.

@Qingfu-Liu
Copy link
Collaborator Author

Qingfu-Liu commented Sep 29, 2022 via email

@yangfanglin
Copy link
Collaborator

@dustinswales @Qingfu-Liu @RuiyuSun My suggestion is to commit this PR without using different radii and number concentrations for convective and MP cloud water and ice. More engineering and science evaluation work needs to be done to enable that capability. This might turned out to be necessary for us to improve cloud radiation interaction with convective cloud water being included in both cloud fraction and radiative transfer calculations. I believe Ruiyu is actively working towards testing and enabling this capability.

@dustinswales
Copy link
Collaborator

@yangfanglin @RuiyuSun @Qingfu-Liu Sounds like a plan.
I opened a new PR, #9, just a rebase PR from the ufs/dev fork that has both the G/GP changes included

@Qingfu-Liu Qingfu-Liu deleted the feature/cloud_rad branch May 10, 2023 15:17
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.

6 participants