-
Notifications
You must be signed in to change notification settings - Fork 268
Refined intergration to Claude AI #53
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
Conversation
…entions and improve clarity
… README and code for consistency
All the models are Valid models, Not sure where you are checking but using latest is not advised even in anthropic documentation: https://docs.anthropic.com/en/docs/about-claude/models/all-models#model-names ![]() |
|
|
+ 'gpt' => 'GPT'
Change from 'gpt' back to 'GPT'
Let's discuss here #55 and come back to this PR once done |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 🚢
Hey |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Thanks for your contribution
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Saw that @dev-opsss had already approved. Approving for him so we could merge.
=> const allowedModels = ['claude-3-7-sonnet-latest', 'claude-3-5-sonnet-20241022', 'claude-3-haiku-20240307'];
I used replaced all to substitute in the models, and tested it just fine :)