Closed roywei closed 6 months ago
Thanks for this PR. I was just thinking we should add Claude.
I'll review this week
This PR looks good. The only thing I am noticing may not be working as expected is that gpt_4_fallback
function may not work with the message structure for Claude. I'll take a closer look tomorrow and approve if all good
nice catch! updated and fallback seems working
@roywei PR looks good now. Made a few slightly adjustments and merged!
Thanks again for this PR. I'll let the community know about it on Twitter and mention you.
I've been wanting to post about it, but I think Anthropic is crashed : ( lol
anthropic.InternalServerError: Error code: 529 - {'type': 'error', 'error': {'type': 'overloaded_error', 'message': 'Overloaded'}}
@roywei With some more testing I still see the memory limit on Anthropics API get it with the screenshot. We may need to compress it more when it surpasses the threshold. I'll add that tomorrow morning or you want to work on it feel free to make a new PR
What does this PR do?
adding support for claude 3 apis, tested with claude 3 opus, and passed evaluation. Few things I had to work around:
Fixes # (issue)
Requirement/Documentation
Type of change
Mandatory Tasks