chore(beep boop 🤖): Bump uv.lock (main) (2026-03-15)#2807
chore(beep boop 🤖): Bump uv.lock (main) (2026-03-15)#2807svcnvidia-nemo-ci wants to merge 1 commit intomainfrom
uv.lock (main) (2026-03-15)#2807Conversation
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
|
/ok to test 32f0c48 |
📝 WalkthroughWalkthroughThis PR updates two version tracking references to point to a new Megatron-LM commit hash. The Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Possibly related PRs
Suggested reviewers
🚥 Pre-merge checks | ✅ 3 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
📝 Coding Plan
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In @.main.commit:
- Line 1: Update the PR title and description to accurately reflect that the
change updates the Megatron-LM submodule by bumping the tracked commit in
.main.commit to f8becec65f47982c80c3d397bef7c3fba65f9efc (a bugfix release:
"Miscellaneous inference bug fixes `#3840`") instead of saying it bumps uv.lock;
edit the PR title and body to mention updating the Megatron-LM submodule commit
and note that this is a non-breaking bugfix release, including the commit hash
f8becec65f47982c80c3d397bef7c3fba65f9efc for traceability.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: a0db21a1-f67c-4153-ae92-c357ddd095cf
⛔ Files ignored due to path filters (1)
uv.lockis excluded by!**/*.lock
📒 Files selected for processing (2)
.main.commit3rdparty/Megatron-LM
| @@ -1 +1 @@ | |||
| 7ca9dc56b1e52a02b495493f2588c85e214b488e | |||
| f8becec65f47982c80c3d397bef7c3fba65f9efc | |||
There was a problem hiding this comment.
🧩 Analysis chain
🏁 Script executed:
#!/bin/bash
# Description: Verify the new Megatron-LM commit exists and get its details
# Check if the commit exists in the submodule
cd 3rdparty/Megatron-LM 2>/dev/null || {
echo "Megatron-LM submodule not found at 3rdparty/Megatron-LM"
exit 1
}
# Fetch the latest changes to ensure we have the commit
git fetch origin
# Verify the new commit exists and show details
echo "=== Checking new commit f8becec65f47982c80c3d397bef7c3fba65f9efc ==="
git cat-file -t f8becec65f47982c80c3d397bef7c3fba65f9efc 2>/dev/null || {
echo "ERROR: Commit f8becec65f47982c80c3d397bef7c3fba65f9efc not found"
exit 1
}
echo -e "\n=== Commit details ==="
git log -1 --oneline f8becec65f47982c80c3d397bef7c3fba65f9efc
echo -e "\n=== Checking old commit 7ca9dc56b1e52a02b495493f2588c85e214b488e ==="
git cat-file -t 7ca9dc56b1e52a02b495493f2588c85e214b488e 2>/dev/null || {
echo "ERROR: Old commit 7ca9dc56b1e52a02b495493f2588c85e214b488e not found"
exit 1
}
echo -e "\n=== Commits between old and new ==="
git log --oneline 7ca9dc56b1e52a02b495493f2588c85e214b488e..f8becec65f47982c80c3d397bef7c3fba65f9efcRepository: NVIDIA-NeMo/Megatron-Bridge
Length of output: 204
🏁 Script executed:
git ls-files | head -20Repository: NVIDIA-NeMo/Megatron-Bridge
Length of output: 739
🏁 Script executed:
cat .main.commit 2>/dev/null || echo "File not found"Repository: NVIDIA-NeMo/Megatron-Bridge
Length of output: 111
🏁 Script executed:
git config --file .gitmodules --name-only --get-regexp path 2>/dev/null | head -20 || echo "No submodules found or .gitmodules doesn't exist"Repository: NVIDIA-NeMo/Megatron-Bridge
Length of output: 106
🏁 Script executed:
fd ".main.commit" --type fRepository: NVIDIA-NeMo/Megatron-Bridge
Length of output: 53
🏁 Script executed:
find . -name ".main.commit" -type f 2>/dev/nullRepository: NVIDIA-NeMo/Megatron-Bridge
Length of output: 85
🏁 Script executed:
git config --file .gitmodules --get submodule.3rdparty/Megatron-LM.urlRepository: NVIDIA-NeMo/Megatron-Bridge
Length of output: 112
🏁 Script executed:
cat .dev.commitRepository: NVIDIA-NeMo/Megatron-Bridge
Length of output: 111
🏁 Script executed:
ls -la | grep "\.commit"Repository: NVIDIA-NeMo/Megatron-Bridge
Length of output: 201
🏁 Script executed:
git submodule statusRepository: NVIDIA-NeMo/Megatron-Bridge
Length of output: 133
🌐 Web query:
site:github.com/NVIDIA/Megatron-LM commit f8becec65f47982c80c3d397bef7c3fba65f9efc
💡 Result:
Commit f8becec65f47982c80c3d397bef7c3fba65f9efc in NVIDIA/Megatron-LM is titled “Miscellaneous inference bug fixes (#3840)”, authored by Keshav Santhanam, dated Fri, Mar 13, 2026 17:49:46 -0700. [1]
It changes 9 files with 5,275 insertions and 4,762 deletions, mainly in Megatron Core inference dynamic context/engine and related Mamba metadata, plus updates to H100 test recipes and multiple inference unit/functional test fixtures (including a large update to golden_values_dev_dgx_h100.json). [1][2]
Sources: [1] [2]
Update PR description to accurately reflect changes.
The PR title and description mention bumping uv.lock, but the actual changes update .main.commit (tracking file for the Megatron-LM submodule) to commit f8becec65f47982c80c3d397bef7c3fba65f9efc. The new commit is a valid bug fix release ("Miscellaneous inference bug fixes #3840") from Megatron-LM and does not introduce breaking changes, but the PR metadata should be corrected to accurately describe what is being updated.
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In @.main.commit at line 1, Update the PR title and description to accurately
reflect that the change updates the Megatron-LM submodule by bumping the tracked
commit in .main.commit to f8becec65f47982c80c3d397bef7c3fba65f9efc (a bugfix
release: "Miscellaneous inference bug fixes `#3840`") instead of saying it bumps
uv.lock; edit the PR title and body to mention updating the Megatron-LM
submodule commit and note that this is a non-breaking bugfix release, including
the commit hash f8becec65f47982c80c3d397bef7c3fba65f9efc for traceability.
|
Superseded by latest nightly bump. |
3 participants
🚀 PR to bump
uv.lockinmain.🤖 This PR will be merged automatically once CI passes.
Summary by CodeRabbit