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/27 15:49:25 UTC
[GitHub] [incubator-tvm] tqchen opened a new pull request #6572: [TEST] Temporary disable test_mutate_parallel
tqchen opened a new pull request #6572:
URL: https://github.com/apache/incubator-tvm/pull/6572
cc @merrymercy .
To unblock flaky related problem in master and other tests
- https://ci.tlcpack.ai/job/tvm/job/master/1656/execution/node/146/log/
----------------------------------------------------------------
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] zhiics commented on a change in pull request #6572: [TEST] Temporary disable test_mutate_parallel
Posted by GitBox <gi...@apache.org>.
zhiics commented on a change in pull request #6572:
URL: https://github.com/apache/incubator-tvm/pull/6572#discussion_r495591503
##########
File path: tests/python/unittest/test_auto_scheduler_evolutionary_search.py
##########
@@ -68,6 +69,7 @@ def predict(self, task, states):
assert found
+@pytest.mark.xfail("flaky")
Review comment:
we probably want to mark it as skip?
----------------------------------------------------------------
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] comaniac commented on pull request #6572: [TEST] Temporary disable test_mutate_parallel
Posted by GitBox <gi...@apache.org>.
comaniac commented on pull request #6572:
URL: https://github.com/apache/incubator-tvm/pull/6572#issuecomment-699658130
IIUC, `xfail` expects the test to fail, but this test is expected to pass, although apparently it may fail.
To unblock other tests, we can skip this test first and find a better way to test parallel mutation.
----------------------------------------------------------------
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 #6572: [TEST] Temporary disable test_mutate_parallel
Posted by GitBox <gi...@apache.org>.
tqchen commented on pull request #6572:
URL: https://github.com/apache/incubator-tvm/pull/6572#issuecomment-699651959
cc @zhiics @merrymercy @comaniac
----------------------------------------------------------------
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] comaniac merged pull request #6572: [TEST] Temporary disable test_mutate_parallel
Posted by GitBox <gi...@apache.org>.
comaniac merged pull request #6572:
URL: https://github.com/apache/incubator-tvm/pull/6572
----------------------------------------------------------------
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