Skip to content

[tests] migrate model-level ModelOpt quantization tests to model tests - #14256

Open
sayakpaul wants to merge 3 commits into
mainfrom
modelopt-model-tests-migration
Open

[tests] migrate model-level ModelOpt quantization tests to model tests#14256
sayakpaul wants to merge 3 commits into
mainfrom
modelopt-model-tests-migration

Conversation

@sayakpaul

Copy link
Copy Markdown
Member

As discussed in #14113 (comment)

Also filed a related issue: NVIDIA/Model-Optimizer#2001

… mixins

Move the model-level tests out of tests/quantization/modelopt/test_modelopt.py
(the ModelOptBaseTesterMixin subclasses) into the standardized model-test mixin
structure:

- Extend ModelOptTesterMixin with keep_modules_in_fp32, training and
  prequantized-serialization coverage, and set disable_conv_quantization in the
  shared MODELOPT_CONFIGS so conv-containing models (SD3 patch embed) don't break.
- Wire concrete SD3 classes: TestSD3TransformerModelOpt and
  TestSD3TransformerModelOptCompile.
- Trim test_modelopt.py to the remaining pipeline-level test_model_cpu_offload
  (kept per quant type, incl. NF4/NVFP4).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@sayakpaul
sayakpaul requested a review from DN6 July 21, 2026 05:06
@github-actions github-actions Bot added size/L PR with diff > 200 LOC tests and removed size/L PR with diff > 200 LOC labels Jul 21, 2026
@github-actions github-actions Bot added the size/L PR with diff > 200 LOC label Jul 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/L PR with diff > 200 LOC tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant