Skip to content

Remove redundant variable already set by hpc-launcher#21

Merged
michaelmckinsey1 merged 2 commits intoLBANN:mainfrom
michaelmckinsey1:cleanup-vars
Mar 5, 2026
Merged

Remove redundant variable already set by hpc-launcher#21
michaelmckinsey1 merged 2 commits intoLBANN:mainfrom
michaelmckinsey1:cleanup-vars

Conversation

@michaelmckinsey1
Copy link
Collaborator

@michaelmckinsey1 michaelmckinsey1 self-assigned this Mar 5, 2026
@michaelmckinsey1 michaelmckinsey1 changed the title Remove redundant variable set by hpc-launcher Remove redundant variable already set by hpc-launcher Mar 5, 2026
fi
done No newline at end of file
done
cd -
Copy link
Collaborator

@PatrickRMiles PatrickRMiles Mar 5, 2026

Choose a reason for hiding this comment

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

is this cd - necessary?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

It gets the user back to the directory they were in after running the script. otherwise they will end up in .venvs/scaffoldvenv-tuo/lib/python3.11/site-packages/torch/lib

@michaelmckinsey1 michaelmckinsey1 merged commit 124ff46 into LBANN:main Mar 5, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants