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
[Intel MKL] Enabling Quantized Conv ops in native format mode #45108
[Intel MKL] Enabling Quantized Conv ops in native format mode #45108
Conversation
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.
Thank you for the PR and I'm so sorry for the delay!
tensorflow/core/kernels/mkl/mkl_quantized_conv_ops_perchannel_test.cc
Outdated
Show resolved
Hide resolved
Thank you for the valuable comments. I have addressed them. |
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.
Thank you for the changes and for cleaning up the registrations even further! I have a few nits.
#include "tensorflow/core/kernels/quantization_utils.h" | ||
|
||
namespace tensorflow { | ||
|
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.
Nit: Please remove this blank line.
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.
Done.
I have addressed the remaining comments. Please have a look. Thank you! |
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.
Thank you for the changes! This PR looks good to me now. I think I can just approve it and remove the ready-to-pull
tag. (Will add it back after #45107 is merged.)
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.
Thank you for the PR and I'm sorry for the delay!
No description provided.