Add: docs/ index and runtime capability survey - #1525
Conversation
docs/ had 35 files flat at its root with no index. The top-level README listed 10 of them, so the other 25 were discoverable only by grep, and four had no inbound link from anywhere in the repo. Only investigations/ carried a README; dfx/ (12 files, 5227 lines), troubleshooting/ and hardware/ had none. - Add docs/README.md indexing every root document and subdirectory, grouped by reader intent, and stating that a new doc registers there in the same commit. This de-orphans install.md, simt-launch.md, profiling-config-naming.md and sim_multi_device_isolation.md. - Add docs/capability-survey.md: a status snapshot of what is shipped, gated, or design-only across the level model (L2 chip up to L4 host-to-host), the CANN launch surface for AICore and AICPU, and the communication engines. Every claim carries a file:line so it can be re-checked, and the page records the snapshot commit because status claims rot faster than architecture. - Move profiling-framework.md, profiling-name-map.md and profiling-config-naming.md into docs/dfx/, which already held 12 profiling documents and linked up to these three nine times via ../. Add docs/dfx/README.md as that directory's index. logging.md stays at the docs/ root: PR hw-native-sys#1475 modifies it, so renaming it here would hand that author a rename-vs-modify conflict. - Fix seven broken links in docs/dfx/pmu-profiling.md, a file already touched by this change: six had the wrong relative depth (../src -> ../../src) and profiling_copy.h moved to src/common/ when hw-native-sys#944 unified the collector framework. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (10)
📝 WalkthroughWalkthroughThe PR expands documentation navigation, adds a runtime capability survey covering topology, launch, and communication engines, creates a DFX documentation index, and corrects relative links across profiling documents. ChangesDocumentation navigation and status
Estimated code review effort: 2 (Simple) | ~10 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Summary
docs/had 35 files flat at its root with no index. The top-level README listed 10 of them, so the other 25 were discoverable only bygrep, and four had no inbound link from anywhere in the repo (install.md,simt-launch.md,profiling-config-naming.md,sim_multi_device_isolation.md). Onlyinvestigations/carried a README —dfx/(12 files, 5,227 lines),troubleshooting/(7) andhardware/(4) had none. Root also keeps growing: #1453 adds a 36th file.Three changes:
docs/README.md— indexes every root document and subdirectory, grouped by reader intent (Start here / Architecture / Kernels / Launch & linking / Multi-chip & comm / Profiling & DFX / Build & test / When something is broken). It also states the rule that a new doc registers there in the same commit, so the index does not silently fall behind. All four orphans are now reachable.docs/capability-survey.md— a status snapshot of what is shipped, gated, or design-only across the level model (L2 chip → L4 host-to-host), the CANN launch surface for AICore/AICPU, and the communication engines. Every claim carries afile:line, and the page records its snapshot commit up front because status claims rot faster than architecture prose.docs/dfx/consolidation — movedprofiling-framework.md,profiling-name-map.mdandprofiling-config-naming.mdintodocs/dfx/, plus a newdocs/dfx/README.mdindex.Why the profiling docs belong in
dfx/docs/dfx/already held 12 profiling documents and already linked upward to these three nine times via../profiling-framework.md/../profiling-name-map.md. Their main consumer was the directory they were not in. Each had 0–1 external references, so the move is cheap.logging.mddeliberately stays at thedocs/root. PR #1475 modifies that file, and renaming it here would hand its author a rename-vs-modify conflict. It can follow once #1475 lands.A full topical regrouping of the remaining root docs (
architecture/,mechanisms/,guides/) is not in this PR: it would touch ~28 reference sites in.cpp/.h/.py/.ymlplus.claude/rules/and skills, and it collides with #1467 (6 core architecture docs), #1475 (9 docs) and #1453. The index captures the orientation benefit at none of that cost.Drive-by fix, disclosed
docs/dfx/pmu-profiling.md— a file this PR already edits — had 7 broken links: 6 with the wrong relative depth (../src→../../src), andprofiling_copy.h, which moved tosrc/common/platform/include/host/when #944 unified the collector framework. Same file, same defect class, individually verified.Testing
Documentation only (
.md); no code, build, or config files changed, so the sim/hardware suites do not apply per the git-commit skill's test matrix.markdownlint-cli2clean across all 19 touched markdown files, using v0.20.0 — the version.pre-commit-config.yaml:83pins, i.e. what CI runstests/st/**,tools/cann-examples/**, one false positive introubleshooting/macos-build.md). This change introduces zero broken linksdocs/README.mdgitdetected all three moves as renames, so history is preserved