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 2020/07/23 21:38:22 UTC

[GitHub] [incubator-tvm] ymwangg opened a new pull request #6130: [AutoTVM][BugFix] Fix autotvm on the conv2d_nchw_winograd.mali operator

ymwangg opened a new pull request #6130:
URL: https://github.com/apache/incubator-tvm/pull/6130


   This PR fixes two bugs related to autotvm on the `conv2d_nchw_winograd.mali` operator.
   - KeyError when computing`kvshape`
   - Duplicate args returned by `autotvm.task.task.TaskTemplate.get_inputs`
   
   Due to the hack term [here]( https://github.com/apache/incubator-tvm/blob/0a1c4c2174e1c4a04ca6e40cd90cdf7c2ef1d90a/topi/python/topi/mali/conv2d.py#L323), both tensor `output` and tensor `Y` point to the same tensor `M`, which causes tensor `M` to be added in the queue twice. Using a hash set can remove duplicate nodes when performing level order traversal.


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [incubator-tvm] kevinthesun commented on pull request #6130: [AutoTVM][BugFix] Fix autotvm on the conv2d_nchw_winograd.mali operator

Posted by GitBox <gi...@apache.org>.

kevinthesun commented on pull request #6130:
URL: https://github.com/apache/incubator-tvm/pull/6130#issuecomment-663766170


   Thanks @ymwangg @comaniac 


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [incubator-tvm] kevinthesun merged pull request #6130: [AutoTVM][BugFix] Fix autotvm on the conv2d_nchw_winograd.mali operator

Posted by GitBox <gi...@apache.org>.

kevinthesun merged pull request #6130:
URL: https://github.com/apache/incubator-tvm/pull/6130


   


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org