[GitHub] [incubator-tvm] t-vi opened a new pull request #5433: fix miopen padding

[GitBox <gi...@apache.org>]

t-vi opened a new pull request #5433:
URL: https://github.com/apache/incubator-tvm/pull/5433


   The MIOpen convolution accidentally passed the pad left + pad right (and top + bottom) instead of just the value applied to both sides. This patch fixes this, and also conditions the use of MIOpen in the op strategy on equality of the padding on the corresponding sides.
   


----------------------------------------------------------------
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] tqchen commented on pull request #5433: fix miopen padding

[GitBox <gi...@apache.org>]

tqchen commented on pull request #5433:
URL: https://github.com/apache/incubator-tvm/pull/5433#issuecomment-619108217


   thanks @t-vi  please rebase


----------------------------------------------------------------
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] masahi commented on pull request #5433: fix miopen padding

[GitBox <gi...@apache.org>]

masahi commented on pull request #5433:
URL: https://github.com/apache/incubator-tvm/pull/5433#issuecomment-619299101


   Thanks @t-vi 


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