Skip to content

set $NVHPC_CUDA_HOME for NVHPC 22.7+#2776

Merged
branfosj merged 2 commits intoeasybuilders:developfrom
SebastianAchilles:20220818140209_new_pr_nvhpc
Aug 19, 2022
Merged

set $NVHPC_CUDA_HOME for NVHPC 22.7+#2776
branfosj merged 2 commits intoeasybuilders:developfrom
SebastianAchilles:20220818140209_new_pr_nvhpc

Conversation

@SebastianAchilles
Copy link
Copy Markdown
Member

@SebastianAchilles SebastianAchilles commented Aug 18, 2022

(created using eb --new-pr)

NVHPC 22.7+ requires NVHPC_CUDA_HOME for external CUDA. CUDA_HOME has been deprecated.

$ nvc -v
nvc-Warning-CUDA_HOME has been deprecated. Please, use NVHPC_CUDA_HOME instead.
Export NVCOMPILER=/p/project/cexalab/easybuild/juwelsbooster/software/NVHPC/22.7/Linux_x86_64/22.7
Export PGI=/p/project/cexalab/easybuild/juwelsbooster/software/NVHPC/22.7
nvc-Warning-No files to process

$ NVHPC_CUDA_HOME=$CUDA_HOME nvc -v
Export NVCOMPILER=/p/project/cexalab/easybuild/juwelsbooster/software/NVHPC/22.7/Linux_x86_64/22.7
Export PGI=/p/project/cexalab/easybuild/juwelsbooster/software/NVHPC/22.7
nvc-Warning-No files to process

CC: @AndiH

@SebastianAchilles SebastianAchilles added this to the 4.x milestone Aug 18, 2022
branfosj
branfosj approved these changes Aug 19, 2022
View reviewed changes
Copy link
Copy Markdown
Member

@branfosj branfosj left a comment

Choose a reason for hiding this comment

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

lgtm

@branfosj
Copy link
Copy Markdown
Member

branfosj commented Aug 19, 2022

Test report by @branfosj

Overview of tested easyconfigs (in order)

  • SUCCESS NVHPC-22.3-CUDA-11.4.1.eb
  • SUCCESS NVHPC-22.7-CUDA-11.4.1.eb

Build succeeded for 2 out of 2 (2 easyconfigs in total)
bear-pg0103u01a.bear.cluster - Linux RHEL 8.6, x86_64, Intel(R) Xeon(R) Gold 6330 CPU @ 2.00GHz (icelake), 1 x NVIDIA NVIDIA A100-PCIE-40GB, 470.57.02, Python 3.6.8
See https://gist.github.com/ef63e6168dc297b22e0fe245876b9017 for a full test report.

@branfosj
Copy link
Copy Markdown
Member

branfosj commented Aug 19, 2022

Going in, thanks @SebastianAchilles!

@branfosj branfosj merged commit 5aafaec into easybuilders:develop Aug 19, 2022
@branfosj branfosj modified the milestones: 4.x, next release (4.6.1?) Aug 19, 2022
@boegel boegel changed the title Set NVHPC_CUDA_HOME for NVHPC 22.7+ set $NVHPC_CUDA_HOME for NVHPC 22.7+ Aug 25, 2022
@pauleonix pauleonix mentioned this pull request Oct 24, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@branfosj branfosj branfosj approved these changes

Assignees

No one assigned

Labels

enhancement

Projects

None yet

Milestone

4.6.1

Development

Successfully merging this pull request may close these issues.

2 participants

@SebastianAchilles @branfosj