Skip to content

Make sure gfs_* DA jobs are not regression tests#1390

Merged
DavidNew-NOAA merged 2 commits into
developfrom
bugfix/gw-ci
Nov 27, 2024
Merged

Make sure gfs_* DA jobs are not regression tests#1390
DavidNew-NOAA merged 2 commits into
developfrom
bugfix/gw-ci

Conversation

@DavidNew-NOAA
Copy link
Copy Markdown
Collaborator

@DavidNew-NOAA DavidNew-NOAA commented Nov 26, 2024

This PR addresses the bug @RussTreadon-NOAA found that gfs_atmanlvar was being run as a regression test and using the same test reference as gdas_atmanlvar in GW PR #3120.

See NOAA-EMC/global-workflow#3120 (comment)

I've moved all activation of testing mode in JCB out of the JCB base YAMLs and into the JCB algorithm YAMLs. I test the RUN variables to make sure it's not equal to gfs.

I re-ran all the regression tests, and they all passed.

Copy link
Copy Markdown
Contributor

@RussTreadon-NOAA RussTreadon-NOAA left a comment

Choose a reason for hiding this comment

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

Did not test but @DavidNew-NOAA did and found that all regression tests pass.

We need the changes in this PR to move g-w PR #3120 forward.

Approve.

@DavidNew-NOAA DavidNew-NOAA merged commit 9b78c7d into develop Nov 27, 2024
@DavidNew-NOAA DavidNew-NOAA deleted the bugfix/gw-ci branch November 27, 2024 13:16
DavidNew-NOAA added a commit that referenced this pull request Jan 16, 2026
This PR addresses the bug @RussTreadon-NOAA found that
```gfs_atmanlvar``` was being run as a regression test and using the
same test reference as ```gdas_atmanlvar``` in GW PR
[#3120](NOAA-EMC/global-workflow#3120).

See
NOAA-EMC/global-workflow#3120 (comment)

I've moved all activation of testing mode in JCB out of the JCB base
YAMLs and into the JCB algorithm YAMLs. I test the ```RUN``` variables
to make sure it's not equal to ```gfs```.

I re-ran all the regression tests, and they all 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.

2 participants