Skip to content

Scheduled removal of guided_* config fields#29326

Merged
DarkLight1337 merged 1 commit intovllm-project:mainfrom
hmellor:remove-guided-decoding-fields
Nov 25, 2025
Merged

Scheduled removal of guided_* config fields#29326
DarkLight1337 merged 1 commit intovllm-project:mainfrom
hmellor:remove-guided-decoding-fields

Conversation

@hmellor
Copy link
Copy Markdown
Member

@hmellor hmellor commented Nov 24, 2025

These fields are scheduled for removal in v0.12.0, which will be the next release.

The config being removed now lives in StructuredOuptutsConfig/StructuredOutputsParams

These fields are scheduled for removal in v0.12.0, which will be the next release.

The config being removed now lives in `StructuredOuptutsConfig`/`StructuredOutputsParams`

Signed-off-by: Harry Mellor <19981378+hmellor@users.noreply.github.com>
@mergify
Copy link
Copy Markdown

mergify bot commented Nov 24, 2025

Documentation preview: https://vllm--29326.org.readthedocs.build/en/29326/

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request effectively removes the deprecated guided_* configuration fields and associated logic across the codebase. The changes are consistent and correctly replace the old API with the new structured_outputs parameters. The removal of deprecated code from EngineArgs, SamplingParams, and the OpenAI protocol classes is clean. The related tests have also been updated appropriately to reflect these changes. The refactoring in vllm/entrypoints/openai/protocol.py to handle response_format is a nice improvement that simplifies the code. Overall, this is a solid cleanup effort with no issues found.

@DarkLight1337 DarkLight1337 enabled auto-merge (squash) November 25, 2025 03:12
@github-actions github-actions bot added the ready ONLY add when PR is ready to merge/full CI is needed label Nov 25, 2025
@DarkLight1337 DarkLight1337 merged commit 316c849 into vllm-project:main Nov 25, 2025
53 of 54 checks passed
@hmellor hmellor deleted the remove-guided-decoding-fields branch November 25, 2025 10:33
devpatelio pushed a commit to SumanthRH/vllm that referenced this pull request Nov 29, 2025
Signed-off-by: Harry Mellor <19981378+hmellor@users.noreply.github.com>
kitaekatt pushed a commit to kitaekatt/vllm that referenced this pull request Dec 1, 2025
Signed-off-by: Harry Mellor <19981378+hmellor@users.noreply.github.com>
nrghosh added a commit to kouroshHakha/ray that referenced this pull request Dec 7, 2025
vLLM renamed guided_decoding to structured_outputs and changed
the embedding API:

- SamplingParams: GuidedDecodingParams -> StructuredOutputsParams,
  guided_decoding -> structured_outputs (vllm-project/vllm#22772,
  vllm-project/vllm#29326)

- Embedding: use encode(pooling_params=...) instead of
  generate(sampling_params=...) for pooling tasks
  (vllm-project/vllm#16188, vllm-project/vllm#25524)

- EngineArgs: guided_decoding_backend -> structured_outputs_config

User-facing "guided_decoding" key in sampling_params dict preserved
for backwards compatibility.

Signed-off-by: Nikhil Ghosh <nikhil@anyscale.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation frontend ready ONLY add when PR is ready to merge/full CI is needed structured-output v1

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants