Skip to content

docs: Adds device vendor contacts if any#464

Merged
yanziz-nvidia merged 1 commit into
mainfrom
yanziz/doc-add-pico-contact
May 8, 2026
Merged

docs: Adds device vendor contacts if any#464
yanziz-nvidia merged 1 commit into
mainfrom
yanziz/doc-add-pico-contact

Conversation

@yanziz-nvidia
Copy link
Copy Markdown
Contributor

@yanziz-nvidia yanziz-nvidia commented May 4, 2026

Summary by CodeRabbit

  • Documentation
    • Improved device overview documentation with enhanced reference links for easier navigation
    • Added new "Device Acquisition" section featuring manufacturer contact information
    • Expanded reference links for manufacturer home pages and device product pages to provide better access to resources

@yanziz-nvidia yanziz-nvidia requested a review from jiwenc-nv May 4, 2026 17:18
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 4, 2026

Important

Review skipped

Auto incremental reviews are disabled on this repository.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: 5cd53a56-378f-46a2-938b-abb3b92f9205

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
📝 Walkthrough

Walkthrough

The pull request updates the ecosystem documentation file to restructure and enhance the device reference information. Device names in three existing tables (XR Headsets and Tracking Peripherals, Standalone Input Devices, and Planned Input Devices) were converted to Sphinx reference-style links. A new "Device Acquisition" section was added, featuring a "Manufacturer Contacts" table listing vendor information and acquisition contact details. Corresponding Sphinx reference target definitions were expanded to include manufacturer home pages and device product page links. A redundant standalone reference block was removed as its content is now consolidated.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: adding device vendor/manufacturer contacts to the documentation. This aligns with the primary objective of the PR.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch yanziz/doc-add-pico-contact

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@docs/source/overview/ecosystem.rst`:
- Around line 146-147: Update the paragraph that currently reads "For inquiries
about acquiring supported or planned devices, please contact the manufacturers
directly. Each device name in the tables above links to the corresponding
manufacturer page." to accurately reflect the actual link targets: change the
wording to indicate that device names link to product pages (or
manufacturer/product pages where applicable) and that not every device name
links to a manufacturer page. Ensure the revised sentence(s) reference "device
name" and "product pages" (or "manufacturer/product pages") so readers
understand the nature of the links defined later in the document.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: 5fdbfba5-a4bd-4a97-bc1f-bb315b02a71b

📥 Commits

Reviewing files that changed from the base of the PR and between a94d1d8 and eb80324.

📒 Files selected for processing (1)
  • docs/source/overview/ecosystem.rst

Comment thread docs/source/overview/ecosystem.rst
@jiwenc-nv
Copy link
Copy Markdown
Collaborator

/preview-docs

@yanziz-nvidia yanziz-nvidia merged commit ef43b37 into main May 8, 2026
112 of 115 checks passed
@yanziz-nvidia yanziz-nvidia deleted the yanziz/doc-add-pico-contact branch May 8, 2026 16:51
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