-
Notifications
You must be signed in to change notification settings - Fork 42
Implement Model-Provider for Claude 3.7 with Thinking #1999
New issue
Have a question about this project? Sign up for a free GitHub account to ope 8000 n 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
Comments
@iwilltry42 I am getting the following error when chatting with agents that are configured to use model -
failed to run: failed calling model for completion: request failed on non-retriable status-code 400: error, status code: 400 (400 Bad Request), message: messages.1.content.0.type: Expected thinking or redacted_thinking, but found text. When thinking is enabled, a final assistant message must start with a thinking block (preceeding the lastmost set of tool_use and tool_result blocks). We recommend you include thinking blocks from previous turns. To avoid this requirement, disable thinking. Please consult our documentation at https://docs.anthropic.com/en/docs/build-with-claude/extended-thinking
Default model used for chatting: |
Hi @sangee2004 I had a couple of chats with the thinking models without hitting this - can you please provide some chat example that I can follow? Or well... I don't have the access anymore to follow this actually 🙈 |
I looked at how we implemented the claude thinking model. This seems wrong. We probably want to do something more "intelligent" than this and figure out how to dynamically turn thinking on and off in chat. Maybe this is a good start in that we can build on top if it. |
Implement a model-provider for Claude 3.7 with thinking as discussed in the Slack thread. Refer to the documentation for extended thinking: <https://docs.anthropic.com/en/docs/build-with-claude/extended-thinking|Extended Thinking Documentation>. The model is still Claude 3.7, but requires additional steps similar to what Daishan did for O1.
The text was updated successfully, but these errors were encountered: