[Feature][Response API] Add streaming support for non-harmony#23741
Merged
DarkLight1337 merged 3 commits intovllm-project:mainfrom Sep 4, 2025
Merged
[Feature][Response API] Add streaming support for non-harmony#23741DarkLight1337 merged 3 commits intovllm-project:mainfrom
DarkLight1337 merged 3 commits intovllm-project:mainfrom
Conversation
8dc2da4 to
b65638e
Compare
b65638e to
3bb6902
Compare
6d9fe9c to
af25d9a
Compare
Contributor
Author
|
@heheda12345 PTAL |
heheda12345
reviewed
Sep 2, 2025
Collaborator
heheda12345
left a comment
There was a problem hiding this comment.
Thanks for your contribution. Some small comments.
77bd0aa to
bc4c5ae
Compare
heheda12345
reviewed
Sep 3, 2025
Collaborator
There was a problem hiding this comment.
Thanks for the quick update. Can you also update the "current_item_id"?
Contributor
Author
There was a problem hiding this comment.
Thank you for the reminder, my apologies for the oversight, fixed.
bc4c5ae to
cf993d1
Compare
heheda12345
approved these changes
Sep 3, 2025
Collaborator
heheda12345
left a comment
There was a problem hiding this comment.
LGTM! Thanks for your contribution.
Collaborator
|
@kebe7jun The v1-test-entrypoints CI failure seems to be related to this PR. Can you take a look? |
auto-merge was automatically disabled
September 4, 2025 01:14
Head branch was pushed to by a user without write access
77ef2de to
30d435e
Compare
Signed-off-by: Kebe <mail@kebe7jun.com>
Signed-off-by: Kebe <mail@kebe7jun.com>
Signed-off-by: Kebe <mail@kebe7jun.com>
30d435e to
3e604da
Compare
4 tasks
eicherseiji
pushed a commit
to eicherseiji/vllm
that referenced
this pull request
Sep 9, 2025
…roject#23741) Signed-off-by: Kebe <mail@kebe7jun.com>
4 tasks
ABC12345anouys
pushed a commit
to ABC12345anouys/vllm
that referenced
this pull request
Sep 25, 2025
…roject#23741) Signed-off-by: Kebe <mail@kebe7jun.com>
mystous
pushed a commit
to mystous/vllm_hybrid
that referenced
this pull request
May 10, 2026
…roject#23741) Signed-off-by: Kebe <mail@kebe7jun.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Purpose
Add streaming support for non-harmony
Related issue #23225
Test Plan
Unit tests and self tests(see result).
Test Result
GPT-OSS Stream output
Qwen3 30B A3B Stream output
Essential Elements of an Effective PR Description Checklist
supported_models.mdandexamplesfor a new model.