You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tvm.apache.org by GitBox <gi...@apache.org> on 2022/04/14 22:12:58 UTC
[GitHub] [tvm] mbrookhart opened a new pull request, #11019: [QNN] fix the type relation and add a test for grouped per-channel quantized convolution
mbrookhart opened a new pull request, #11019:
URL: https://github.com/apache/tvm/pull/11019
cc @anijain2305 @masahi @jwfromm
It looks like there was a bug in the type relation and no tests that hit it.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@tvm.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [tvm] mbrookhart closed pull request #11019: [QNN] fix the type relation and add a test for grouped per-channel quantized convolution
Posted by GitBox <gi...@apache.org>.
mbrookhart closed pull request #11019: [QNN] fix the type relation and add a test for grouped per-channel quantized convolution
URL: https://github.com/apache/tvm/pull/11019
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@tvm.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [tvm] AndrewZhaoLuo commented on pull request #11019: [QNN] fix the type relation and add a test for grouped per-channel quantized convolution
Posted by GitBox <gi...@apache.org>.
AndrewZhaoLuo commented on PR #11019:
URL: https://github.com/apache/tvm/pull/11019#issuecomment-1099821671
I'll try updating https://github.com/apache/tvm/pull/10162. I think I got the tests to pass but was a little suspicious since I think there might be a deeper problem: see https://github.com/apache/tvm/issues/10294. But maybe the codepaths happen to work out (it appears my last attempt passed CI somehow)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@tvm.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [tvm] masahi commented on pull request #11019: [QNN] fix the type relation and add a test for grouped per-channel quantized convolution
Posted by GitBox <gi...@apache.org>.
masahi commented on PR #11019:
URL: https://github.com/apache/tvm/pull/11019#issuecomment-1099738407
Possibly related issues
https://github.com/apache/tvm/pull/10162
https://github.com/apache/tvm/issues/10088
https://github.com/apache/tvm/issues/8299
https://github.com/apache/tvm/issues/7878
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@tvm.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [tvm] mbrookhart commented on pull request #11019: [QNN] fix the type relation and add a test for grouped per-channel quantized convolution
Posted by GitBox <gi...@apache.org>.
mbrookhart commented on PR #11019:
URL: https://github.com/apache/tvm/pull/11019#issuecomment-1099747011
Ooh, I hadn't seen Andrew's PR. It makes the same type relation change. @AndrewZhaoLuo want to fix the merge conflict and maybe pull in my test and I'll close this in favor of yours?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@tvm.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org