-
Notifications
You must be signed in to change notification settings - Fork 168
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update JGDAS ENKF ECEN job #3050
base: develop
Are you sure you want to change the base?
Update JGDAS ENKF ECEN job #3050
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why are there no changes for COM_ATMOS_ANALYSIS_STAT
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I second @CoryMartin-NOAA 's comment. Changes in this PR look good so far but there are other COM
variables in scripts/exgdas_enkf_ecen.sh
that also need to be updated to their COMIN
/COMOUT
versions. Please update those variables as well, thanks!
@CoryMartin-NOAA @KateFriedman-NOAA |
@mingshichen-noaa Thanks for committing those additional updates. Question, the testing you did included these |
Kate, |
I agree with @KateFriedman-NOAA those are links to files produced by executables and thus should be COMOUT |
Kate, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think the changes are correct now. Approve pending successful completion of CI tests and approval by other reviewers.
CI Passed on Hercules in Build# 1
|
Description
NCO has requested that each COM variable specify whether it is an input or an output. This completes that process for the global jgdas enkf ecen job.
Refs #2451
Type of change
Change characteristics
How has this been tested?
Checklist