fix: remove ModelOption.THINKING from automatic mapping because it's explicitely handled in line #417 (which was causing parameter conflicts) (#124)

HendrikStrobelt · web-flow · commit b5c2a394e3bc · 2025-09-08T09:32:53.000-07:00
diff --git a/mellea/backends/openai.py b/mellea/backends/openai.py
@@ -117,7 +117,6 @@ def __init__(
         self.from_mellea_model_opts_map_chats = {
             ModelOption.SEED: "seed",
             ModelOption.MAX_NEW_TOKENS: "max_completion_tokens",
-            ModelOption.THINKING: "reasoning_effort",
         }
 
         # See notes above.

Original file line number	Diff line number	Diff line change
`@@ -117,7 +117,6 @@ def __init__(`
`117`	`117`	`self.from_mellea_model_opts_map_chats = {`
`118`	`118`	`ModelOption.SEED: "seed",`
`119`	`119`	`ModelOption.MAX_NEW_TOKENS: "max_completion_tokens",`
`120`		`- ModelOption.THINKING: "reasoning_effort",`
`121`	`120`	`}`
`122`	`121`
`123`	`122`	`# See notes above.`