Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Handle DeepSeek reasoning_content in streamed responses#2174

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Conversation

tarruda
Copy link
Contributor

@tarrudatarruda commentedJul 10, 2025
edited
Loading

Fix#2048

I adapted the test from test_openai.py and useduv run -m pytest --record-mode=rewrite tests/models/test_deepseek -k stream (which is the same command used bymake update-vcr-tests) to generate theyaml file, however the response body is written as a quoted JSON string. How can I re-run the tool and generate a multiline YAML string like in the other existing yaml?

@tarrudatarrudaforce-pushed thehandle_reasoning_content_in_streamed_responses branch from45e38da to53bb1daCompareJuly 10, 2025 16:25
@DouweM
Copy link
Contributor

@tarruda Thanks!

the response body is written as a quoted JSON string.

That's fine, a few other streaming cassettes look the same, and they're not really meant for human consumption anyway.

tarruda reacted with thumbs up emoji

@DouweMDouweM self-assigned thisJul 10, 2025
@DouweMDouweM merged commit78f08f8 intopydantic:mainJul 10, 2025
18 checks passed
@tarrudatarruda deleted the handle_reasoning_content_in_streamed_responses branchJuly 10, 2025 18:21
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers
No reviews
Assignees

@DouweMDouweM

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

The reasoning content from deepseek-reasoner is not included in streamed runs but included in non-streamed runs
2 participants
@tarruda@DouweM

[8]ページ先頭

©2009-2025 Movatter.jp