search

mem0ai / mem0

The Memory layer for your AI apps
https://mem0.ai
Apache License 2.0
21.35k stars 1.94k forks source link

OpenAI embeder from langchain has more options than chroma #1455

Open PranavPuranik opened 2 months ago

PranavPuranik commented 2 months ago

🐛 Describe the bug

We should be importing the embedder for openai from langchain not chromadb - https://github.com/embedchain/embedchain/blob/main/embedchain/embedder/openai.py#L4

Openai vs chromadb funciton

It gives us more flexibility to call the openai embedding functionality over current implementation of chroma - especially http proxy settings.

tanmaysankhe commented 2 months ago

I will work on this.

Dev-Khant commented 2 months ago

@tanmaysankhe Feel free to work on this.

SumantxD commented 2 months ago

@Dev-Khant can you review my PR #1468