Skip to content

Fix/missing libs in docs#3190

Merged
angeloskath merged 6 commits intoml-explore:mainfrom
ChristophePRAT:fix/missing-libs-in-docs
Mar 3, 2026
Merged

Fix/missing libs in docs#3190
angeloskath merged 6 commits intoml-explore:mainfrom
ChristophePRAT:fix/missing-libs-in-docs

Conversation

@ChristophePRAT
Copy link
Copy Markdown
Contributor

Proposed changes

As mentioned in issue #3103, I updated the installation documentations by adding 3 required dependencies:

  • libblas-dev
  • liblapack-dev
  • liblapacke-dev

They are most oftentimes already installed but not always, hence the need to add them.

Checklist

Put an x in the boxes that apply.

  • I have read the CONTRIBUTING document
  • I have run pre-commit run --all-files to format my code / installed pre-commit prior to committing changes
  • I have added tests that prove my fix is effective or that my feature works (N/A)
  • I have updated the necessary documentation (if needed)

Copilot AI review requested due to automatic review settings March 2, 2026 19:50
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the “Build from source” installation documentation to include missing Linux BLAS/LAPACK development dependencies, addressing the build failures reported in issue #3103.

Changes:

  • Document Linux build dependencies: BLAS/LAPACK development packages (libblas-dev, liblapack-dev, liblapacke-dev).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Copy Markdown
Collaborator

@zcbenz zcbenz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me, thanks!

@angeloskath angeloskath merged commit f145ece into ml-explore:main Mar 3, 2026
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants