Skip to content

Fix two minor bugs in MTP implementation for hybrid models - #3194

Merged
ko3n1g merged 2 commits into
NVIDIA:mainfrom
deepakn94:dnarayanan/mtp_bugfixes
Feb 2, 2026
Merged

Fix two minor bugs in MTP implementation for hybrid models#3194
ko3n1g merged 2 commits into
NVIDIA:mainfrom
deepakn94:dnarayanan/mtp_bugfixes

Conversation

@deepakn94

Copy link
Copy Markdown
Contributor

This fixes broken functional tests from #2363

@deepakn94

Copy link
Copy Markdown
Contributor Author

/ok to test bf5e49e

Signed-off-by: Deepak Narayanan <dnarayanan@nvidia.com>
Signed-off-by: Deepak Narayanan <dnarayanan@nvidia.com>

@ko3n1g ko3n1g left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems to fix the issue

@ko3n1g
ko3n1g merged commit e836e62 into NVIDIA:main Feb 2, 2026
48 of 50 checks passed
ko3n1g added a commit that referenced this pull request Feb 2, 2026
sancha added a commit to sancha/Megatron-LM that referenced this pull request Feb 2, 2026
arendu pushed a commit to arendu/Megatron-LM that referenced this pull request Feb 5, 2026
daiyaanarfeen pushed a commit to daiyaanarfeen/Megatron-LM that referenced this pull request Feb 23, 2026
Signed-off-by: Deepak Narayanan <dnarayanan@nvidia.com>
daiyaanarfeen pushed a commit to daiyaanarfeen/Megatron-LM that referenced this pull request Feb 23, 2026
@deepakn94
deepakn94 deleted the dnarayanan/mtp_bugfixes branch April 2, 2026 16:34
yangbofun pushed a commit to xlm-research/Megatron-LM that referenced this pull request May 22, 2026
Signed-off-by: Deepak Narayanan <dnarayanan@nvidia.com>
yangbofun pushed a commit to xlm-research/Megatron-LM that referenced this pull request May 22, 2026
terminator123 pushed a commit to 021ai/Megatron-LM that referenced this pull request Aug 3, 2026
Signed-off-by: Deepak Narayanan <dnarayanan@nvidia.com>
terminator123 pushed a commit to 021ai/Megatron-LM that referenced this pull request Aug 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants