Johnson9009 commented on pull request #8614:
URL: https://github.com/apache/tvm/pull/8614#issuecomment-890672455


   Below two CI test cases failed, the root cause are all CUDA hasn't 
implemented and register "conv2d_NCHWc_strategy", so these test cases are 
testing wrong target. Before this PR, the default "conv2d_NCHWc_strategy" will 
use "hls.py", in another words, CUDA conv2d_NCHWc_strategy use that of "hls.py" 
implicitly, so it will not report the error.
   
   ```shell
   1. tests/python/relay/test_any.py::test_any_conv2d_NCHWc
   2. 
tests/python/relay/test_pass_alter_op_layout.py::test_alter_layout_strided_slice
   ```
   All of these 2 test cases are decorated by "@tvm.testing.uses_gpu", the 1st 
one is decorated by PR #6598, the 2nd is decorated by PR #6331.
   
   @zhiics, @comaniac, @tkonolige, @tqchen, please help to see how should we 
change these test cases.
   
   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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to