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 2023/01/06 22:49:55 UTC
[GitHub] [tvm] Mousius opened a new pull request, #13717: [Tests] Replace pytest.main with tvm.testing.main
Mousius opened a new pull request, #13717:
URL: https://github.com/apache/tvm/pull/13717
This ensures that if you want to run a specific test script then at least it's reasonably consistent and as people copy test files they'll use the new function :smile_cat:
--
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: commits-unsubscribe@tvm.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [tvm] junrushao merged pull request #13717: [Tests] Replace pytest.main with tvm.testing.main
Posted by GitBox <gi...@apache.org>.
junrushao merged PR #13717:
URL: https://github.com/apache/tvm/pull/13717
--
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: commits-unsubscribe@tvm.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [tvm] tvm-bot commented on pull request #13717: [Tests] Replace pytest.main with tvm.testing.main
Posted by GitBox <gi...@apache.org>.
tvm-bot commented on PR #13717:
URL: https://github.com/apache/tvm/pull/13717#issuecomment-1374228814
<!---bot-comment-->
Thanks for contributing to TVM! Please refer to the contributing guidelines https://tvm.apache.org/docs/contribute/ for useful information and tips. Please request code reviews from [Reviewers](https://github.com/apache/incubator-tvm/blob/master/CONTRIBUTORS.md#reviewers) by @-ing them in a comment.
<sub>Generated by [tvm-bot](https://github.com/apache/tvm/blob/main/ci/README.md#github-actions)</sub>
--
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: commits-unsubscribe@tvm.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [tvm] mehrdadh commented on pull request #13717: [Tests] Replace pytest.main with tvm.testing.main
Posted by GitBox <gi...@apache.org>.
mehrdadh commented on PR #13717:
URL: https://github.com/apache/tvm/pull/13717#issuecomment-1376182941
@Mousius I'm not sure what's the reason behind it. @driazati could help here.
Thanks for sending the fix!
--
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: commits-unsubscribe@tvm.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [tvm] Mousius commented on pull request #13717: [Tests] Replace pytest.main with tvm.testing.main
Posted by GitBox <gi...@apache.org>.
Mousius commented on PR #13717:
URL: https://github.com/apache/tvm/pull/13717#issuecomment-1374229958
As promised @mehrdadh 😸 I'm fairly sure this is going to fail with some `undefined tvm`. Out of interest do we know why we don't encourage people running tests directly using the `pytest` CLI?
--
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: commits-unsubscribe@tvm.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [tvm] driazati commented on pull request #13717: [Tests] Replace pytest.main with tvm.testing.main
Posted by GitBox <gi...@apache.org>.
driazati commented on PR #13717:
URL: https://github.com/apache/tvm/pull/13717#issuecomment-1376194605
I’ve always used pytest directly (and that’s what is tested in CI), I don’t think it really makes a huge difference but it’s nice to have the test files be directly runnable
--
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: commits-unsubscribe@tvm.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org