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

[production/RRFS.v1] fix zero cloud fraction for RRFS ensemble members #801

Conversation

JiliDong-NOAA
Copy link
Contributor

Description

This PR will address zero cloud fraction issue for RRFS ensemble members using GFS PBL and G-F convection by adding SGS cloud to ccpp SDF and changing conv_cf_opt to Xu-Randall for convective cloud fraction computation for those members.

The change to fv3atm is to add conv_cf_opt as an input namelist parameter for runtime switch (conv_cf_opt=1 for GFS PBL members) and add sgs cloud to related ccpp SDFs.

Issue(s) addressed

#800

Testing

How were these changes tested?
What compilers / HPCs was it tested with?
Intel on Hera.
Are the changes covered by regression tests? (If not, why? Do new tests need to be added?)
No. No RTs using RRFS ensemble physics suites.
Have the ufs-weather-model regression test been run? On what platform?
Not yet. But no changes expected for RT baseline.

  • Will the code updates change regression test baseline? If yes, why? Please show the baseline directory below.
    No
  • Please commit the regression test log files in your ufs-weather-model branch

Dependencies

ufs-community/ccpp-physics#186

@grantfirl
Copy link
Collaborator

@jkbk2004 @JiliDong-NOAA This PR can be closed since it was combined into #802.

MatthewPyle-NOAA pushed a commit that referenced this pull request Mar 19, 2024
…es RRFS.v1 (#802)

* "point to the mynn pbl, HAILCAST diagnostic updates; allocate rho_dry to zero size when not in use"
* "point to the smoke updates for RRFS.v1, and update input for smoke"
* "merge fv3atm PR #801 from Jili (clouds fix)"
* "point to hashes ufs-community/ccpp-physics@85fabc0 and NOAA-EMC/GFDL_atmos_cubed_sphere@fab198c"
@grantfirl
Copy link
Collaborator

@JiliDong-NOAA Since this was merged in #802, please close this PR. As the developer, you should have permissions to do that.

grantfirl pushed a commit to grantfirl/fv3atm that referenced this pull request Oct 29, 2024
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.

2 participants