Bug fix for ccpp-physics routine sfcsub.F

[GeorgeGayno-NOAA]

Description

No changes to fv3atm other than pointing to a new version of the ccpp physics to
address a bug in sfcsub.F routine.

Issue(s) addressed

Fixes ufs-community/ccpp-physics#199

Testing

Full set of ufs-weather-model regression tests were run on Hera and passed.

Dependencies

• sfcsub.F - Fix the setting of model landmask when interpolating GLDAS data ufs-community/ccpp-physics#202
• Bug fix for ccpp-physics routine sfcsub.F ufs-community/ufs-weather-model#2253

[DusanJovic-NOAA]

The title of this PR is "Test fix to ...". Is this just for testing the fix, or is this intended to be actually merged into develop branch? @GeorgeGayno-NOAA

[GeorgeGayno-NOAA]

The title of this PR is "Test fix to ...". Is this just for testing the fix, or is this intended to be actually merged into develop branch? @GeorgeGayno-NOAA

According to the CCPP repository wiki page (see step 4), any changes to CCPP must also include PRs for FV3ATM and UFS weather model:

https://github.com/ufs-community/ccpp-physics/wiki#instructions-for-ccpp-physics-code-developers

I did not make any changes to FV3ATM itself.

[DusanJovic-NOAA]

The title of this PR is "Test fix to ...". Is this just for testing the fix, or is this intended to be actually merged into develop branch? @GeorgeGayno-NOAA

According to the CCPP repository wiki page (see step 4), any changes to CCPP must also include PRs for FV3ATM and UFS weather model:

https://github.com/ufs-community/ccpp-physics/wiki#instructions-for-ccpp-physics-code-developers

I did not make any changes to FV3ATM itself.

I was actually asking about the title itself. The "Test fix to ..." implies that this is just for testing. But since this needs to be merged into fv3atm the title should be "Bug fix for ccpp-physics routine sfcsub.F" like the corresponding ufs PR.

[grantfirl]

@jkbk2004 @DusanJovic-NOAA This is combined into #831 and therefore it can be closed.