Skip to content
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 breaking changes due to updated Anthropic SDK #452

Merged
merged 2 commits into from
Jun 5, 2024

Conversation

eitanturok
Copy link
Contributor

@eitanturok eitanturok commented May 30, 2024

Anthropic just moved their tool use from beta to main so we have to change the import from anthropic.types.beta.tools import ToolUseBlock to from anthropic.types import ToolUseBlock. You cannot run the eval without this change as things break.

Also, my IDE automatically sorted the imported packages and removed some extra spaces -- this explains all the other changes.

@eitanturok
Copy link
Contributor Author

@justinwangx, @HuanzhiMao

@eitanturok eitanturok changed the title Update anthropic SDK Fix breaking changes due to updated Anthropic SDK May 30, 2024
Copy link
Collaborator

@HuanzhiMao HuanzhiMao 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 PR @eitanturok!

@ShishirPatil LGTM, ready to merge.

@eitanturok
Copy link
Contributor Author

Also, this situation would be avoided if you pin your package versions. As a best practice, might be worthwhile doing this...

@HuanzhiMao
Copy link
Collaborator

Also, this situation would be avoided if you pin your package versions. As a best practice, might be worthwhile doing this...

Yes, good point. We will add that.

Copy link
Owner

@ShishirPatil ShishirPatil left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks @eitanturok

@ShishirPatil ShishirPatil merged commit 33cabef into ShishirPatil:main Jun 5, 2024
@eitanturok eitanturok deleted the anthropic-patch branch June 6, 2024 18:40
devanshamin pushed a commit to devanshamin/gorilla that referenced this pull request Jul 9, 2024
Anthropic just moved their tool use from beta to main so we have to
change the import `from anthropic.types.beta.tools import ToolUseBlock`
to `from anthropic.types import ToolUseBlock`. You cannot run the eval
without this change as things break.

Also, my IDE automatically sorted the imported packages and removed some
extra spaces -- this explains all the other changes.
aw632 pushed a commit to vinaybagade/gorilla that referenced this pull request Aug 22, 2024
Anthropic just moved their tool use from beta to main so we have to
change the import `from anthropic.types.beta.tools import ToolUseBlock`
to `from anthropic.types import ToolUseBlock`. You cannot run the eval
without this change as things break.

Also, my IDE automatically sorted the imported packages and removed some
extra spaces -- this explains all the other changes.
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.

None yet

3 participants