Update ccpp-physics to include sfcsub cleanup

[ClaraDraper-NOAA]

Description

ccpp-physics hash update for PR ufs-community/ccpp-physics#343

Do PRs in upstream repositories need to be merged first?

ufs-community/ccpp-physics#343

[gspetro-NOAA]

Could we get reviews on this PR so that we can process WM parent PR 3076?

[DusanJovic-NOAA]

We usually require developers to also update .gitmodules to point to fork/branch where the changes are made. Then, after the corresponding submodule is merged (ccpp-physics in this case), you will revert the changes in the .gitmodule to point to develop branch and update submodule commit hash.

[ClaraDraper-NOAA]

Gillian, The test_changes.list has: hafs_regional_storm_following_1nest_atm_ocn_wav_mom6 intel Is this expected?

…

On Thu, Feb 5, 2026 at 9:53 AM Gillian Petro ***@***.***> wrote: *gspetro-NOAA* left a comment (NOAA-EMC/ufsatm#1062) <#1062 (comment)> Could we get reviews on this PR so that we can process WM parent PR 3076? — Reply to this email directly, view it on GitHub <#1062 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AH7BYD5DFR4K3WMKU3KFGK34KNKPXAVCNFSM6AAAAACTWDE2JCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZTQNJUGE2DKMRWGQ> . You are receiving this because you authored the thread.Message ID: ***@***.***>

-- -------------------------------------------------------------------- Clara Draper Physical Scientist NOAA Environmental Modeling Center College Park, MD --------------------------------------------------------------------

[gspetro-NOAA]

@ClaraDraper-NOAA It is not, but sometimes a few tests will time out, which is not usually cause for concern. What does the log say?

[ClaraDraper-NOAA]

hafs_regional_storm_following_1nest_atm_ocn_wav_mom6

baseline dir = /scratch4/NAGAPE/epic/role-epic/UFS-WM_RT/NEMSfv3gfs/develop-20260126/hafs_regional_storm_following_1nest_atm_ocn_wav_mom6_intel
working dir = /scratch4/BMC/gsienkf/Clara.Draper/ufs_testing//RT_RUNDIRS/Clara.Draper/FV3_RT/rt_3431193/hafs_regional_storm_following_1nest_atm_ocn_wav_mom6_intel
Checking test hafs_regional_storm_following_1nest_atm_ocn_wav_mom6_intel results ....
Comparing atmf003.nc .....USING NCCMP......OK
Comparing sfcf003.nc .....USING NCCMP......OK
Comparing atm.nest02.f003.nc .....USING NCCMP......OK
Comparing sfc.nest02.f003.nc .....USING NCCMP......OK
Comparing RESTART/20200825.150000.MOM.res.nc .....USING NCCMP......OK
Comparing RESTART/20200825.150000.MOM.res_1.nc .....USING NCCMP......OK
Comparing 20200825.150000.out_grd.ww3 .....USING CMP......OK
Comparing 20200825.150000.out_pnt.ww3.nc .....USING NCCMP......OK

0: The total amount of wall time = 1062.311798
0: The maximum resident set size (KB) = 1086716

Test hafs_regional_storm_following_1nest_atm_ocn_wav_mom6_intel PASS

[gspetro-NOAA]

@DusanJovic-NOAA I believe Clara has made the requested changes. Could you take a look?

[DusanJovic-NOAA]

@DusanJovic-NOAA I believe Clara has made the requested changes. Could you take a look?

Looks good. Thanks @ClaraDraper-NOAA

[rhaesung]

@ClaraDraper-NOAA ccpp-physics PR has been merged. Please update the submodule pointer for ccpp-physics (ufs-community/ccpp-physics@b73b97b) and revert the .gitmodules file in preparation for the merge.

[grantfirl]

@rhaesung @DusanJovic-NOAA This was combined into #1060 and merged already, right? If so, this can be closed.

[DusanJovic-NOAA]

Merged via #1060