-
Notifications
You must be signed in to change notification settings - Fork 211
Enable global-workflow to run C768C384 GSI on Gaea-C5 and C6 #2990
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
Changes from all commits
f190cbf
daedec8
2b7f661
116edd7
0a95b82
5c45bc9
569e388
7db0b48
e3de0e4
7a41553
00fd832
1ba8b69
f9b6516
b53d848
3da4d04
73d87d5
6f0dccd
830a0bf
6ed90e2
340aced
8f1d6e6
5a81985
7310f74
5b0e1ae
d9b6af7
d4d97d5
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 | ||||||||
|---|---|---|---|---|---|---|---|---|---|---|
|
|
@@ -4,7 +4,6 @@ | |||||||||
|
|
||||||||||
| case ${step} in | ||||||||||
| "prep") | ||||||||||
| # Run on two nodes (requires ~400GB total) | ||||||||||
| tasks_per_node=7 | ||||||||||
| ;; | ||||||||||
|
|
||||||||||
|
|
@@ -21,6 +20,65 @@ case ${step} in | |||||||||
| esac | ||||||||||
| ;; | ||||||||||
|
|
||||||||||
| "eupd") | ||||||||||
| # update ntasks to 80 and threads_per_task to 20 | ||||||||||
| case ${CASE} in | ||||||||||
| "C768") | ||||||||||
| export ntasks=80 | ||||||||||
| export threads_per_task=20 | ||||||||||
| ;; | ||||||||||
| *) | ||||||||||
| ;; | ||||||||||
| esac | ||||||||||
| export tasks_per_node=$(( max_tasks_per_node / threads_per_task )) | ||||||||||
| ;; | ||||||||||
|
|
||||||||||
| "analcalc") | ||||||||||
| # decrease tasks_per_node 127 to 64 | ||||||||||
| case ${CASE} in | ||||||||||
|
|
||||||||||
| "C768") | ||||||||||
| export tasks_per_node=64 | ||||||||||
| ;; | ||||||||||
| *) | ||||||||||
| ;; | ||||||||||
| esac | ||||||||||
| ;; | ||||||||||
|
|
||||||||||
| "upp") | ||||||||||
| # decrease tasks_per_node 120 to 60 | ||||||||||
| case ${CASE} in | ||||||||||
|
|
||||||||||
| "C768") | ||||||||||
| export tasks_per_node=60 | ||||||||||
| ;; | ||||||||||
| *) | ||||||||||
| ;; | ||||||||||
| esac | ||||||||||
| ;; | ||||||||||
|
|
||||||||||
| "fcst") | ||||||||||
| # increase WRTTASK_PER_GROUP_PER_THREAD_PER_TILE_{GDAS,GFS} | ||||||||||
| case ${CASE} in | ||||||||||
| "C768") | ||||||||||
| export WRTTASK_PER_GROUP_PER_THREAD_PER_TILE_GDAS=20 | ||||||||||
| export WRTTASK_PER_GROUP_PER_THREAD_PER_TILE_GFS=25 | ||||||||||
| (( WRTTASK_PER_GROUP_PER_THREAD_GDAS = WRTTASK_PER_GROUP_PER_THREAD_PER_TILE_GDAS * 6 )) | ||||||||||
| (( WRTTASK_PER_GROUP_PER_THREAD_GFS = WRTTASK_PER_GROUP_PER_THREAD_PER_TILE_GFS * 6 )) | ||||||||||
| export WRTTASK_PER_GROUP_PER_THREAD_GDAS | ||||||||||
| export WRTTASK_PER_GROUP_PER_THREAD_GFS | ||||||||||
|
DavidHuber-NOAA marked this conversation as resolved.
|
||||||||||
| (( ntasks_quilt_gdas = WRITE_GROUP_GDAS * WRTTASK_PER_GROUP_PER_THREAD_GDAS )) | ||||||||||
| (( ntasks_quilt_gfs = WRITE_GROUP_GFS * WRTTASK_PER_GROUP_PER_THREAD_GFS )) | ||||||||||
| export ntasks_quilt_gdas | ||||||||||
| export ntasks_quilt_gfs | ||||||||||
|
Comment on lines
+64
to
+71
Contributor
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. @aerorahul Since this is duplicating calculations from config.ufs, I wonder if these should be moved from config.ufs to config.resources OR if we should allow the basic variables (i.e. global-workflow/parm/config/gfs/config.ufs Lines 283 to 286 in a42c833
to export WRITE_GROUP_GDAS=${WRITE_GROUP_GDAS:-2}
export WRTTASK_PER_GROUP_PER_THREAD_PER_TILE_GDAS=${WRTTASK_PER_GROUP_PER_THREAD_PER_TILE_GDAS:-15}
export WRITE_GROUP_GFS=${WRITE_GROUP_GFS:-4}
export WRTTASK_PER_GROUP_PER_THREAD_PER_TILE_GFS=${WRTTASK_PER_GROUP_PER_THREAD_PER_TILE_GFS:-20}Then, instead of re-sourcing config.resources, Alex would re-source config.ufs, then config.resources, while only defining
Contributor
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. @WalterKolczynski-NOAA as well ^^^.
Contributor
Author
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. Morning @WalterKolczynski-NOAA @aerorahul C5 is down probably through tomorrow (Wed) but just wanted to check if you had a chance to look at this. I think the less we have to calculate in config.resources.$MACH the better. Thanks.
Contributor
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 favor the second approach of allowing overrides, but want to hear from @aerorahul
Contributor
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. @DavidHuber-NOAA's recommendation makes sense.
Contributor
Author
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. Thanks @WalterKolczynski-NOAA and @aerorahul. Hello @DavidHuber-NOAA ...My hands are tied with the C5 OS upgrade, but I was looking through this and writing out all the steps..Everything looks good for your second option until the re-sourcing of config.ufs. In config.fcst, config.ufs is sourced via
Contributor
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. @DavidBurrows-NCO Yes, you are correct about sourcing config.fcst -- that's the way to go about it rather than config.ufs. |
||||||||||
| if [[ "${gaea_sourced_resources:-false}" == false ]]; then | ||||||||||
| export gaea_sourced_resources=true | ||||||||||
| source "${EXPDIR}/config.resources" "${step}" | ||||||||||
| fi | ||||||||||
| ;; | ||||||||||
| *) | ||||||||||
| ;; | ||||||||||
| esac | ||||||||||
| ;; | ||||||||||
|
|
||||||||||
| *) | ||||||||||
| ;; | ||||||||||
|
|
||||||||||
|
|
||||||||||
Uh oh!
There was an error while loading. Please reload this page.