Skip to content

Fix broken links in docs#2287

Merged
maanug-nv merged 2 commits intomainfrom
maanug/fix-links
Feb 9, 2026
Merged

Fix broken links in docs#2287
maanug-nv merged 2 commits intomainfrom
maanug/fix-links

Conversation

@maanug-nv
Copy link
Copy Markdown
Contributor

@maanug-nv maanug-nv commented Feb 9, 2026

What does this PR do ?

Aimed to preserve the target content. i.e., old links for CP and Dist opt pointed to the technical detail pages.

Changelog

  • Add specific line by line info of high level changes in this PR.

GitHub Actions CI

See the CI sectionin the Contributing doc for how to trigger the CI. A Nvidia developer will need to approve and trigger the CI for external contributors.

Before your PR is "Ready for review"

Pre checks:

  • Make sure you read and followed Contributor guidelines
  • Did you write any new necessary tests?
  • Did you add or update any necessary documentation?
  • Does the PR affect components that are optional to install? (Ex: Numba, Pynini, Apex etc)
    • Reviewer: Does the PR have correct import guards for all optional libraries?

If you haven't finished some of the above items you can still open "Draft" PR.

Additional Information

  • Related to # (issue)

Summary by CodeRabbit

  • Documentation
    • Updated parallelisms documentation with references to the latest Megatron Core Developer Guide sections for Distributed Optimizer, Tensor Parallelism, and Context Parallelism features.

Signed-off-by: Maanu Grover <maanug@nvidia.com>
@maanug-nv maanug-nv requested a review from ananthsub February 9, 2026 21:33
@maanug-nv maanug-nv enabled auto-merge (squash) February 9, 2026 21:33
@maanug-nv maanug-nv self-assigned this Feb 9, 2026
@maanug-nv maanug-nv added the docs-only With great power comes great responsibility. label Feb 9, 2026
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Feb 9, 2026

📝 Walkthrough

Walkthrough

Updated three external documentation links in the Megatron Core developer guide to reflect new URL paths for distributed optimizer, tensor parallelism, and context parallelism resources.

Changes

Cohort / File(s) Summary
Documentation Links
docs/parallelisms.md
Updated three Megatron Core external documentation links to newer paths: distributed optimizer, tensor parallelism, and context parallelism references. Minor wording adjustments to reflect updated destination URLs.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Suggested labels

docs-only

Suggested reviewers

  • cuichenx
🚥 Pre-merge checks | ✅ 4
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'Fix broken links in docs' is directly related to and accurately summarizes the main change: updating three broken external documentation links in docs/parallelisms.md to their correct URLs.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Test Results For Major Changes ✅ Passed PR contains only minor documentation changes (URL updates in markdown file) with no impact on code behavior, performance, or numerics.

✏️ 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
  • Commit unit tests in branch maanug/fix-links

No actionable comments were generated in the recent review. 🎉


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

Signed-off-by: Maanu Grover <maanug@nvidia.com>
@maanug-nv maanug-nv merged commit 40aa462 into main Feb 9, 2026
24 checks passed
@maanug-nv maanug-nv deleted the maanug/fix-links branch February 9, 2026 23:52
sowmen pushed a commit to sowmen/Megatron-Bridge that referenced this pull request Feb 11, 2026
Signed-off-by: Maanu Grover <maanug@nvidia.com>
Signed-off-by: sowmen <sowmendipta@gmail.com>
ko3n1g pushed a commit that referenced this pull request Feb 11, 2026
Signed-off-by: Maanu Grover <maanug@nvidia.com>
ko3n1g pushed a commit that referenced this pull request Feb 11, 2026
Signed-off-by: Maanu Grover <maanug@nvidia.com>
Signed-off-by: oliver könig <okoenig@nvidia.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs-only With great power comes great responsibility.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants