-
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
Add GEFS atm-only fcst job support #1731
Add GEFS atm-only fcst job support #1731
Conversation
Makes the minor changes necessary to run GEFS fcst job only from pre-staged ICs. Also many unnecessary DA settings are removed from the GEFS version of `config.base`. Resolves NOAA-EMC#826
Automated global-workflow Testing Results:
|
Automated global-workflow Testing Results:
|
jobs/JGLOBAL_FORECAST
Outdated
@@ -17,15 +17,23 @@ rCDUMP=${CDUMP} | |||
[[ ${CDUMP} = "gfs" ]] && export rCDUMP="gdas" | |||
|
|||
# Forecast length for GFS forecast | |||
if [ ${CDUMP} = "gfs" ]; then | |||
case ${CDUMP} in | |||
gfs | gefs) |
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.
In the gefs
tree, does FHMAX_GFS
still make sense?
Should the line 23 be something like:
current_var=FHMAX_${CDUMP^^}; declare -x FHMAX=${current_var}
And so on.
Beyond this point, there shouldn't be any references to _gfs
, _gefs
etc.
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.
These kind of things are woven throughout the workflow, well beyond the little portion I am working on here.
Automated global-workflow Testing Results:
|
The `config.efcs` was using the DA diag tables for GEFS when it should just use the normal diag table from `config.fcst`. Also removed more unneeded DA settings from the GEFS side. Refs: NOAA-EMC#826
Automated global-workflow Testing Results:
|
Automated global-workflow Testing Results:
|
@aerorahul said he had a fix for the vrfy file move issue? |
His solution was to put the commit hash as part of the pslot. |
Oh, I see, didn't know those where related, makes total sense now, ok I"ll test that. |
Well, the path has the pslot in it, and if you use the commit hash that will always be unique unless you rerun a PR without any changes. That said, given |
Automated global-workflow Testing Results:
|
Automated global-workflow Testing Results:
|
Automated global-workflow Testing Results:
|
Automated global-workflow Testing Results:
|
Automated global-workflow Testing Results:
|
Automated global-workflow Testing Results:
|
Automated global-workflow Testing Results:
|
Automated global-workflow Testing Results:
|
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.
looks good.
Description
Makes the minor changes necessary to run GEFS atm-only fcst job only from pre-staged ICs. Also many unnecessary DA settings are removed from the GEFS version of
config.base
. A few settings are retained either because they are needed, or are expected to be needed to support running from reanalysis (e.g.IAU_FHROT
).Resolves #826
Type of change
How Has This Been Tested?
Checklist