Skip to content

Update eobs ATMGES variable in JJOB#4663

Open
CatherineThomas-NOAA wants to merge 3 commits intoNOAA-EMC:dev/gfs.v17from
CatherineThomas-NOAA:eobs_jjob
Open

Update eobs ATMGES variable in JJOB#4663
CatherineThomas-NOAA wants to merge 3 commits intoNOAA-EMC:dev/gfs.v17from
CatherineThomas-NOAA:eobs_jjob

Conversation

@CatherineThomas-NOAA
Copy link
Contributor

Description

@lgannoaa discovered a silent failure in the v17 real time in the eobs JJOB JGLOBAL_ENKF_SELECT_OBS:

/lfs/h2/emc/gfstemp/emc.global/git/global-workflow/retrov17_01/dev/jobs/JGLOBAL_ENKF_SELECT_OBS: line 62: ATMGES: unbound variable

JGLOBAL_ENKF_SELECT_OBS[62]LEVS=
JGLOBAL_ENKF_SELECT_OBS[63]export err=1
JGLOBAL_ENKF_SELECT_OBS[63]err=1

The variable ATMGES is not defined in this JJOB. This PR replaces this variable with ATMGES_ENSMEAN, eliminating the error and properly defining LEVS.

This PR brings this fix into the dev/gfs.v17 branch. The fix will be added to the develop branch separately.

Refs #4615

Type of change

  • Bug fix (fixes something broken)

Change characteristics

  • Is this change expected to change outputs (e.g. value changes to existing outputs, new files stored in COM, files removed from COM, filename changes, additions/subtractions to archives)? NO
  • Is this a breaking change (a change in existing functionality)? NO
  • Does this change require a documentation update? NO
  • Does this change require an update to any of the following submodules? NO

How has this been tested?

All generate_workflow -G tests pass on WCOSS2 and Gaea C6 (results will be posted in the comments). v17 low res test results are identical with current dev/gfs.v17 on WCOSS2.

Checklist

  • Any dependent changes have been merged and published
  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have documented my code, including function, input, and output descriptions
  • My changes generate no new warnings
  • New and existing tests pass with my changes
  • This change is covered by an existing CI test or a new one has been added
  • Any new scripts have been added to the .github/CODEOWNERS file with owners
  • I have made corresponding changes to the system documentation if necessary

@CatherineThomas-NOAA
Copy link
Contributor Author

WCOSS2 -G Tests

/lfs/h2/emc/ptmp/catherine.thomas/RUNTESTS_eobs2/EXPDIR/C48_ATM_eobs2
   CYCLE         STATE           ACTIVATED              DEACTIVATED     
202103231200        Done    Mar 12 2026 01:00:37    Mar 12 2026 02:00:46
 
/lfs/h2/emc/ptmp/catherine.thomas/RUNTESTS_eobs2/EXPDIR/C48mx500_3DVarAOWCDA_eobs2
   CYCLE         STATE           ACTIVATED              DEACTIVATED     
202103241800        Done    Mar 12 2026 01:00:45    Mar 12 2026 01:20:35
202103250000        Done    Mar 12 2026 01:00:45    Mar 12 2026 02:55:32
 
/lfs/h2/emc/ptmp/catherine.thomas/RUNTESTS_eobs2/EXPDIR/C48mx500_hybAOWCDA_eobs2
   CYCLE         STATE           ACTIVATED              DEACTIVATED     
202103241800        Done    Mar 12 2026 01:00:18    Mar 12 2026 01:20:21
202103250000        Done    Mar 12 2026 01:00:18    Mar 12 2026 02:30:11
 
/lfs/h2/emc/ptmp/catherine.thomas/RUNTESTS_eobs2/EXPDIR/C48_S2SW_extended_eobs2
   CYCLE         STATE           ACTIVATED              DEACTIVATED     
202103231200        Done    Mar 12 2026 01:00:22    Mar 12 2026 03:50:14
202103231800        Done    Mar 12 2026 01:00:22    Mar 12 2026 03:30:26
 
