Skip to content

Cloud cover parameter changes for GFSv17#1027

Merged
DusanJovic-NOAA merged 5 commits into
NOAA-EMC:developfrom
RuiyuSun:moorthi_cover
Oct 27, 2025
Merged

Cloud cover parameter changes for GFSv17#1027
DusanJovic-NOAA merged 5 commits into
NOAA-EMC:developfrom
RuiyuSun:moorthi_cover

Conversation

@RuiyuSun
Copy link
Copy Markdown
Contributor

@RuiyuSun RuiyuSun commented Oct 16, 2025

Description

There is a low cloud bias in the current version of the GFSv17. The formula used in the cloud cover calculation is based Xu/Randall (1996). To increase the cloud cover in the GFSv17, a different set of parameters is used in the formulation. The increase in the cloud cover will also help to reduce the downward SW at the surface.

Issue(s) addressed

This PR

  • fixes noaa-emc/fv3atm/issues/<1026>

Testing

How were these changes tested?
This change has been used in a GFSv17 RETRO-like experiment on WCOSS2. RT tests are being conducted on the URSA.

What compilers / HPCs was it tested with?
Intel

Are the changes covered by regression tests?
YES

Have the ufs-weather-model regression test been run? On what platform?
The RT tests were conducted on the URSA.

  • Will the code updates change regression test baseline? - Please show the baseline directory below.

This change will change the baseline.
BASELINE DIRECTORY: /scratch4/NAGAPE/epic/role-epic/UFS-WM_RT/NEMSfv3gfs/develop-20251007

  • Please commit the regression test log files in your ufs-weather-model branch
    DONE.

Dependencies

If testing this branch requires non-default branches in other repositories, list them.
Those branches should have matching names (ideally)
https://github.com/RuiyuSun/ccpp-physics/tree/moorthi_cover
ufs-community/ccpp-physics#325

@DusanJovic-NOAA
Copy link
Copy Markdown
Collaborator

@RuiyuSun Please add corresponding ccpp-physics PR as a dependency

@gspetro-NOAA
Copy link
Copy Markdown
Collaborator

We are hoping to process the WM priority GFS v17 parent PR #2940 before the end of the week. Could we get approvals on this UFSATM subcomponent PR?

@RuiyuSun , from Dusan's comment above, it seems like you should add the CCPP physics PR as a dependency in your PR description above.

@RuiyuSun
Copy link
Copy Markdown
Contributor Author

@DusanJovic-NOAA @gspetro-NOAA The dependency has been added.

@gspetro-NOAA
Copy link
Copy Markdown
Collaborator

Could we get approvals on this PR so that we can process its high-priority GFSv17 parent WM PR#2940 by the end of the week? Thanks! :)

@grantfirl
Copy link
Copy Markdown
Collaborator

@RuiyuSun The ccpp-physics PR was merged. This is ready to revert .gitmodules and point to the latest ufs/dev commit of ccpp-physics: ufs-community/ccpp-physics@291b4aa

@RuiyuSun
Copy link
Copy Markdown
Contributor Author

RuiyuSun commented Oct 27, 2025

@RuiyuSun The ccpp-physics PR was merged. This is ready to revert .gitmodules and point to the latest ufs/dev commit of ccpp-physics: ufs-community/ccpp-physics@291b4aa

I see it is pointing to the correct ccpp/physics and branch

@dpsarmie
Copy link
Copy Markdown
Collaborator

I see it is pointing to the correct ccpp/physics and branch

Jong had gone ahead and updated the hash. This just needs to get merged into the UFSATM now that everything is approved and updated. @DusanJovic-NOAA @BrianCurtis-NOAA

@DusanJovic-NOAA DusanJovic-NOAA merged commit 25f0376 into NOAA-EMC:develop Oct 27, 2025
8 checks passed
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