Merge pull request OpenInterpreter#710 from InterwebAlchemy/feature/t…

…emp-0 fix: default to 0 if temperature is unset Former-commit-id: 286228d Former-commit-id: 75856f9f325953bf2d5c305b662bdd3a2b64bc22 Former-commit-id: bf2b0882e15a49aae48c733eace3b89299a36453 [formerly c78c6349aec79caac5a5f801e7381a027b096789] Former-commit-id: 240b431b1d75afba86ebda6bd09995a227e1382a
joshuavial · Oct 28, 2023 · bfc28e2 · bfc28e2
2 parents 23c33e9 + 31c8cca
commit bfc28e2
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/interpreter/llm/setup_openai_coding_llm.py b/interpreter/llm/setup_openai_coding_llm.py
@@ -80,7 +80,9 @@ def coding_llm(messages):
             params["max_tokens"] = interpreter.max_tokens
         if interpreter.temperature is not None:
             params["temperature"] = interpreter.temperature
-
+        else:
+            params["temperature"] = 0.0
+
         # These are set directly on LiteLLM
         if interpreter.max_budget:
             litellm.max_budget = interpreter.max_budget

diff --git a/interpreter/llm/setup_text_llm.py b/interpreter/llm/setup_text_llm.py
@@ -103,6 +103,8 @@ def base_llm(messages):
             params["max_tokens"] = interpreter.max_tokens
         if interpreter.temperature is not None:
             params["temperature"] = interpreter.temperature
+        else:
+            params["temperature"] = 0.0
 
         # These are set directly on LiteLLM
         if interpreter.max_budget: