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

Feat Request: Different OpenAI API Support #592

Open
egelhaus opened this issue Feb 6, 2025 · 4 comments
Open

Feat Request: Different OpenAI API Support #592

egelhaus opened this issue Feb 6, 2025 · 4 comments
Labels
type: feature-request New feature or request

Comments

@egelhaus
Copy link
Collaborator

egelhaus commented Feb 6, 2025

Support for different kind of APIs because there are multiple Providers which support the OpenAI API, so just be able to change the API URL in the env.
I can't raise an PR.

@egelhaus egelhaus added the type: feature-request New feature or request label Feb 6, 2025
@egelhaus egelhaus changed the title Feat Request: OpenRouter API Support Feat Request: Different OpenAI API Support Feb 6, 2025
@cchance27
Copy link

cchance27 commented Feb 6, 2025

Seems like this would be a quick change, just gotta update the 3 files that reference the OpenAI module... to add the

const openai = new OpenAI({
apiKey: process.env.OPENAI_API_KEY || 'sk-proj-',
baseURL: process.env.OPENAI_BASE_URL || 'https://api.openai.com/v1',
});

would do it but not home at moment, would also allow you to point it to local ollama and other instances

@egelhaus
Copy link
Collaborator Author

egelhaus commented Feb 6, 2025

Yeah @cchance27 , but we would also need to configure the model, since it won't be compatible with local Instances.

@cchance27
Copy link

Ah True True, Not to mention system messages might want to be different for the various prompts, might want to expose or move those to a config or conf in the db to be adjustable, i tend to find ENV variables are great for the base_url, key, model name, but if it ends up being that things like the system messages etc should be adjustable maybe its best to have those manageable from the admin section perhaps?

@egelhaus
Copy link
Collaborator Author

egelhaus commented Feb 7, 2025

Yeah, that would be good

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: feature-request New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants