Skip to content

[tmva][sofie] Fix sofie tests when blas is not found #21188

Merged
guitargeek merged 2 commits intoroot-project:v6-32-00-patchesfrom
lmoneta:v6-32-00-patches
Feb 8, 2026
Merged

[tmva][sofie] Fix sofie tests when blas is not found #21188
guitargeek merged 2 commits intoroot-project:v6-32-00-patchesfrom
lmoneta:v6-32-00-patches

Conversation

@lmoneta
Copy link
Member

@lmoneta lmoneta commented Feb 7, 2026

Fix sofie tests when blas is not found but GSL cblas is found

Whenc blas from gsl is found tmva-cpu is ON but sofie needs blas not cblas , therefore the tests need to be disabled in this case

…SL is found

Whencblas from gsl is found tmva-cpu is ON but sofie needs blas not cblas , therefore the tests nuts be disabled in this case
@lmoneta lmoneta self-assigned this Feb 7, 2026
@lmoneta lmoneta requested a review from bellenot as a code owner February 7, 2026 18:01
@guitargeek guitargeek added the clean build Ask CI to do non-incremental build on PR label Feb 7, 2026
@guitargeek guitargeek closed this Feb 7, 2026
@guitargeek guitargeek reopened this Feb 7, 2026
@github-actions
Copy link

github-actions bot commented Feb 7, 2026

Test Results

     8 files       8 suites   1d 7h 59m 55s ⏱️
 2 562 tests  2 562 ✅ 0 💤 0 ❌
19 926 runs  19 926 ✅ 0 💤 0 ❌

Results for commit a725eb5.

♻️ This comment has been updated with latest results.

@guitargeek guitargeek requested a review from dpiparo as a code owner February 7, 2026 19:42
@guitargeek guitargeek force-pushed the v6-32-00-patches branch 2 times, most recently from fb5f322 to 440ffeb Compare February 7, 2026 23:24
Both OpenBLAS and Felxiblas caused problems, so now we need to force
using the LAPACK BLAS, for which there is no BLA_VENDOR value.
@guitargeek guitargeek merged commit 4a60c67 into root-project:v6-32-00-patches Feb 8, 2026
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

clean build Ask CI to do non-incremental build on PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants