-
Notifications
You must be signed in to change notification settings - Fork 120
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
[develop] Bring change of production/AQM.v7 back to develop #828
Changes from 9 commits
777c0a3
6d19532
1158568
2acbe92
0217016
e500744
cc2031f
1da7eb6
57896b3
8b4e998
818b8f3
0c9b8a4
5f1be8e
193656c
e0f79fa
f8e479e
7e5c1c0
53a7248
0055e36
cec9168
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -30,7 +30,7 @@ | |
# | ||
. $USHdir/source_util_funcs.sh | ||
source_config_for_task "task_get_extrn_ics|task_get_extrn_lbcs" ${GLOBAL_VAR_DEFNS_FP} | ||
. $USHdir/job_preamble.sh | ||
. $USHdir/job_preamble.sh "TRUE" | ||
# | ||
#----------------------------------------------------------------------- | ||
# | ||
|
@@ -221,8 +221,7 @@ esac | |
#----------------------------------------------------------------------- | ||
# | ||
if [ $RUN_ENVIR = "nco" ]; then | ||
export EXTRN_MDL_STAGING_DIR="${COMINext}" | ||
mkdir_vrfy -p "${EXTRN_MDL_STAGING_DIR}" | ||
export EXTRN_MDL_STAGING_DIR="${EXTRN_MDL_STAGING_DIR:-${DATA}}" | ||
else | ||
export EXTRN_MDL_STAGING_DIR="${COMIN}/${EXTRN_MDL_NAME}/for_${ICS_OR_LBCS}" | ||
mkdir_vrfy -p "${EXTRN_MDL_STAGING_DIR}" | ||
|
@@ -245,7 +244,7 @@ Call to ex-script corresponding to J-job \"${scrfunc_fn}\" failed." | |
# | ||
#----------------------------------------------------------------------- | ||
# | ||
job_postamble | ||
job_postamble "FALSE" | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Similar comment as above. I see that this function does accept TRUE as an action here, but this seems to be an undefined keyword that could be literally anything. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I add a new function to |
||
# | ||
#----------------------------------------------------------------------- | ||
# | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,3 +1,5 @@ | ||
load(pathJoin("PrgEnv-intel", os.getenv("PrgEnv_intel_ver"))) | ||
load(pathJoin("craype", os.getenv("craype_ver"))) | ||
load(pathJoin("intel", os.getenv("intel_ver"))) | ||
load(pathJoin("python", os.getenv("python_ver"))) | ||
load(pathJoin("prod_util", os.getenv("prod_util_ver"))) |
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.
It doesn't look like
job_preamble.sh
should accept an argument. If it does and I missed it, please add documentation about it in that script.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.
@christinaholtNOAA, it accepts an argument for the shared job id. If an argument exists, a shared pid is used.
I'll add a comment to the script.