-
-
Notifications
You must be signed in to change notification settings - Fork 10.8k
TST: Azure testing with NumPy #12036
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
Looks like dgelsd is the problem. I'd really like to see windows testing, appveyor is a real bottleneck for us. |
+10 for Azure Pipelines. Discussed it with quite a few people, and Steve Dower gave an overview of features, MS's motivations, and what's coming next. Everyone was very positive and enthusiastic about it. |
I added Windows / Python 3.6 full tests to matrix; 1 error and a few failures; worse if I try more than 1 core for testing; tempfile stuff and a few other things. I'll try to sort them out. |
|
I've been able to reduce test skips on the azure mac os image now: But the failures look real and are also reported for macos 10.13 recently for macports. |
One user tracked down the source line in Numpy -- I'll see if I can debug that if nobody beats me to it; there's a comment mentioning |
What version of Lapack is being tested there? That comment was introduced by me in 92f1d12. I think there are all sorts of bugs in the lapack workspace calculations, which is what motivated my upgrade of lapack_lite a while back. I think the situation now is that OSX ships with a more broken lapack than the one bundled with numpy. |
Can we make the Mac build with the NumPy fallback code? I know we can on Linux, but have never tried on Mac. |
I'll have to check re: lapack version -- the show_config style commands don't always make this stuff obvious to me. What is meant by "fallback code?" Are there any low-level hooks that can be used in C source files to probe LAPACK version during pre-processing / guard the logic for handling the bugs? I see online that |
IIRC, on the Mac use of accelerate/veclib is hardwired in, see
which disables linking to the external libraries and uses unoptimized numpy code instead. |
Support for also ACCELERATE=None was iirc added recently.
…On September 28, 2018 3:31:28 AM UTC, Charles Harris ***@***.***> wrote:
IIRC, on the Mac use of accelerate/veclib is hardwired in, see
`numpy/distutils/system_info.py`. On other platforms one can do
```
BLAS=None LAPACK=None python setup.py build
```
which disables linking to the external libraries and uses unoptimized
numpy code instead.
|
Can we close this? |
Yeah, the broader setup effort has been completed; some more challenging refinements may remain for i.e., MacOS |
This was discussed a bit on the mailing list--I've done a little testing on my fork of NumPy.
A sample PR shows the github integration in action for a MacOS job. The
numpy_azure
branch contains the.yml
file, and the PR is opened against that.MacOS tests don't all pass there yet. Looks like it took 3 minutes, 17 seconds for the full run.
The text was updated successfully, but these errors were encountered: