Skip to content

Combined updates from UFS-dev PR325 and PR326#633

Merged
grantfirl merged 5 commits into
NCAR:mainfrom
hertneky:ufs-dev-PR325
Nov 11, 2025
Merged

Combined updates from UFS-dev PR325 and PR326#633
grantfirl merged 5 commits into
NCAR:mainfrom
hertneky:ufs-dev-PR325

Conversation

@hertneky
Copy link
Copy Markdown
Collaborator

@hertneky hertneky commented Nov 4, 2025

This PR catches the NCAR:main branch up with changes from the ufs-community:ufs/dev branch.

Associated ufs/dev PR:

Associated ufsatm PR:

Associated NCAR PR:


TESTS CONDUCTED: SCM RTs

REGRESSION TEST CHANGES:

@grantfirl grantfirl marked this pull request as ready for review November 10, 2025 15:21
Copy link
Copy Markdown
Collaborator

@grantfirl grantfirl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved pending input from Dom on the physics PR.

Copy link
Copy Markdown
Member

@scrasmussen scrasmussen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me, thanks!

@grantfirl
Copy link
Copy Markdown
Collaborator

@scrasmussen You may want to re-review this because I added some changes to GFS_typedefs to generalize the cloud fraction changes to be host controlled by request from NRL.

@grantfirl
Copy link
Copy Markdown
Collaborator

FYI, RTs with the configurable code changes are slightly different than beforehand. I looked at the differences, and they are tiny (near precision level) and only with release compilation flags, so I'm pretty confident that this is optimization related.

@grantfirl grantfirl merged commit 56fe933 into NCAR:main Nov 11, 2025
12 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.

3 participants