-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
community[patch]: anthropic add tool call support new tools api #5640
Merged
jacoblee93
merged 13 commits into
langchain-ai:main
from
LordMsz:community-bedrock-anthropic-add-tool-call-support-new-tools-api-5639
Jun 7, 2024
Merged
Changes from 2 commits
Commits
Show all changes
13 commits
Select commit
Hold shift + click to select a range
93aa70c
5639 community/bedrock anthropic add tool call support new tools api
LordMsz 24f22d9
Re-use most of parsing logic from @langchain/anthropic directly; expo…
LordMsz 1d5f944
Fix build errors
LordMsz cac36c0
Updated docs, BedrockChat partial support for Tool Calling, Anthropic…
LordMsz 039d0c7
Merge branch 'main' of https://github.com/hwchase17/langchainjs into …
jacoblee93 81386fc
Small updates
jacoblee93 d2ecb03
Format
jacoblee93 0155009
Revert Anthropic
jacoblee93 78a8aaa
Merge branch 'main' of https://github.com/hwchase17/langchainjs into …
jacoblee93 208bba3
Refactor to remove dep
jacoblee93 8f71915
Delegate streaming tool calls to invoke for now
jacoblee93 c7b06e8
Fix
jacoblee93 e60732e
Bump core version
jacoblee93 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1 +1,2 @@ | ||
export * from "./chat_models.js"; | ||
export * from "./output_parsers.js"; | ||
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
This change is biggest question for me - if it's inline with overall "strategy" that I can export this from Anthropic and use it in community? Any thoughts?
I only need single function from there, so that could be a minor copy-paste I assume.
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.
Yeah I think that's fine. Maybe we can prefix it with an underscore to show that it shouldn't be imported directly.