Skip to content

CI: removes azure pipelines - #30232

Merged
rgommers merged 12 commits into
numpy:mainfrom
andyfaff:az2
Nov 17, 2025
Merged

rgommers merged 12 commits into
numpy:mainfrom
andyfaff:az2

Conversation

@andyfaff

Copy link
Copy Markdown
Member

I force pushed to #30225 whilst closed, which meant that I wasn't able to reopen it. Sorry for the noise.

Comment thread .github/workflows/linux.yml Outdated
Comment thread .github/workflows/linux.yml Outdated
@mattip

mattip commented Nov 16, 2025

Copy link
Copy Markdown
Member

The linux pypy run failed, something about underflow in a check of the conj ufunc. I reran it, let's see if it is transient.

@rgommers

Copy link
Copy Markdown
Member

With gh-30222 merged, this one can be updated now.

@rgommers rgommers left a comment

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.

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.

Comment thread .github/workflows/linux.yml Outdated

spin config-openblas --with-scipy-openblas=32
export PKG_CONFIG_PATH=/numpy/.openblas
python -m pip install . -v -Csetup-args="-Dallow-noblas=true"

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 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.

@rgommers rgommers left a comment

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.

LGTM now, thanks @andyfaff. I'll apply one last tiny tweak to the job name, then merge.

Comment thread .github/workflows/linux.yml Outdated
[skip ci]
@rgommers
rgommers merged commit 02c8c3d into numpy:main Nov 17, 2025
@rgommers rgommers added this to the 2.4.0 release milestone Nov 17, 2025
@andyfaff
andyfaff deleted the az2 branch November 17, 2025 10:58
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants