Skip to content

cp: Fix: nccl-ub in ddp path (3181) into main - #3182

Merged
youngeunkwon0405 merged 2 commits into
mainfrom
cherry-pick-3181-into-main
Jan 31, 2026
Merged

cp: Fix: nccl-ub in ddp path (3181) into main#3182
youngeunkwon0405 merged 2 commits into
mainfrom
cherry-pick-3181-into-main

Conversation

@ko3n1g

@ko3n1g ko3n1g commented Jan 30, 2026

Copy link
Copy Markdown
Contributor

[🤖]: Hi @youngeunkwon0405 👋

We've cherry-picked Fix: nccl-ub in ddp path (#3181) into main for you! 🚀

Please review and approve this cherry-pick at your convenience!

Signed-off-by: Youngeun Kwon <youngeunk@nvidia.com>
@ko3n1g
ko3n1g requested review from a team as code owners January 30, 2026 23:33
@copy-pr-bot

copy-pr-bot Bot commented Jan 30, 2026

Copy link
Copy Markdown

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@ko3n1g
ko3n1g requested a review from a team January 30, 2026 23:34
@github-actions github-actions Bot added the Final Review PR is in the "final review" stage label Jan 30, 2026
@youngeunkwon0405

Copy link
Copy Markdown
Member

/ok to test 410714e

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 4 lines in your changes missing coverage. Please review.
✅ Project coverage is 68.72%. Comparing base (14b70c7) to head (410714e).
⚠️ Report is 5 commits behind head on main.

Files with missing lines Patch % Lines
megatron/core/distributed/param_and_grad_buffer.py 0.00% 4 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3182      +/-   ##
==========================================
- Coverage   68.73%   68.72%   -0.01%     
==========================================
  Files         339      339              
  Lines       44712    44719       +7     
==========================================
+ Hits        30733    30734       +1     
- Misses      13979    13985       +6     
Flag Coverage Δ
unit-test 68.72% <0.00%> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@youngeunkwon0405
youngeunkwon0405 added this pull request to the merge queue Jan 31, 2026
Merged via the queue into main with commit 5d0a7fd Jan 31, 2026
49 of 51 checks passed
@youngeunkwon0405
youngeunkwon0405 deleted the cherry-pick-3181-into-main branch January 31, 2026 04:51
daiyaanarfeen pushed a commit to daiyaanarfeen/Megatron-LM that referenced this pull request Feb 23, 2026
Signed-off-by: Youngeun Kwon <youngeunk@nvidia.com>
Co-authored-by: Youngeun Kwon <youngeunk@nvidia.com>
yangbofun pushed a commit to xlm-research/Megatron-LM that referenced this pull request May 22, 2026
Signed-off-by: Youngeun Kwon <youngeunk@nvidia.com>
Co-authored-by: Youngeun Kwon <youngeunk@nvidia.com>
terminator123 pushed a commit to 021ai/Megatron-LM that referenced this pull request Aug 3, 2026
Signed-off-by: Youngeun Kwon <youngeunk@nvidia.com>
Co-authored-by: Youngeun Kwon <youngeunk@nvidia.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cherry-pick Final Review PR is in the "final review" stage

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants