[chfeng-cs]

Description

Seven SVG image links in the legacy performance-tuning-guide docs were broken. The affected files live at: docs/source/legacy/performance/performance-tuning-guide/ , such as https://github.com/NVIDIA/TensorRT-LLM/blob/main/docs/source/legacy/performance/performance-tuning-guide/deciding-model-sharding-strategy.md

Each link used ../../media/<image>.svg, which resolves to: docs/source/legacy/media/ . This path does not exist.

The actual SVG files are at docs/source/media/. Because the guide is nested one level deeper than expected, the path was missing one ../ level.

Fix

Changed all 7 occurrences from ../../media/ → ../../../media/, which correctly resolves to docs/source/media/ where the files actually live.

Files changed:

File	Links fixed
deciding-model-sharding-strategy.md	2 (Pipeline_Parallel_Vis.svg, Tensor_Parallelism_Vis.svg)
tuning-max-batch-size-and-max-num-tokens.md	5 (TRTLLM_Scheduler_Vis_1~4.svg, TRTLLM_Scheduler_Vis_Chunked_Context_1.svg)

No content changes — path corrections only.

Summary by CodeRabbit

• Documentation
  • Updated several performance-tuning guide illustrations so images display correctly with the new document structure.
  • Adjusted figure placement and spacing in the affected sections for better readability.

[coderabbitai]

📝 Walkthrough

Walkthrough

Updated embedded figure paths in two legacy performance tuning guide pages so the referenced images point to the current media directory.

Changes

Legacy performance guide image updates

Layer / File(s)	Summary
Relative media path updates docs/source/legacy/performance/performance-tuning-guide/deciding-model-sharding-strategy.md, docs/source/legacy/performance/performance-tuning-guide/tuning-max-batch-size-and-max-num-tokens.md	Embedded visualization links were updated from ../../media/... to ../../../media/... in the sharding guide and in the scheduler and chunked-context sections of the tuning guide.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description check	✅ Passed	The description clearly explains the issue and fix, but it omits the template's Test Coverage and PR Checklist sections.
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.
Title check	✅ Passed	The title accurately summarizes the main change: fixing broken SVG image links in the legacy performance tuning guide.

_{✏️ 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.}

[github-actions]

@chfeng-cs this pull request has had no activity for more than 120 days. Is it still active? Please post an update when you can.

[karljang]

Thank you for bringing this to our attention.
However, I’m closing this PR because it seems to have been superseded by #19007, which provides more coverage.