-
Notifications
You must be signed in to change notification settings - Fork 5.6k
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
fix AnthropicClient to make it work with ConversableAgent #2807
Conversation
@microsoft-github-policy-service agree |
Thanks for the contribution! I think you may need to control the scope of modified files. Multiple files irrelevant to this PR are modified. I think maybe only the notebook itself should be modified. You can follow the contribution guidelines here: https://microsoft.github.io/autogen/docs/contributor-guide/contributing/ |
Hi @teilomillet, thanks for creating this. I've tried to run it with the current Anthropic library, v0.28.0 and I can't run the second cell:
Just checking if it's working for you? |
I am trying to make this work, and no, it's not working for me, same error I guessModuleNotFoundError Traceback (most recent call last) ModuleNotFoundError: No module named 'anthropic.types.beta' |
It is an improvement but it doesn't seem to fully resolve issues, which I believe are associated with Autogen's underlying reliance on the OpenAI format. For example, if I run the two-agent example here, I get the following error:
|
@teilomillet thanks for the PR , we enhanced the support of AutoGen with Anthropic Client #2946. changes made in this PR are no longer necessary. any issues faced, please reach out to us on discord. |
Why are these changes needed?
The create in AnthropicClient wasn't conform the Anthropic API.
Related issue number
Checks