Add portable CI workflow without MKL dependency - #32
Merged
Conversation
Add a second CI workflow that builds and tests the BLAS-free portable kernels without Intel MKL, so the -DCQR_WITH_MKL=OFF path stays healthy and gives fast feedback without the heavy libmkl-dev install. Configures with -DCQR_WITH_MKL=OFF in RelWithDebInfo, then builds and runs the four portable CTest suites across the gcc and clang matrix. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01K6WaceLhu1dpQMkiNCXnX6
CMake, Ninja, GCC, and Clang all ship on the ubuntu-latest runner image, and the -DCQR_WITH_MKL=OFF build needs nothing beyond a C++17 compiler. Remove the apt install step entirely so the BLAS-free smoke check skips both the toolchain and the libmkl-dev install. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01K6WaceLhu1dpQMkiNCXnX6
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
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
This PR adds a new GitHub Actions CI workflow that tests the codebase in a portable configuration without the Intel MKL library dependency. This provides a quick smoke test to ensure the project builds and tests successfully with only standard C++17 compiler support.
Key Changes
.github/workflows/ci-portable.ymlworkflow that:-DCQR_WITH_MKL=OFFto verify portable (BLAS-free) functionalityImplementation Details
libmkl-devdependency installation, keeping this a lightweight smoke testhttps://claude.ai/code/session_01K6WaceLhu1dpQMkiNCXnX6