Skip to content
GitLab
Menu
Projects
Groups
Snippets
Loading...
Help
Help
Support
Community forum
Keyboard shortcuts
?
Submit feedback
Contribute to GitLab
Sign in
Toggle navigation
Menu
Open sidebar
OpenDAS
vllm_cscc
Commits
3d64d366
Unverified
Commit
3d64d366
authored
Jun 08, 2025
by
Richard Zou
Committed by
GitHub
Jun 08, 2025
Browse files
[Misc] Change tests/compile to use VLLM_V1 by default (#19302)
Signed-off-by:
rzou
<
zou3519@gmail.com
>
parent
eaa2e510
Changes
2
Hide whitespace changes
Inline
Side-by-side
Showing
2 changed files
with
2 additions
and
15 deletions
+2
-15
tests/compile/conftest.py
tests/compile/conftest.py
+0
-15
tests/compile/piecewise/test_simple.py
tests/compile/piecewise/test_simple.py
+2
-0
No files found.
tests/compile/conftest.py
deleted
100644 → 0
View file @
eaa2e510
# SPDX-License-Identifier: Apache-2.0
# SPDX-FileCopyrightText: Copyright contributors to the vLLM project
import
pytest
# TEST V1: this should be removed. Right now V1 overrides
# all the torch compile logic. We should re-enable this
# as we add torch compile support back to V1.
@
pytest
.
fixture
(
scope
=
"function"
,
autouse
=
True
)
def
use_v0_only
(
monkeypatch
):
"""
Since this module is V0 only, set VLLM_USE_V1=0 for
all tests in the module.
"""
monkeypatch
.
setenv
(
'VLLM_USE_V1'
,
'0'
)
tests/compile/piecewise/test_simple.py
View file @
3d64d366
...
...
@@ -13,6 +13,7 @@ from vllm.compilation.counter import compilation_counter
from
vllm.compilation.decorators
import
support_torch_compile
from
vllm.config
import
(
CompilationConfig
,
CompilationLevel
,
VllmConfig
,
set_current_vllm_config
)
from
vllm.envs
import
VLLM_USE_V1
from
vllm.utils
import
direct_register_custom_op
global_counter
=
0
...
...
@@ -76,6 +77,7 @@ class SillyModel(nn.Module):
def
_test_simple_piecewise_compile
(
*
,
use_inductor
):
assert
VLLM_USE_V1
vllm_config
=
VllmConfig
(
compilation_config
=
CompilationConfig
(
level
=
CompilationLevel
.
PIECEWISE
,
...
...
Write
Preview
Markdown
is supported
0%
Try again
or
attach a new file
.
Attach a file
Cancel
You are about to add
0
people
to the discussion. Proceed with caution.
Finish editing this message first!
Cancel
Please
register
or
sign in
to comment