Skip to content

Commit

Permalink
fix(runlocal): only check slurm job id on biowulf and frce
Browse files Browse the repository at this point in the history
to enforce interactive node only
  • Loading branch information
kelly-sovacool committed Jan 2, 2025
1 parent a89b1ff commit 191b11e
Showing 1 changed file with 6 additions and 2 deletions.
8 changes: 6 additions & 2 deletions charlie
Original file line number Diff line number Diff line change
Expand Up @@ -338,8 +338,6 @@ function printbinds(){

function runlocal() {
runcheck
# TODO do not assume $SLURM_JOB_ID exists, may run on other platform without slurm e.g. eddie
if [ "$SLURM_JOB_ID" == "" ];then err "runlocal can only be done on an interactive node"; exit 1; fi
run "local"
}

Expand Down Expand Up @@ -431,6 +429,12 @@ function run() {

if [ "$1" == "local" ];then

if [ "$PLATFORM" == "biowulf" ] || [ "$PLATFORM" == "fnlcr"]; then
if [ "$SLURM_JOB_ID" == "" ]; then
err "runlocal can only be done on an interactive node";
exit 1;
fi
fi
preruncleanup

$EXPORT_SING_CACHE_DIR_CMD
Expand Down

0 comments on commit 191b11e

Please sign in to comment.