nncf icon indicating copy to clipboard operation
nncf copied to clipboard

Fix SQ algorithm ports usage

Open nikita-malininn opened this issue 1 year ago • 1 comments

Changes

  • Fixed SmoothQuant algorithm to work with Split ports correctly;

Reason for changes

  • Bugfix

Related tickets

  • 140351

Tests

nikita-malininn avatar May 08 '24 13:05 nikita-malininn

Codecov Report

Attention: Patch coverage is 0% with 2 lines in your changes are missing coverage. Please review.

Project coverage is 29.87%. Comparing base (359447b) to head (44c5617).

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff              @@
##           develop    #2672       +/-   ##
============================================
- Coverage    62.02%   29.87%   -32.16%     
============================================
  Files          495      495               
  Lines        45969    45970        +1     
============================================
- Hits         28514    13735    -14779     
- Misses       17455    32235    +14780     
Files Coverage Δ
.../quantization/algorithms/smooth_quant/algorithm.py 21.84% <0.00%> (-72.31%) :arrow_down:

... and 264 files with indirect coverage changes

Flag Coverage Δ
COMMON ?
ONNX ?
OPENVINO ?
TENSORFLOW 29.87% <0.00%> (-0.01%) :arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Components Coverage Δ
common 76.35% <ø> (-12.10%) :arrow_down:
torch 0.01% <ø> (-32.84%) :arrow_down:
tensorflow 93.74% <ø> (ø)
onnx 0.00% <ø> (-93.07%) :arrow_down:
openvino 0.00% <ø> (-94.25%) :arrow_down:
ptq 15.18% <0.00%> (-65.49%) :arrow_down:

codecov[bot] avatar May 08 '24 13:05 codecov[bot]

@alexsu52, please, review.

nikita-malininn avatar May 27 '24 09:05 nikita-malininn