Skip to content

update fix submodule hash to include gfs.v16.3.9 updates#621

Merged
CoryMartin-NOAA merged 2 commits into
NOAA-EMC:developfrom
RussTreadon-NOAA:feature/updatefix
Sep 14, 2023
Merged

update fix submodule hash to include gfs.v16.3.9 updates#621
CoryMartin-NOAA merged 2 commits into
NOAA-EMC:developfrom
RussTreadon-NOAA:feature/updatefix

Conversation

@RussTreadon-NOAA
Copy link
Copy Markdown
Contributor

@RussTreadon-NOAA RussTreadon-NOAA commented Sep 13, 2023

Description
This PR updates the fix submodule to bring in GFS v16.3.9 updates to global_convinfo.txt and global_ozinfo.txt. Issue #620 provides additional information on the GFS v16.3.9 updates.

Please note the following

  • The change to global_ozinfo.txt alters analysis results if ompsnp observation from NOAA-20, NOAA-21, or NPP are processed by gsi.x or enkf.x.
  • The change to global_convinfo.txt alters analysis results if PlanetIQ GPSRO (type 267) is processed by gsi.x or enkf.x.

Fixes #620

Type of change

  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

See the Note above for potential impact on analysis results.

How Has This Been Tested?
ctests have been run on Hera, Orion, and WCOSS2 (Cactus) with results posted in issue #620.

Checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code

@RussTreadon-NOAA RussTreadon-NOAA self-assigned this Sep 13, 2023
WalterKolczynski-NOAA pushed a commit to NOAA-EMC/global-workflow that referenced this pull request Sep 14, 2023
…1858)

GFS v16.3.9 was implemented 20230911 12Z.  The following changes were implemented into
operations:
- turns off ingest of NOAA-21 radiances into the GSI,
-  PlanetIQ GPSRO and NOAA-20 OMPS (Ozone Mapping and Profiler Suite) Ozone retrievals
will be actively assimilated,
-  OMPS observation errors will be adjusted.

g-w PR #1852 brought these changes into g-w branch [`dev/gfs.v16`](https://github.com/NOAA-EMC/global-workflow/tree/dev/gfs.v16).
This PR brings the same changes into g-w `develop`.

Fixes #1856
Fixes #1857

Related to, but not dependent on, GSI [#621](NOAA-EMC/GSI#621)
@RussTreadon-NOAA
Copy link
Copy Markdown
Contributor Author

Ctests have been run on Orion with results posted and discussed in issue #620. No tests failed due to generating different analysis results.

This result was at first puzzling. The ozinfo change altered the observation error for ompsnp data. The global ctests assimilate this data. A closer look at the ctest scripts show that both the updat and contrl jobs use the same fix files, those from updat. Given this, the no-analysis impact result makes sense.

As a test, modify the global_3dvar test to use the fix files associated with gsi version being run: updat gsi.x using updat fix, contrl gsi.x using contrl fix. With this change, the ozone penalties differ and, therefore, the analyses differ. This is expected.

The Orion ctests confirm the appropriate behavior.

Since by default the ctest scripts use the updat fix opt to not run ctests on Hera or WCOSS2. The no-impact result observed on Orion will be likewise found on these platforms.

@RussTreadon-NOAA RussTreadon-NOAA marked this pull request as ready for review September 14, 2023 10:10
@HaixiaLiu-NOAA HaixiaLiu-NOAA self-requested a review September 14, 2023 11:59
@HaixiaLiu-NOAA
Copy link
Copy Markdown
Contributor

@RussTreadon-NOAA Thank you for the detailed documentations in this PR and in issue #620 . The changes to the 2 fix files look good to me.

Copy link
Copy Markdown
Contributor

@HaixiaLiu-NOAA HaixiaLiu-NOAA left a comment

Choose a reason for hiding this comment

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

changes look good. Approving

@RussTreadon-NOAA
Copy link
Copy Markdown
Contributor Author

g-w PR #1858 added the GFS v16.3.9 GSI fix directory to their archive of staged GSI fix files.

For the same of completeness update GSI_BINARY_SOURCE_DIR in modulefiles/gsi_$MACHINE.lua to point at EID directory 20230911. Done at 1176751.

Copy link
Copy Markdown
Contributor

@XuanliLi-NOAA XuanliLi-NOAA left a comment

Choose a reason for hiding this comment

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

convinfo file looks to me.

@RussTreadon-NOAA
Copy link
Copy Markdown
Contributor Author

Thank you @HaixiaLiu-NOAA and @XuanliLi-NOAA for your reviews and approvals. I'll work with the Handling Review team to get this PR merged into develop.

@CoryMartin-NOAA CoryMartin-NOAA merged commit 008c63c into NOAA-EMC:develop Sep 14, 2023
@RussTreadon-NOAA RussTreadon-NOAA deleted the feature/updatefix branch September 14, 2023 15:36
xyzemc pushed a commit to xyzemc/GSI_develop-v16-tms that referenced this pull request Feb 27, 2026
**Description**
This PR updates the `fix` submodule to bring in GFS v16.3.9 updates to
`global_convinfo.txt` and `global_ozinfo.txt`. Issue NOAA-EMC#620 provides
additional information on the GFS v16.3.9 updates.

Please **note** the following
- The change to `global_ozinfo.txt` alters analysis results if ompsnp
observation from NOAA-20, NOAA-21, or NPP are processed by gsi.x or
`enkf.x`.
- The change to `global_convinfo.txt` alters analysis results if
PlanetIQ GPSRO (type 267) is processed by `gsi.x` or `enkf.x`.

Fixes NOAA-EMC#620 

**Type of change**
- [x] Breaking change (fix or feature that would cause existing
functionality to not work as expected)

See the **Note** above for potential impact on analysis results.

**How Has This Been Tested?**
ctests have been run on Hera, Orion, and WCOSS2 (Cactus) with results
posted in issue NOAA-EMC#620.
  
**Checklist**
- [x] My code follows the style guidelines of this project
- [x] I have performed a self-review of my own code
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.

Update GSI fix submodule to bring in GFS v16.3.9 changes

4 participants