Skip to content

Conversation

WoosukKwon
Copy link
Collaborator

No description provided.

Signed-off-by: Woosuk Kwon <woosuk.kwon@berkeley.edu>
@WoosukKwon WoosukKwon added the ready ONLY add when PR is ready to merge/full CI is needed label Sep 18, 2025
Copy link
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 a number of deprecated tests associated with the V0 engine, which is a good cleanup effort. The deleted test files are all specific to V0 features, and their removal is appropriate. The one modification made in tests/engine/test_short_mm_context.py introduces load_format="dummy", which is a sensible optimization to speed up tests that don't require actual model weights. The changes are correct and align with the goal of deprecating the V0 engine.

@WoosukKwon WoosukKwon merged commit 2c3c1bd into main Sep 18, 2025
31 of 32 checks passed
@WoosukKwon WoosukKwon deleted the woosuk/remove-v0-engine-tests branch September 18, 2025 02:38
debroy-rh pushed a commit to debroy-rh/vllm that referenced this pull request Sep 19, 2025
Signed-off-by: Woosuk Kwon <woosuk.kwon@berkeley.edu>
FeiDaLI pushed a commit to FeiDaLI/vllm that referenced this pull request Sep 25, 2025
Signed-off-by: Woosuk Kwon <woosuk.kwon@berkeley.edu>
charlifu pushed a commit to ROCm/vllm that referenced this pull request Sep 25, 2025
Signed-off-by: Woosuk Kwon <woosuk.kwon@berkeley.edu>
Signed-off-by: charlifu <charlifu@amd.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready ONLY add when PR is ready to merge/full CI is needed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant