-
Notifications
You must be signed in to change notification settings - Fork 171
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 marine hybrid envar #3041
base: develop
Are you sure you want to change the base?
Add marine hybrid envar #3041
Conversation
ci/cases/pr/C48mx500_hybAOWCDA.yaml
Outdated
idate: 2021032418 | ||
edate: 2021032500 | ||
nens: 3 | ||
gfs_cyc: 0 |
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.
gfs_cyc: 0 | |
interval: 0 |
Co-authored-by: AndrewEichmann-NOAA <[email protected]>
WCOSS2 (Cactus) C48mx500_3DVarAOWCDA g-w CI Install Prior to launching C48mx500_3DVarAOWCDA g-w CI make the following change in working copy of
With these changes in place, successfully run C48mx500_3DVarAOWCDA g-w CI
The following marine DA jobs successfully ran
|
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 @guillaumevernieres !
@AndrewEichmann-NOAA feel free to push your wcoss updates here if there are more (as a PR into my branch). |
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 to me.
Co-authored-by: David Huber <[email protected]>
Co-authored-by: David Huber <[email protected]>
Co-authored-by: David Huber <[email protected]>
ush/forecast_postdet.sh
Outdated
# GEFS perturbations | ||
if [[ "${RUN}" == "gefs" ]]; then | ||
# to ensure it does not interfere with the GFS | ||
if (( MEMBER > 0 )) && [[ "${ODA_INCUPD:-False}" == "True" ]]; then | ||
${NCP} "${COMIN_OCEAN_ANALYSIS}/${RUN}.t${cyc}z.ocninc.nc" "${DATA}/INPUT/mom6_increment.nc" \ | ||
|| ( echo "FATAL ERROR: Unable to copy ensemble MOM6 increment, ABORT!"; exit 1 ) | ||
fi | ||
fi # if [[ "${RUN}" == "gefs" ]]; then | ||
# TODO if [[ $RUN} == "gefs" ]] block maybe be needed | ||
# to ensure it does not interfere with the GFS when ensemble is updated in the GFS |
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.
Should these comments be removed now?
workflow/rocoto/gfs_tasks.py
Outdated
dep_dict = {'type': 'task', 'name': f'{self.run.replace("enkf", "")}_ocnanalecen'} | ||
deps.append(rocoto.add_dependency(dep_dict)) |
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 this needs to be conditional on marine analysis running. As-is it would add a dependency even if we were running without marine turned on at all.
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.
Yes, that's the intent @WalterKolczynski-NOAA . Re-centering around the deterministic should happen, with or without the marine DA turned on.
Is efcs
used outside of the DA cycling context (atmos or otherwise)?
Companion PR to NOAA-EMC/global-workflow#3041 This PR needs an update to `jcb-gdas` (NOAA-EMC/jcb-gdas#46)
Co-authored-by: Walter Kolczynski - NOAA <[email protected]>
Description
What the title says.
Main features added:
ci test
that runs 1.5 cycle of the hybrid envar with the coupled UFSyamls
to allow running the hybrid envar GFSv17 prototype at c384/0.25 for the det and C192/0.25 for the ens. membersIssues addressed:
Dependencies:
Type of change
Change characteristics
How has this been tested?
global-worflow
ci
onHercules
andHera
at various stages of the development.Checklist