Skip to content

Commit

Permalink
Move WCOSS2 LD_LIBRARY_PATH patches to load_ufsda_modules.sh (#3236)
Browse files Browse the repository at this point in the history
This PR moves the `LD_LIBRARY_PATH` patches currently needed to run
GDASApp on WCOSS2 from `WCOSS2.env` to `load_ufsda_modules.sh`. With
this change, the `LD_LIBRARY_PATH` patch is only applied to WCOSS2
GDASApp jobs.

This change was suggested by @aerorahul 

Resolves #3232
  • Loading branch information
RussTreadon-NOAA authored Jan 17, 2025
1 parent 3a5af9a commit 4c6c6a4
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 5 deletions.
5 changes: 0 additions & 5 deletions env/WCOSS2.env
Original file line number Diff line number Diff line change
Expand Up @@ -13,11 +13,6 @@ step=$1
export launcher="mpiexec -l"
export mpmd_opt="--cpu-bind verbose,core cfp"

# TODO: Add path to GDASApp libraries and cray-mpich as temporary patches
# TODO: Remove LD_LIBRARY_PATH lines as soon as permanent solutions are available
export LD_LIBRARY_PATH="${LD_LIBRARY_PATH}:${HOMEgfs}/sorc/gdas.cd/build/lib"
export LD_LIBRARY_PATH="${LD_LIBRARY_PATH}:/opt/cray/pe/mpich/8.1.19/ofi/intel/19.0/lib"

# Calculate common resource variables
# Check first if the dependent variables are set
if [[ -n "${ntasks:-}" && -n "${max_tasks_per_node:-}" && -n "${tasks_per_node:-}" ]]; then
Expand Down
4 changes: 4 additions & 0 deletions ush/load_ufsda_modules.sh
Original file line number Diff line number Diff line change
Expand Up @@ -38,6 +38,10 @@ case "${MACHINE_ID}" in
#TODO: Remove LMOD_TMOD_FIND_FIRST line when spack-stack on WCOSS2
if [[ "${MACHINE_ID}" == "wcoss2" ]]; then
export LMOD_TMOD_FIND_FIRST=yes
# TODO: Add path to GDASApp libraries and cray-mpich as temporary patches
# TODO: Remove LD_LIBRARY_PATH lines as soon as permanent solutions are available
export LD_LIBRARY_PATH="${LD_LIBRARY_PATH}:${HOMEgfs}/sorc/gdas.cd/build/lib"
export LD_LIBRARY_PATH="${LD_LIBRARY_PATH}:/opt/cray/pe/mpich/8.1.19/ofi/intel/19.0/lib"
fi
module load "${MODS}/${MACHINE_ID}"
ncdump=$( command -v ncdump )
Expand Down

0 comments on commit 4c6c6a4

Please sign in to comment.