Skip to content

Conversation

@KanTakahiro
Copy link
Contributor

@KanTakahiro KanTakahiro commented Jun 26, 2025

Related GitHub Issue

Closes: #5136

Description

Change the maxTokens value of qwen/qwen3-32b model to 40960 from 131072.
The current config set maxTokens to 131072, when the output token over 40960, it will cause a 400 error.

Test Procedure

I test this change by comparing the before/after of modifying packages/types/src/providers/groq.ts.

Pre-Submission Checklist

  • Issue Linked: This PR is linked to an approved GitHub Issue (see "Related GitHub Issue" above).
  • Scope: My changes are focused on the linked issue (one major feature/fix per PR).
  • Self-Review: I have performed a thorough self-review of my code.
  • Testing: New and/or updated tests have been added to cover my changes (if applicable).
  • Documentation Impact: I have considered if my changes require documentation updates (see "Documentation Updates" section below).
  • Contribution Guidelines: I have read and agree to the Contributor Guidelines.

Screenshots / Videos

Documentation Updates

Additional Notes

Get in Touch


Important

Update maxTokens for qwen/qwen3-32b model in groq.ts to prevent 400 error.

  • Behavior:
    • Update maxTokens for qwen/qwen3-32b model in groq.ts from 131072 to 40960 to prevent 400 error when output tokens exceed 40960.

This description was created by Ellipsis for 47c3cfa. You can customize this summary. It will automatically update as commits are pushed.

@KanTakahiro KanTakahiro requested review from cte, jr and mrubens as code owners June 26, 2025 08:29
@dosubot dosubot bot added size:XS This PR changes 0-9 lines, ignoring generated files. bug Something isn't working labels Jun 26, 2025
@hannesrudolph hannesrudolph added the Issue/PR - Triage New issue. Needs quick review to confirm validity and assign labels. label Jun 26, 2025
@Maxxoto
Copy link

Maxxoto commented Jun 26, 2025

nicee 👍

@daniel-lxs daniel-lxs moved this from Triage to PR [Needs Prelim Review] in Roo Code Roadmap Jun 26, 2025
@hannesrudolph hannesrudolph added PR - Needs Preliminary Review and removed Issue/PR - Triage New issue. Needs quick review to confirm validity and assign labels. labels Jun 26, 2025
Copy link
Member

@daniel-lxs daniel-lxs left a comment

Choose a reason for hiding this comment

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

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Jun 26, 2025
@daniel-lxs daniel-lxs moved this from PR [Needs Prelim Review] to PR [Needs Review] in Roo Code Roadmap Jun 26, 2025
@mrubens mrubens merged commit c797c9a into RooCodeInc:main Jun 26, 2025
32 checks passed
@github-project-automation github-project-automation bot moved this from New to Done in Roo Code Roadmap Jun 26, 2025
@github-project-automation github-project-automation bot moved this from PR [Needs Review] to Done in Roo Code Roadmap Jun 26, 2025
hannesrudolph pushed a commit that referenced this pull request Jul 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working lgtm This PR has been approved by a maintainer PR - Needs Review size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

Groq provider : Qwen3-32b - max_tokens must be less than or equal to 40960

5 participants