Skip to content
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 nccl, cvmfs, cuda-mpi modules, update mpich and gpu modules #32

Merged
merged 6 commits into from
Mar 17, 2023

Conversation

lastephey
Copy link
Collaborator

Adds a podman-hpc version of the current shifter NCCL ss11 module

WIP since it still needs to be fully tested on muller

@lastephey lastephey requested a review from danfulton February 14, 2023 22:23
@lastephey lastephey changed the title WIP: start adding nccl ss11 module WIP: start adding nccl and cvmfs modules Feb 14, 2023
@lastephey lastephey removed the request for review from danfulton February 15, 2023 01:56
@lastephey lastephey force-pushed the lastephey/add-nccl-module branch from 2eae08f to 7275aa1 Compare March 15, 2023 23:37
@lastephey lastephey changed the title WIP: start adding nccl and cvmfs modules add nccl, cvmfs, cuda-mpi modules, update mpich and gpu modules Mar 15, 2023
@lastephey
Copy link
Collaborator Author

Ok should be ready for review @danfulton and @scanon. To work correctly requires #41

Should now match what is in https://gitlab.nersc.gov/nersc/csg/nersc-cle/-/merge_requests/3885

@lastephey lastephey requested review from danfulton and scanon March 15, 2023 23:44
@danfulton
Copy link
Member

Otherwise, LGTM.

etc/modules.d/nccl.yaml Show resolved Hide resolved
@danfulton
Copy link
Member

LGTM

@danfulton danfulton closed this Mar 17, 2023
@danfulton danfulton reopened this Mar 17, 2023
@danfulton
Copy link
Member

Oops, I meant to close my review not the PR... sorry.

@danfulton danfulton merged commit c3362cc into main Mar 17, 2023
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.

3 participants