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

Implemented function calls #67

Open
wants to merge 5 commits into
base: beta
Choose a base branch
from

Conversation

Plytas
Copy link
Contributor

@Plytas Plytas commented Jan 10, 2025

Implemented function calling as per https://ai.google.dev/gemini-api/docs/function-calling

@mayconmedeiroz
Copy link

@aydinfatih, is there anything missing for this PR to be approved? I really need function calls in my project, and this implementation would be extremely helpful. Thank you!

@Plytas
Copy link
Contributor Author

Plytas commented Feb 3, 2025

@mayconmedeiroz while this is being merged, you should be able to install my fork to test it out. Feedback welcome.

@mayconmedeiroz
Copy link

@mayconmedeiroz while this is being merged, you should be able to install my fork to test it out. Feedback welcome.

Yes, I'm using it in a project, and it works very well.

Choose a reason for hiding this comment

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

One small detail is that the ToolConfig is missing in the testing.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks! Seems like I forgot to add withToolConfig() to the interface as well. Updated now.

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.

2 participants