Unverified Commit b0476a06 authored by Keyang Ru's avatar Keyang Ru Committed by GitHub
Browse files

[router][quick fix] Add minimal option for reasoning effort in spec (#12711)

parent ffba61a1
...@@ -85,6 +85,7 @@ fn default_reasoning_effort() -> Option<ReasoningEffort> { ...@@ -85,6 +85,7 @@ fn default_reasoning_effort() -> Option<ReasoningEffort> {
#[derive(Debug, Clone, Deserialize, Serialize)] #[derive(Debug, Clone, Deserialize, Serialize)]
#[serde(rename_all = "snake_case")] #[serde(rename_all = "snake_case")]
pub enum ReasoningEffort { pub enum ReasoningEffort {
Minimal,
Low, Low,
Medium, Medium,
High, High,
......
...@@ -277,6 +277,8 @@ impl HarmonyBuilder { ...@@ -277,6 +277,8 @@ impl HarmonyBuilder {
"high" => ReasoningEffort::High, "high" => ReasoningEffort::High,
"medium" => ReasoningEffort::Medium, "medium" => ReasoningEffort::Medium,
"low" => ReasoningEffort::Low, "low" => ReasoningEffort::Low,
// Harmony does not support minimal reasoning effort
"minimal" => ReasoningEffort::Low,
_ => ReasoningEffort::Medium, _ => ReasoningEffort::Medium,
}); });
...@@ -302,6 +304,7 @@ impl HarmonyBuilder { ...@@ -302,6 +304,7 @@ impl HarmonyBuilder {
ResponsesReasoningEffort::High => ReasoningEffort::High, ResponsesReasoningEffort::High => ReasoningEffort::High,
ResponsesReasoningEffort::Medium => ReasoningEffort::Medium, ResponsesReasoningEffort::Medium => ReasoningEffort::Medium,
ResponsesReasoningEffort::Low => ReasoningEffort::Low, ResponsesReasoningEffort::Low => ReasoningEffort::Low,
ResponsesReasoningEffort::Minimal => ReasoningEffort::Low,
}); });
self.build_system_message(reasoning_effort, with_custom_tools) self.build_system_message(reasoning_effort, with_custom_tools)
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment