broune commented on issue #4367: [Tests] Dump duration of Pytests. URL: https://github.com/apache/incubator-tvm/pull/4367#issuecomment-556265407 What I like about having a single script that knows how to run pytest on TVM is that it prevents duplication of the logic for this, including environment variables, into the 8 files changed in this change here and then the docker and makes it easy to change in one place without recreating dockers. This logic is currently out of sync across these files and it is not clear how much of that is on purpose and how much is drift (the motivation for the settings is not documented in the files). Though thinking more about it I'm mainly happy that we get to see test times and whichever way that achieves that is an improvement in my view.
---------------------------------------------------------------- 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: [email protected] With regards, Apache Git Services
