Skip to content

RFC: GRPO recipes - #924

Draft
burtenshaw wants to merge 2 commits into
huggingface:mainfrom
burtenshaw:ben/rfc-grpo-harness-recipes
Draft

RFC: GRPO recipes#924
burtenshaw wants to merge 2 commits into
huggingface:mainfrom
burtenshaw:ben/rfc-grpo-harness-recipes

Conversation

@burtenshaw

Copy link
Copy Markdown
Collaborator

This PR adds an RFC for framework-specific GRPO harness training recipes.

@sergiopaniego

sergiopaniego commented Jul 31, 2026

Copy link
Copy Markdown
Member

Hey, coming back to this one. I've opened #1028 and I'm leaving some comments here (iterated with CC) so we can iterate and merge.

The big change since this was filed is that the reference implementation already landed on the TRL side:

I checked it against the "What The Recipe Must Specify" list and it covers every bullet, with one caveat about where the validation evidence lives (more below). So I'd say the two open questions are answered de facto: OpenCode is the first reference harness (Pi next), and coding tasks with a held-out verifier are the smoke and representative task.

With that, a few suggestions to get this merged:

  1. Keep the recipe contract (the checklist) in this RFC, and add the template header (Status / Created / Authors / RFC ID) so it matches the other RFCs.
  2. Let recipes live where they already live: each framework maintains its own in its repo, in its own idioms, and OpenEnv's deliverable is a curated index. We already do exactly this in docs/source/tutorials/index.md, which lists TRL, Unsloth and Inspect AI workflows and even links to a notebook hosted in the TRL repo. What was missing is one row: the black-box OpenCode + AsyncGRPO recipe was not discoverable from OpenEnv at all (the BrowserGym harness tutorial covers the earlier pattern where the trainer keeps sampling). That's what docs(tutorials): add black-box coding agent training tutorial (OpenCode + AsyncGRPO) #1028 adds.
  3. Make the validation bar explicit with two statuses: runnable (the script runs end to end) and validated (a training run with metrics, before/after evals or a checkpoint). The OpenCode recipe has in fact been validated end to end on Qwen3 (Async grpo OpenEnv harness rollout trl#6420, results thread), but that evidence lives in a PR body and an X thread today, while the Wordle recipe keeps a results section next to the recipe itself. So the bar should also say where the evidence gets captured, and a status column in the index implements all of this with zero extra machinery.

One last thing worth stating in the text: the "no trajectory serialization format yet" non-goal points at #941, so this RFC effectively layers on top of the interception contract there. A one-line cross-reference would help the two merge in a sane order.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants