-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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(liteLLM): Add support for Azure OpenAI, Palm, Claude-2, Llama2, CodeLlama (100+LLMs) #1097
Conversation
@StanGirard can you please take a look at this PR when possible ? Happy to add more docs/tests if this initial commit looks good😊 |
Looks really cool. I don’t have my pc with me currently but here are a few notes:
|
@ishaan-jaff i think instead of replacing the current import you should add .env variable and with if statement allow both openai as well as litellm |
The goal is to replace openai with a project that allows anyone :) |
@@ -1,5 +1,6 @@ | |||
pymupdf==1.22.3 | |||
langchain==0.0.247 | |||
langchain==0.0.281 |
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.
langchain 0.0.281 includes the latest version of ChatLiteLLM too
@StanGirard done, good catch. Also bumped the version of langchain to include the latest
This is an experimental feature using Just want to re-iterate that the dashboard is experimental and our focus is entirely on making LiteLLM really good at I/O |
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.
@gozineb looks good to me. Can you check that everything works ? Especially with the version update of langchain ?
Should only need to test:
Uploading
Chat with a brain and unplug.
@@ -1,5 +1,6 @@ | |||
pymupdf==1.22.3 | |||
langchain==0.0.247 | |||
langchain==0.0.281 | |||
litellm==0.1.531 |
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.
Do we still needlitellm
import ?
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.
I think so :) langchain is just a wrapper. It doesn't come with the packages.
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.
Sounds good
@StanGirard looks good to me ! Merging this as soon as the checks pass ! |
…CodeLlama (100+LLMs) (#1097) * v0 litellm * bump versions
what about PALM |
we support palm too https://docs.litellm.ai/docs/providers |
Description
This PR adds support for the above mentioned LLMs using LiteLLM https://github.com/BerriAI/litellm/
Example:
docs on langchain integration https://python.langchain.com/docs/integrations/chat/litellm
In addition LiteLLM client allows you to n [Experimental + Optional Features]:
Here's a link to a live demo of litellm client: https://admin.litellm.ai/
Please include a summary of the changes and the related issue. Please also include relevant motivation and context.
Checklist before requesting a review
Please delete options that are not relevant.
Screenshots (if appropriate):