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

Different text embedding model #37

Open
loftusa opened this issue Dec 29, 2022 · 4 comments
Open

Different text embedding model #37

loftusa opened this issue Dec 29, 2022 · 4 comments

Comments

@loftusa
Copy link

loftusa commented Dec 29, 2022

I see that this package uses text-davinci-003 as its standard model: https://github.com/Methexis-Inc/terminal-copilot/blob/main/copilot/main.py#L186

but OpenAI recommends the new text-embedding-ada-002 for nearly all use cases, which is substantially cheaper and better: https://beta.openai.com/docs/guides/embeddings/embedding-models

Regardless, one of the code models (e.g., code-davinci-002) is pretrained specifically on code examples and might be better.

Are there any plans to switch, or is there a good reason we're using text-davinci-003?

@github-actions
Copy link

Hey! Thanks for sharing feedback with the community!

@JoelKronander
Copy link
Collaborator

@loftusa in my experience text-davinci-003 has worked best. Feel free to make a PR for adding a command line flag to switch to a different model though, should be a great starting PR.!

@shantanuo
Copy link

I think it is important to be able to change the model if I am not getting the expected command. For e.g.

copilot kill the orphaned python process --model=codex

Ref: https://github.com/yasyf/gpt-do

@2mawi2
Copy link
Contributor

2mawi2 commented Mar 19, 2023

@JoelKronander @shantanuo this can be closed after the GPT migration

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

4 participants