Skip to content

fix(azure-ai): add max_tokens limit for DeepSeek V4 models - #849

Merged
prabhash-varma merged 2 commits into
mainfrom
add/deepseek-v4-max-tokens
Jul 20, 2026
Merged

fix(azure-ai): add max_tokens limit for DeepSeek V4 models#849
prabhash-varma merged 2 commits into
mainfrom
add/deepseek-v4-max-tokens

Conversation

@prabhash-varma

Copy link
Copy Markdown
Contributor

Description

Please include a summary of the changes and the related issue. Please also include relevant motivation and context.

  • New model pricing
  • Update existing pricing
  • New model configuration
  • Bug fix

Source Verification

https://api-docs.deepseek.com/quick_start/pricing/

Important

Please include a link to the official pricing page from the provider. Simple "I heard it from somewhere" or screenshot sources are not accepted.

Checklist

  • I have validated the JSON using jq or an online validator
  • I have verified that prices are in cents per token (not dollars)
  • I have included the source link above
  • I have signed the CLA (if first-time contributor)

Related Issue

Fixes # (issue)

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the Azure AI model configuration to explicitly cap max_tokens for the new DeepSeek V4 model entries, aligning request parameter validation with the provider’s documented limits.

Changes:

  • Add a max_tokens maxValue of 384000 for DeepSeek-V4-Flash.
  • Add a max_tokens maxValue of 384000 for DeepSeek-V4-Pro.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread general/azure-ai.json Outdated
Comment thread general/azure-ai.json Outdated

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

@prabhash-varma
prabhash-varma merged commit 57a5db8 into main Jul 20, 2026
3 checks passed
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.

3 participants