Skip to content
Merged
Show file tree
Hide file tree
Changes from 2 commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 9 additions & 0 deletions components/src/dynamo/vllm/tests/conftest.py
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,15 @@ def pytest_ignore_collect(collection_path, config):
if filename.startswith("test_vllm_"):
if importlib.util.find_spec("vllm") is None:
return True # vllm not available, skip this file
# Omni tests import dynamo.vllm.omni.* which transitively imports
# vllm_omni at module load — that import raises on platforms vllm_omni
# doesn't support (e.g. CPU-only sample-runtime runners), and the
# exception type isn't always ImportError so each file's local
# try/except guard isn't enough.
parts = collection_path.parts
if "omni" in parts and filename.startswith("test_"):
if importlib.util.find_spec("vllm_omni") is None:
return True
return None


Expand Down
1 change: 0 additions & 1 deletion lib/llm/src/preprocessor.rs
Original file line number Diff line number Diff line change
Expand Up @@ -1578,7 +1578,6 @@ impl OpenAIPreprocessor {
choice.delta.refusal = None;
choice.delta.reasoning_content = None;
choice.finish_reason = None;
choice.stop_reason = None;
choice.logprobs = None;
true
} else {
Expand Down
1 change: 0 additions & 1 deletion lib/llm/tests/postprocessor_parsing_stream.rs
Original file line number Diff line number Diff line change
Expand Up @@ -329,7 +329,6 @@ fn mock_multi_choice_content_chunk(
reasoning_content: None,
},
finish_reason: None,
stop_reason: None,
logprobs: None,
})
.collect();
Expand Down
Loading