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

feat(google): added gemini api as provider #79

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

leftmove
Copy link

@leftmove leftmove commented Nov 19, 2024

What:

Adding support for the Gemini API.

Why:

Extra extensibility for LLMs.

How:

Fiddling around with the code to copy the implementation of the other LLMs.

Checklist:

  • Documentation
  • Tests
  • Ready to be merged

Probably not ready to be merged. I made this implementation because I needed it for my personal project.

Therefore, my timeline is a bit rushed and I didn't write any tests (documentation was added where needed, but not a lot was needed).

I don't plan to add any tests or further documentation, only bug fixes. This pull request will likely require work on others' parts to be fully ingratiated with the codebase - but I figure it might be at least a little useful as it is now.

Sorry I could not contribute more. Thanks for the awesome library.

#78 (comment)

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.

1 participant