[GitHub] lebeg commented on issue #12378: Disabled flaky test: test_mkldnn.test_activation

2018-08-29 Thread GitBox
lebeg commented on issue #12378: Disabled flaky test: test_mkldnn.test_activation URL: https://github.com/apache/incubator-mxnet/pull/12378#issuecomment-416905521 Would that be alright with you @pengzhao-intel @luobao-intel?

[GitHub] lebeg commented on issue #12378: Disabled flaky test: test_mkldnn.test_activation

2018-08-29 Thread GitBox
lebeg commented on issue #12378: Disabled flaky test: test_mkldnn.test_activation URL: https://github.com/apache/incubator-mxnet/pull/12378#issuecomment-416904080 @pengzhao-intel my intent with this PR is disabling the test, not to make it pass. As far as I have understood from https://gi

[GitHub] lebeg commented on issue #12378: Disabled flaky test: test_mkldnn.test_activation

2018-08-29 Thread GitBox
lebeg commented on issue #12378: Disabled flaky test: test_mkldnn.test_activation URL: https://github.com/apache/incubator-mxnet/pull/12378#issuecomment-416876126 We have continues failures on the CI for that: http://jenkins.mxnet-ci.amazon-ml.com/blue/organizations/jenkins/incubator-mx

[GitHub] lebeg commented on issue #12378: Disabled flaky test: test_mkldnn.test_activation

2018-08-29 Thread GitBox
lebeg commented on issue #12378: Disabled flaky test: test_mkldnn.test_activation URL: https://github.com/apache/incubator-mxnet/pull/12378#issuecomment-416875237 @pengzhao-intel Sure) But it's better to disable a flaky test first and so that PR's from others are not blocked and then add i