-
Notifications
You must be signed in to change notification settings - Fork 123
feat: use allow changing the Copilot model used #2 #373
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
Conversation
language server now expects to be run directly through language-server.js
I was not sure on the config placement, let me know if you prefer it elsewhere! |
When I set wrong model name, I can see on LSP logs that it fallbacks to default one. It would be nice to validate it in on setup and print error message. |
Hmm the downside to validating it before the LSP means that we would need to add future models whenever they come out. Though it took some time for 4o to come out so it might not be much maintenance. I'll wait for @zbirenbaum to chip in on this one as I am divided on this! |
Anyone know what to set it for, for Claude 3.7? And is there a way to check which model is being used? |
Claude 3.7 is only for the chat, not the autocomplete. |
nothing yet? |
Can we get this merged @zbirenbaum? I was surprised to learn that I've been using GPT 3.5 this whole time |
Assuming @zbirenbaum is too busy to review and merge the PRs (life hapens!), you can always use my repo in your return {
"AntoineGS/copilot.lua",
config = function()
require("copilot").setup {
copilot_model = "gpt-4o-copilot",
}
end,
} It does include some other changes that are either in PRs or going to be on PRs: |
Love it! Thank you! |
No problem, |
Is this going to add support for models like Claude that is supported in vscode? |
We should sticky this 😂 |
Sorry, my bad for bring up old wounds :) |
Depends on PR #372
Addition of github_model option, used as such:
Big thank to @trixnz for figuring out the required configs!
Fixes #365