/lfs/h2/emc/ptmp/catherine.thomas/RUNTESTS_eobs2/EXPDIR/C96_atm3DVar_extended_eobs2
   CYCLE         STATE           ACTIVATED              DEACTIVATED     
202112201800        Done    Mar 12 2026 01:00:28    Mar 12 2026 01:20:26
202112210000        Done    Mar 12 2026 01:00:28    Mar 12 2026 11:55:37
202112210600        Done    Mar 12 2026 01:00:28    Mar 12 2026 13:05:50
202112211200        Done    Mar 12 2026 01:25:28    Mar 12 2026 14:15:41
202112211800        Done    Mar 12 2026 12:00:42    Mar 12 2026 16:50:29
 
/lfs/h2/emc/ptmp/catherine.thomas/RUNTESTS_eobs2/EXPDIR/C96C48_hybatmDA_eobs2
   CYCLE         STATE           ACTIVATED              DEACTIVATED     
202112201800        Done    Mar 12 2026 01:00:24    Mar 12 2026 01:20:25
202112210000        Done    Mar 12 2026 01:00:24    Mar 12 2026 03:15:28
202112210600        Done    Mar 12 2026 01:00:24    Mar 12 2026 03:25:32
 
/lfs/h2/emc/ptmp/catherine.thomas/RUNTESTS_eobs2/EXPDIR/C96C48_hybatmsnowDA_eobs2
   CYCLE         STATE           ACTIVATED              DEACTIVATED     
202112201200        Done    Mar 12 2026 01:00:51    Mar 12 2026 01:20:47
202112201800        Done    Mar 12 2026 01:00:51    Mar 12 2026 03:25:36
202112210000        Done    Mar 12 2026 01:00:51    Mar 12 2026 03:25:36
 
/lfs/h2/emc/ptmp/catherine.thomas/RUNTESTS_eobs2/EXPDIR/C96C48_hybatmsoilDA_eobs2
   CYCLE         STATE           ACTIVATED              DEACTIVATED     
202205150600        Done    Mar 12 2026 01:00:18    Mar 12 2026 01:25:14
202205151200        Done    Mar 12 2026 01:00:18    Mar 12 2026 03:30:20
202205151800        Done    Mar 12 2026 01:00:18    Mar 12 2026 03:40:16
 
/lfs/h2/emc/ptmp/catherine.thomas/RUNTESTS_eobs2/EXPDIR/C96C48mx500_S2SW_cyc_gfs_eobs2
   CYCLE         STATE           ACTIVATED              DEACTIVATED     
202112201200        Done    Mar 12 2026 01:00:15    Mar 12 2026 01:20:13
202112201800        Done    Mar 12 2026 01:00:15    Mar 16 2026 15:05:16
202112210000        Done    Mar 12 2026 01:00:15    Mar 16 2026 15:25:16
202112211800        Done    Mar 12 2026 01:25:18    Mar 16 2026 15:35:10

@CatherineThomas-NOAA
Copy link
Contributor Author

Gaea C6 -G Tests

/gpfs/f6/gfs-cpu/world-shared/Catherine.Thomas/tmp/RUNTESTS_eobs2/EXPDIR/C48_ATM_eobs2
   CYCLE         STATE           ACTIVATED              DEACTIVATED     
202103231200        Done    Mar 12 2026 01:30:10    Mar 12 2026 02:30:23
 
/gpfs/f6/gfs-cpu/world-shared/Catherine.Thomas/tmp/RUNTESTS_eobs2/EXPDIR/C48mx500_3DVarAOWCDA_eobs2
   CYCLE         STATE           ACTIVATED              DEACTIVATED     
202103241800        Done    Mar 12 2026 01:30:10    Mar 12 2026 01:50:13
202103250000        Done    Mar 12 2026 01:30:10    Mar 12 2026 03:10:09
 
