[doc] update reasoning doc#18306
Conversation
|
👋 Hi! Thank you for contributing to the vLLM project. 💬 Join our developer Slack at https://slack.vllm.ai to discuss your PR in #pr-reviews, coordinate on features in #feat- channels, or join special interest groups in #sig- channels. Just a reminder: PRs would not trigger full CI run by default. Instead, it would only run Once the PR is approved and ready to go, your PR reviewer(s) can run CI to test the changes comprehensively before merging. To run CI, PR reviewers can either: Add 🚀 |
There was a problem hiding this comment.
Can you change this into vllm/reasoning/deepseek_r1_reasoning_parser.py instead?
Thanks
There was a problem hiding this comment.
hi, updated, PTAL
There was a problem hiding this comment.
Just need to change L291, Please keep the example here.
Signed-off-by: reidliu41 <reid201711@gmail.com>
73afe29 to
b9e2bf7
Compare
Signed-off-by: reidliu41 <reid201711@gmail.com>
aarnphm
left a comment
There was a problem hiding this comment.
Thanks. cc @DarkLight1337
Signed-off-by: reidliu41 <reid201711@gmail.com> Co-authored-by: reidliu41 <reid201711@gmail.com> Signed-off-by: Yuqi Zhang <yuqizhang@google.com>
Since PR #14428 updated, so try to update the
reasoning_outputs.md.