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

llms: Implement tool calling, including parallel tool call request support #709

Merged
merged 12 commits into from
Mar 31, 2024

Conversation

tmc
Copy link
Owner

@tmc tmc commented Mar 21, 2024

This adds tool implementation which generalizes function calling.

This focuses on the openai inference backend. Related filed issues: #716

Fixes #712, Refs #507

@tmc tmc changed the title tools: Implement tool calling openai: Implement tool calling Mar 21, 2024
@tmc tmc force-pushed the tool-calls branch 4 times, most recently from e4226eb to d4b2583 Compare March 24, 2024 20:42
@tmc tmc requested a review from eliben March 24, 2024 22:14
@tmc tmc changed the title openai: Implement tool calling llms: Implement tool calling, including parallel tool call request support Mar 24, 2024
llms/openai/openaillm.go Show resolved Hide resolved
@@ -4,7 +4,7 @@ go 1.21

toolchain go1.21.4

require github.com/tmc/langchaingo v0.1.7
require github.com/tmc/langchaingo v0.1.8-0.20240324221257-f3c3fcc40b49
Copy link
Collaborator

Choose a reason for hiding this comment

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

nit: the tag doesn't exist yet so this won't work for others

you're probably testing this locally with a go.work file? so this line can be omitted for now and later all examples can be updated with the script when 0.1.8 is tagged?

Copy link
Owner Author

Choose a reason for hiding this comment

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

This is pointing at this branch temporarily so this can build ahead of these changes merging.

@tmc tmc marked this pull request as ready for review March 26, 2024 20:39
@tmc
Copy link
Owner Author

tmc commented Mar 26, 2024

Rebased to main, marking ready for review.

@tmc tmc merged commit 4174692 into main Mar 31, 2024
3 checks passed
@tmc tmc deleted the tool-calls branch March 31, 2024 23:34
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.

llms: Evolve interface to enable multiple tool calls
2 participants