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/11 10:39:47 UTC
[GitHub] [incubator-tvm] Johnson9009 opened a new pull request #6454: [TEST] Fix Some Failed Test Cases and Tutorials of The Issue #6453
Johnson9009 opened a new pull request #6454:
URL: https://github.com/apache/incubator-tvm/pull/6454
I found the bug firstly from the recent version of tutorial "tutorials/optimize/opt_gemm.py", and I have tried my best to find other test cases and tutorials that will fail with the same reason, for each of these python script, I first run it to ensure it have this error behavior, then I add the fix string `import tvm.testing`, finally I run it again to ensure the error behavior is gone.
Pleae notice some of these test cases still maybe fail but the reason will have nothing to do with the problem described in issue #6453.
----------------------------------------------------------------
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] leandron commented on pull request #6454: [TEST] Fix Some Failed Test Cases and Tutorials of The Issue #6453
Posted by GitBox <gi...@apache.org>.
leandron commented on pull request #6454:
URL: https://github.com/apache/incubator-tvm/pull/6454#issuecomment-691032137
@Johnson9009 thanks for the PR - You're correct, this is a side-effect of that change. I wonder why we didn't see this in CI.
cc @tqchen
----------------------------------------------------------------
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 merged pull request #6454: [TEST] Fix Some Failed Test Cases and Tutorials of The Issue #6453
Posted by GitBox <gi...@apache.org>.
tqchen merged pull request #6454:
URL: https://github.com/apache/incubator-tvm/pull/6454
----------------------------------------------------------------
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 #6454: [TEST] Fix Some Failed Test Cases and Tutorials of The Issue #6453
Posted by GitBox <gi...@apache.org>.
tqchen commented on pull request #6454:
URL: https://github.com/apache/incubator-tvm/pull/6454#issuecomment-691138542
----------------------------------------------------------------
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] leandron commented on pull request #6454: [TEST] Fix Some Failed Test Cases and Tutorials of The Issue #6453
Posted by GitBox <gi...@apache.org>.
leandron commented on pull request #6454:
URL: https://github.com/apache/incubator-tvm/pull/6454#issuecomment-691032137
@Johnson9009 thanks for the PR - You're correct, this is a side-effect of that change. I wonder why we didn't see this in CI.
cc @tqchen
----------------------------------------------------------------
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 merged pull request #6454: [TEST] Fix Some Failed Test Cases and Tutorials of The Issue #6453
Posted by GitBox <gi...@apache.org>.
tqchen merged pull request #6454:
URL: https://github.com/apache/incubator-tvm/pull/6454
----------------------------------------------------------------
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 #6454: [TEST] Fix Some Failed Test Cases and Tutorials of The Issue #6453
Posted by GitBox <gi...@apache.org>.
tqchen commented on pull request #6454:
URL: https://github.com/apache/incubator-tvm/pull/6454#issuecomment-691138542
I think the reason is that the tests are run together, and some previous tests already imports the testing
----------------------------------------------------------------
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] leandron commented on pull request #6454: [TEST] Fix Some Failed Test Cases and Tutorials of The Issue #6453
Posted by GitBox <gi...@apache.org>.
leandron commented on pull request #6454:
URL: https://github.com/apache/incubator-tvm/pull/6454#issuecomment-691032137
----------------------------------------------------------------
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 #6454: [TEST] Fix Some Failed Test Cases and Tutorials of The Issue #6453
Posted by GitBox <gi...@apache.org>.
tqchen commented on pull request #6454:
URL: https://github.com/apache/incubator-tvm/pull/6454#issuecomment-691138542
----------------------------------------------------------------
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] leandron commented on pull request #6454: [TEST] Fix Some Failed Test Cases and Tutorials of The Issue #6453
Posted by GitBox <gi...@apache.org>.
leandron commented on pull request #6454:
URL: https://github.com/apache/incubator-tvm/pull/6454#issuecomment-691032137
----------------------------------------------------------------
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 merged pull request #6454: [TEST] Fix Some Failed Test Cases and Tutorials of The Issue #6453
Posted by GitBox <gi...@apache.org>.
tqchen merged pull request #6454:
URL: https://github.com/apache/incubator-tvm/pull/6454
----------------------------------------------------------------
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 merged pull request #6454: [TEST] Fix Some Failed Test Cases and Tutorials of The Issue #6453
Posted by GitBox <gi...@apache.org>.
tqchen merged pull request #6454:
URL: https://github.com/apache/incubator-tvm/pull/6454
----------------------------------------------------------------
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 #6454: [TEST] Fix Some Failed Test Cases and Tutorials of The Issue #6453
Posted by GitBox <gi...@apache.org>.
tqchen commented on pull request #6454:
URL: https://github.com/apache/incubator-tvm/pull/6454#issuecomment-691206257
Thanks @Johnson9009 @leandron !
----------------------------------------------------------------
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