Skip to content

Commit 67450fe

Browse files
Merge pull request #14146 from ProjectAlita/gpt_5_unsupported_params
GPT-5: Drop unsupported params
2 parents 8e72f99 + 76dd9d8 commit 67450fe

File tree

2 files changed

+23
-1
lines changed

2 files changed

+23
-1
lines changed

litellm/llms/openai/chat/gpt_5_transformation.py

Lines changed: 12 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,18 @@ def get_supported_openai_params(self, model: str) -> list:
2828
base_gpt_series_params.extend(gpt_5_only_params)
2929
if not supports_tool_choice(model=model):
3030
base_gpt_series_params.remove("tool_choice")
31-
return base_gpt_series_params
31+
32+
non_supported_params = [
33+
"logprobs",
34+
"top_p",
35+
"presence_penalty",
36+
"frequency_penalty",
37+
"top_logprobs",
38+
]
39+
40+
return [
41+
param for param in base_gpt_series_params if param not in non_supported_params
42+
]
3243

3344
def map_openai_params(
3445
self,

tests/test_litellm/llms/openai/test_gpt5_transformation.py

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -41,3 +41,14 @@ def test_gpt5_temperature_error(config: OpenAIConfig):
4141
model="gpt-5",
4242
drop_params=False,
4343
)
44+
45+
46+
def test_gpt5_unsupported_params_drop(config: OpenAIConfig):
47+
assert "top_p" not in config.get_supported_openai_params(model="gpt-5")
48+
params = config.map_openai_params(
49+
non_default_params={"top_p": 0.5},
50+
optional_params={},
51+
model="gpt-5",
52+
drop_params=True,
53+
)
54+
assert "top_p" not in params

0 commit comments

Comments
 (0)