Skip to content

[LLM] Wire MATH and Countdown into GRPO and Expert Iteration scripts#3546

Open
vmoens wants to merge 1 commit intogh/vmoens/238/basefrom
gh/vmoens/238/head
Open

[LLM] Wire MATH and Countdown into GRPO and Expert Iteration scripts#3546
vmoens wants to merge 1 commit intogh/vmoens/238/basefrom
gh/vmoens/238/head

Conversation

@vmoens
Copy link
Collaborator

@vmoens vmoens commented Mar 5, 2026

Stack from ghstack (oldest at bottom):

Add GRPO config files for math and countdown datasets, and update
grpo_utils.py and ei_utils.py to support the new dataset choices.

All four datasets (gsm8k, ifeval, math, countdown) are now selectable
via the env.dataset config key.

Made-with: Cursor

[ghstack-poisoned]
vmoens added a commit that referenced this pull request Mar 5, 2026
Add GRPO config files for math and countdown datasets, and update
grpo_utils.py and ei_utils.py to support the new dataset choices.

All four datasets (gsm8k, ifeval, math, countdown) are now selectable
via the env.dataset config key.

Made-with: Cursor
ghstack-source-id: 8713155
Pull-Request: #3546
@pytorch-bot
Copy link

pytorch-bot bot commented Mar 5, 2026

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/rl/3546

Note: Links to docs will display an error until the docs builds have been completed.

❌ 4 New Failures, 1 Cancelled Job, 6 Unrelated Failures

As of commit 783db8a with merge base 491ed0f (image):

NEW FAILURES - The following jobs have failed:

CANCELLED JOB - The following job was cancelled. Please retry:

BROKEN TRUNK - The following jobs failed but were present on the merge base:

👉 Rebase onto the `viable/strict` branch to avoid these failures

This comment was automatically generated by Dr. CI and updates every 15 minutes.

@github-actions
Copy link
Contributor

github-actions bot commented Mar 5, 2026

⚠️ PR Title Label Error

Unknown or invalid prefix [LLM].

Current title: [LLM] Wire MATH and Countdown into GRPO and Expert Iteration scripts

Supported Prefixes (case-sensitive)

Your PR title must start with exactly one of these prefixes:

Prefix Label Applied Example
[BugFix] BugFix [BugFix] Fix memory leak in collector
[Feature] Feature [Feature] Add new optimizer
[Doc] or [Docs] Documentation [Doc] Update installation guide
[Refactor] Refactoring [Refactor] Clean up module imports
[CI] CI [CI] Fix workflow permissions
[Test] or [Tests] Tests [Tests] Add unit tests for buffer
[Environment] or [Environments] Environments [Environments] Add Gymnasium support
[Data] Data [Data] Fix replay buffer sampling
[Performance] or [Perf] Performance [Performance] Optimize tensor ops
[BC-Breaking] bc breaking [BC-Breaking] Remove deprecated API
[Deprecation] Deprecation [Deprecation] Mark old function
[Quality] Quality [Quality] Fix typos and add codespell

Note: Common variations like singular/plural are supported (e.g., [Doc] or [Docs]).

@github-actions
Copy link
Contributor

github-actions bot commented Mar 5, 2026

⚠️ PR Title Label Error

Unknown or invalid prefix [LLM].

Current title: [LLM] Wire MATH and Countdown into GRPO and Expert Iteration scripts

Supported Prefixes (case-sensitive)

Your PR title must start with exactly one of these prefixes:

Prefix Label Applied Example
[BugFix] BugFix [BugFix] Fix memory leak in collector
[Feature] Feature [Feature] Add new optimizer
[Doc] or [Docs] Documentation [Doc] Update installation guide
[Refactor] Refactoring [Refactor] Clean up module imports
[CI] CI [CI] Fix workflow permissions
[Test] or [Tests] Tests [Tests] Add unit tests for buffer
[Environment] or [Environments] Environments [Environments] Add Gymnasium support
[Data] Data [Data] Fix replay buffer sampling
[Performance] or [Perf] Performance [Performance] Optimize tensor ops
[BC-Breaking] bc breaking [BC-Breaking] Remove deprecated API
[Deprecation] Deprecation [Deprecation] Mark old function
[Quality] Quality [Quality] Fix typos and add codespell

Note: Common variations like singular/plural are supported (e.g., [Doc] or [Docs]).

@meta-cla meta-cla bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Mar 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. sota-implementations/

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant