mem0 icon indicating copy to clipboard operation
mem0 copied to clipboard

Fixes for LLM providers LiteLLM and TogetherAI

Open Ataraksia opened this issue 1 year ago • 2 comments
trafficstars

Description

Added LiteLLM and TogetherAI to the LLM provider validator list so they wouldn't error out as unsupported, and then additionally renamed TogetherAI so it better match both the name of the LLM as well as fit the example in the docs.

Fixes # (issue)

Type of change

  • [x] Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

Checklist:

Maintainer Checklist

Ataraksia avatar Jul 19 '24 23:07 Ataraksia

Saw the PR you referenced had some changes I failed to include, specifically another provider, and a change to the container type. I also felt it made sense to sort the list of providers alphabetically, given its increased length just to make it easier to parse.

Ataraksia avatar Jul 22 '24 17:07 Ataraksia

CLA assistant check
All committers have signed the CLA.

CLAassistant avatar Jul 26 '24 02:07 CLAassistant

Hey @Ataraksia Thank you for working on this and for pointing out the doc issue. But here we would keep together as it's the standard name we use across Mem0 and Embedchain.

Therefore, closing this issue. Please feel free to work on other issues and let us know if you need any help.

Dev-Khant avatar Aug 01 '24 19:08 Dev-Khant