[GitHub] eric-haibin-lin commented on issue #10112: [MXNET-16] [DO NOT MERGE] Move submodules to 3rdparty

2018-03-15 Thread GitBox
eric-haibin-lin commented on issue #10112: [MXNET-16] [DO NOT MERGE] Move submodules to 3rdparty URL: https://github.com/apache/incubator-mxnet/pull/10112#issuecomment-373427664 Oh right, I didn't notice my branch is already behind master. Will do rebase

[GitHub] eric-haibin-lin commented on issue #10112: [MXNET-16] [DO NOT MERGE] Move submodules to 3rdparty

2018-03-15 Thread GitBox
eric-haibin-lin commented on issue #10112: [MXNET-16] [DO NOT MERGE] Move submodules to 3rdparty URL: https://github.com/apache/incubator-mxnet/pull/10112#issuecomment-373406907 You mean CI may not clean up the workspace if it is corrupted in some case? If so maybe we want to prevent it

[GitHub] eric-haibin-lin commented on issue #10112: [MXNET-16] [DO NOT MERGE] Move submodules to 3rdparty

2018-03-15 Thread GitBox
eric-haibin-lin commented on issue #10112: [MXNET-16] [DO NOT MERGE] Move submodules to 3rdparty URL: https://github.com/apache/incubator-mxnet/pull/10112#issuecomment-373406256 No, I didn’t merge anything. I just added two commits on top of master

[GitHub] eric-haibin-lin commented on issue #10112: [MXNET-16] [DO NOT MERGE] Move submodules to 3rdparty

2018-03-14 Thread GitBox
eric-haibin-lin commented on issue #10112: [MXNET-16] [DO NOT MERGE] Move submodules to 3rdparty URL: https://github.com/apache/incubator-mxnet/pull/10112#issuecomment-373212201 @marcoabreu do you know why this happens on CI? ``` GitHub has been notified of this commit’s build result