Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
Add new init scripts for new initialization module #668
Add new init scripts for new initialization module #668
Changes from 5 commits
d512675
7f71c28
1e2f154
a0c4377
68aea9f
788375d
b14cef8
1aa8cb8
0b6bc54
4aa7d00
71543e3
ea32ffe
bb3c8bd
385aee0
461c941
aee306c
6d7fde5
fcf4937
032cdae
33b6b92
d4bc3bc
63482f5
7f95d5e
a2ce628
2ab75c3
7c798d4
8d74aea
596f605
462f180
68ad3af
bccd55a
7ad306a
1b38cec
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The module load should not be necessary, they should be loaded when Lmod is initialised
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not 100% though...https://lmod.readthedocs.io/en/latest/070_standard_modules.html#providing-a-standard-set-of-modules-for-all-users
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In my test cases the
EESSI/2023.06
module was not loaded automatically, that why I put it in.Just setting
StdEnv.lua
doesn't do the trick also.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Was there already an Lmod loaded when you
source
d it?module load $LMOD_SYSTEM_DEFAULT_MODULES
is a little brittle as this won't work if there is more than module in the variable (I think). It's probably best to take the official route as per the Lmod docs (shell dependent):There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No, Lmod was not loaded at that moment. Only loaded with the script.