fix: maintain backward compatibility for remove_reasoning field

This commit is contained in:
Paul Gauthier (aider) 2025-03-08 17:37:37 -08:00
parent f8a7854efa
commit 14e37a82ab
2 changed files with 2 additions and 1 deletions

View file

@ -351,6 +351,7 @@ class Model(ModelSettings):
self.examples_as_sys_msg = True self.examples_as_sys_msg = True
self.use_temperature = False self.use_temperature = False
self.reasoning_tag = "think" self.reasoning_tag = "think"
self.remove_reasoning = "think" # For backward compatibility
return # <-- return # <--
if ("llama3" in model or "llama-3" in model) and "70b" in model: if ("llama3" in model or "llama-3" in model) and "70b" in model:

View file

@ -285,7 +285,7 @@ class TestModels(unittest.TestCase):
self.assertTrue(model.use_repo_map) self.assertTrue(model.use_repo_map)
self.assertTrue(model.examples_as_sys_msg) self.assertTrue(model.examples_as_sys_msg)
self.assertFalse(model.use_temperature) self.assertFalse(model.use_temperature)
self.assertEqual(model.remove_reasoning, "think") self.assertEqual(model.reasoning_tag, "think")
# Test provider/deepseek-v3 case # Test provider/deepseek-v3 case
model = Model("anotherprovider/deepseek-v3") model = Model("anotherprovider/deepseek-v3")