From dbae9214b9cd15e1097168eda1be3f1115dd3f40 Mon Sep 17 00:00:00 2001 From: Pierre F Date: Mon, 25 Aug 2025 19:45:19 +0200 Subject: [PATCH] qwen hermes tool calling : fixed grammar rules names --- common/chat.cpp | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/common/chat.cpp b/common/chat.cpp index c230eecaa4..67db927a5c 100644 --- a/common/chat.cpp +++ b/common/chat.cpp @@ -1824,20 +1824,20 @@ static common_chat_params common_chat_params_init_hermes_2_pro(const common_chat "( \"```\\n\" | \"```json\\n\" | \"```xml\\n\" ) space " + wrappable_tool_call + " space \"```\" space "); auto tool_call = builder.add_rule("tool_call", string_join(tool_call_alts, " | ")); - builder.add_rule("thinking_start", "\"\""); - builder.add_rule("thinking_content", "[^\\x00]*"); - builder.add_rule("thinking_end", "\"\" space"); + builder.add_rule("thinking-start", "\"\""); + builder.add_rule("thinking-content", "[^\\x00]*"); + builder.add_rule("thinking-end", "\"\" space"); //thinking grammar logic depending on if thinking_forced_open was to true (so already opened (and maybe closed)) and if thinking is even allowed std::string thinking_grammar_logic = ""; // thinking tag was closed or not supported/wanted if (extra_context["enable_thinking"]) { if (data.thinking_forced_open) { //thinking tag was already opened by used so we don't need to add it again - thinking_grammar_logic = "thinking_content thinking_end "; + thinking_grammar_logic = "thinking-content thinking-end "; } else { - thinking_grammar_logic = "thinking_start thinking_content thinking_end "; + thinking_grammar_logic = "thinking-start thinking-content thinking-end "; } }