Skip to content

RL: unify rollout generation into one pipeline - #6495

Merged
tdene merged 5 commits into
NVIDIA:mainfrom
tdene:tde/rl_pipeline_refactor
Aug 13, 2026
Merged

RL: unify rollout generation into one pipeline #6495
tdene merged 5 commits into
NVIDIA:mainfrom
tdene:tde/rl_pipeline_refactor

Conversation

@tdene

@tdene tdene commented Aug 12, 2026

Copy link
Copy Markdown
Contributor
  • I, the PR author, have personally reviewed every line of this PR.

What does this PR do?

Instead of 1 pipeline per environment, this has 1 pipeline total.

Issue tracking

For PRs from open-source community contributors:

  • New features: a linked issue is required. Please open a feature request and reference it here before submitting the PR.
  • Small updates (bug fixes, minor improvements): a linked issue is recommended and will accelerate the PR review process.

Linked issue:

Contribution process

Pre-checks

  • I have added relevant unit tests
  • I have added relevant functional tests
  • I have added proper typing to my code Typing guidelines
  • I have added relevant documentation
  • I have run the autoformatter.sh on my PR

Code review

Feel free to message or comment @NVIDIA/mcore-oncall to help accelerate your merge into main. The less complex your PR is, the faster it will be approved and merged!

All PRs start as draft. If you open a non-draft PR, it will be automatically converted to draft.

Step 1: Mark PR as "Ready for Review"

  1. When your PR is ready, click Ready for Review.
  2. An oncall reviewer is auto-assigned and expert reviewers are notified based on your changes.
    • Some PRs may jump straight to step 2. This is determined by .github/CODEOWNERS.

⚠️ Only mark as ready once merge-conflicts are resolved and the CI is passing.
Final Review might get declined if these requirements are not fulfilled.

Step 2: Final Review

For PRs that change megatron/core, once all expert reviewers have approved, the Final Review label is applied automatically and final reviewers are assigned.

For PRs outside megatron/core, this step is skipped.

Step 3: Approved

Once all required reviewers have approved, the Approved label is applied automatically.

Merge

Any member of mcore-engineers will be able to merge your PR.

tdene added 2 commits August 12, 2026 12:59
Signed-off-by: Teodor-Dumitru Ene <teodord.ene@gmail.com>
Signed-off-by: Teodor-Dumitru Ene <teodord.ene@gmail.com>
@copy-pr-bot

copy-pr-bot Bot commented Aug 12, 2026

Copy link
Copy Markdown

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

Contributors can view more details about this message here.

Comment thread megatron/rl/agent/weighted_multi_task.py
Comment thread megatron/rl/agent/weighted_multi_task.py Outdated
Comment thread megatron/rl/agent/api.py Outdated
Comment thread megatron/rl/agent/api.py Outdated
Comment thread megatron/rl/rl_utils.py
Comment thread megatron/rl/agent/api.py Outdated
Comment thread megatron/rl/agent/api.py Outdated
Comment thread megatron/rl/agent/api.py Outdated
Comment thread megatron/rl/agent/api.py Outdated
Comment thread megatron/rl/agent/weighted_multi_task.py Outdated
tdene added 2 commits August 13, 2026 03:09
Signed-off-by: Teodor-Dumitru Ene <teodord.ene@gmail.com>
Signed-off-by: Teodor-Dumitru Ene <teodord.ene@gmail.com>
@lauradang

Copy link
Copy Markdown
Contributor

This is a big change, so approving assuming you have run the gitlab ci/cd tests and maybe a quick test on the cluster if possible (I'm sure you have that going already)

@svcnvidia-nemo-ci svcnvidia-nemo-ci added the Approved All necessary approvals have been made label Aug 13, 2026
@tdene

tdene commented Aug 13, 2026

Copy link
Copy Markdown
Contributor Author

This is a big change, so approving assuming you have run the gitlab ci/cd tests and maybe a quick test on the cluster if possible (I'm sure you have that going already)

This run included these changes: https://wandb.ai/adlr/megatron-rl?nw=9vg5y1477c9

And I've got a more recent one from today as well, if I go dig through wandb.

@tdene
tdene enabled auto-merge August 13, 2026 08:39
@svcnvidia-nemo-ci

Copy link
Copy Markdown
Contributor

🔄 Merge queue validation started!

You can track the progress here: https://github.com/NVIDIA/Megatron-LM/actions/runs/31699429702

Merged via the queue into NVIDIA:main with commit 87bf520 Aug 13, 2026
89 checks passed
@tdene
tdene deleted the tde/rl_pipeline_refactor branch August 13, 2026 13:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Approved All necessary approvals have been made complexity: high

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants