Repository navigation
Unify training inference through Chat Completions #3660
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
Changes from all commits
321ff13
d84aeae
1ca61be
ecb34e9
56fb11a
b4aed52
c573018
a48cf6f
d1facb0
4b65792
a954a22
317e6f3
5d1f3f0
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -50,6 +50,7 @@ attn = "flash_attention_2" | |
| [inference.vllm] | ||
| max_model_len = 8192 | ||
| data_parallel_size = 4 | ||
| [orchestrator.renderer] | ||
| reasoning_parser = "qwen3" | ||
|
|
||
| [inference.renderer] | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
For the Hendrycks runs using Useful? React with 👍 / 👎. |
||
| name = "default" | ||
| reasoning_parser = "think" | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -36,5 +36,5 @@ max_completion_tokens = 128 | |
|
|
||
| [inference] | ||
|
|
||
| [orchestrator.renderer] | ||
| [inference.renderer] | ||
| name = "prime-qwen3" | ||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This block configures only the managed policy server, while the frozen teacher at
localhost:8001is started separately bytests/integration/test_reverse_text_rl_sft.py:56-66without--renderer.name prime-qwen3. Because the teacher model’s-RLsuffix does not resolve to the intended renderer, the new server-owned training-record path renders teacher prompts with the auto/default renderer rather than the policy-compatible renderer, so this SFT integration can produce mismatched CE targets. Fresh evidence beyond the existing tokenizer comment is that the equivalent debug launch now explicitly passes--renderer.name prime-qwen3, but the fixture launch was not updated.Useful? React with 👍 / 👎.