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

sync with develop (20240923) #3459

Merged
merged 37 commits into from
Sep 24, 2024
Merged

sync with develop (20240923) #3459

merged 37 commits into from
Sep 24, 2024

Conversation

boegel
Copy link
Member

@boegel boegel commented Sep 23, 2024

No description provided.

Flamefire and others added 30 commits April 5, 2024 16:25
…k, there the req_py_majver and friends are guaranteed to be defined
Add two tests for the max_py_* parameters
When no compute capability is set but NVPTX is enabled, trying to figure
out the NVPTX architecture fails with the error:

```
File "easybuild/easyblocks/g/gcc.py", line 431, in map_nvptx_capability
  return sorted_gcc_cc[0]
IndexError: list index out of range
```

The error occurs because of an insufficient check for an unset CUDA
compute capability. This commit changes the checked conditions, so that
empty lists are also correctly handled.

Signed-off-by: Jan André Reuter <j.reuter@fz-juelich.de>
GCC: Fix NVPTX mapping error with no CC set
…ax_python_ver_system_toolchain

allow Python bundles and packages to specify a maximum Python version for the system toolchain
…w_pr_gromacs

Ignore Python from virtualenvs in GROMACS configure via `-DPython3_FIND_VIRTUALENV=STANDARD`
enhance custom easyblock for NCCL: add licence to NCCL installation
@boegel boegel added update EasyBuild-5.0 EasyBuild 5.0 labels Sep 23, 2024
@boegel boegel added this to the 5.0 milestone Sep 23, 2024
Copy link
Contributor

@Micket Micket left a comment

Choose a reason for hiding this comment

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

lgtm

@Micket Micket merged commit a0b0958 into easybuilders:5.0.x Sep 24, 2024
19 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants