Skip to content

ufs-dev PR301#617

Merged
grantfirl merged 4 commits into
NCAR:mainfrom
hertneky:ufs-dev-PR301
Sep 23, 2025
Merged

ufs-dev PR301#617
grantfirl merged 4 commits into
NCAR:mainfrom
hertneky:ufs-dev-PR301

Conversation

@hertneky
Copy link
Copy Markdown
Collaborator

@hertneky hertneky commented Sep 17, 2025

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

Associated ufs/dev PR:

Associated fv3atm PR:

Associated NCAR PR:


REGRESSION TEST CHANGES: NONE

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.

Looks good. I thought that it may have been necessary to add the SCM as a "dycore", but it wouldn't do much good at this point, I don't think, since some UFS physics that the SCM calls still need FV3-specific grid info, even if the SCM "fakes" it (e.g. tile number, etc.). If we need to do something different for the SCM in the future, we can do so at that time.

@grantfirl grantfirl merged commit 7cd20a4 into NCAR:main Sep 23, 2025
13 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.

2 participants