Add model architecture support to quantization job parameters#7007
Merged
Conversation
jpggvilaca
reviewed
Jul 10, 2026
🐳 Docker image sizes
|
⏱️ Backend import time —
|
| Accelerator | Cumulative (s) | Self (s) |
|---|---|---|
cpu |
4.365 | 0.003 |
xpu |
4.548 | 0.003 |
cuda |
4.557 | 0.004 |
Contributor
There was a problem hiding this comment.
Pull request overview
This PR extends the quantization job contract so the backend receives the model architecture identifier, resolves it to a user-friendly architecture name, and exposes richer model metadata (name + architecture) for quantization jobs so the UI can display it on the jobs card.
Changes:
- UI: include
model_architecture_idwhen submitting quantization jobs and plumb the value through the quantization dialog. - Backend: add
model_architecture_id/model_architecture_nametoQuantizationJobParamsand include{id,name,architecture}in quantization job metadata. - Tests: update UI and backend unit tests to cover the new parameters/metadata.
Reviewed changes
Copilot reviewed 10 out of 10 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| application/ui/tests/models/model-details.spec.ts | Updates mocked quantization job payload to include model_architecture_id. |
| application/ui/src/features/models/model-listing/model-variants/quantization-row.component.tsx | Adds modelArchitectureId prop and passes it into the dialog. |
| application/ui/src/features/models/model-listing/model-variants/quantization-dialog/quantization-dialog.component.tsx | Sends model_architecture_id in the quantization job submission payload. |
| application/ui/src/features/models/model-listing/model-variants/model-variant-tabs.component.tsx | Wires model.architecture into QuantizationRow. |
| application/backend/tests/unit/models/jobs/test_quantization_job.py | Updates quantization job params fixture for new required fields. |
| application/backend/tests/unit/execution/quantization/test_getitune_quantizer.py | Updates quantization params in multiple tests for new required fields. |
| application/backend/tests/unit/api/routers/test_jobs.py | Patches manifest lookup and asserts architecture id/name propagation into submitted job params. |
| application/backend/app/models/jobs/quantization_job.py | Adds architecture fields and a computed model_name used for metadata. |
| application/backend/app/api/schemas/jobs/quantization.py | Extends request params and enriches quantization job metadata model fields. |
| application/backend/app/api/routers/jobs.py | Resolves architecture name from manifest during quantize job submission and stores it in params. |
📊 Test coverage report
|
itallix
previously approved these changes
Jul 13, 2026
github-merge-queue
Bot
removed this pull request from the merge queue due to failed status checks
Jul 13, 2026
github-merge-queue
Bot
removed this pull request from the merge queue due to failed status checks
Jul 13, 2026
leoll2
previously approved these changes
Jul 13, 2026
dwesolow
dismissed stale reviews from leoll2, itallix, and jpggvilaca
via
July 14, 2026 09:01
f38d37c
A-Artemis
enabled auto-merge
July 14, 2026 09:25
leoll2
approved these changes
Jul 14, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Adds the model name to the jobs card for quantization.
Checklist