Skip to content

Conversation

@mattip
Copy link
Member

@mattip mattip commented Nov 3, 2025

Fixes #30092.

Rather than update the OpenBLAS version, this PR uses a patched version of OpenBLAS that incorporate the fix from OpenMathLib/OpenBLAS#5520. That should minimize the risk in updating the dependency so close to the new release cut-off date.

@github-actions github-actions bot added the 36 - Build Build related PR label Nov 3, 2025
@mattip mattip added 09 - Backport-Candidate PRs tagged should be backported 36 - Build Build related PR and removed 36 - Build Build related PR labels Nov 3, 2025
@mattip
Copy link
Member Author

mattip commented Nov 3, 2025

@FreddieWitherden could you try this out on your system?

Edit: it should be enough to update scipy-openblas64 on the reproducer from #30092, since that seems to be a local build and not a PyPI install.

@FreddieWitherden
Copy link

@FreddieWitherden could you try this out on your system?

Edit: it should be enough to update scipy-openblas64 on the reproducer from #30092, since that seems to be a local build and not a PyPI install.

Will action this evening and get back to you.

@charris
Copy link
Member

charris commented Nov 3, 2025

Note that this will also be needed in the numpy-release repo which currently uses 0.3.30.0.2. The maintenance branch also builds onn macos-13, which I think should be good through Dec.

@andyfaff
Copy link
Member

andyfaff commented Nov 3, 2025

I think the macos-13 limit is fully retired by Dec 4th.

@FreddieWitherden
Copy link

@FreddieWitherden could you try this out on your system?

Edit: it should be enough to update scipy-openblas64 on the reproducer from #30092, since that seems to be a local build and not a PyPI install.

I can confirm this resolves the deadlock issue.

Regards, Freddie.

@mattip
Copy link
Member Author

mattip commented Nov 4, 2025

Note that this will also be needed in the numpy-release repo

I opened numpy/numpy-release#21 to simplify that.

Copy link
Member

@rgommers rgommers left a comment

Choose a reason for hiding this comment

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

@rgommers rgommers merged commit 0fdb197 into numpy:main Nov 4, 2025
76 checks passed
@rgommers rgommers added this to the 2.4.0 release milestone Nov 4, 2025
charris pushed a commit to charris/numpy that referenced this pull request Nov 5, 2025
@charris charris removed the 09 - Backport-Candidate PRs tagged should be backported label Nov 5, 2025
charris added a commit that referenced this pull request Nov 5, 2025
BLD: use scipy-openblas 0.3.30.7 (#30132)
cakedev0 pushed a commit to cakedev0/numpy that referenced this pull request Dec 5, 2025
IndifferentArea pushed a commit to IndifferentArea/numpy that referenced this pull request Dec 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

36 - Build Build related PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

BUG: Binary Builds Deadlock due to OpenBLAS threading issue with fork

5 participants