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
[BUG][ConvertLayout] Fix qnn.conv2d layout conversion too many values to unpack #6442
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.
After applying this patch, I can successfully run the tests from #6302, that were reproducing this problem.
LGTM
0e8261d
to
bb4cb09
Compare
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
please wait until we merge #6448 first. |
… to unpack This patch follows a previous bugfix in apache#6419. I made a very simple oversight for qnn.conv2d in that tinfos also contains qnn parameters. Therefore, we need to extract data_info and weight_info differently. Change-Id: Ib0ad01f427543371380d0bb604a77b5e0ec1103d
bb4cb09
to
8dacecf
Compare
Thanks @lhutton1 @anijain2305 @comaniac @leandron |
After apache#6442 was merged, black started complaining about formatting of test_pass_convert_op_layout.py. Format here. Change-Id: I04346fa06e22b722b619488b895b47c6943e3fd9
After #6442 was merged, black started complaining about formatting of test_pass_convert_op_layout.py. Format here. Change-Id: I04346fa06e22b722b619488b895b47c6943e3fd9
… to unpack (apache#6442) This patch follows a previous bugfix in apache#6419. I made a very simple oversight for qnn.conv2d in that tinfos also contains qnn parameters. Therefore, we need to extract data_info and weight_info differently. Change-Id: Ib0ad01f427543371380d0bb604a77b5e0ec1103d
After apache#6442 was merged, black started complaining about formatting of test_pass_convert_op_layout.py. Format here. Change-Id: I04346fa06e22b722b619488b895b47c6943e3fd9
… to unpack (apache#6442) This patch follows a previous bugfix in apache#6419. I made a very simple oversight for qnn.conv2d in that tinfos also contains qnn parameters. Therefore, we need to extract data_info and weight_info differently. Change-Id: Ib0ad01f427543371380d0bb604a77b5e0ec1103d
After apache#6442 was merged, black started complaining about formatting of test_pass_convert_op_layout.py. Format here. Change-Id: I04346fa06e22b722b619488b895b47c6943e3fd9
… to unpack (apache#6442) This patch follows a previous bugfix in apache#6419. I made a very simple oversight for qnn.conv2d in that tinfos also contains qnn parameters. Therefore, we need to extract data_info and weight_info differently. Change-Id: Ib0ad01f427543371380d0bb604a77b5e0ec1103d
After apache#6442 was merged, black started complaining about formatting of test_pass_convert_op_layout.py. Format here. Change-Id: I04346fa06e22b722b619488b895b47c6943e3fd9
This patch follows a previous bugfix in #6419. I made a very simple oversight for qnn.conv2d in that tinfos also contains qnn parameters. Therefore, we need to extract data_info and weight_info differently.
cc @leandron @anijain2305 @JoeyTPChou