Skip to content
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

LiteLLM ModelClient Integration #269

Open
PhiBrandon opened this issue Nov 20, 2024 · 2 comments
Open

LiteLLM ModelClient Integration #269

PhiBrandon opened this issue Nov 20, 2024 · 2 comments
Labels
[adalflow] suggest integration Add a new model_client, db retriever, etc in /adalflow help wanted Need helps with input, discussion, review, and PR submission.

Comments

@PhiBrandon
Copy link
Contributor

Description & Motivation

Given the landscape of Inference providers it seems like a good idea to have a way to interact with then using a relatively 'proven' library. Developing a ModelClient for LiteLLM would allow users to interact 100+ LLMs with more coming online everyday.

Pitch

There was a previous attempt at this, that has since been updated. https://github.com/SylphAI-Inc/AdalFlow/tree/litellm. This was implemented prior to optimization updates. I want to add this issue here for communication on this ModelClient.

Provider POC

Needs active outreach.

Additional context

No response

@PhiBrandon PhiBrandon changed the title LiteLLM ModelClient Integration LiteLLM ModelClient Integration [adalflow] suggest integration Nov 20, 2024
@PhiBrandon PhiBrandon changed the title LiteLLM ModelClient Integration [adalflow] suggest integration LiteLLM ModelClient Integration Nov 20, 2024
@liyin2015 liyin2015 added the [adalflow] suggest integration Add a new model_client, db retriever, etc in /adalflow label Nov 21, 2024
@liyin2015
Copy link
Member

liyin2015 commented Nov 21, 2024

the litellm team is not responsive

@PhiBrandon do you want to create a pr? Do users really find litellm useful? Let's wait to see if people will upvote for it.

I feel it will be more secure to use each provider's sdk (and often they are more supportive on the integration) and easier to maintain rather than introducing a 3rd party proxy

Note: Do not create a pr until we put up "ready-for-PR" label

@PhiBrandon
Copy link
Contributor Author

Based on discord vote this wouldn't be the best use of time at the moment.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[adalflow] suggest integration Add a new model_client, db retriever, etc in /adalflow help wanted Need helps with input, discussion, review, and PR submission.
Projects
None yet
Development

No branches or pull requests

2 participants