Skip to content

[Dev] Cherry-pick: M-FSDP: Cancel erroneous grad accumulation check (#3629) - #3729

Merged
yaox12 merged 1 commit into
NVIDIA:devfrom
Victarry:denliu/cherry-pick-3629-dev
Mar 6, 2026
Merged

[Dev] Cherry-pick: M-FSDP: Cancel erroneous grad accumulation check (#3629)#3729
yaox12 merged 1 commit into
NVIDIA:devfrom
Victarry:denliu/cherry-pick-3629-dev

Conversation

@Victarry

@Victarry Victarry commented Mar 6, 2026

Copy link
Copy Markdown

Summary

Cherry-pick of #3629 from main into dev.

  • Removes the incorrect assert that blocked EP + gradient_accumulation_fusion in Megatron-FSDP. Since Transformer Engine v2.10, these features are compatible, so the restriction is no longer needed.

Original PR: #3629
Original Author: @shjwudp

Made with Cursor

@Victarry
Victarry requested review from a team as code owners March 6, 2026 02:30
@copy-pr-bot

copy-pr-bot Bot commented Mar 6, 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.

@yaox12
yaox12 enabled auto-merge March 6, 2026 02:42
@Victarry

Victarry commented Mar 6, 2026

Copy link
Copy Markdown
Author

/ok to test 0eb36e3

@Victarry

Victarry commented Mar 6, 2026

Copy link
Copy Markdown
Author

/ok to test 0eb36e3

@yaox12
yaox12 added this pull request to the merge queue Mar 6, 2026
@svcnvidia-nemo-ci

Copy link
Copy Markdown
Contributor

🔄 Merge queue validation started!

You can track the progress here: https://github.com/NVIDIA/Megatron-LM/actions/runs/22753600664

@svcnvidia-nemo-ci

Copy link
Copy Markdown
Contributor

🔄 Merge queue validation started!

You can track the progress here: https://github.com/NVIDIA/Megatron-LM/actions/runs/22755241911

Merged via the queue into NVIDIA:dev with commit 597f0d8 Mar 6, 2026
49 checks passed
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.

4 participants