-
-
Notifications
You must be signed in to change notification settings - Fork 11.7k
CI: removes azure pipelines #30232
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
base: main
Are you sure you want to change the base?
CI: removes azure pipelines #30232
Conversation
Co-authored-by: Matti Picus <[email protected]>
Co-authored-by: Matti Picus <[email protected]>
|
The linux pypy run failed, something about underflow in a check of the |
|
With gh-30222 merged, this one can be updated now. |
rgommers
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @andyfaff. This is almost ready, just one issue with BLAS detection.
The job is also quite a bit faster; runtime goes from 13 to 8 minutes, which is nice.
| spin config-openblas --with-scipy-openblas=32 | ||
| export PKG_CONFIG_PATH=/numpy/.openblas | ||
| python -m pip install . -v -Csetup-args="-Dallow-noblas=true" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should say false. And it would have caught the problem that I was suggested it could, since the build log shows:
Run-time dependency scipy-openblas found: NO (tried pkgconfig)
It's not clear from the Azure logs whether scipy-openblas32 was successfully found in the Azure version of this job unfortunately, because the build log is hidden there. I suspect also not; the path /numpy/.openblas looks wrong, it shouldn't start with a slash.
I force pushed to #30225 whilst closed, which meant that I wasn't able to reopen it. Sorry for the noise.