Skip to content

MAINT: update openblas to 0.3.33.112.0 - #31649

Merged
ngoldbaum merged 5 commits into
numpy:mainfrom
mattip:openblas-0.3.33.112
Jun 16, 2026
Merged

ngoldbaum merged 5 commits into
numpy:mainfrom
mattip:openblas-0.3.33.112

Conversation

@mattip

@mattip mattip commented Jun 15, 2026

Copy link
Copy Markdown
Member

Fixes #31618 and maybe #29442

Will need a parallel PR to numpy-release, but I think the CI tests here are more extensive, so the two should be looked at together.

@mattip

mattip commented Jun 16, 2026

Copy link
Copy Markdown
Member Author

ppc64 and s390x are failing the test_blas_gemm_thread_safety test.

@ngoldbaum

Copy link
Copy Markdown
Member

For both the ppc64le and s390x builds:

Called: `/usr/bin/pkg-config --modversion openblas` -> 0
stdout:
0.3.26

So the test I added probably needs to be xfailed if OpenBLAS is too old.

if mismatches and _openblas_predates_gemm_fix(blas_name, blas_version):
pytest.xfail(
f"OpenBLAS version ({blas_version}) predates first OpenBLAS "
"version with a fix (0.3.33.112)"

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

this isn't a real upstream version but I can't think of situation where this would be incorrect and where we wouldn't want to know about it

@ngoldbaum

Copy link
Copy Markdown
Member

Looks like this is working. @mattip is the way I spelled the test skip OK with you?

@mattip

mattip commented Jun 16, 2026

Copy link
Copy Markdown
Member Author

Ahh right, i dont build those versions of scipy-openblas

All good

@ngoldbaum

Copy link
Copy Markdown
Member

All good

Alright, in it goes. Thanks Matti!

@ngoldbaum
ngoldbaum merged commit 64131f2 into numpy:main Jun 16, 2026
83 of 85 checks passed
@charris

charris commented Jun 20, 2026

Copy link
Copy Markdown
Member

@mattip Should this be backported to 2.5.x, or are we good to go there.

@ngoldbaum

Copy link
Copy Markdown
Member

This should be backported, IMO

@charris charris added this to the 2.5.0 Release milestone Jun 20, 2026
@charris charris removed the 08 - Backport Used to tag backport PRs label Jun 20, 2026
@charris charris removed this from the 2.5.0 Release milestone Jun 20, 2026
@charris

charris commented Jun 20, 2026

Copy link
Copy Markdown
Member

I put up #31704 with the backport. If yes, just merge it.

charris pushed a commit that referenced this pull request Jun 21, 2026
charris added a commit that referenced this pull request Jun 21, 2026
MAINT: update openblas to 0.3.33.112.0 (#31649)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

BUG: np.dot is thread-unsafe for non-contiguous input with OpenBLAS

3 participants