-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Allow exp of default model #1301
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
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR adds experimentation support for the default language model selection in the GitHub Copilot Chat extension. The change allows A/B testing of different default models through the experimentation service.
- Introduces dependency injection for the experimentation service
- Implements hierarchical default model selection logic with experimentation override
- Restructures the default endpoint selection to prioritize experimental models over system defaults
@lramos15 do you know how these changes interact with these variables that also change the default model in agent mode? To my understanding, they've only been used for OSWE internal flights. It seems like the intention is the same, so we can switch OSWE to use the chat.defaultLanguageModel variable when they do a next iteration:
|
@karthiknadig Would know more about these, but I don't think so |
@jul-stevenson |
Thanks, @karthiknadig! @isidorn and I discussed moving all default model experiments to a single experiment which covers all users. Since |
Only the default model part becomes useless. The other things it does is still useful, like acting as a proxy and allowing you to change visible name of the model in the picker. |
Allows experimentation of default model via extension