Unverified Commit 56cb1baa authored by Umut Polat's avatar Umut Polat Committed by GitHub
Browse files

[Misc] Use VLLMValidationError in batch, pooling, and tokenize protocol validators (#36256)


Signed-off-by: default avatarumut-polat <52835619+umut-polat@users.noreply.github.com>
parent f3403243
...@@ -54,6 +54,7 @@ from vllm.entrypoints.pooling.score.protocol import ( ...@@ -54,6 +54,7 @@ from vllm.entrypoints.pooling.score.protocol import (
ScoreResponse, ScoreResponse,
) )
from vllm.entrypoints.utils import create_error_response from vllm.entrypoints.utils import create_error_response
from vllm.exceptions import VLLMValidationError
from vllm.logger import init_logger from vllm.logger import init_logger
from vllm.reasoning import ReasoningParserManager from vllm.reasoning import ReasoningParserManager
from vllm.utils import random_uuid from vllm.utils import random_uuid
...@@ -86,9 +87,10 @@ class BatchTranscriptionRequest(TranscriptionRequest): ...@@ -86,9 +87,10 @@ class BatchTranscriptionRequest(TranscriptionRequest):
def validate_no_file(cls, data: Any): def validate_no_file(cls, data: Any):
"""Ensure file field is not provided in batch requests.""" """Ensure file field is not provided in batch requests."""
if isinstance(data, dict) and "file" in data: if isinstance(data, dict) and "file" in data:
raise ValueError( raise VLLMValidationError(
"The 'file' field is not supported in batch requests. " "The 'file' field is not supported in batch requests. "
"Use 'file_url' instead." "Use 'file_url' instead.",
parameter="file",
) )
return data return data
...@@ -116,9 +118,10 @@ class BatchTranslationRequest(TranslationRequest): ...@@ -116,9 +118,10 @@ class BatchTranslationRequest(TranslationRequest):
def validate_no_file(cls, data: Any): def validate_no_file(cls, data: Any):
"""Ensure file field is not provided in batch requests.""" """Ensure file field is not provided in batch requests."""
if isinstance(data, dict) and "file" in data: if isinstance(data, dict) and "file" in data:
raise ValueError( raise VLLMValidationError(
"The 'file' field is not supported in batch requests. " "The 'file' field is not supported in batch requests. "
"Use 'file_url' instead." "Use 'file_url' instead.",
parameter="file",
) )
return data return data
......
...@@ -11,6 +11,7 @@ from vllm.entrypoints.chat_utils import ( ...@@ -11,6 +11,7 @@ from vllm.entrypoints.chat_utils import (
ChatTemplateContentFormatOption, ChatTemplateContentFormatOption,
) )
from vllm.entrypoints.openai.engine.protocol import OpenAIBaseModel from vllm.entrypoints.openai.engine.protocol import OpenAIBaseModel
from vllm.exceptions import VLLMValidationError
from vllm.renderers import ChatParams, merge_kwargs from vllm.renderers import ChatParams, merge_kwargs
from vllm.utils import random_uuid from vllm.utils import random_uuid
from vllm.utils.serial_utils import EmbedDType, EncodingFormat, Endianness from vllm.utils.serial_utils import EmbedDType, EncodingFormat, Endianness
...@@ -147,9 +148,9 @@ class ChatRequestMixin(OpenAIBaseModel): ...@@ -147,9 +148,9 @@ class ChatRequestMixin(OpenAIBaseModel):
@classmethod @classmethod
def check_generation_prompt(cls, data): def check_generation_prompt(cls, data):
if data.get("continue_final_message") and data.get("add_generation_prompt"): if data.get("continue_final_message") and data.get("add_generation_prompt"):
raise ValueError( raise VLLMValidationError(
"Cannot set both `continue_final_message` and " "Cannot set both `continue_final_message` and "
"`add_generation_prompt` to True." "`add_generation_prompt` to True.",
) )
return data return data
......
...@@ -17,6 +17,7 @@ from vllm.entrypoints.openai.chat_completion.protocol import ( ...@@ -17,6 +17,7 @@ from vllm.entrypoints.openai.chat_completion.protocol import (
from vllm.entrypoints.openai.engine.protocol import ( from vllm.entrypoints.openai.engine.protocol import (
OpenAIBaseModel, OpenAIBaseModel,
) )
from vllm.exceptions import VLLMValidationError
from vllm.renderers import ChatParams, TokenizeParams, merge_kwargs from vllm.renderers import ChatParams, TokenizeParams, merge_kwargs
...@@ -120,9 +121,9 @@ class TokenizeChatRequest(OpenAIBaseModel): ...@@ -120,9 +121,9 @@ class TokenizeChatRequest(OpenAIBaseModel):
@classmethod @classmethod
def check_generation_prompt(cls, data): def check_generation_prompt(cls, data):
if data.get("continue_final_message") and data.get("add_generation_prompt"): if data.get("continue_final_message") and data.get("add_generation_prompt"):
raise ValueError( raise VLLMValidationError(
"Cannot set both `continue_final_message` and " "Cannot set both `continue_final_message` and "
"`add_generation_prompt` to True." "`add_generation_prompt` to True.",
) )
return data return data
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment