Skip to content

Rename 'max_tokens' to 'max_completion_tokens'#112

Open
andresterc wants to merge 1 commit into
modxcms:mainfrom
andresterc:patch-1
Open

Rename 'max_tokens' to 'max_completion_tokens'#112
andresterc wants to merge 1 commit into
modxcms:mainfrom
andresterc:patch-1

Conversation

@andresterc

Copy link
Copy Markdown

max_tokens is deprecated for new models

max_tokens is deprecated for new models
@theboxer

theboxer commented Jun 16, 2026

Copy link
Copy Markdown
Member

@andresterc the fix is not that simple, since the max_tokens are used in a lot more places and more importantly, different services use different name, so the meta max_tokens is adjusted on service (openai, google, anthropic) level to the correct one for the newest model.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants