docs: add H200 and B200 to GPU architecture detection list - #304
docs: add H200 and B200 to GPU architecture detection list#304ntheanh201 wants to merge 1 commit into
Conversation
H200 and B200 are supported: both have entries in the gpu-defaults table (pkg/catalog/entries/_lib/gpu-defaults.yaml) and are listed in docs/concepts/catalog.md, but the README feature list omitted them. Verified H200 detection on a 3-node on-prem HGX H200 cluster. Signed-off-by: The Anh Nguyen <ntheanh201@gmail.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Enterprise Run ID: 📒 Files selected for processing (1)
Included review availability: Your plan provides up to 12 included reviews per hour; 11 remain after this review. 📝 WalkthroughWalkthroughThe README Features section now lists the L40S GPU architecture with the previously documented supported architectures. Estimated code review effort: 1 (Trivial) | ~2 minutes Merge Risk: 🟡 Moderate · up to The README adds L40S rather than the requested H200 and B200 entries, leaving the intended GPU architecture documentation incomplete and potentially misleading users about supported hardware. 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
Summary
The README's platform/GPU detection feature list omitted H200 and B200, even though both are supported: both have entries in the GPU defaults table (
pkg/catalog/entries/_lib/gpu-defaults.yaml), anddocs/concepts/catalog.mdalready states "Supported GPU architectures include GB200, GB300, H100, H200, and B200". The stale list made an on-prem HGX H200 cluster look unsupported at first glance — it is not (verified there).Related Issue
Trivial docs fix, exempt per the template.
Type of Change
Component(s) Affected
Testing
nvidia.com/gpu.product: NVIDIA-H200are detected, andnvcrectl certification render --platform onpremforcommunication/nccl-all-reducerenders a valid 3-node × 8-GPU WorkflowChecklist
git commit -s)make manifests generaterun (if*_types.gowas modified) — n/a