Skip to content

ufs-dev 332#1195

Merged
grantfirl merged 20 commits intoNCAR:mainfrom
scrasmussen:ufs-dev-332
Feb 26, 2026
Merged

ufs-dev 332#1195
grantfirl merged 20 commits intoNCAR:mainfrom
scrasmussen:ufs-dev-332

Conversation

BinLiu-NOAA and others added 18 commits December 12, 2025 10:55
nesting (moving-nesting) configurations, while reading in
lw/sw_file_gas/clouds files.
options of tc_rain and tc_snow for adjustments of max terminal fall
speeds of rain and snow under TC conditions.
Add RRTMGP initialization flags and revert hardcoding of mpiroot
….meta so that convective adjustment time can be tunable, introducing two parameters cat_adj_deep and cat_adj_shal for this
Clean up sfcsub.F: remove unused code, comments, and developer tags
Copy link
Collaborator

@mdtoyNOAA mdtoyNOAA 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.

Copy link
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.

Approving because the changes are identical to the ufs-community PR, noting that the RRTMGP moving nest MPI problem is using a temporary solution: see ufs-community#352 that was added to address the issue more completely in the future.

@grantfirl grantfirl requested a review from climbfuji February 24, 2026 18:41
# Default codeowners for files that don't have specific owners:

* @grantfirl @rhaesung @Qingfu-Liu @dustinswales @mzhangw
* @grantfirl @rhaesung @Qingfu-Liu @dustinswales @mzhangw @climbfuji
Copy link
Collaborator

Choose a reason for hiding this comment

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

Thanks!

mpirank, & !< Current MPI rank
mpiroot !< Master MPI rank
mpiroot1 !< Master MPI rank
integer :: mpiroot
Copy link
Collaborator

Choose a reason for hiding this comment

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

I am still a little uncomfortable with a locally defined MPI communicator, but for now (until I understand this better), I think it's ok.

Copy link
Member

Choose a reason for hiding this comment

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

Ditto. But I had no way of testing alternatives.
There is now(?) a HAFSv2 RT with GP enabled, so I will be able to revisit this.

Copy link
Collaborator

Choose a reason for hiding this comment

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

I actually couldn't find a RT that runs the HAFS_v2 suite. I know that the SDF was added, but I don't see an RT for it.

@grantfirl grantfirl merged commit a2aa6cc into NCAR:main Feb 26, 2026
3 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.

9 participants