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.

Sign up for GitHub

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

Jump to bottom

[LiteLLM Fix] Fix using litellm.api_base, litellm.api_key, litellm.api_version #284

Merged
merged 4 commits into from
Sep 12, 2023
Merged

[LiteLLM Fix] Fix using litellm.api_base, litellm.api_key, litellm.api_version #284

merged 4 commits into from
Sep 12, 2023

Conversation

ishaan-jaff
Copy link
Contributor

@ishaan-jaff ishaan-jaff commented Sep 11, 2023 •
edited by jordanbtucker
Loading

Users reported not being able to correctly use

 litellm.api_type = self.azure_api_type 
 litellm.api_base = self.azure_api_base 
 litellm.api_version = self.azure_api_version 
 litellm.api_key = self.api_key

This PR bumps the version of litellm to ensure this setting works

Litellm 0.1.590 has the fix

Fixes #263
Fixes #258

@ishaan-jaff
Copy link
Contributor Author

cc @KillianLucas @jordanbtucker @codeacme17

This was referenced Sep 11, 2023
Closed
Closed
@ishaan-jaff
Copy link
Contributor Author

fixed poetry lock errors

@Taik Taik mentioned this pull request Sep 11, 2023
Merged
@TanmayDoesAI TanmayDoesAI merged commit b457a62 into OpenInterpreter:main Sep 12, 2023
joshuavial pushed a commit to joshuavial/open-interpreter that referenced this pull request Sep 15, 2023
@ishaan-jaff
[LiteLLM Fix] Fix using litellm.api_base, litellm.api_key, litellm.ap…
e99231e 
…i_version (OpenInterpreter#284)

Fixed OpenInterpreter#245 

* bump litellm

* bump litellm version

* fix failing build try ^0.1.589

* bump poetry lock version + litellm
joshuavial pushed a commit to joshuavial/open-interpreter that referenced this pull request Nov 16, 2023
@ishaan-jaff
[LiteLLM Fix] Fix using litellm.api_base, litellm.api_key, litellm.ap…
1587cb3 
…i_version (OpenInterpreter#284)

Fixed OpenInterpreter#245 

* bump litellm

* bump litellm version

* fix failing build try ^0.1.589

* bump poetry lock version + litellm



Former-commit-id: b457a62
Former-commit-id: 421e5eac2ebe7d67c93c106957dbd1818e126870
Former-commit-id: d278f8c2c7e1922e04aa4953c7e3be7c5cff23df [formerly 3a31e648db9558fe36a0e8910e9a87d264407663]
Former-commit-id: f0b189d7622260f8e23d8156f59f0d7501e88c9e
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@KillianLucas KillianLucas Awaiting requested review from KillianLucas

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

API_KEY can not find Error with Azure API in version 0.1.3 & 0.1.2
2 participants
@ishaan-jaff @TanmayDoesAI