/gpfs/f6/gfs-cpu/world-shared/Catherine.Thomas/tmp/RUNTESTS_eobs2/EXPDIR/C48mx500_hybAOWCDA_eobs2
   CYCLE         STATE           ACTIVATED              DEACTIVATED     
202103241800        Done    Mar 12 2026 01:30:10    Mar 12 2026 01:50:13
202103250000        Done    Mar 12 2026 01:30:10    Mar 12 2026 02:45:32
 
/gpfs/f6/gfs-cpu/world-shared/Catherine.Thomas/tmp/RUNTESTS_eobs2/EXPDIR/C48_S2SW_eobs2
   CYCLE         STATE           ACTIVATED              DEACTIVATED     
202103231200        Done    Mar 12 2026 01:30:10    Mar 12 2026 02:30:23
 
/gpfs/f6/gfs-cpu/world-shared/Catherine.Thomas/tmp/RUNTESTS_eobs2/EXPDIR/C96_atm3DVar_eobs2
   CYCLE         STATE           ACTIVATED              DEACTIVATED     
202112201800        Done    Mar 12 2026 01:30:10    Mar 12 2026 01:50:13
202112210000        Done    Mar 12 2026 01:30:10    Mar 12 2026 03:30:32
202112210600        Done    Mar 12 2026 01:30:10    Mar 12 2026 03:50:07
 
/gpfs/f6/gfs-cpu/world-shared/Catherine.Thomas/tmp/RUNTESTS_eobs2/EXPDIR/C96C48_hybatmDA_eobs2
   CYCLE         STATE           ACTIVATED              DEACTIVATED     
202112201800        Done    Mar 12 2026 01:30:10    Mar 12 2026 01:50:13
202112210000        Done    Mar 12 2026 01:30:10    Mar 12 2026 03:20:16
202112210600        Done    Mar 12 2026 01:30:10    Mar 12 2026 03:35:08
 
/gpfs/f6/gfs-cpu/world-shared/Catherine.Thomas/tmp/RUNTESTS_eobs2/EXPDIR/C96C48_hybatmsnowDA_eobs2
   CYCLE         STATE           ACTIVATED              DEACTIVATED     
202112201200        Done    Mar 12 2026 01:30:10    Mar 12 2026 01:50:13
202112201800        Done    Mar 12 2026 01:30:10    Mar 12 2026 03:20:16
202112210000        Done    Mar 12 2026 01:30:10    Mar 12 2026 03:30:32
 
/gpfs/f6/gfs-cpu/world-shared/Catherine.Thomas/tmp/RUNTESTS_eobs2/EXPDIR/C96C48_hybatmsoilDA_eobs2
   CYCLE         STATE           ACTIVATED              DEACTIVATED     
202205150600        Done    Mar 12 2026 01:30:10    Mar 12 2026 01:50:13
202205151200        Done    Mar 12 2026 01:30:10    Mar 12 2026 03:20:16
202205151800        Done    Mar 12 2026 01:30:10    Mar 12 2026 03:50:07
 
/gpfs/f6/gfs-cpu/world-shared/Catherine.Thomas/tmp/RUNTESTS_eobs2/EXPDIR/C96C48mx500_S2SW_cyc_gfs_eobs2
   CYCLE         STATE           ACTIVATED              DEACTIVATED     
202112201200        Done    Mar 12 2026 01:30:10    Mar 12 2026 01:50:13
202112201800        Done    Mar 12 2026 01:30:10    Mar 12 2026 04:10:09
202112210000        Done    Mar 12 2026 01:30:10    Mar 12 2026 04:35:33

@CatherineThomas-NOAA CatherineThomas-NOAA marked this pull request as ready for review March 16, 2026 18:24
Copy link
Contributor

@TravisElless-NOAA TravisElless-NOAA left a comment

Choose a reason for hiding this comment

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

Looks great to me. Thanks @CatherineThomas-NOAA!

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.

3 participants