[Bugfix] yield when stage response#287
[Bugfix] yield when stage response#287Bounty-hunter wants to merge 1 commit intovllm-project:mainfrom
Conversation
|
please update the execution time comparison |
Gaohan123
left a comment
There was a problem hiding this comment.
Thanks for a nice catch!. Please use git commit -s --amend to pass the DCO check.
fake0fan
left a comment
There was a problem hiding this comment.
LGTM. But this is just the first step, ensuring that different requests can be switched between processes during execution. We will continue to solve parallel issues and out-of-order return results.
Okay, but I will try to address it together with #293 |
#293 track this problem |
|
nice catch |
|
@Bounty-hunter fix DCO |
I will close this pr. when it resolved, new problem has emerged where responses are being mismatched across concurrent requests, as describe in #293, I both of them in #301 , please help to review |
PLEASE FILL IN THE PR DESCRIPTION HERE ENSURING ALL CHECKLIST ITEMS (AT THE BOTTOM) HAVE BEEN CONSIDERED.
Purpose
As described in #286
, the execution becomes blocked while waiting for the stage response. We need to yield control so that other tasks can proceed.
Test Plan
Result:

The log as follow:
Test Result
Essential Elements of an Effective PR Description Checklist
supported_models.mdandexamplesfor a new model.BEFORE SUBMITTING, PLEASE READ https://github.com/vllm-project/vllm-omni/blob/main/CONTRIBUTING.md (anything written below this line will be removed by GitHub Actions)