Skip to content

[Docs] Fix build-from-source env prerequisites in README#645

Merged
sjfeng1999 merged 1 commit into
ROCm:mainfrom
Arist12:docs/fix-build-from-source-prereqs
Jun 4, 2026
Merged

[Docs] Fix build-from-source env prerequisites in README#645
sjfeng1999 merged 1 commit into
ROCm:mainfrom
Arist12:docs/fix-build-from-source-prereqs

Conversation

@Arist12
Copy link
Copy Markdown
Contributor

@Arist12 Arist12 commented Jun 3, 2026

Clarify that nanobind/numpy/pybind11 are only installed by build_llvm.sh (not build.sh), and document the pytest/pandas/ROCm-torch deps needed to run tests and examples.

Motivation

The README's "Build from source" / "Run tests" instructions are missing a few environment-prep steps, so following them top-to-bottom in a clean environment fails. This PR makes the prerequisites accurate.

Technical Details

No code changes.

What was wrong

  1. Misleading build-deps note. The prerequisites claim nanobind, numpy, pybind11 are "installed by the build scripts", but only scripts/build.sh does not.

  2. torch, pytest and pandas undocumented. They are prerequisites to run the tests bash scripts/run_tests.sh but are not documented.

Submission Checklist

Clarify that nanobind/numpy/pybind11 are only installed by build_llvm.sh
(not build.sh), and document the pytest/pandas/ROCm-torch deps needed to
run tests and examples.
@sjfeng1999 sjfeng1999 merged commit a36f926 into ROCm:main Jun 4, 2026
9 checks passed
@Arist12 Arist12 deleted the docs/fix-build-from-source-prereqs branch June 4, 2026 06:36
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.

2 participants