-
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
Enable cycling atmosphere DA in 3DEnVar mode with coupled ensemble. #1718
Changes from 16 commits
8c2a515
5e8e1f6
c6d3935
bab5b67
2c80256
eb57158
10503ec
4243202
4a6b0c1
711a088
cd73e26
5398408
ae821b7
725fe84
a41c0aa
69d8f80
f645ab4
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 |
---|---|---|
|
@@ -5,17 +5,17 @@ | |
|
||
echo "BEGIN: config.efcs" | ||
|
||
# TODO: the _ENKF counterparts need to be defined in config.base | ||
export DO_AERO=${DO_AERO_ENKF:-"NO"} | ||
export DO_OCN=${DO_OCN_ENKF:-"NO"} | ||
export DO_ICE=${DO_ICE_ENKF:-"NO"} | ||
export DO_WAVE=${DO_WAVE_ENKF:-"NO"} | ||
# TODO: the _ENKF counterparts need to be defined in config.base; for now inherit from config.base | ||
export DO_AERO=${DO_AERO_ENKF:-${DO_AERO:-"NO"}} | ||
export DO_OCN=${DO_OCN_ENKF:-${DO_OCN:-"NO"}} | ||
export DO_ICE=${DO_ICE_ENKF:-${DO_ICE:-"NO"}} | ||
export DO_WAVE=${DO_WAVE_ENKF:-${DO_WAVE:-"NO"}} | ||
|
||
# TODO: Possibly need OCNRES_ENKF, ICERES_ENKF, WAVRES_ENKF too | ||
if [[ ${DO_OCN} == "YES" ]]; then | ||
case "${CASE_ENS}" in | ||
"C48") export OCNRES=500;; | ||
"C96") export OCNRES=100;; | ||
"C96") export OCNRES=500;; | ||
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. 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. This is set because the deterministic and the ensemble share the same ocean resolution, unlike the atmosphere where we have dual resolutions. I am sure we can support dual resolution ocean as well, but its not worth the hassle at this moment. |
||
"C192") export OCNRES=050;; | ||
"C384") export OCNRES=025;; | ||
"C768") export OCNRES=025;; | ||
|
@@ -81,18 +81,4 @@ else | |
export restart_interval="6" | ||
fi | ||
|
||
# wave model | ||
export cplwav=.false. | ||
|
||
# ocean model resolution | ||
case "${CASE_ENS}" in | ||
"C48") export OCNRES=500;; | ||
"C96") export OCNRES=100;; | ||
"C192") export OCNRES=050;; | ||
"C384") export OCNRES=025;; | ||
"C768") export OCNRES=025;; | ||
*) export OCNRES=025;; | ||
esac | ||
export ICERES=${OCNRES} | ||
|
||
echo "END: config.efcs" |
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 not support
OCNRES=100
as well?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's supported, it's just not a default for any ATM resolution.
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.
OK, thank you for the clarification @WalterKolczynski-NOAA.
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.
We need to revisit this section in
config.base
as more combinations are possible, but the limitation here is the bash configuration.