Skip to content

remove CompressionParameter#4108

Merged
andrey-churkin merged 1 commit into
openvinotoolkit:developfrom
AlexanderDokuchaev:ad/del_compressed_parameter
Jun 23, 2026
Merged

remove CompressionParameter#4108
andrey-churkin merged 1 commit into
openvinotoolkit:developfrom
AlexanderDokuchaev:ad/del_compressed_parameter

Conversation

@AlexanderDokuchaev

@AlexanderDokuchaev AlexanderDokuchaev commented Jun 19, 2026

Copy link
Copy Markdown
Collaborator

Changes

Removed CompressionParameter

Reason for changes

Unused

Related tickets

Tests

@github-actions github-actions Bot added the NNCF PT Pull requests that updates NNCF PyTorch label Jun 19, 2026
@AlexanderDokuchaev AlexanderDokuchaev marked this pull request as ready for review June 22, 2026 06:39
@AlexanderDokuchaev AlexanderDokuchaev requested a review from a team as a code owner June 22, 2026 06:39
Copilot AI review requested due to automatic review settings June 22, 2026 06:39

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 removes the CompressionParameter abstraction from the PyTorch backend and eliminates the now-unused compression_lr_multiplier field from quantizer specs/config serialization, simplifying quantizer parameter handling and related tests.

Changes:

  • Deleted CompressionParameter from nncf.torch.layer_utils and replaced its uses with plain torch.nn.Parameter.
  • Removed compression_lr_multiplier from PTQuantizerSpec constructor/state, and from min-max backend quantizer construction.
  • Updated serialization test fixture to reflect the new quantizer config schema; updated mypy exclude list accordingly.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.

Show a summary per file
File Description
tests/torch/function_hook/test_serialization.py Updates expected quantizer module config (drops compression_lr_multiplier).
src/nncf/torch/quantization/layers.py Removes CompressionParameter usage and drops compression_lr_multiplier from PTQuantizerSpec.
src/nncf/torch/layer_utils.py Removes the CompressionParameter class (leaves only StatefulModuleInterface).
src/nncf/quantization/algorithms/min_max/torch_backend.py Stops passing compression_lr_multiplier when creating PTQuantizerSpec.
pyproject.toml Removes src/nncf/torch/layer_utils.py from mypy’s exclude list.

@andrey-churkin andrey-churkin merged commit 4735c26 into openvinotoolkit:develop Jun 23, 2026
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

NNCF PT Pull requests that updates NNCF PyTorch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants