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

Allow openai endpoint configuration via config #35

Open

CuSO4Deposit wants to merge 4 commits into piersolenski:main from CuSO4Deposit:allow-openai-endpoint-configuration

CuSO4Deposit commented Sep 22, 2024

Continue #18 .

When a user wants to use other API-compatible services or use a local instance of GPT, might be needed to redefine the OpenAI base URL.

All the nil check code has been removed for the following reasons:

As mentioned, we have a fallback in place, so the check is not needed.
The openai_base_url should not be manually set by all users. Only users who want to use a custom URL will intentionally set it, as they know it’s necessary. Others who use the original endpoint shouldn't receive any warnings about the base URL.

Karim Shakirov and others added 4 commits

September 22, 2024 18:41


          add base_url to config, implement usage of base_url in requests


          rm commented code

27a7e63


          Update README

172d42a


          remove unused code since base_url is always set

1644ade

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet