You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mxnet.apache.org by ns...@apache.org on 2018/08/18 00:42:22 UTC

[incubator-mxnet] branch master updated: [MXNET-703] Remove race condition in onnx build (#12225)

This is an automated email from the ASF dual-hosted git repository.

nswamy pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git


The following commit(s) were added to refs/heads/master by this push:
     new 1355bd7  [MXNET-703]  Remove race condition in onnx build (#12225)
1355bd7 is described below

commit 1355bd7821f8575a4cde9e0459e5a6ef680116a1
Author: Kellen Sunderland <ke...@gmail.com>
AuthorDate: Sat Aug 18 02:42:12 2018 +0200

    [MXNET-703]  Remove race condition in onnx build (#12225)
    
    Note: This commit decreases the number of workers during the onnx build
    but this should have minimal impact on overall build times.  Building the
    onnx native lib only takes a few seconds.
    
    This is a work around, the correct fix ha sbeen applied to onnx here:
    https://github.com/onnx/onnx/pull/934/files and will be updated in
    mxnet after sufficient testing.
---
 ci/docker/runtime_functions.sh | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/ci/docker/runtime_functions.sh b/ci/docker/runtime_functions.sh
index a0c4723..2341674 100755
--- a/ci/docker/runtime_functions.sh
+++ b/ci/docker/runtime_functions.sh
@@ -431,7 +431,8 @@ build_ubuntu_gpu_tensorrt() {
         -DCMAKE_CXX_FLAGS=-I/usr/include/python${PYVER}\
         -DBUILD_SHARED_LIBS=ON ..\
         -G Ninja
-    ninja -v
+    ninja -j 1 -v onnx/onnx.proto
+    ninja -j 1 -v
     export LIBRARY_PATH=`pwd`:`pwd`/onnx/:$LIBRARY_PATH
     export CPLUS_INCLUDE_PATH=`pwd`:$CPLUS_INCLUDE_PATH
     popd