scriptoriumadmin/litellm
1
0
Fork 0
mirror of https://github.com/BerriAI/litellm.git synced 2026-10-10 03:28:53 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 86

fix: fix stop param indentation, remove continue, add noqa PLR0915

Browse source
This commit is contained in:
Monesh Ram 2026-02-24 09:32:10 +05:30 • committed by GitHub
parent 397877720b
commit 81d8fd4707
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 10 additions and 12 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

22
litellm/llms/bedrock/chat/converse_transformation.py
View file

@ -875,7 +875,7 @@ class AmazonConverseConfig(BaseConfig):
optional_params: dict,
model: str,
drop_params: bool,
) -> dict:
) -> dict: # noqa: PLR0915
is_thinking_enabled = self.is_thinking_enabled(non_default_params)
for param, value in non_default_params.items():
@ -891,17 +891,15 @@ class AmazonConverseConfig(BaseConfig):
optional_params["maxTokens"] = value
if param == "stream":
optional_params["stream"] = value
if param == "stop":
if isinstance(value, str):
if len(value) == 0: # converse raises error for empty strings
continue
value = [value]
elif isinstance(value, list):
# Filter out empty strings from list (Bedrock rejects them)
value = [v for v in value if isinstance(v, str) and len(v) > 0]
if not value: # Skip if empty list or all empty strings
continue
optional_params["stopSequences"] = value
if param == "stop":
if isinstance(value, str):
if len(value) > 0: # skip empty strings (converse rejects them)
optional_params["stopSequences"] = [value]
elif isinstance(value, list):
# Filter out empty strings from list (Bedrock rejects them)
filtered = [v for v in value if isinstance(v, str) and len(v) > 0]
if filtered: # only set if non-empty after filtering
optional_params["stopSequences"] = filtered
if param == "temperature":
optional_params["temperature"] = value
if param == "top_p":