Skip to content

Commit 197d986

Browse files
DavidePagliericopybara-github
authored andcommitted
Remove top_p from gpt models (deprecated from GPT-5)
PiperOrigin-RevId: 808950671 Change-Id: I4eb46df3438ddaeeb15f3dad8abce3bd5473aa80
1 parent b65e6e9 commit 197d986

1 file changed

Lines changed: 1 addition & 2 deletions

File tree

concordia/language_model/base_gpt_model.py

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -58,7 +58,7 @@ def _sample_text(
5858
timeout: float = language_model.DEFAULT_TIMEOUT_SECONDS,
5959
seed: int | None = None,
6060
) -> str:
61-
del terminators # Unused for OpenAI models.
61+
del terminators, top_p # Unused for OpenAI models.
6262

6363
messages = [
6464
{
@@ -90,7 +90,6 @@ def _sample_text(
9090
messages=messages,
9191
temperature=temperature,
9292
max_completion_tokens=max_tokens,
93-
top_p=top_p,
9493
timeout=timeout,
9594
seed=seed,
9695
reasoning_effort=reasoning_effort,

0 commit comments

Comments
 (0)