Skip to content

Require --env-url in the Wordle GRPO script - #1537

Merged
sergiopaniego merged 3 commits into
openenv:mainfrom
sergiopaniego:wordle-require-env-url
Nov 3, 2026
Merged

sergiopaniego merged 3 commits into
openenv:mainfrom
sergiopaniego:wordle-require-env-url

Conversation

@sergiopaniego

@sergiopaniego sergiopaniego commented Oct 30, 2026 •

Copy link
Copy Markdown
Collaborator

Each GRPO step opens 64 sessions, and openenv/wordle allows 8 (MAX_CONCURRENT_ENVS). With the official Space as the default, a run without --env-url fails partway into the first step. Now grpo.py requires --env-url, so it fails at startup, and the README and tutorial already say to pass the URL of your copy.

evaluate.py keeps its default, since it plays one game at a time. Both scripts now call get_token() once.

Same change as in the TRL Wordle example (huggingface/trl#7932).


Note

Low Risk
CLI and test-only changes for the Wordle example scripts; no changes to auth logic beyond avoiding a duplicate token lookup.

Overview
GRPO training now requires --env-url at startup (no default to the public openenv-wordle Space). That avoids a mid-step failure when a run opens many concurrent sessions against a Space capped at 8 environments while GRPO needs 64.

evaluate.py still defaults --env-url for single-game runs. evaluate.py and grpo.py both call get_token() once when building optional auth headers for private Spaces.

Tests pass --env-url when exercising grpo.py.

Reviewed by Cursor Bugbot for commit 4c8463d. Bugbot is set up for automated code reviews on this repo. Configure here.

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

Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 3158933. Configure here.

Comment thread examples/wordle/grpo.py Resolved
@sergiopaniego
sergiopaniego merged commit f6c0043 into openenv:main Nov 3, 2026
12 checks passed
@sergiopaniego
sergiopaniego deleted the wordle-require-env-url branch November 3, 2026 14:50
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.

1 participant