Skip to content

fix(llm): use DEFAULT_LLM_MODEL instead of ROUTING_MODEL in think_tool/rlm (#3948)#3960

Merged
mrveiss merged 1 commit intoDev_new_guifrom
fix/issue-3948
Apr 8, 2026
Merged

fix(llm): use DEFAULT_LLM_MODEL instead of ROUTING_MODEL in think_tool/rlm (#3948)#3960
mrveiss merged 1 commit intoDev_new_guifrom
fix/issue-3948

Conversation

@mrveiss
Copy link
Copy Markdown
Owner

@mrveiss mrveiss commented Apr 8, 2026

Closes #3948

ROUTING_MODEL (1B) is for routing/classification only. Think tool reasoning,
RLM benchmarking, and RLM type responses require the full DEFAULT_LLM_MODEL
for complex reasoning tasks.

Changes:

  • think_tool.py: ThinkToolConfig.model
  • rlm/benchmark.py: function signatures
  • rlm/types.py: RequestResponse.model

…l/rlm (#3948)

ROUTING_MODEL (1B) is for routing/classification only. Think tool reasoning,
RLM benchmark testing, and RLM types require the full DEFAULT_LLM_MODEL
for complex reasoning tasks.

Files updated:
- agent_loop/think_tool.py: ThinkToolConfig.model
- rlm/benchmark.py: multiple function signatures
- rlm/types.py: RequestResponse.model

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@mrveiss mrveiss merged commit d99af3f into Dev_new_gui Apr 8, 2026
3 of 4 checks passed
@mrveiss mrveiss deleted the fix/issue-3948 branch April 8, 2026 07:19
@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 8, 2026

✅ SSOT Configuration Compliance: Passing

🎉 No hardcoded values detected that have SSOT config equivalents!

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.

1 participant