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 2021/01/11 20:56:03 UTC

[GitHub] [tvm] areusch commented on pull request #7250: Remove check_correctness in AutoTVM, which is busted

areusch commented on pull request #7250:
URL: https://github.com/apache/tvm/pull/7250#issuecomment-758217721


   @merrymercy @antinucleon should this maybe be done instead by overriding some part of Runner? I don't think we should keep it as a generic mechanism in the API if it's only useful for 1 backend. at minimum, we need to provide a warning or add more checks to verify that we are testing the right thing when we do `check_correctness`. I spent several hours trying to debug this because i thought i'd broken it.


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