-
Notifications
You must be signed in to change notification settings - Fork 3.8k
Add reasoning and reasoningBudgetTokens parameters to config schemas and docs #5041
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add reasoning and reasoningBudgetTokens parameters to config schemas and docs #5041
Conversation
✅ Deploy Preview for continuedev ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
|
@ferenci84 nice Also before we merge would this potentially conflict with a UI that would allow users to adjust reasoning tokens ad hoc in the future? I know either @FallDownTheSystem or @ferenci84 did a PR with this at some point. Something to consider |
|
@RomneyDa Not sure if you think of bumping the version number of the @continuedev/config-yaml, but if so, I think it should be set when the merging is done, because any change I do now, will soon become obsolete. I didn't find any other modifications related to reasoning. Also verified that adding reasoning and reasoningBudgetTokens works with a yaml config like this: Before this modification, I was not able to get reasoning to work with the yaml config, although it was working with the json. |
|
I opened a similar PR but also included prompt caching. We have migrated to yaml and then immediately noticed this regression. |
|
Thank you @ferenci84 ! |
|
Hi @ferenci84, yesterday we shared some updates with our contributors about how we're aiming to improve the contribution process. Part of this included the addition of a Contributor License Agreement (CLA) to protect both contributors and the project. I figure there's a good chance you'll open another PR since you've been contributing a lot of great code lately, but still wanted to send this message right away. Could you please take a moment to sign, or if you have any questions send me a message? (either here or nate@continue.dev would work) To do so, you just need to post a comment below with the following text: ❤️ Thank you for the work you've done on Continue, and let me know if you have any suggestions on how we can make the project even better! |
|
I have read the CLA Document and I hereby sign the CLA You can retrigger this bot by commenting recheck in this Pull Request. Posted by the CLA Assistant Lite bot. |
Description
Adds reasoning and reasoningBudgetTokens params to config schemas and docs.
Issue: #4918
Checklist