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

[QNN] Support different qnn params between in/out tensor in leaky_relu #12116

Merged
merged 7 commits into from
Jul 22, 2022

Conversation

zhaoyang-star
Copy link
Contributor

@zhaoyang-star zhaoyang-star commented Jul 16, 2022

@AndrewZhaoLuo AndrewZhaoLuo self-assigned this Jul 19, 2022
@@ -1192,15 +1192,13 @@ def leaky_relu(x, alpha, scale, zero_point):
The scale of the quantized expr.
Copy link
Contributor

Choose a reason for hiding this comment

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

change this and desc. for zero point to 'scale of the input quantized expr'

Copy link
Contributor Author

@zhaoyang-star zhaoyang-star Jul 20, 2022

Choose a reason for hiding this comment

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

Done

python/tvm/relay/qnn/op/qnn.py Outdated Show resolved Hide resolved
src/relay/qnn/op/leaky_relu.cc Outdated Show resolved Hide resolved
src/relay/qnn/op/leaky_relu.cc Outdated Show resolved Hide resolved
src/relay/qnn/op/leaky_relu.cc Outdated Show resolved Hide resolved
src/relay/qnn/op/leaky_relu.cc Outdated Show resolved Hide resolved
src/relay/qnn/op/leaky_relu.cc Outdated Show resolved Hide resolved
@zhaoyang-star
Copy link
Contributor Author

zhaoyang-star commented Jul 21, 2022

Please review again. @AndrewZhaoLuo
Thank you.

@AndrewZhaoLuo AndrewZhaoLuo merged commit 76155c2 into apache:main Jul 22, 2022
@zhaoyang-star zhaoyang-star deleted the leaky_relu_qnn_update branch July 26, 2022 11:05
xinetzone pushed a commit to daobook/tvm that referenced this pull request Nov 25, 2022
apache#12116)

* [QNN] Support different qnn params between in/out tensor in leaky_relu

* format code

* format code

* fix bug

* fix format

* fix format

* fix
mikeseven pushed a commit to mikeseven/tvm that referenced this pull request Sep 27, 2023
apache#12116)

* [QNN] Support different qnn params between in/out tensor in leaky_relu

* format code

* format code

* fix bug

* fix format

* fix format

* fix
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants