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/09 16:30:31 UTC

[GitHub] [incubator-tvm] giuseros opened a new pull request #6027: Bug fix] Fix in arm_cpu/conv2d_alter_op for NHWC quantized

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


   Few minor typos to be fixed in topi/arm_cpu/conv2d_alter_op.py for the
   NHWC quantized route:
   - Kernel shape was misread (CO, IC, KH, KW) -> (KH, KW, IC, OC)
   - Pad along the K dimension was misspelled: pad_k -> pad_K
   - Workload name was wrong: "conv2d_NHWC_int8_without_tranform.arm_cpu"
     -> "conv2d_NHWC_quantized_without_transform.arm_cpu"
   
   This submission fixes those errors and add a further test for conv2d_alter_op.py
   
   Change-Id: I0622df05f1d4d15311946f6e75f1840a34815a5b


----------------------------------------------------------------
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] icemelon9 commented on pull request #6027: Bug fix] Fix in arm_cpu/conv2d_alter_op for NHWC quantized

Posted by GitBox <gi...@apache.org>.
icemelon9 commented on pull request #6027:
URL: https://github.com/apache/incubator-tvm/pull/6027#issuecomment-656806953


   Thanks @giuseros 


----------------------------------------------------------------
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] icemelon9 merged pull request #6027: Bug fix] Fix in arm_cpu/conv2d_alter_op for NHWC quantized

Posted by GitBox <gi...@apache.org>.
icemelon9 merged pull request #6027:
URL: https://github.com/apache/incubator-tvm/pull/6027


   


----------------------------------------------------------------
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] giuseros commented on pull request #6027: Bug fix] Fix in arm_cpu/conv2d_alter_op for NHWC quantized

Posted by GitBox <gi...@apache.org>.
giuseros commented on pull request #6027:
URL: https://github.com/apache/incubator-tvm/pull/6027#issuecomment-656807720


   Thank you @icemelon9 !


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