Repository navigation
Conversation
The experiments on the numpy.linalg crashes on Windows on ARM (see #1003) run in their own pull requests. Remove the Azure pipeline and the wheel builds, the workflows that run the tests, so that these pull requests do not use the CI queues. Not for merging.
Maps the sizes at which numpy.linalg calls, SciPy's LAPACK and BLAS, and the LAPACK and BLAS routines of numpy's OpenBLAS (called directly with ctypes) crash. A child process walks through the sizes, and after a crash or a hang a new child continues with the next size, so a map takes minutes rather than a process for every size. Each size that crashed is then tried alone. Options set the OpenBLAS threads in the environment or at run time, the stack of the thread that makes the calls, and faulthandler. Not for merging.
Maps the sizes at which inv, solve, det, cholesky, qr, eigh, svd, lstsq and matmul crash, with OPENBLAS_NUM_THREADS 1, 2 and 4 and unset. Not for merging.
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
|
|
||
| result["scipy"] = scipy.__version__ | ||
| result["scipy openblas"] = runtime("scipy") | ||
| except ImportError: |
With 2 threads, inv and solve hang at 23 and 24 rather than crash, and the map skipped them after two hangs. Map 2 and 3 threads over sizes 1 to 40 with a 15 s timeout, going on after every hang. Add --timeout and --max-hangs, put a hang between calls on the next size, and list the other outcomes as ranges. Not for merging.
bashtage
added a commit
that referenced
this pull request
Nov 4, 2026
numpy.linalg.inv and solve crash in the OpenBLAS of numpy's win_arm64 wheels only when OpenBLAS starts with 1 thread (see #1004 to #1007), so do not set OPENBLAS_NUM_THREADS, OMP_NUM_THREADS or MKL_NUM_THREADS, and fail if any thread limit is set or OpenBLAS reports fewer than 2 threads. Run the tests that the other CI runs with pytest-xdist, then the slow tests, on Python 3.13 and 3.14 with numba and 3.15 without it (no win_arm64 wheels). statsmodels has no win_arm64 wheels for 3.12. Remove the crash isolation, which moved to #1004 to #1007.
This was referenced Nov 4, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Experimental, not for merging. One of four experiments on the access violations in
numpy.linalgon Windows on ARM that #1003 found. WithOPENBLAS_NUM_THREADS=1,invandsolveof small positive definite matrices crash in numpy 2.5.4's OpenBLAS, and numpy alone reproduces it. Each experiment has its own pull request, so that they run at the same time and each can change without restarting the others: #1003 (arch's tests), #1004 (the calls and threads), #1005 (the numpy versions), #1006 (the OpenBLAS routines) and #1007 (SciPy, the stack and faulthandler).Like #1003, this removes the Azure pipeline (
azure-pipelines.yml) and the wheel builds (cron-test.yml), so that no tests run here. arch is not built: the job installs only numpy (and SciPy where needed) from wheels.What this one runs
Every
numpy.linalgcall, to find which crash and at which sizes:invof the identity, andinv,solve(one and three right-hand sides),det,cholesky,qr,eigh,svd,lstsqandmatmul(matrix and vector) of random positive definite matrices. One job for eachOPENBLAS_NUM_THREADS: 1, 2, 4 and unset (OpenBLAS picks). numpy 2.5.4, Python 3.14.selftestcrashes at size 5 on purpose, to show that crashes are found on ARM.How it works
ci/github-actions/winarm_blas.py, the same in all four pull requests, makes every call in a child process. A child walks through the sizes (1 to 64, then every 8th to 160) and printsstart CALL Nbefore each call. When it crashes or hangs (60 s), the size is recorded and a new child goes on with the next size. Each size that crashed is then tried alone, in a new process. So a map takes minutes rather than a process for every size, which is what made the size map in #1003 so slow. The matrices are built without BLAS (einsum), so the call under test is the only BLAS call.The job passes when the map is complete, whether or not anything crashed. The results are in the job summary, in the log as they come, and in the
report.mdartifact. Each report starts with the builds and what OpenBLAS reports at run time (kernel, threads, configuration) and the CPU.Checked locally (x64)
actionlintandcheck-jsonschema(GitHub workflow schema) report no problems. The script is clean under ruff, black, isort, flake8, bandit and pylint.selftestcrashes at size 5 and is found, the walk goes on, and the crash is found again alone.🤖 Generated with Claude Code