Skip to content

Conversation

@taras-yemets
Copy link
Contributor

Add examples of thinking models configuration (for more context see this thread).

@vercel
Copy link

vercel bot commented Apr 29, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
sourcegraph-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 30, 2025 11:06am

Copy link
Contributor

@loujar loujar left a comment

Choose a reason for hiding this comment

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

Thanks for the updates. It doesn't need to be tackled in this PR, but I think we should reconcile our context window recommendations with the expanded context window length for Sonnet 3.7 we are documenting here
cc: @MaedahBatool @aramaraju

@taras-yemets taras-yemets merged commit cf3a634 into main Apr 30, 2025
2 of 3 checks passed
@taras-yemets taras-yemets deleted the ty/upd-modelconfig-examples branch April 30, 2025 11:01
@aramaraju
Copy link
Contributor

Thanks for the updates. It doesn't need to be tackled in this PR, but I think we should reconcile our context window recommendations with the expanded context window length for Sonnet 3.7 we are documenting here cc: @MaedahBatool @aramaraju

@MaedahBatool @alexromano Haven't we updated the context window for all the models already? If so, I agree with Louis, we should update them. Or are we still rolling them out in phases? The modelConfig examples/docs can still have the expanded limits as it's more likely the BYOK customers on that page, so we shouldn't have cost concerns.

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.

5 participants