-
Notifications
You must be signed in to change notification settings - Fork 258
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
Updates to aerosol RTs - new dust and fcsav #1170
Changes from 46 commits
1d390b5
63db579
e8ef421
98a8780
7d17cb7
f1ab9da
6be35ed
45f4f99
2df8f42
9be0d61
ff647b7
54af806
d85d71a
f96d43d
9765e0c
2114229
63d9c09
4826456
d89b1af
ee05082
8588c75
b302d57
04e3773
cc9160b
60d8750
2e77acc
fdef213
0f36755
ccfb273
0cbcfc8
bcdb7fd
4b73284
51a4d6e
73062ee
f37c38b
745c5d3
afc526e
e81cb28
b4214c4
26717e7
f6a0e5b
a3da492
0c1d249
debd75d
4ed3fe0
c7e17e7
eb249b9
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Large diffs are not rendered by default.
Large diffs are not rendered by default.
Large diffs are not rendered by default.
Large diffs are not rendered by default.
Large diffs are not rendered by default.
Large diffs are not rendered by default.
Large diffs are not rendered by default.
Large diffs are not rendered by default.
Large diffs are not rendered by default.
Large diffs are not rendered by default.
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -485,7 +485,7 @@ if [[ $TESTS_FILE =~ '35d' ]] || [[ $TESTS_FILE =~ 'weekly' ]]; then | |
TEST_35D=true | ||
fi | ||
|
||
BL_DATE=20220516 | ||
BL_DATE=20220517 | ||
if [[ $MACHINE_ID = hera.* ]] || [[ $MACHINE_ID = orion.* ]] || [[ $MACHINE_ID = cheyenne.* ]] || [[ $MACHINE_ID = gaea.* ]] || [[ $MACHINE_ID = jet.* ]] || [[ $MACHINE_ID = s4.* ]]; then | ||
RTPWD=${RTPWD:-$DISKNM/NEMSfv3gfs/develop-${BL_DATE}/${RT_COMPILER^^}} | ||
else | ||
|
@@ -531,10 +531,9 @@ mkdir ${LOG_DIR} | |
if [[ $ROCOTO == true ]]; then | ||
|
||
ROCOTO_XML=${PATHRT}/rocoto_workflow.xml | ||
ROCOTO_STATE=${PATHRT}/rocoto_workflow.state | ||
ROCOTO_DB=${PATHRT}/rocoto_workflow.db | ||
|
||
rm -f $ROCOTO_XML $ROCOTO_DB $ROCOTO_STATE *_lock.db | ||
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. Why is $ROCOTO_STATE removed? 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. @DusanJovic-NOAA Honestly I don't know when that was changed. It may have come from when I pulled in the latest changes from the develop branch. Looking back a the log history I didn't modify that. I have no problem adding it back in if that is what we should do. 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. @DusanJovic-NOAA it seems to me that these lines were not there until @SamuelTrahanNOAA 's PR 1195. 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. @bbakernoaa Please revert those changes. It is okay to make these changes at this point in the commit since they do not affect the regression test results. |
||
rm -f $ROCOTO_XML $ROCOTO_DB *_lock.db | ||
|
||
if [[ $MACHINE_ID = wcoss ]]; then | ||
QUEUE=dev | ||
|
@@ -881,7 +880,7 @@ else | |
|
||
rm -f fv3_*.x fv3_*.exe modules.fv3_* keep_tests.tmp | ||
[[ ${KEEP_RUNDIR} == false ]] && rm -rf ${RUNDIR_ROOT} | ||
[[ ${ROCOTO} == true ]] && rm -f ${ROCOTO_XML} ${ROCOTO_DB} ${ROCOTO_STATE} *_lock.db | ||
[[ ${ROCOTO} == true ]] && rm -f ${ROCOTO_XML} ${ROCOTO_DB} *_lock.db | ||
[[ ${TEST_35D} == true ]] && rm -f tests/cpld_bmark*_20* | ||
[[ ${SINGLE_NAME} != '' ]] && rm -f rt.conf.single | ||
fi | ||
|
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 is this line removed?