Skip to content

[EXAMPLES] Update wordle to new openenv release#4791

Merged
sergiopaniego merged 3 commits intomainfrom
update-wrodle-to-openenv
Jan 9, 2026
Merged

[EXAMPLES] Update wordle to new openenv release#4791
sergiopaniego merged 3 commits intomainfrom
update-wrodle-to-openenv

Conversation

@burtenshaw
Copy link
Collaborator

This PR make small changes to the wordle openenv example so it's compatible with openenv-0.2.0

  • update docstrings with latest install methods
  • uses one method for interaction (url)
  • fixes vLLM utils @kashif

@HuggingFaceDocBuilderDev

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

Copy link
Collaborator

@kashif kashif left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

GuidedDecodingParams is the better super class for structured output and there will be no breaking change due to this.

Copy link
Member

@sergiopaniego sergiopaniego left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! I'll update the rest of the scripts/notebooks in a follow-up PR to use the updated OpenEnv version

@sergiopaniego sergiopaniego merged commit 10c300a into main Jan 9, 2026
13 of 14 checks passed
@sergiopaniego sergiopaniego deleted the update-wrodle-to-openenv branch January 9, 2026 11:41
if is_vllm_available():
from vllm import SamplingParams
from vllm.sampling_params import StructuredOutputsParams
from vllm.sampling_params import GuidedDecodingParams
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We will have to revert this, see #4117

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ah damn!

@burtenshaw
Copy link
Collaborator Author

Sorry about this @qgallouedec. I should have been clearer that I needed those changes reviewed. 🤦

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.

5 participants