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

Fix need of OpenAI API key if using Litellm #621

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

abhishekbhakat
Copy link
Contributor

Fixes the need of unused OpenAI API key if Litellm is configured after #604.

.gitignore Outdated Show resolved Hide resolved
@abhishekbhakat abhishekbhakat force-pushed the fix-need-of-openai-api-key-if-using-litellm branch from 8a2fc2f to 82a4061 Compare December 6, 2024 06:20
@abhishekbhakat abhishekbhakat force-pushed the fix-need-of-openai-api-key-if-using-litellm branch from 82a4061 to 1885d15 Compare December 6, 2024 06:22
@abhishekbhakat
Copy link
Contributor Author

@code-cntrl dropped the .gitignore
See if you can merge this.

I want the PR #575 to close but having to say you have to set a dummy key for OPENAI_API_KEY to use anything from LiteLLm makes it a friction.

@lucasliet
Copy link

in the case of #575, how can I provide the api_key from gemini in this implementation?

it would be good to have usage example on readme

@abhishekbhakat
Copy link
Contributor Author

abhishekbhakat commented Dec 6, 2024

@lucasliet you need to set GEMINI_API_KEY as env variable. As per https://docs.litellm.ai/docs/providers/gemini.

I explained in #604 (comment). Which works once you set LiteLLM as True

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.

3 participants