Skip to content

Conversation

violetch24
Copy link
Contributor

@violetch24 violetch24 commented Feb 2, 2024

Type of Change

Migrate static quant ipex backend to 3.x API
API changed

Description

  • Change UTs to pytest
  • Add more UTs to cover more parameters and functions
  • Refine quantize function to cover older version ipex
  • Refine docstring and coding format
  • Check accuracy

Expected Behavior & Potential Risk

pass UT and CI

How has this PR been tested?

Dependency Change?

@violetch24 violetch24 marked this pull request as ready for review February 4, 2024 08:46
@violetch24 violetch24 requested review from xin3he and yiliu30 February 4, 2024 08:46
Copy link
Contributor

@yiliu30 yiliu30 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

left some comments. If any of them are unclear, let's discuss the details offline.

@yiliu30 yiliu30 added INC3.X PyTorch Related to PyTorch F/W labels Feb 4, 2024
violetch24 and others added 25 commits February 5, 2024 15:23
Signed-off-by: Cheng, Zixuan <[email protected]>
Signed-off-by: Cheng, Zixuan <[email protected]>
Conflicts:
	neural_compressor/torch/quantization/algorithm_entry.py
	neural_compressor/torch/quantization/quantize.py
Signed-off-by: Cheng, Zixuan <[email protected]>
Signed-off-by: Cheng, Zixuan <[email protected]>
Signed-off-by: Cheng, Zixuan <[email protected]>
Signed-off-by: Cheng, Zixuan <[email protected]>
Signed-off-by: Cheng, Zixuan <[email protected]>
Signed-off-by: Cheng, Zixuan <[email protected]>
Signed-off-by: Cheng, Zixuan <[email protected]>
Signed-off-by: yiliu30 <[email protected]>
@chensuyue chensuyue merged commit 191383e into master Feb 7, 2024
@chensuyue chensuyue deleted the zixuan/ipex_static branch February 7, 2024 05:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
INC3.X PyTorch Related to PyTorch F/W
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants