Merge pull request #459 from kreneskyp/function_call_linting

empty function_call prop passed to IxChatOpenAI is now ignored
kreneskyp · Feb 24, 2024 · 2470762 · 2470762
2 parents 0971c15 + eb9a9ea
commit 2470762
Showing 1 changed file with 6 additions and 3 deletions.
diff --git a/ix/runnable/llm.py b/ix/runnable/llm.py
@@ -32,9 +32,12 @@ def bind(self, **kwargs: Any) -> Runnable[Input, Output]:
         """
         kwargs = kwargs.copy()
 
-        if "function_call" in kwargs:
-            if isinstance(kwargs["function_call"], str):
-                kwargs["function_call"] = {"name": kwargs["function_call"]}
+        function_call = kwargs.pop("function_call", None)
+        if function_call:
+            if isinstance(function_call, str):
+                kwargs["function_call"] = {"name": function_call}
+            else:
+                kwargs["function_call"] = function_call
 
         if "functions" in kwargs:
             kwargs["functions"] = [to_openai_fn(obj) for obj in kwargs["functions"]]