fix #17317: corrected condition to include tools in boto3 client for …

…Bedrock
run-llama · Dec 24, 2024 · 139f1d4 · 139f1d4
1 parent cbec20b
commit 139f1d4
Showing 1 changed file with 2 additions and 12 deletions.
diff --git a/...rations/llms/llama-index-llms-bedrock-converse/llama_index/llms/bedrock_converse/utils.py b/...rations/llms/llama-index-llms-bedrock-converse/llama_index/llms/bedrock_converse/utils.py
@@ -337,12 +337,7 @@ def converse_with_retry(
         {
             k: v
             for k, v in kwargs.items()
-            if (
-                k != "tools"
-                or k != "guardrail_identifier"
-                or k != "guardrail_version"
-                or k != "trace"
-            )
+            if k not in ["tools", "guardrail_identifier", "guardrail_version", "trace"]
         },
     )
 
@@ -395,12 +390,7 @@ async def converse_with_retry_async(
         {
             k: v
             for k, v in kwargs.items()
-            if (
-                k != "tools"
-                or k != "guardrail_identifier"
-                or k != "guardrail_version"
-                or k != "trace"
-            )
+            if k not in ["tools", "guardrail_identifier", "guardrail_version", "trace"]
         },
     )