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

Refactor OCO_BASE_PATHs configs and others to be a single config OCO_API_URL #358

Closed
jeromeleong opened this issue Jul 6, 2024 · 5 comments

Comments

@jeromeleong
Copy link

Description

While setting up OpenCommit, I'm experiencing issues with the OCO_OPENAI_BASE_PATH configuration. I've set it to https://api.xxx.xxx/v1/ and provided the API key, but it's not working as expected.

Suggested Solution

Can you provide some specific examples of how to correctly use OCO_OPENAI_BASE_PATH, especially when using a custom API endpoint?
Are there any additional steps or configurations needed when using a custom API endpoint instead of the default OpenAI one?

Alternatives

No response

Additional Context

No response

@di-sukharev
Copy link
Owner

lets start with what do you mean by not working as expected?

@di-sukharev
Copy link
Owner

@jeromeleong :)

@di-sukharev
Copy link
Owner

we need to make OCO_OPENAI_BASE_PATH and OCO_FLOWISE_API_URL and all variables solving the same thing (path to an endpoint) — be one variable OCO_API_URL

if smdb introduces a PR before me — please refer to this ticket and add a README docs on how to use the OCO_API_URL. It's used on Flowise and Ollama engines, and also in openAi engine and Azure too

@di-sukharev
Copy link
Owner

there are now many similar configs which should be refactored to a single one, e.g.:
OCO_OPENAI_BASE_PATH
OCO_GEMINI_BASE_PATH
OCO_API_URL
OCO_OLLAMA_API_URL
OCO_FLOWISE_ENDPOINT

they should all be a single config key, we'll keep the OCO_API_URL instead of all the others. I'm renaming the ticket to serve this purpose. We will also add docs to readme. @matscube wdyt?

@di-sukharev di-sukharev changed the title Usage and Examples for OCO_OPENAI_BASE_PATH Refactor OCO_BASE_PATHs configs and others to be a single config OCO_API_URL Sep 1, 2024
@di-sukharev
Copy link
Owner

done

update the latest version just merged #412

run npm i -g opencommit@latest

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

No branches or pull requests

2 participants