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 2022/08/22 08:51:38 UTC
[GitHub] [tvm] leandron opened a new issue, #12529: [Bug] Missing integration tests in ci_gpu image
leandron opened a new issue, #12529:
URL: https://github.com/apache/tvm/issues/12529
When looking at recent CI test run (which I'm enabling torch/ONNX tests on AArch64), I noticed an issue with a missing `transformers` package while executing `test_meta_schedule_integration_extract_from_bert_base `, causing the integration tests to fail on AArch64.
Investigating the issue a bit deeper, it seems that integration tests are **not running on** `ci_GPU` image, which is the one that is supposed to contain `torch` and therefore would reproduce the problem.
I'm raising this issue just because I won't have the time in the next weeks to enable that package and test the GPU image and test it to solve the problem. So I create this PR https://github.com/apache/tvm/pull/12528 to make the test to be properly skipped if `transformers` is not installed, but leaving this issue so that we can investigate two potential problems:
1. Integration tests not running in the GPU environment
2. Missing `transformers` installation, if that is what we want (pending to be confirmed, I didn't fully investigate)
cc @driazati @areusch @Mousius
--
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.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] [Bug][CI] integration tests not currently running in ci_gpu image [tvm]
Posted by "tqchen (via GitHub)" <gi...@apache.org>.
tqchen closed issue #12529: [Bug][CI] integration tests not currently running in ci_gpu image
URL: https://github.com/apache/tvm/issues/12529
--
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] gigiblender commented on issue #12529: [Bug][CI] integration tests not currently running in ci_gpu image
Posted by GitBox <gi...@apache.org>.
gigiblender commented on issue #12529:
URL: https://github.com/apache/tvm/issues/12529#issuecomment-1228834455
> @gigiblender is it possible to leverage #12436 to add another GH [check](https://docs.github.com/en/rest/checks) to assert we aren't skipping a couple of critical integration tests in a PR?
Sure thing. We could change that script to pass a list of critical tests to ensure that some tests are not skipped and call it in a GH check.
--
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] leandron commented on issue #12529: [Bug][CI] integration tests not currently running in ci_gpu image
Posted by GitBox <gi...@apache.org>.
leandron commented on issue #12529:
URL: https://github.com/apache/tvm/issues/12529#issuecomment-1222103418
ONNX testing that runs as part of Integration tests is also not running anywhere.
https://github.com/apache/tvm/pull/12532
--
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] areusch commented on issue #12529: [Bug][CI] integration tests not currently running in ci_gpu image
Posted by GitBox <gi...@apache.org>.
areusch commented on issue #12529:
URL: https://github.com/apache/tvm/issues/12529#issuecomment-1227802187
@gigiblender is it possible to leverage https://github.com/apache/tvm/pull/12436 to add another GH [check](https://docs.github.com/en/rest/checks) to assert we aren't skipping a couple of critical integration tests in a PR?
--
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