Skip to content

Vitalyk/multiturn v2 - #3167

Merged
ko3n1g merged 3 commits into
NVIDIA:mainfrom
yobibyte:vitalyk/multiturn-v2
Jan 30, 2026
Merged

Vitalyk/multiturn v2#3167
ko3n1g merged 3 commits into
NVIDIA:mainfrom
yobibyte:vitalyk/multiturn-v2

Conversation

@yobibyte

Copy link
Copy Markdown
Contributor

What does this PR do?

Why was #2966 reverted?

We failed a gitlab CI test on timings. Every second iteration of the post-change test was extremely fast.

Why did it happen?

This was the configuration of a faulty test:

gpt/gpt_grpo_tp4_pp1_dp2_8b_throughput/model_config.yaml
78:  --global-batch-size: 2
79:  --grpo-group-size: 2
80:  --grpo-prompts-per-step: 2

We sampled 4 rollouts each before and were supposed to consume 2 rollouts per training step (global bs = 2).
Before #2966, sequence packing had a bug which made it sample every iteration no matter on whether we had consumed all the samples or not.
The huge timings you saw in the test were mostly for inference, training step did not take long.

Why didn't it fail on Github CI?

Our github CI used a correct batch size which made it consume all data that was sampled and not being affected by a bug I fixed in #2966.

@yobibyte
yobibyte requested a review from a team as a code owner January 30, 2026 11:46
@copy-pr-bot

copy-pr-bot Bot commented Jan 30, 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.

@ko3n1g
ko3n1g requested a review from a team January 30, 2026 11:46
@yobibyte

Copy link
Copy Markdown
Contributor Author

/ok to test ee6228e

@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.

Fast-merging this replay

@ko3n1g
ko3n1g merged commit b9ee19e into NVIDIA:main Jan 30, 2026
54 of 58 checks passed
daiyaanarfeen pushed a commit to daiyaanarfeen/Megatron-LM that referenced this pull request Feb 23, 2026
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
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