Skip to content

data: DeepSeek-V4 thinking mode for regeneration and training - #1069

Draft
yushengsu-thu wants to merge 1 commit into
sgl-project:mainfrom
yushengsu-thu:v4-thinking-regen
Draft

yushengsu-thu wants to merge 1 commit into
sgl-project:mainfrom
yushengsu-thu:v4-thinking-regen

Conversation

@yushengsu-thu

Copy link
Copy Markdown
Contributor

Makes DeepSeek-V4 thinking-mode data usable end to end.

  • scripts/regenerate_train_data.py: --reasoning save and --reasoning disable only sent Qwen's enable_thinking, but SGLang reads chat_template_kwargs.thinking for DeepSeek-V4, so save got chat-mode replies with no reasoning_content and every row errored. Both keys are now sent.
  • specforge/data/template.py: adds deepseek-v4-thinking, the existing deepseek-v4 template with enable_thinking=True, so training renders the reasoning.

Tests: devbox CPU run of tests/test_data/test_deepseek_v4_template.py and tests/test_data/test_template_registry.py: 14 passed. No new test case. The regenerate change is in use on a live V4-Flash 0731 regeneration: 33,392 of 100,000 prompts processed, 29,967 written so far.

Regeneration sends the thinking key SGLang reads for DeepSeek-V4; deepseek-v4-thinking renders reasoning in training.
Copilot AI balanced review requested due to automatic review settings November 2, 2026 22:55

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota. Try requesting a review at a lower effort level.

@yushengsu-thu
yushengsu-thu marked this pull request as draft November 3, 2026 00:57
@yushengsu-thu
yushengsu-thu marked this pull request as ready for review November 3, 2026 04:24
@yushengsu-thu
yushengsu-thu marked this pull request as draft November 3, 2026 04:25

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants