-
Notifications
You must be signed in to change notification settings - Fork 59
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
Enable OpenAI other compatible APIs #15
base: main
Are you sure you want to change the base?
Conversation
Don't merge this yet. Still debugging |
Seems to be working now. Seems like llama3 has some hard coded text that where the "```python" continue from the prompt doesn't work. Added a regex to extract based on code markdown code blocks "```" Seems to be working on my end but would appreciate a second pair of eyes. Set Note the client object is not per model type, so there is no way to mix models between open ones and openai models. |
Thanks @rapatel0. Could you take a look at some of my comments in the code? |
I can't anything in the review changes on github. Where are the comments? |
@rapatel0 Just tagged you. |
Hmm, that's odd! I'm not sure why it's not showing up. I'll just write it here:
(1) add a space between elif and the string Sorry for the nit.
|
Two changes:
OPENAI_API_BASE_URL
as environmental variable.Tested with ollama and llama3