Skip to content

avoid test failure on AVX512 systems by using -O2 instead of -O3 for CASTEP 24.1 + 25.1.2#24378

Merged
smoors merged 2 commits intoeasybuilders:developfrom
branfosj:20251027152333_new_pr_CASTEP2512
Oct 30, 2025
Merged

avoid test failure on AVX512 systems by using -O2 instead of -O3 for CASTEP 24.1 + 25.1.2#24378
smoors merged 2 commits intoeasybuilders:developfrom
branfosj:20251027152333_new_pr_CASTEP2512

Conversation

@branfosj
Copy link
Copy Markdown
Member

@branfosj branfosj commented Oct 27, 2025 •
edited
Loading

(created using eb --new-pr)

see #24377

@github-actions github-actions Bot added change manual_download Sources require manual download labels Oct 27, 2025
@branfosj branfosj added bug fix and removed change labels Oct 27, 2025
@Thyre Thyre added 2023b 2024a issues & PRs related to 2024a common toolchains labels Oct 27, 2025
branfosj
branfosj commented Oct 27, 2025
View reviewed changes
Comment thread easybuild/easyconfigs/c/CASTEP/CASTEP-24.1-foss-2023b.eb Outdated
Micket
Micket approved these changes Oct 27, 2025
View reviewed changes
Copy link
Copy Markdown
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
Copy link
Copy Markdown
Contributor

Micket commented Oct 28, 2025

test reports please?

@branfosj
Copy link
Copy Markdown
Member Author

branfosj commented Oct 28, 2025

test reports please?

I can test report CASTEP-24.1-foss-2023b.eb but I do not have the source for CASTEP-25.1.2-foss-2024a.eb. I think @smoors has the sources and hopefully can test report.

My test report will be tomorrow - we have maintenance today.

@smoors
Copy link
Copy Markdown
Contributor

smoors commented Oct 28, 2025

Test report by @smoors
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
node600.hydra.os - Linux Rocky Linux 9.6, x86_64, AMD EPYC 9384X 32-Core Processor (zen4), Python 3.9.21
See https://gist.github.com/smoors/26aa302974cedfb224f89c8c4f135715 for a full test report.

@branfosj
Copy link
Copy Markdown
Member Author

branfosj commented Oct 29, 2025

Test report by @branfosj
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
bear-pg0105u03b - Linux RHEL 8.10, x86_64, Intel(R) Xeon(R) Platinum 8360Y CPU @ 2.40GHz (icelake), Python 3.6.8
See https://gist.github.com/branfosj/6c5c1406014782ebe00e6f36c26ffcda for a full test report.

@smoors smoors added this to the next release (5.2.0?) milestone Oct 30, 2025
smoors
smoors approved these changes Oct 30, 2025
View reviewed changes
Copy link
Copy Markdown
Contributor

@smoors smoors left a comment

Choose a reason for hiding this comment

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

lgtm

@smoors
Copy link
Copy Markdown
Contributor

smoors commented Oct 30, 2025

Going in, thanks @branfosj!

@smoors smoors merged commit 440c3bc into easybuilders:develop Oct 30, 2025
8 checks passed
@branfosj branfosj deleted the 20251027152333_new_pr_CASTEP2512 branch October 30, 2025 15:58
@boegel boegel changed the title avoid test failure on AVX512 system in CASTEP avoid test failure on AVX512 systems by using -O2 instead of -O3 for CASTEP 24.1 + 25.1.2 Dec 19, 2025
@boegel boegel removed the change label Dec 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Micket Micket Micket approved these changes

@smoors smoors smoors approved these changes

Assignees

No one assigned

Labels

2023b 2024a issues & PRs related to 2024a common toolchains bug fix manual_download Sources require manual download

Projects

None yet

Milestone

5.2.0

Development

Successfully merging this pull request may close these issues.

5 participants

@branfosj @Micket @smoors @boegel @Thyre