-
Notifications
You must be signed in to change notification settings - Fork 64
{2023.06}[2023b] RAxML 8.2.13 #1196
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
base: main
Are you sure you want to change the base?
Conversation
PetrKralCZ
commented
Sep 18, 2025
bot: build repo:eessi.io-2023.06-software instance:eessi-bot-mc-aws for:arch=x86_64/amd/zen2 |
Unable to download or merge changes between the source branch and the destination branch. |
1 similar comment
Unable to download or merge changes between the source branch and the destination branch. |
@PetrKralCZ please fix the merge conflict |
bot: build repo:eessi.io-2023.06-software instance:eessi-bot-mc-aws for:arch=x86_64/amd/zen2 |
New job on instance
|
New job on instance
|
This needs special attention for Arm it seems:
|
options: | ||
# see https://github.com/easybuilders/easybuild-easyconfigs/pull/22371 | ||
from-commit: 29f82e43229663c22f0c76cb3fc7b6dd5c407cd7 | ||
- RAxML-8.2.13-gompi-2023b-avx2.eb |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is not going to work on systems that don't have AVX2, you need to create a variant of https://github.com/easybuilders/easybuild-easyconfigs/blob/develop/easybuild/easyconfigs/r/RAxML/RAxML-8.2.13-gompi-2023a-standard.eb with a newer toolchain
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actually, this is a case where I think we should diverge from the "same modules everywhere" goal.
The AVX2 variant of RAxML is definitely worth installing for x86_64
CPUs, but clearly doesn't make any sense for aarch64
.
So we should install "fake" modules for the latter, and get them to produce a meaningful error to explain why that installation is not available on aarch64
targets?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think we need to make this exception, we need a better easyconfig. For example, https://github.com/stamatak/standard-RAxML/blob/master/Makefile.HYBRID.gcc has the mechanism to support both x86 and Arm, we should really be using that.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PR for this particular case in easybuilders/easybuild-easyconfigs#23975
RAxML does seem to have some intrinsics available, but the specific code targeted Haswell and hasn't been touched in 10 years, I think a native CPU compilation is a reasonable option
This should be closed RAxML was deployed with this pr #1032 |
This is |
Discussion on how to handle this and similar cases see https://gitlab.com/eessi/support/-/issues/196 |
PR opened to make a more universal version of RAxML: easybuilders/easybuild-easyconfigs#23975 |