Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly
Appearance settings

[None][doc] Fix invalid links in tech blogs.#12425

Merged
nv-guomingz merged 1 commit intoNVIDIA:mainNVIDIA/TensorRT-LLM:mainfrom
nv-guomingz:user/guomingz/fix_invalid_lognv-guomingz/TensorRT-LLM:user/guomingz/fix_invalid_logCopy head branch name to clipboard
Mar 22, 2026
Merged

[None][doc] Fix invalid links in tech blogs.#12425
nv-guomingz merged 1 commit intoNVIDIA:mainNVIDIA/TensorRT-LLM:mainfrom
nv-guomingz:user/guomingz/fix_invalid_lognv-guomingz/TensorRT-LLM:user/guomingz/fix_invalid_logCopy head branch name to clipboard

Conversation

@nv-guomingz
Copy link
Copy Markdown
Collaborator

@nv-guomingz nv-guomingz commented Mar 21, 2026 •
edited by coderabbitai Bot
Loading

Summary by CodeRabbit

  • Documentation
    • Updated internal documentation cross-references across multiple blog posts and guides to reflect standardized file naming conventions, ensuring all links correctly point to their intended resources.

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.

@nv-guomingz
[None][doc] Fix invalid links in tech blogs.
ac3b3ad 
Signed-off-by: nv-guomingz <137257613+nv-guomingz@users.noreply.github.com>
@nv-guomingz nv-guomingz requested a review from a team as a code owner March 21, 2026 17:21
@nv-guomingz
Copy link
Copy Markdown
Collaborator Author

nv-guomingz commented Mar 21, 2026

/bot run

@nv-guomingz nv-guomingz enabled auto-merge (squash) March 21, 2026 17:21
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Mar 21, 2026 •
edited
Loading

📝 Walkthrough

Walkthrough

Blog documentation files were updated to standardize internal hyperlink paths by renaming referenced blog files to use zero-padded numbering format (e.g., blog1_ → blog01_, blog2_ → blog02_). All changes are hyperlink references only; no content or structure modifications.

Changes

Cohort / File(s) Summary
Blog documentation link standardization
docs/source/blogs/Best_perf_practice_on_DeepSeek-R1_in_TensorRT-LLM.md, docs/source/blogs/tech_blog/blog02_DeepSeek_R1_MTP_Implementation_and_Optimization.md, docs/source/blogs/tech_blog/blog03_Optimizing_DeepSeek_R1_Throughput_on_NVIDIA_Blackwell_GPUs.md, docs/source/blogs/tech_blog/blog04_Scaling_Expert_Parallelism_in_TensorRT-LLM.md, docs/source/blogs/tech_blog/blog05_Disaggregated_Serving_in_TensorRT-LLM.md		 Updated hyperlinks to reference blog files with zero-padded numbering scheme (e.g., blog1_ → blog01_, blog2_ → blog02_).
Additional blog documentation link updates
docs/source/blogs/tech_blog/blog08_Scaling_Expert_Parallelism_in_TensorRT-LLM_part2.md, docs/source/blogs/tech_blog/blog12_Combining_Guided_Decoding_and_Speculative_Decoding.md, docs/source/blogs/tech_blog/blog14_Scaling_Expert_Parallelism_in_TensorRT-LLM_part3.md, docs/source/blogs/tech_blog/blog15_Optimizing_DeepSeek_V32_on_NVIDIA_Blackwell_GPUs.md		 Updated multiple internal hyperlinks to referenced blog files to use zero-padded numbering format (blog1_... → blog01_..., blog2_... → blog02_..., etc.).
Feature and legacy documentation link updates
docs/source/features/parallel-strategy.md, docs/source/legacy/advanced/disaggregated-service.md		 Updated hyperlinks in feature and legacy documentation sections to reference blog files with zero-padded numbering format.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The PR description is incomplete. Required sections 'Description' and 'Test Coverage' are blank, leaving the rationale and test strategy unexplained. Fill in the Description section explaining why the blog links needed to be renamed and Test Coverage section documenting how the link fixes were validated.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and specifically describes the main change: fixing invalid links in tech blogs by updating file references to use zero-padded numbering.
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

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

Tip

CodeRabbit can suggest fixes for GitHub Check annotations.

Configure the reviews.tools.github-checks setting to adjust the time to wait for GitHub Checks to complete.

@tensorrt-cicd
Copy link
Copy Markdown
Collaborator

tensorrt-cicd commented Mar 21, 2026

PR_Github #39798 [ run ] triggered by Bot. Commit: ac3b3ad Link to invocation

@tensorrt-cicd
Copy link
Copy Markdown
Collaborator

tensorrt-cicd commented Mar 21, 2026

PR_Github #39798 [ run ] completed with state DISABLED
CI server is currently disabled for scheduled maintenance. Estimated completion time: 9 PM PST on 3/21.

Link to invocation

@nv-guomingz
Copy link
Copy Markdown
Collaborator Author

nv-guomingz commented Mar 22, 2026

/bot run

@tensorrt-cicd
Copy link
Copy Markdown
Collaborator

tensorrt-cicd commented Mar 22, 2026

PR_Github #39824 [ run ] triggered by Bot. Commit: ac3b3ad Link to invocation

@tensorrt-cicd
Copy link
Copy Markdown
Collaborator

tensorrt-cicd commented Mar 22, 2026

PR_Github #39824 [ run ] completed with state SUCCESS. Commit: ac3b3ad
/LLM/main/L0_MergeRequest_PR pipeline #31001 completed with status: 'SUCCESS'

CI Report

Link to invocation

@nv-guomingz nv-guomingz merged commit 3353334 into NVIDIA:main Mar 22, 2026
9 of 10 checks passed
longcheng-nv pushed a commit to longcheng-nv/TensorRT-LLM that referenced this pull request Mar 31, 2026
@nv-guomingz @longcheng-nv
[None][doc] Fix invalid links in tech blogs. (NVIDIA#12425)
cbb7cdb 
Signed-off-by: nv-guomingz <137257613+nv-guomingz@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@arysef arysef arysef approved these changes

@Shixiaowei02 Shixiaowei02 Awaiting requested review from Shixiaowei02 Shixiaowei02 is a code owner automatically assigned from NVIDIA/trt-llm-doc-owners

Assignees

@nv-guomingz nv-guomingz

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@nv-guomingz @tensorrt-cicd @arysef
Morty Proxy This is a proxified and sanitized view of the page, visit original site.