- 
                Notifications
    You must be signed in to change notification settings 
- Fork 326
dynamic model switch #2211
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?
dynamic model switch #2211
Conversation
Signed-off-by: Ubuntu <azureuser@denvr-inf.kifxisxbiwme5gt4kkwqsfdjuh.dx.internal.cloudapp.net>
| Dependency Review✅ No vulnerabilities or license issues found.Scanned FilesNone | 
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 enables dynamic model switching functionality across CodeGen and ChatQnA services. The changes modify how model identifiers are passed through the request pipeline, allowing clients to specify which model to use rather than being hardcoded to default values.
Key changes:
- Modified model assignment in input alignment to use dynamic model from request instead of hardcoded constants
- Updated request handling to pass model parameter with proper fallback to default values
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
| File | Description | 
|---|---|
| CodeGen/codegen.py | Updates model assignment to use dynamic input and adds model parameter to request handling with fallback | 
| ChatQnA/chatqna.py | Similar dynamic model assignment changes and fallback logic for request processing | 
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
for more information, see https://pre-commit.ci
| This PR is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 7 days. | 
| This PR is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 7 days. | 
Description
Dynamic model switching enabled.
Issues
#2209
Type of change
List the type of change like below. Please delete options that are not relevant.
Dependencies
NA
Tests
NA