Fix not working with Gemini models

mem0ai / mem0

The Memory layer for your AI apps

https://mem0.ai

Apache License 2.0

22.94k stars 2.11k forks source link

Open lh0x00 opened 1 week ago

lh0x00 commented 1 week ago

This PR's did the following:

Fixed missing gemini in LLM's provider check.
Handled case where model returns json in code block.
Fixed Gemini model not working: removed unnecessary noop, updated function to parse function calls returning Gemini (based on their source code/docs).

All tested by my app working properly again.

Please delete options that are not relevant.

CLAassistant commented 1 week ago

All committers have signed the CLA.

lh0x00 commented 1 week ago

@Dev-Khant Gemini will not be available for now, I hope you consider and merge it soon if possible.

lh0x00 commented 1 week ago

@spike-spiegel-21 @Dev-Khant While working on the library I resolved some related bugs so that Gemini could work with mem0 as documented.

lh0x00 commented 1 week ago

@Dev-Khant I updated to resolve test fails.