Skip to content

Conversation

loujar
Copy link
Contributor

@loujar loujar commented Sep 5, 2024

remove references to invalid provide id anthropic-direct

Pull Request approval

Although pull request approval is not enforced for this repository in order to reduce friction, merging without a review will generate a ticket for the docs team to review your changes. So if possible, have your pull request approved before merging.

Copy link

vercel bot commented Sep 5, 2024

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 Sep 5, 2024 7:46pm

Copy link
Contributor

@aramaraju aramaraju left a comment

Choose a reason for hiding this comment

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

Minor comment on whether the example should be using 3.5 Sonnet, but that's an aside.

Thanks for jumping on this fix, @loujar !

"modelOverrides": [
{
"modelRef": "anthropic-direct::2023-06-01::claude-3-sonnet",
"modelRef": "anthropic::2023-06-01::claude-3-sonnet",
Copy link
Contributor

Choose a reason for hiding this comment

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

Should we just change this to 3.5 Sonnet in case people are copy pasting this?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is a good suggestion. I updated the model in the example to Claude Sonnet 3.5 and also bumped up the maxInputTokens value to 45k based on our previous conversations about the max supported context window.

@loujar loujar merged commit dcd7b38 into main Sep 5, 2024
5 checks passed
@loujar loujar deleted the lsj/llm-config-edits branch September 5, 2024 22:56
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.

2 participants