{numlib}[GCC/12.3.0] OpenBLAS v0.3.23#18012
Conversation
|
@boegelbot please test @ generoso |
|
@bartoldeman: Request for testing this PR well received on login1 PR test command '
Test results coming soon (I hope)... Details- notification for comment with ID 1573700931 processed Message to humans: this is just bookkeeping information for me, |
This comment was marked as outdated.
This comment was marked as outdated.
|
Test report by @bartoldeman |
|
Test report by @branfosj |
|
@boegelbot please test @ jsc-zen2 |
Updates by the bot instance
|
|
@branfosj: Request for testing this PR well received on jsczen2l1.int.jsc-zen2.easybuild-test.cluster PR test command '
Test results coming soon (I hope)... Details- notification for comment with ID 1574776325 processed Message to humans: this is just bookkeeping information for me, |
|
Test report by @branfosj |
|
Test report by @boegel |
|
Test report by @boegel |
|
More info on the failing test on and from Let's not block the merging of this PR over that, since this looks specific to I have no idea if @bartoldeman Do you happen have any insight here? I'm hoping you're up for reporting this upstream, since you have more experience with OpenBLAS |
|
@boegelbot please test @ generoso |
|
@boegel: Request for testing this PR well received on login1 PR test command '
Test results coming soon (I hope)... Details- notification for comment with ID 1574983668 processed Message to humans: this is just bookkeeping information for me, |
|
Test report by @boegelbot |
|
Test report by @boegel |
|
Test report by @boegel |
|
Test report by @boegel |
|
Hmm, on our Intel Haswell + AMD Rome & Milan systems, I consistently get this a double precision numerical error: Problem on In all these cases, the updated OpenBLAS from easybuilders/easybuild-easyblocks#2944 was used. |
@branfosj I'm confused, how can this be a successful test report if there are numerical errors? |
We allow up to 150 failing numerical errors: But 0 other errors. See also the code in https://github.com/easybuilders/easybuild-easyblocks/blob/develop/easybuild/easyblocks/o/openblas.py#L96 |
|
ok, it seems to fail on avx2 arches (Haswell, zen2, zen2), but not avx512 *lake. Let me try if I can reproduce that. |
|
I've reproduced it locally, will try to debug later. |
|
so it seems that one test was sensitive to using FMA or not; the matrix is nearly singular and was failing a threshold during Gauss elimination with FMA, but not without FMA in daxpy. Just reordering the order of test matrices makes the matrix better behaved and it passes. I'll have to take this up with upstream LAPACK to have a better grip on things. |
|
Test report by @branfosj |
|
@boegelbot please test @ generoso |
|
@bartoldeman: Request for testing this PR well received on login1 PR test command '
Test results coming soon (I hope)... Details- notification for comment with ID 1576772585 processed Message to humans: this is just bookkeeping information for me, |
|
Test report by @boegelbot |
|
Test report by @branfosj |
|
@akesandgren mentioned some testing routines may be overoptimized.. I'll check if changing that may fix it as well. |
|
I tried compiling everything in MATGEN and EIG with -O0, makes no difference, workaround is still necessary, so easiest to just stick with what we have now. |
|
Test report by @casparvl |
|
Test report by @boegel |
|
@boegelbot please test @ generoso |
|
@bartoldeman: Request for testing this PR well received on login1 PR test command '
Test results coming soon (I hope)... Details- notification for comment with ID 1580906608 processed Message to humans: this is just bookkeeping information for me, |
|
Test report by @boegelbot |
|
Test report by @bartoldeman |
|
Test report by @boegel |
|
Test report by @boegel |
|
Test report by @boegel |
|
Test report by @boegel |
|
Test report by @boegel |
|
Test report by @boegel |
|
@bartoldeman Trouble on POWER9 (emulated) now, apparently, while it worked fine before (cfr. #18012 (comment)). I won't this block the merge, if it's worth following up, let's open an issue or follow-up PR... |
|
Going in, thanks @bartoldeman! |
|
POWER fails in the same place as Haswell without the patch, it's odd indeed. |
|
Test report by @boegel |
|
@Flamefire Any thoughts on the failing test on POWER? (if we should follow up on this, please open an issue) |
|
Failing POWER test fixed in #19495 |
(created using
eb --new-pr)depends on:
make sharedis necessary and sufficient with OpenBLAS 0.3.23 + recent parallel build fixes easybuild-easyblocks#2944fixes #18017