Skip to content

[https://nvbugs/5949098][doc] Fixing docs links - #11912

Merged
pcastonguay merged 1 commit into
NVIDIA:release/1.2from
pcastonguay:doc_fix_md_links
Mar 4, 2026
Merged

[https://nvbugs/5949098][doc] Fixing docs links#11912
pcastonguay merged 1 commit into
NVIDIA:release/1.2from
pcastonguay:doc_fix_md_links

Conversation

@pcastonguay

@pcastonguay pcastonguay commented Mar 4, 2026

Copy link
Copy Markdown
Collaborator

Summary by CodeRabbit

  • Documentation
    • Updated external documentation links to current NVIDIA resources, including TensorRT-LLM Dynamo backend references and NeMo Framework multimodal documentation paths.

Description

Test Coverage

PR Checklist

Please review the following before submitting your PR:

  • PR description clearly explains what and why. If using CodeRabbit's summary, please make sure it makes sense.

  • PR Follows TRT-LLM CODING GUIDELINES to the best of your knowledge.

  • Test cases are provided for new code paths (see test instructions)

  • Any new dependencies have been scanned for license and vulnerabilities

  • CODEOWNERS updated if ownership changes

  • Documentation updated as needed

  • Update tava architecture diagram if there is a significant design change in PR.

  • The reviewers assigned automatically/manually are appropriate for the PR.

  • Please check this after reviewing the above items as appropriate for this PR.

GitHub Bot Help

To see a list of available CI bot commands, please comment /bot help.

Signed-off-by: Patrice Castonguay <55748270+pcastonguay@users.noreply.github.com>
@pcastonguay
pcastonguay requested a review from a team as a code owner March 4, 2026 15:09
@pcastonguay

Copy link
Copy Markdown
Collaborator Author

/bot skip --comment "Doc only changes"

@coderabbitai

coderabbitai Bot commented Mar 4, 2026

Copy link
Copy Markdown
Contributor
📝 Walkthrough

Walkthrough

This pull request updates documentation hyperlinks across multiple files to reflect changes in NVIDIA documentation URLs. Four documentation files are modified to point to updated documentation paths for TensorRT-LLM, Dynamo, and NeVA resources without any changes to content or functionality.

Changes

Cohort / File(s) Summary
Documentation Link Updates
docs/source/blogs/tech_blog/blog5_Disaggregated_Serving_in_TensorRT-LLM.md, docs/source/features/disagg-serving.md, triton_backend/all_models/multimodal/Deprecation_notice.md
Updated NVIDIA documentation URLs for TensorRT-LLM resources from old docs.nvidia.com paths to new docs.dynamo.nvidia.com structure.
NeVA Documentation Link
examples/models/core/multimodal/README.md
Updated NeVA documentation URL from generic docs.nvidia.com path to versioned NVIDIA NeMo framework path (two link occurrences updated).

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The PR description is incomplete. While the author checked the final checklist item, they did not fill in the required Description section or provide Test Coverage details. Add a clear description explaining what documentation links were fixed and why. Include specific details about which links were updated and the reason for the changes. Add Test Coverage section explaining how the link changes are validated.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly describes the main change: fixing documentation links across multiple files. It follows the required format with NVBugs ticket ID and [doc] type prefix.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

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

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #37717 [ skip ] triggered by Bot. Commit: bad52c6 Link to invocation

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #37717 [ skip ] completed with state SUCCESS. Commit: bad52c6
Skipping testing for commit bad52c6

Link to invocation

@pcastonguay
pcastonguay merged commit f06eaaa into NVIDIA:release/1.2 Mar 4, 2026
9 of 10 checks passed
dominicshanshan pushed a commit to dominicshanshan/TensorRT-LLM that referenced this pull request Mar 5, 2026
Signed-off-by: Patrice Castonguay <55748270+pcastonguay@users.noreply.github.com>
dominicshanshan pushed a commit to dominicshanshan/TensorRT-LLM that referenced this pull request Mar 5, 2026
Signed-off-by: Patrice Castonguay <55748270+pcastonguay@users.noreply.github.com>
Signed-off-by: Wangshanshan <30051912+dominicshanshan@users.noreply.github.com>
dominicshanshan pushed a commit to dominicshanshan/TensorRT-LLM that referenced this pull request Mar 5, 2026
Signed-off-by: Patrice Castonguay <55748270+pcastonguay@users.noreply.github.com>
Signed-off-by: Wangshanshan <30051912+dominicshanshan@users.noreply.github.com>
dominicshanshan pushed a commit to dominicshanshan/TensorRT-LLM that referenced this pull request Mar 5, 2026
Signed-off-by: Patrice Castonguay <55748270+pcastonguay@users.noreply.github.com>
Signed-off-by: Wangshanshan <30051912+dominicshanshan@users.noreply.github.com>
dominicshanshan pushed a commit to dominicshanshan/TensorRT-LLM that referenced this pull request Mar 5, 2026
Signed-off-by: Patrice Castonguay <55748270+pcastonguay@users.noreply.github.com>
Signed-off-by: Wangshanshan <30051912+dominicshanshan@users.noreply.github.com>
dominicshanshan pushed a commit to dominicshanshan/TensorRT-LLM that referenced this pull request Mar 5, 2026
Signed-off-by: Patrice Castonguay <55748270+pcastonguay@users.noreply.github.com>
Signed-off-by: Wangshanshan <30051912+dominicshanshan@users.noreply.github.com>
dominicshanshan pushed a commit to dominicshanshan/TensorRT-LLM that referenced this pull request Mar 5, 2026
Signed-off-by: Patrice Castonguay <55748270+pcastonguay@users.noreply.github.com>
Signed-off-by: Wangshanshan <30051912+dominicshanshan@users.noreply.github.com>
chzblych pushed a commit that referenced this pull request Mar 7, 2026
Signed-off-by: Patrice Castonguay <55748270+pcastonguay@users.noreply.github.com>
Signed-off-by: Wangshanshan <30051912+dominicshanshan@users.noreply.github.com>
tianyuz-nv pushed a commit to wanqian-nv/TensorRT-LLM that referenced this pull request Mar 19, 2026
Signed-off-by: Patrice Castonguay <55748270+pcastonguay@users.noreply.github.com>
Signed-off-by: Wangshanshan <30051912+dominicshanshan@users.noreply.github.com>
limin2021 pushed a commit to limin2021/TensorRT-LLM that referenced this pull request Mar 19, 2026
Signed-off-by: Patrice Castonguay <55748270+pcastonguay@users.noreply.github.com>
Signed-off-by: Wangshanshan <30051912+dominicshanshan@users.noreply.github.com>
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