Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Feature]: separating reasoning and answer for reasoning models #12602

Closed
1 task done
jamesbraza opened this issue Jan 31, 2025 · 1 comment
Closed
1 task done

[Feature]: separating reasoning and answer for reasoning models #12602

jamesbraza opened this issue Jan 31, 2025 · 1 comment

Comments

@jamesbraza
Copy link

🚀 The feature, motivation and pitch

When hitting the DeepSeek R1 API at deepseek-reasoner (link), the API responds with reasoning and the answer in separate aspects of the payload. This enables clients like:

  • LiteLLM to house reasoning and the answer in separate fields (link).
  • OpenRouter's include_reasoning opt-in parameter (link).

It would be really useful if vllm serve had some capability for this.

Alternatives

Manually post-processing a response is error prone and arduous. It's better to do it within the model serving code.

Additional context

DeepSeek R1 model on Hugging Face

Before submitting a new issue...

  • Make sure you already searched for relevant issues, and asked the chatbot living at the bottom right corner of the documentation page, which can answer lots of frequently asked questions.
@jamesbraza
Copy link
Author

Actually this was done (tho currently unreleased) in #12473

It's docs: https://docs.vllm.ai/en/latest/features/reasoning_outputs.html

vllm serve ... --enable-reasoning --reasoning-parser xyz

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant