-
Notifications
You must be signed in to change notification settings - Fork 170
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
Check the DO_JEDISNOWDA condition before adding snow DA analysis to the list #2471
Check the DO_JEDISNOWDA condition before adding snow DA analysis to the list #2471
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.
Thank you for adding the conditional. This will work.
However, in config.base
, the variable is named DO_JEDILANDDA
. It should be renamed to DO_JEDISNOWDA
. Would you mind changing that variable as well to be DO_JEDISNOWDA
instead of DO_JEDILANDDA
?
ush/hpssarch_gen.sh
Outdated
echo "${COM_SNOW_ANALYSIS/${ROTDIR}\//}/*0000.sfc_data.tile4.nc" | ||
echo "${COM_SNOW_ANALYSIS/${ROTDIR}\//}/*0000.sfc_data.tile5.nc" | ||
echo "${COM_SNOW_ANALYSIS/${ROTDIR}\//}/*0000.sfc_data.tile6.nc" | ||
if [[ ${DO_JEDISNOWDA} = "YES" ]]; then |
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.
if [[ ${DO_JEDISNOWDA} = "YES" ]]; then | |
if [[ "${DO_JEDISNOWDA:-}" == "YES" ]]; then |
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.
Thanks @aerorahul and done the changes as suggested.
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.
Also changed the DO_JEDILANDDA
to DO_JEDISNOWDA
.
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.
At somepoint we should review what needs to be archived for the sfc_data
files. We have sfcanl_data
tile files also being archived.
* upstream/develop: Update fbwind for COM refactor (NOAA-EMC#2479) Update parm files for atm product (NOAA-EMC#2486) Fix paths in prepocnobs task (NOAA-EMC#2459) Add oceanalecn to workflow generator (NOAA-EMC#2409) Add support for CADS IR cloud detection scheme in the GSI (NOAA-EMC#2478) Improve error messaging to resolve bugzilla (NOAA-EMC#2468) Check the DO_JEDISNOWDA condition before adding snow DA analysis to the list (NOAA-EMC#2471)
Description
This PR added checking the DO_JEDISNOWDA condition before adding snow DA analysis to the archive list.
Resolves #2469
Type of change
Change characteristics
Checklist