FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

CI Downgrade libopenblas to 0.3.33 to avoid regression on Windows by betatim · Pull Request #34778 · scikit-learn/scikit-learn · GitHub

CI Downgrade libopenblas to 0.3.33 to avoid regression on Windows - #34778

Merged
lesteve merged 12 commits into
scikit-learn:mainfrom
betatim:windows-openblas-bug-0.3.33
Aug 21, 2026
Merged

CI Downgrade libopenblas to 0.3.33 to avoid regression on Windows#34778
lesteve merged 12 commits into
scikit-learn:mainfrom
betatim:windows-openblas-bug-0.3.33

Conversation

betatim commented Aug 19, 2026
edited
Loading

Copy link
Copy Markdown
Member

Reference Issues/PRs

Fixes #34717

What does this implement/fix? Explain your changes.

With libopenblas 0.3.34 we see intermittent failures on windows. Let's see if using an older version makes them go away.

We need to re-run the windows CI job a few times to see if the failure is really gone or not.

AI thinks OpenMathLib/OpenBLAS#5954 is related/the same problem.

AI usage disclosure

I used AI assistance for:

  • Research and understanding

First, second, third, fourth, fifth, sixth, seventh, eight and ninth run succeeded. Commits with a red cross can be the result of stopping the CI before all jobs ran. The important thing is to check the status of the windows job.

With libopenblas 0.3.34 we see intermittent failures on windows. Let's
see if using an older version makes them go away.

betatim commented Aug 20, 2026

Copy link
Copy Markdown
Member Author

@lesteve want to have a look? It seems to pass/only fail less than 1 in 10 times.

lesteve commented Aug 21, 2026

Copy link
Copy Markdown
Member

AFAICT from this AI-generated script all the Windows CI build succeeded

 repo="scikit-learn/scikit-learn"
  job="Windows x64 pymin_conda_forge_openblas"

  gh api "repos/$repo/pulls/34778/commits" --paginate --jq '.[].sha' |
  while read -r sha; do
      gh api "repos/$repo/commits/$sha/check-runs" --paginate \
          --jq ".check_runs[]
              | select(.name == \"$job\")
              | [\"$sha\", .status, (.conclusion // \"pending\"), .html_url]
              | @tsv"
              | cat
  done  

Thanks for testing this! It's hard to know whether the OpenBLAS issue is related but empirically it seems to fix the problem. We'll keep an eye out for the OpenBLAS 0.3.35 release and hope the problem doesn't come back.

lesteve changed the title CI Downgrade libopenblas to 0.3.33 because of a regression CI Downgrade libopenblas to 0.3.33 to avoid regression on Windows Aug 21, 2026
lesteve merged commit d0fc1d9 into scikit-learn:main Aug 21, 2026
39 checks passed

lesteve commented Aug 21, 2026

Copy link
Copy Markdown
Member

One less issue to look at for my week on Triage (next week), really appreciated @betatim 🙏!

betatim deleted the windows-openblas-bug-0.3.33 branch August 21, 2026 07:45

betatim commented Aug 21, 2026

Copy link
Copy Markdown
Member Author

🤝

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

⚠️ CI failed on Unit tests Windows x64 pymin_conda_forge_openblas (last failure: Aug 11, 2026) ⚠️

2 participants


Back | FazBrowse Home | New Git URL