-
Notifications
You must be signed in to change notification settings - Fork 612
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
feat: support for custom toolchoice in LLMs #1102
Merged
Merged
Changes from 18 commits
Commits
Show all changes
45 commits
Select commit
Hold shift + click to select a range
e0e83e2
support for custom tooluse in anthropic
jayeshp19 7379ca4
support for custom tooluse in anthropic
jayeshp19 86bee50
support for custom tooluse in anthropic
jayeshp19 4cd8a40
Merge branch 'main' of https://github.com/livekit/agents into jp/anth…
jayeshp19 b4d5292
openai parallel tool call impl
jayeshp19 9dc1e07
wip
jayeshp19 344d040
tool choice wip
jayeshp19 d441874
wip
jayeshp19 e2ff9da
wip
jayeshp19 b5611c5
tool choice wip
jayeshp19 efbc8ed
tool choice wip
jayeshp19 6e23f77
wip
jayeshp19 9ae405d
added test cases
jayeshp19 ce9ca43
added test cases
jayeshp19 62070bd
assistant llm tool choice param
jayeshp19 a1c0dbe
minor
jayeshp19 45da685
Merge branch 'main' of https://github.com/livekit/agents into jp/anth…
jayeshp19 439e2ff
updates
jayeshp19 06e6555
updates
jayeshp19 7d64c10
updates
jayeshp19 69d6a68
updates
jayeshp19 6e4c2f3
updates
jayeshp19 9cfb364
updates
jayeshp19 31006c2
updates
jayeshp19 4845c59
updates
jayeshp19 47ac432
updates
jayeshp19 fcc1e97
updates
jayeshp19 cf98684
updates
jayeshp19 dce1ad5
typecheck
jayeshp19 5a924ff
typecheck
jayeshp19 2dbcdbe
typecheck
jayeshp19 01b573a
typecheck
jayeshp19 ec8ff13
updates
jayeshp19 1c472d8
typecheck
jayeshp19 475330c
updates
jayeshp19 d7618bc
minor
jayeshp19 e726c56
minor
jayeshp19 d648ec7
minor
jayeshp19 a89616b
Update livekit-plugins/livekit-plugins-anthropic/livekit/plugins/anth…
jayeshp19 b051534
Merge branch 'main' of https://github.com/livekit/agents into jp/anth…
jayeshp19 9edfe29
updates
jayeshp19 d6d3cff
updates
jayeshp19 b2e984b
updates
jayeshp19 0aba89e
updates
jayeshp19 7490a95
updates
jayeshp19 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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,5 @@ | ||
--- | ||
"livekit-plugins-anthropic": patch | ||
--- | ||
|
||
support for custom tool use in anthropic |
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
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
Oops, something went wrong.
Oops, something went wrong.
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.
what is the difference between
tool
andrequired
?What is
any
used for?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.
required
(OpenAI specific param) - the model must call one or more tools.any
(Anthropic specific param) - the model must call one or more tools.tool
(Anthropic specific param) - the model must call specific toolThere 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.
can we merge
any
andrequired
?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.
Yes, we can, but I think we may need to document this otherwise, it might confuse users.
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.
I've updated code accordingly