-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
[OneDNN] cpu_quantize_pass fix #56303
[OneDNN] cpu_quantize_pass fix #56303
Conversation
你的PR提交成功,感谢你对开源项目的贡献! |
|
✅ This PR's description meets the template requirements! |
cacfdc6
to
0b3d693
Compare
fea2f85
to
622e67a
Compare
…to avoid dump error
@vivienfanghuagood Hi, can you please take a review on this PR? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
* add cpu quantize pass unit test
PR types
Bug fixes
PR changes
Others
Description
In MobileNetV2 unit test, there is an error of
Attribute fuse_alpha is not found
. We found during upgrade OneDNN v3, we usefuse_beta
in relu6, there is no need to multi scale_out to getfuse_alpha
. So the related code can be removed. And the unit test has been passed.