-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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 Gemini (#1490) #1965
Conversation
merge main from mem0ai/mem0
I have a doubt. I noticed there is already a |
Hey Thanks for the PR, you should |
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.
Make sure to make changes to mint.json
for updating doc page.
merge main from mem0
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.
Make sure that .lock
file remains same as there is no change in pyproject.toml
.
I see there have been a few changes in the |
Hey @into-the-night |
Wow that was pretty sneaky but something new to learn about line-endings. here you go @Dev-Khant, no diffs in |
LGTM!! Thanks @into-the-night for the quick resolution and for the contribution! |
Thank you so much for your patience😁 we go up brick by brick! |
Description
Added support for Gemini using
google-generativeai
module. (sorry it took so long 👍)Fixes #1490
Type of change
How Has This Been Tested?
Also added unit test for Gemini in
tests/llms/test_gemini_llm.py
Please delete options that are not relevant.
Checklist:
Maintainer Checklist