Skip to content
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

Restructure the bufr sounding job #2853

Merged
merged 11 commits into from
Sep 7, 2024
2 changes: 1 addition & 1 deletion env/WCOSS2.env
Original file line number Diff line number Diff line change
Expand Up @@ -223,7 +223,7 @@ elif [[ "${step}" = "postsnd" ]]; then
export OMP_NUM_THREADS=1

export NTHREADS_POSTSND=${NTHREADS1}
export APRUN_POSTSND="${APRUN} --depth=${NTHREADS_POSTSND} --cpu-bind depth"
export mpmd_opt="-ppn 21 ${mpmd_opt}"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@BoCui-NOAA @WalterKolczynski-NOAA Was it intentional to get rid of APRUN_POSTSND? This is referenced by ush/gfs_bufr_netcdf.sh.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, the 'APRUN_POSTSND' setting is no longer needed, and the ush/gfs_bufr_netcdf.sh script is no longer in use.


export NTHREADS_POSTSNDCFP=${threads_per_task_postsndcfp:-1}
[[ ${NTHREADS_POSTSNDCFP} -gt ${max_threads_per_task} ]] && export NTHREADS_POSTSNDCFP=${max_threads_per_task}
Expand Down
6 changes: 3 additions & 3 deletions parm/config/gfs/config.resources
Original file line number Diff line number Diff line change
Expand Up @@ -1094,9 +1094,9 @@ case ${step} in

"postsnd")
walltime="02:00:00"
ntasks=40
threads_per_task=8
tasks_per_node=10
export ntasks=141
threads_per_task=6
export tasks_per_node=21
export ntasks_postsndcfp=9
export tasks_per_node_postsndcfp=1
postsnd_req_cores=$(( tasks_per_node * threads_per_task ))
Expand Down
File renamed without changes.
File renamed without changes.
Loading
Loading