search

opensouls / terminal-copilot

A smart terminal assistant that helps you find the right command.
Apache License 2.0
573 stars 43 forks source link

Different text embedding model #37

Open loftusa opened 1 year ago

loftusa commented 1 year ago

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[bot] commented 1 year ago

Hey! Thanks for sharing feedback with the community!

JoelKronander commented 1 year ago

@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 commented 1 year ago

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 commented 1 year ago

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