Repository navigation
[Refactor] Stage Proc/Client for LLM and DiT #5441
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Closed
Closed
Changes from all commits
Commits
Show all changes
42 commits
Select commit
Hold shift + click to select a range
d7848ed
refactor stage proc and client
chickeyton a1915dc
Fix StageLLMCoreProcManager kwarg key to match run_stage_core param
chickeyton 40d1c0a
bugfix
chickeyton f6590a8
enhancement
chickeyton 1f1d0d9
fix arg name
chickeyton 25a1e19
Port main's post-refactor stage changes into vllm_omni/engine/stage
chickeyton af0fdf6
Fix rebase gap: rename OmniEngineCoreRequest in model_intermediate_bu…
chickeyton a52cc79
Fix stale test doubles in test_orchestrator for refactored stage pool…
chickeyton ab41b0c
Remove 6 legacy stage modules superseded by engine/stage subpackage
chickeyton 31e347f
Remove stage_client.py by dropping its type-only Protocol surface
chickeyton 73175dc
Make InlineStageDiffusionClient a true StageCoreClientBase subclass
chickeyton 4d997f1
Fix test_run_llm_to_diffusion assertion for typed diffusion request
chickeyton d4c85aa
Simplify engine/stage/__init__.py: drop unused re-export layer
chickeyton 0b01b0d
Relocate diffusion stage modules to vllm_omni/diffusion/stage/
chickeyton 0846abe
Port main's preprocess_add_request into relocated StageLLMCoreProc
chickeyton 1a9178a
Fix post-rebase stage test breakage surfaced on server 1
chickeyton 9734b1b
Drop redundant quotes on make_async_mp_client annotations
chickeyton 78e6fb8
Apply pre-commit: ruff format/lint, wrap long log lines, add test marks
chickeyton 06b45c1
Adopt centralized vllm_omni.metrics helpers in StageReplicaPool
chickeyton 693a869
Port submit_interaction (midway prompt update #4652) into relocated s…
chickeyton f807bf6
Merge branch 'main' into clean_stage_proc_rebase
chickeyton ba9e3dd
update doc
chickeyton 78c7fad
Merge branch 'clean_stage_proc_rebase' of https://github.com/chickeyt…
chickeyton 759e580
fix review comments
chickeyton 42fe2f2
Merge remote-tracking branch 'origin/main' into clean_stage_proc_rebase
chickeyton 2712a63
Merge remote-tracking branch 'origin/main' into clean_stage_proc_rebase
chickeyton cf4288e
Apply ruff-format to deprecation warning string
chickeyton ae8320a
[docs] Add stage type extensibility example to async omni architecture
chickeyton ea167a1
Merge remote-tracking branch 'origin/main' into clean_stage_proc_rebase
chickeyton 0fef82f
[refactor] Replace Any with concrete types in stage client/pool signa…
chickeyton 44ddf83
Merge branch 'main' into clean_stage_proc_rebase
chickeyton 8d389d9
Merge branch 'main' into clean_stage_proc_rebase
chickeyton 455865e
Merge remote-tracking branch 'origin/main' into clean_stage_proc_rebase
chickeyton 268b1b3
Merge branch 'main' into clean_stage_proc_rebase
chickeyton c9bf498
Merge remote-tracking branch 'origin/main' into clean_stage_proc_rebase
chickeyton 8e6af27
Merge remote-tracking branch 'origin/clean_stage_proc_rebase' into cl…
chickeyton 40be871
Merge branch 'main' into clean_stage_proc_rebase
chickeyton 0aa670e
Merge branch 'main' of https://github.com/vllm-project/vllm-omni into…
chickeyton 90d4fad
Merge remote-tracking branch 'origin/clean_stage_proc_rebase' into cl…
chickeyton 0802e23
Merge branch 'main' of https://github.com/vllm-project/vllm-omni into…
chickeyton 2cdb989
Merge branch 'main' of https://github.com/vllm-project/vllm-omni into…
chickeyton fd880bf
Merge branch 'main' of https://github.com/vllm-project/vllm-omni into…
chickeyton File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
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
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
Oops, something went wrong.
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[P3] Update the later data-structures section too
This correctly documents
source_outputs, but the “Data Structures in Stage Transitions” section immediately below still tells authors to readstage_list[source_stage_id].engine_outputs, which no longer exists after this refactor. Please update that section to describesource_outputsitself (and its element shape) so contributors do not implement the removed API.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed. The "Data Structures in Stage Transitions" section and the "Key Points" list
no longer reference the removed
stage_list[source_stage_id].engine_outputs. Bothnow describe
source_outputsdirectly and its element shape, which I verifiedagainst a real consumer (
stage_input_processors/dynin_omni.py:for source_output in source_outputs: output = source_output.outputs[0]; output.multimodal_output):request_idandoutputs(per-sequence; typicallyoutputs[0])token_ids/cumulative_token_ids,multimodal_output, andprompt_token_idsThere was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed, thanks!