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/01/05 01:56:18 UTC

[GitHub] [tvm] xiaolong18 opened a new pull request #9835: [Fix Bug] fix the bug of pool_impl_nd when computing avgpool_nd whith ceil_mode and count_include_pad are True

xiaolong18 opened a new pull request #9835:
URL: https://github.com/apache/tvm/pull/9835


   * Added the offset[i] for getting the correct  boundary
   * Added corresponding test case
   
   Thanks for contributing to TVM!   Please refer to guideline https://tvm.apache.org/docs/contribute/ for useful information and tips. After the pull request is submitted, please request code reviews from [Reviewers](https://github.com/apache/incubator-tvm/blob/master/CONTRIBUTORS.md#reviewers) by @ them in the pull request thread.
   


-- 
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] xiaolong18 commented on pull request #9835: [Fix Bug] fix the bug of pool_impl_nd when computing avgpool_nd whith ceil_mode and count_include_pad are True

Posted by GitBox <gi...@apache.org>.
xiaolong18 commented on pull request #9835:
URL: https://github.com/apache/tvm/pull/9835#issuecomment-1011806049


   @masahi  so sorry , i forget to add new input_shape as [1,1,1,9] 


-- 
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 edited a comment on pull request #9835: [Fix Bug] fix the bug of pool_impl_nd when computing avgpool_nd whith ceil_mode and count_include_pad are True

Posted by GitBox <gi...@apache.org>.
masahi edited a comment on pull request #9835:
URL: https://github.com/apache/tvm/pull/9835#issuecomment-1010629842


   I ran the test case locally and it passed without your change in topi. Is this expected?


-- 
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] xiaolong18 commented on pull request #9835: [Fix Bug] fix the bug of pool_impl_nd when computing avgpool_nd whith ceil_mode and count_include_pad are True

Posted by GitBox <gi...@apache.org>.
xiaolong18 commented on pull request #9835:
URL: https://github.com/apache/tvm/pull/9835#issuecomment-1011805930


   > I ran the test case locally and it passed without your change in topi. Is this expected?
   so sorry , i forget to add new  input_shape as [1,1,1,9] .


-- 
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 #9835: [Fix Bug] fix the bug of pool_impl_nd when computing avgpool_nd whith ceil_mode and count_include_pad are True

Posted by GitBox <gi...@apache.org>.
masahi commented on pull request #9835:
URL: https://github.com/apache/tvm/pull/9835#issuecomment-1010629842


   The test case passed without your change in topi. Is this expected?


-- 
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 merged pull request #9835: [Fix Bug] fix the bug of pool_impl_nd when computing avgpool_nd whith ceil_mode and count_include_pad are True

Posted by GitBox <gi...@apache.org>.
masahi merged pull request #9835:
URL: https://github.com/apache/tvm/pull/9835


   


-- 
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] xiaolong18 commented on pull request #9835: [Fix Bug] fix the bug of pool_impl_nd when computing avgpool_nd whith ceil_mode and count_include_pad are True

Posted by GitBox <gi...@apache.org>.
xiaolong18 commented on pull request #9835:
URL: https://github.com/apache/tvm/pull/9835#issuecomment-1011818112


   @masahi   i have added new input_shape, Thanks !


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