Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Migrate TF Smooth Quant to 3.x API #1594

Merged
merged 20 commits into from
Feb 5, 2024
Merged

Migrate TF Smooth Quant to 3.x API #1594

merged 20 commits into from
Feb 5, 2024

Conversation

zehao-intel
Copy link
Contributor

Type of Change

Feature

Description

This is a separate PR which only includes TF SQ.
The Static Quant part will be in another PR with auto tune.

How has this PR been tested?

UT

Dependency Change?

No

@zehao-intel zehao-intel added INC3.X TensorFlow Related to Tensorflow/Keras F/W labels Feb 2, 2024
Signed-off-by: zehao-intel <[email protected]>

add pre_optimize for sq

Signed-off-by: zehao-intel <[email protected]>

fix ut issues

Signed-off-by: zehao-intel <[email protected]>

fix disable_random

Signed-off-by: zehao-intel <[email protected]>

[pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

fix pylint

Signed-off-by: zehao-intel <[email protected]>

fix import

Signed-off-by: zehao-intel <[email protected]>

[pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

modify requirements.txt

Signed-off-by: zehao-intel <[email protected]>

[pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

fix import

Signed-off-by: zehao-intel <[email protected]>

[pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci
Signed-off-by: zehao-intel <[email protected]>

Signed-off-by: zehao-intel <[email protected]>

fix tensorflow_model

Signed-off-by: zehao-intel <[email protected]>

[pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

add init to graph_rewriter

Signed-off-by: zehao-intel <[email protected]>

[pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

[pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci
@chensuyue chensuyue requested a review from yiliu30 February 2, 2024 14:54
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, let's discuss the details offline.

zehao-intel and others added 4 commits February 4, 2024 14:28
Signed-off-by: chensuyue <[email protected]>

[pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

add more deps for pylint scan

Signed-off-by: chensuyue <[email protected]>

test for pylint

Signed-off-by: chensuyue <[email protected]>

[pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

fix import

Signed-off-by: zehao-intel <[email protected]>
@zehao-intel
Copy link
Contributor Author

zehao-intel commented Feb 4, 2024

All issues in the conversation have been solved.

@zehao-intel zehao-intel requested a review from chensuyue February 5, 2024 07:51
@chensuyue chensuyue merged commit 3882e9c into master Feb 5, 2024
20 of 22 checks passed
@chensuyue chensuyue deleted the zehao/sq3.0 branch February 5, 2024 16:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
INC3.X TensorFlow Related to Tensorflow/Keras F/W
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants