Make HPL easyblock intel-aware + allow running the xhpl benchmark as a test#3917
Make HPL easyblock intel-aware + allow running the xhpl benchmark as a test#3917boegel merged 6 commits intoeasybuilders:developfrom
xhpl benchmark as a test#3917Conversation
|
@boegelbot please test @ jsc-zen3 |
|
@Crivella: Request for testing this PR well received on jsczen3l1.int.jsc-zen3.fz-juelich.de PR test command '
Test results coming soon (I hope)... Details- notification for comment with ID 3258815718 processed Message to humans: this is just bookkeeping information for me, |
|
Test report by @boegelbot Overview of tested easyconfigs (in order)
Build succeeded for 18 out of 18 (10 easyconfigs in total) |
|
Test report by @Crivella Overview of tested easyconfigs (in order)
Build succeeded for 11 out of 11 (11 easyconfigs in total) |
Thyre
left a comment
There was a problem hiding this comment.
LGTM.
Let me throw together an Intel + OpenMPI build real quick to also test that, but I would expect to see this succeed as well.
I guess you've tested the residual test check before implementing the configure option? It's hard to test this when we have CCNOOPT set 😄
I had done a test locally by commenting that section and the intel ones would fail I am also doing 2 tests on a single HPL with |
|
Test report by @Thyre Overview of tested easyconfigs (in order)
Build succeeded for 1 out of 1 (1 easyconfigs in total) |
|
Test report by @Thyre Overview of tested easyconfigs (in order)
Build succeeded for 3 out of 3 (3 easyconfigs in total) |
|
Hm, we get this warning during the test step: Should we use |
|
Ran with Test report by @Crivella Overview of tested easyconfigs (in order)
Build succeeded for 1 out of 1 (1 easyconfigs in total) |
|
Ran with Test report by @Crivella Overview of tested easyconfigs (in order)
Build succeeded for 1 out of 1 (1 easyconfigs in total) |
|
Thanks @ocaisa for the suggestion of using Gonna try and implement it here |
|
@boegelbot please test @ jsc-zen3 |
|
Concerning testing without the 2025b succeeds because we have the flag in the EC right now |
|
@Crivella: Request for testing this PR well received on jsczen3l1.int.jsc-zen3.fz-juelich.de PR test command '
Test results coming soon (I hope)... Details- notification for comment with ID 3265449064 processed Message to humans: this is just bookkeeping information for me, |
|
Test report by @boegelbot Overview of tested easyconfigs (in order)
Build succeeded for 18 out of 18 (10 easyconfigs in total) |
xhpl benchmark as a testsxhpl benchmark as a tests
xhpl benchmark as a testsxhpl benchmark as a test
|
Test report by @boegel Overview of tested easyconfigs (in order)
Build succeeded for 10 out of 10 (10 easyconfigs in total) |
xhpl benchmark as a testxhpl benchmark as a test
Potential fix for
I think we could also not run the benchmark as a a test instead of pinning everything to 1 core (if we do not wanna deal with pinning with every different type of MPI)