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

expand support to Claude, Cohere, Llama2 and Azure #142

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

krrishdholakia
Copy link

@krrishdholakia krrishdholakia commented Aug 4, 2023

Hi @GreyDGL,

Saw you expanded support to LocalLLMs - and wanted to see how to add support for other LLM providers. I'm working on litellm (simple library to standardize LLM API Calls - https://github.com/BerriAI/litellm) and was wondering if we could be helpful.

Added support for Anthropic, Llama2, and Cohere by replacing the raw openai.chatCompletion.create function with completion. The code is pretty similar to the OpenAI class - as litellm follows the same pattern as the openai-python sdk.

Would love to know if this helps.

@krrishdholakia
Copy link
Author

Happy to make any relevant changes if required

@vmayoral
Copy link
Collaborator

vmayoral commented Aug 6, 2023

@krrishdholakia did you tested this out? can you report on any experiments conducted with it?

@krrishdholakia
Copy link
Author

krrishdholakia commented Aug 7, 2023

Hey @vmayoral tested it with Anthropic (claude-instant-1).
Screenshot 2023-08-07 at 12 22 51 PM

@krrishdholakia
Copy link
Author

The PR checks for environment keys for anthropic/cohere/replicate in addition to the openai keys.

Depending on which model is passed in by the user it routes to the correct spec.

@ishaan-jaff
Copy link

@vmayoral any updates on this PR ?

Are you exploring building a hosted version of Pentestgpt ?

@krrishdholakia
Copy link
Author

Bump on this? @vmayoral @GreyDGL

@AkechiShiro
Copy link

Is this the PR that fixes the issue #77 ? @GreyDGL

Yellowwaves

This comment was marked as abuse.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants