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 cuda_cc_space_sep template that does not have periods #4583

Merged
merged 8 commits into from
Jul 26, 2024

Conversation

casparvl
Copy link
Contributor

E.g. '80 90' if cuda compute capabilities is '8.0,9.0'

Can be used in e.g. Cuda-Samples as value to SMS argument, see the readme at https://github.com/NVIDIA/cuda-samples

Caspar van Leeuwen and others added 4 commits July 22, 2024 16:34
@branfosj branfosj added this to the release after 4.9.2 milestone Jul 26, 2024
@branfosj branfosj merged commit fbf5d39 into easybuilders:develop Jul 26, 2024
37 checks passed
@boegel boegel changed the title Add cuda_cc_space_sep variant that does not have periods Add cuda_cc_space_sep template that does not have periods Jul 31, 2024
@@ -94,9 +94,11 @@
"as specify by the --sysroot configuration option"),
('mpi_cmd_prefix', "Prefix command for running MPI programs (with default number of ranks)"),
('cuda_compute_capabilities', "Comma-separated list of CUDA compute capabilities, as specified via "
"--cuda-compute-capabilities configuration option or via cuda_compute_capabilities easyconfig parameter"),
"--cuda-compute-capabilities configuration option or via cuda_cc easyconfig parameter"),
Copy link
Member

Choose a reason for hiding this comment

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

@casparvl Maybe I'm missing something, but there's no cuda_cc easyconfig parameter in EasyBuild 4.x (nor in the 5.0.x branch), so this change is incorrect?

Copy link
Member

Choose a reason for hiding this comment

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

being fixed in #4589

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants