Skip to content

Conversation

@casparvl
Copy link
Collaborator

@casparvl casparvl commented Jun 4, 2024

Currently, the host_injections directory was introduced as part of the Advanced Usage => Accelerator support => GPUs. I think we better introduce the concept of host_injections being a location that can be used for site-specific EESSI configuration first. Since this is about configuration, it makes sense to me to do this as part of the Installation instructions. I also imagine that the Installation chapter would be the only chapter that sysadmins read, so in my opinion it should contain everything needed to install and configure EESSI to it's full functionality (including GPU support, MPI tuning, etc).

I've also added new documentation on how to write Lmod hooks in host_injections. This basically provides the documentation for the functionality added in this PR.

@casparvl casparvl requested a review from xinan1911 July 1, 2024 13:32
Copy link
Contributor

@xinan1911 xinan1911 left a comment

Choose a reason for hiding this comment

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

Looks good to me!

@xinan1911 xinan1911 merged commit 7cc0b4a into EESSI:main Jul 2, 2024
@xinan1911
Copy link
Contributor

Merged and closed.

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