phoenix-oss/llama-stack-mirror
0
0
Fork 1
mirror of https://github.com/meta-llama/llama-stack.git synced 2025-08-03 09:21:45 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

[#391] Add support for json structured output for vLLM

Browse source
This commit is contained in:
Aidan Do 2024-11-26 09:40:17 +00:00
parent 4e6c984c26
commit 1801aa145d
2 changed files with 13 additions and 0 deletions
Download patch file Download diff file Expand all files Collapse all files

11
llama_stack/providers/remote/inference/vllm/vllm.py
View file

@ -100,6 +100,7 @@ class VLLMInferenceAdapter(Inference, ModelsProtocolPrivate):
tool_prompt_format=tool_prompt_format, tool_prompt_format=tool_prompt_format,
stream=stream, stream=stream,
logprobs=logprobs, logprobs=logprobs,
response_format=response_format,
) )
if stream: if stream:
return self._stream_chat_completion(request, self.client) return self._stream_chat_completion(request, self.client)
@ -180,6 +181,16 @@ class VLLMInferenceAdapter(Inference, ModelsProtocolPrivate):
self.formatter, self.formatter,
) )
if fmt := request.response_format:
if fmt.type == ResponseFormatType.json_schema.value:
input_dict["extra_body"] = {
"guided_json": request.response_format.json_schema
}
elif fmt.type == ResponseFormatType.grammar.value:
raise NotImplementedError("Grammar response format not supported yet")
else:
raise ValueError(f"Unknown response format {fmt.type}")
return { return {
"model": request.model, "model": request.model,
**input_dict, **input_dict,

2
llama_stack/providers/tests/inference/test_text_inference.py
View file

@ -139,6 +139,7 @@ class TestInference:
"remote::tgi", "remote::tgi",
"remote::together", "remote::together",
"remote::fireworks", "remote::fireworks",
"remote::vllm",
): ):
pytest.skip( pytest.skip(
"Other inference providers don't support structured output in completions yet" "Other inference providers don't support structured output in completions yet"
@ -198,6 +199,7 @@ class TestInference:
"remote::fireworks", "remote::fireworks",
"remote::tgi", "remote::tgi",
"remote::together", "remote::together",
"remote::vllm",
"remote::nvidia", "remote::nvidia",
): ):
pytest.skip("Other inference providers don't support structured output yet") pytest.skip("Other inference providers don't support structured output yet")