Skip to content

ci: bump Megatron-Bridge to 0c565c9a0 - #3568

Merged
yfw merged 2 commits into
NVIDIA-NeMo:mainfrom
tdene:tde/bump_mbridge_0c565c9
Aug 11, 2026
Merged

ci: bump Megatron-Bridge to 0c565c9a0#3568
yfw merged 2 commits into
NVIDIA-NeMo:mainfrom
tdene:tde/bump_mbridge_0c565c9

Conversation

@tdene

@tdene tdene commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

What does this PR do ?

Add a one line overview of what this PR aims to accomplish.

Issues

List issues that this PR closes (syntax):

Usage

  • You can potentially add a usage example below
# Add a code snippet demonstrating how to use this

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 run the unit tests and functional tests locally? Visit our Testing Guide for how to run tests
  • Did you add or update any necessary documentation? Visit our Document Development Guide for how to write, build and test the docs.

Additional Information

  • ...

@tdene
tdene requested review from a team as code owners August 10, 2026 11:47
@copy-pr-bot

copy-pr-bot Bot commented Aug 10, 2026

Copy link
Copy Markdown

Auto-sync is disabled for ready for review pull requests in this repository. Workflows must be run manually.

Contributors can view more details about this message here.

@github-actions

github-actions Bot commented Aug 10, 2026

Copy link
Copy Markdown

✅ Submodule Fast-Forward Check Results

Check based on commit: 2ee53ea (PR #3568 from tde/bump_mbridge_0c565c9)

✅ Submodules that are properly updated:

Megatron-Bridge: ✅ PR branch is ahead of main branch (fast-forward)

All submodule changes look good! ✨

@tdene tdene added the CI:L1 Run doctests, unit tests, and functional tests label Aug 10, 2026
@tdene

tdene commented Aug 10, 2026

Copy link
Copy Markdown
Contributor Author

/ok to test 2ee53ea

@ZhiyuLi-Nvidia ZhiyuLi-Nvidia self-assigned this Aug 10, 2026
@svcnvidia-nemo-ci

svcnvidia-nemo-ci commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

Superseded by a newer MBridge bump PR.

@tdene tdene reopened this Aug 11, 2026
@github-actions

github-actions Bot commented Aug 11, 2026

Copy link
Copy Markdown

✅ Submodule Fast-Forward Check Results

Check based on commit: 2ee53ea (PR #3568 from tde/bump_mbridge_0c565c9)

✅ Submodules that are properly updated:

Megatron-Bridge: ✅ PR branch is ahead of main branch (fast-forward)

All submodule changes look good! ✨

@yfw yfw mentioned this pull request Aug 11, 2026
Merged
@tdene tdene added CI:Lfast Runs a fast test suite and re-use nightly `main` container (but sync dependencies to PRs version) CI:L0 Run doctests and unit tests and removed CI:L1 Run doctests, unit tests, and functional tests CI:Lfast Runs a fast test suite and re-use nightly `main` container (but sync dependencies to PRs version) labels Aug 11, 2026
yfw
yfw previously approved these changes Aug 11, 2026
View reviewed changes
@tdene
tdene force-pushed the tde/bump_mbridge_0c565c9 branch from 2ee53ea to 0d3e7d1 Compare August 11, 2026 20:34
@tdene

tdene commented Aug 11, 2026

Copy link
Copy Markdown
Contributor Author

/ok to test 0d3e7d1

@tdene tdene added CI:Lfast Runs a fast test suite and re-use nightly `main` container (but sync dependencies to PRs version) and removed CI:L0 Run doctests and unit tests labels Aug 11, 2026
@github-actions

github-actions Bot commented Aug 11, 2026

Copy link
Copy Markdown

✅ Submodule Fast-Forward Check Results

Check based on commit: 0d3e7d1 (PR #3568 from tde/bump_mbridge_0c565c9)

✅ Submodules that are properly updated:

Megatron-Bridge: ✅ PR branch is ahead of main branch (fast-forward)

All submodule changes look good! ✨

tdene added 2 commits August 11, 2026 15:35
@tdene
chore: bump Megatron-Bridge to 0c565c9a0
7ab55a5 
Signed-off-by: Teodor-Dumitru Ene <teodord.ene@gmail.com>
@tdene
Fix isinstance against concrete MFSDP classes
aae969b 
Signed-off-by: Teodor-Dumitru Ene <teodord.ene@gmail.com>
@tdene
tdene force-pushed the tde/bump_mbridge_0c565c9 branch from 0d3e7d1 to aae969b Compare August 11, 2026 20:36
@tdene

tdene commented Aug 11, 2026

Copy link
Copy Markdown
Contributor Author

/ok to test aae969b

@github-actions

github-actions Bot commented Aug 11, 2026

Copy link
Copy Markdown

✅ Submodule Fast-Forward Check Results

Check based on commit: aae969b (PR #3568 from tde/bump_mbridge_0c565c9)

✅ Submodules that are properly updated:

Megatron-Bridge: ✅ PR branch is ahead of main branch (fast-forward)

All submodule changes look good! ✨

@yfw
yfw enabled auto-merge (squash) August 11, 2026 21:35
@yfw
yfw merged commit 46ab18c into NVIDIA-NeMo:main Aug 11, 2026
82 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@yfw yfw yfw approved these changes

Assignees

@ZhiyuLi-Nvidia ZhiyuLi-Nvidia

Labels

CI:Lfast Runs a fast test suite and re-use nightly `main` container (but sync dependencies to PRs version)

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@tdene @svcnvidia-nemo-ci @yfw @ZhiyuLi-Nvidia