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/09/15 04:38:02 UTC

[GitHub] [incubator-tvm] roastduck opened a new issue #6473: Missing import in python source?

roastduck opened a new issue #6473:
URL: https://github.com/apache/incubator-tvm/issues/6473


   Please have a look at this file:
   
   https://github.com/apache/incubator-tvm/blob/dd37ed50c4fda2e402c52a5cb44d15597f9d7cc4/python/tvm/topi/bifrost/gemm.py
   
   I think a `from tvm import te` is missing, since `te` is directly used around this file.
   
   I don't know how this file passes the tests, but Python reports an error to me.


----------------------------------------------------------------
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 issue #6473: [TOPI] Missing import in python source?

Posted by GitBox <gi...@apache.org>.
tqchen commented on issue #6473:
URL: https://github.com/apache/incubator-tvm/issues/6473#issuecomment-692759775


   good catch, please send a PR to fix the problem. It is due to how pytest running everything together and file being imported elsewere


----------------------------------------------------------------
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 issue #6473: Missing import in python source?

Posted by GitBox <gi...@apache.org>.
masahi commented on issue #6473:
URL: https://github.com/apache/incubator-tvm/issues/6473#issuecomment-692461592


   I saw a similar issue, in my case `tvm.testing` was not imported but CI doesn't complain. if I run the test directly it fails with due to missing import. I guess it relates to how pytest resolves imports.


----------------------------------------------------------------
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 edited a comment on issue #6473: Missing import in python source?

Posted by GitBox <gi...@apache.org>.
masahi edited a comment on issue #6473:
URL: https://github.com/apache/incubator-tvm/issues/6473#issuecomment-692461592


   I saw a similar issue, in my case `tvm.testing` was not imported but CI doesn't complain. if I run the test directly it fails due to missing import. I guess it relates to how pytest resolves imports.


----------------------------------------------------------------
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 closed issue #6473: [TOPI] Missing import in python source?

Posted by GitBox <gi...@apache.org>.
tqchen closed issue #6473:
URL: https://github.com/apache/incubator-tvm/issues/6473


   


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