Skip to content

chore(beep boop 🤖): Bump uv.lock (main, mcore-dev) (2026-04-13)#3297

Open
svcnvidia-nemo-ci wants to merge 2 commits intomainfrom
bump-ci-container-2026-04-13-main-dev
Open

chore(beep boop 🤖): Bump uv.lock (main, mcore-dev) (2026-04-13)#3297
svcnvidia-nemo-ci wants to merge 2 commits intomainfrom
bump-ci-container-2026-04-13-main-dev

Conversation

@svcnvidia-nemo-ci
Copy link
Copy Markdown
Contributor

@svcnvidia-nemo-ci svcnvidia-nemo-ci commented Apr 13, 2026

🚀 PR to bump uv.lock in main.

🤖 This PR will be merged automatically once CI passes.

Summary by CodeRabbit

  • Chores
    • Updated internal dependencies and development references.

This release contains no user-facing changes or new features.

Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@svcnvidia-nemo-ci
Copy link
Copy Markdown
Contributor Author

/ok to test 3898362

@copy-pr-bot
Copy link
Copy Markdown

copy-pr-bot bot commented Apr 13, 2026

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.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Apr 13, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: d7fc0177-f1b9-43f8-9779-e424fc3a8ee5

📥 Commits

Reviewing files that changed from the base of the PR and between 8dfe249 and 3898362.

⛔ Files ignored due to path filters (1)
  • uv.lock is excluded by !**/*.lock
📒 Files selected for processing (2)
  • .dev.commit
  • 3rdparty/Megatron-LM

📝 Walkthrough

Walkthrough

This pull request updates the Megatron-LM submodule reference to a new commit hash and synchronizes the repository's commit tracking file to match this submodule update.

Changes

Cohort / File(s) Summary
Submodule and Commit Tracking Updates
.dev.commit, 3rdparty/Megatron-LM
Updated Megatron-LM submodule pointer and .dev.commit file to reference commit c0c4fdc45e9f0f8047e29f2cb8669613169752c3 (previously c72c4599012297cfbd1d57e006b544478b6bbf78 and 7cbc68ce96fc586d989ebaf864afe4f648474414 respectively).

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

  • Megatron-Bridge#2968 — Updates both the Megatron-LM submodule pointer and the commit-tracking file in the same manner as this PR.
  • Megatron-Bridge#3191 — Modifies the same 3rdparty/Megatron-LM submodule reference with a different commit hash.
  • Megatron-Bridge#3210 — Updates the Megatron-LM submodule pointer to track a new upstream commit.

Suggested reviewers

  • yaoyu-33
🚥 Pre-merge checks | ✅ 3 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Title check ⚠️ Warning The title mentions bumping 'uv.lock' but the actual changes are to '.dev.commit' and a Megatron-LM submodule, with no changes to 'uv.lock' present in the changeset. Update the title to accurately reflect the actual changes: bumping the '.dev.commit' hash and Megatron-LM submodule version, not 'uv.lock'.
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
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 administrative changes: .dev.commit update, submodule version bump, and uv.lock update. No functional code modifications or breaking changes.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch bump-ci-container-2026-04-13-main-dev

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

@yaoyu-33 yaoyu-33 added area:build Dependencies, packaging, images, and environment setup needs-review PR is ready for code review and waiting on a reviewer labels Apr 13, 2026
…r and hybrid_context_parallel

state.py: wrap `EnergyMonitor` import in try/except (not yet in mcore dev),
add runtime guard before instantiation.

initialize.py: use inspect.signature to conditionally pass
`hybrid_context_parallel` kwarg to initialize_model_parallel (only in
mcore dev, not yet in main).

Signed-off-by: yaoyu-33 <yaoyu.094@gmail.com>
Made-with: Cursor
@yaoyu-33
Copy link
Copy Markdown
Contributor

/ok to test 41dbccd

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:build Dependencies, packaging, images, and environment setup full-test-suite needs-review PR is ready for code review and waiting on a reviewer

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants