Skip to content

PR 3103 Updating Documentation#3104

Closed
MillaFleurs wants to merge 1 commit intoml-explore:mainfrom
MillaFleurs:documentation-dependencies
Closed

PR 3103 Updating Documentation#3104
MillaFleurs wants to merge 1 commit intoml-explore:mainfrom
MillaFleurs:documentation-dependencies

Conversation

@MillaFleurs
Copy link
Copy Markdown
Contributor

Proposed changes

Please include a description of the problem or feature this PR is addressing. If there is a corresponding issue, include the issue #.

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
  • I have updated the necessary documentation (if needed)

Comment thread docs/src/install.rst
Comment on lines +89 to +92
- The following libraries through brew, or by using the Accelerate framework
- libblas-dev
- liblapack-dev
- liblapacke-dev
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.

These are not required on macOS.. it will use the Accelerate framework by default.

Comment thread docs/src/install.rst
Build Requirements Linux
^^^^^^^^^^^^^^^^^^

- A C++ compiler with C++20 support (e.g. Clang >= 15.0)
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.

Suggested change
- A C++ compiler with C++20 support (e.g. Clang >= 15.0)
- A C++ compiler with C++20 support (e.g. GCC >= 13)

Comment thread docs/src/install.rst
^^^^^^^^^^^^^^^^^^

- A C++ compiler with C++20 support (e.g. Clang >= 15.0)
- `cmake <https://cmake.org/>`_ -- version 3.25 or later, and ``make``
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.

Suggested change
- `cmake <https://cmake.org/>`_ -- version 3.25 or later, and ``make``
- `cmake <https://cmake.org/>`_ 3.25 or later

Comment thread docs/src/install.rst

- A C++ compiler with C++20 support (e.g. Clang >= 15.0)
- `cmake <https://cmake.org/>`_ -- version 3.25 or later, and ``make``
- The following libraries through brew, or by using the Accelerate framework
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.

Suggested change
- The following libraries through brew, or by using the Accelerate framework
- The following libraries:

Comment thread docs/src/install.rst
- A C++ compiler with C++20 support (e.g. Clang >= 15.0)
- `cmake <https://cmake.org/>`_ -- version 3.25 or later, and ``make``
- The following libraries through brew, or by using the Accelerate framework
- build-essential
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.

Suggested change
- build-essential

@zcbenz
Copy link
Copy Markdown
Collaborator

zcbenz commented Mar 17, 2026

I'm closing this PR since latest docs already has some of the information here.

@zcbenz zcbenz closed this Mar 17, 2026
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.

3 participants