Skip to content
This repository was archived by the owner on Jul 4, 2025. It is now read-only.

feat: add reasoning_budget parameter to params map #2211

Merged
merged 3 commits into from
Jun 14, 2025
Merged

feat: add reasoning_budget parameter to params map #2211

merged 3 commits into from
Jun 14, 2025

Conversation

@qnixsynapse
Copy link
Contributor

@qnixsynapse qnixsynapse commented Jun 13, 2025

Describe Your Changes

  • Added reasoning_budget parameter to enable and disable thinking. It takes one of two values. -1 for unlimited thinking and 0 for disable thinking.

Fixes Issues

  • Closes #
  • Closes #

Self Checklist

  • Added relevant comments, esp in complex areas
  • Updated docs (for bug fixes / features)
  • Created issues for follow-up changes or refactoring needed

@qnixsynapse qnixsynapse requested a review from louis-jan June 13, 2025 06:42
@qnixsynapse qnixsynapse enabled auto-merge (squash) June 13, 2025 06:55
louis-jan
louis-jan approved these changes Jun 14, 2025
View reviewed changes
Copy link
Contributor

@louis-jan louis-jan left a comment

Choose a reason for hiding this comment

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

LGTM

@qnixsynapse qnixsynapse merged commit aab3586 into dev Jun 14, 2025
@qnixsynapse qnixsynapse deleted the toggle_think branch June 14, 2025 05:33
@github-project-automation github-project-automation bot moved this to QA in Jan Jun 14, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

1 more reviewer

@louis-jan louis-jan louis-jan approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

Jan
Status: QA

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@qnixsynapse @louis-jan