[GitHub] larroy commented on issue #13840: [CI][URGENT] Fix permissions of ci/docker/install/ubuntu_publish.sh

2019-01-15 Thread GitBox
larroy commented on issue #13840: [CI][URGENT] Fix permissions of ci/docker/install/ubuntu_publish.sh URL: https://github.com/apache/incubator-mxnet/pull/13840#issuecomment-454482782 Check any recent build from the docker cache, or right now from master.

[GitHub] marcoabreu commented on issue #13883: Raspberrypi 3 Failed to build docker image

2019-01-15 Thread GitBox
marcoabreu commented on issue #13883: Raspberrypi 3 Failed to build docker image URL: https://github.com/apache/incubator-mxnet/issues/13883#issuecomment-454482343 Hi, could you try pulling from master? We recently had to delete 'mxnetci/dockcross-linux-armv7:09182018'

[GitHub] lebeg commented on issue #13574: Replaced rand_r with std:: random generation

2019-01-15 Thread GitBox
lebeg commented on issue #13574: Replaced rand_r with std:: random generation URL: https://github.com/apache/incubator-mxnet/pull/13574#issuecomment-454480064 @mxnet-label-bot Update [pr-awaiting-merge] This is an automated

[GitHub] lebeg commented on issue #13574: Replaced rand_r with std:: random generation

2019-01-15 Thread GitBox
lebeg commented on issue #13574: Replaced rand_r with std:: random generation URL: https://github.com/apache/incubator-mxnet/pull/13574#issuecomment-454479932 I think this PR is good to merge, can somebody do this? This is an

[GitHub] marcoabreu commented on issue #13840: [CI][URGENT] Fix permissions of ci/docker/install/ubuntu_publish.sh

2019-01-15 Thread GitBox
marcoabreu commented on issue #13840: [CI][URGENT] Fix permissions of ci/docker/install/ubuntu_publish.sh URL: https://github.com/apache/incubator-mxnet/pull/13840#issuecomment-454479596 Waiting with merge until Kellen gives his green light, considering his open question.

[GitHub] marcoabreu commented on a change in pull request #13884: [CI][URGENT] Fixes for docker cache generation

2019-01-15 Thread GitBox
marcoabreu commented on a change in pull request #13884: [CI][URGENT] Fixes for docker cache generation URL: https://github.com/apache/incubator-mxnet/pull/13884#discussion_r247989069 ## File path: ci/docker_cache.py ## @@ -34,11 +32,11 @@ import build as build_util

[GitHub] larroy commented on a change in pull request #13884: [CI][URGENT] Fixes for docker cache generation

2019-01-15 Thread GitBox
larroy commented on a change in pull request #13884: [CI][URGENT] Fixes for docker cache generation URL: https://github.com/apache/incubator-mxnet/pull/13884#discussion_r247988533 ## File path: ci/docker_cache.py ## @@ -34,11 +32,11 @@ import build as build_util from

[GitHub] larroy commented on a change in pull request #13884: [CI][URGENT] Fixes for docker cache generation

2019-01-15 Thread GitBox
larroy commented on a change in pull request #13884: [CI][URGENT] Fixes for docker cache generation URL: https://github.com/apache/incubator-mxnet/pull/13884#discussion_r247988895 ## File path: ci/docker_cache.py ## @@ -240,16 +234,36 @@ def script_name() -> str:

[GitHub] larroy commented on issue #13884: [CI][URGENT] Fixes for docker cache generation

2019-01-15 Thread GitBox
larroy commented on issue #13884: [CI][URGENT] Fixes for docker cache generation URL: https://github.com/apache/incubator-mxnet/pull/13884#issuecomment-454478476 @marcoabreu this is the isolated fix: https://github.com/apache/incubator-mxnet/pull/13840

[GitHub] larroy commented on a change in pull request #13884: [CI][URGENT] Fixes for docker cache generation

2019-01-15 Thread GitBox
larroy commented on a change in pull request #13884: [CI][URGENT] Fixes for docker cache generation URL: https://github.com/apache/incubator-mxnet/pull/13884#discussion_r247988533 ## File path: ci/docker_cache.py ## @@ -34,11 +32,11 @@ import build as build_util from

[GitHub] aaronmarkham commented on issue #13887: Temporarily disable website builds

2019-01-15 Thread GitBox
aaronmarkham commented on issue #13887: Temporarily disable website builds URL: https://github.com/apache/incubator-mxnet/pull/13887#issuecomment-454476813 Is anyone looking at the underlying problem? Why are there OS errors? I checked the results of the last 30 builds. 16 failed with this

[GitHub] marcoabreu commented on issue #13884: [CI][URGENT] Fixes for docker cache generation

2019-01-15 Thread GitBox
marcoabreu commented on issue #13884: [CI][URGENT] Fixes for docker cache generation URL: https://github.com/apache/incubator-mxnet/pull/13884#issuecomment-454475606 Could you separate the urgent fix (changing the executable bit) from the rest of the changes in this PR so we can merge the

[GitHub] aaronmarkham commented on issue #13833: Website build is flaky

2019-01-15 Thread GitBox
aaronmarkham commented on issue #13833: Website build is flaky URL: https://github.com/apache/incubator-mxnet/issues/13833#issuecomment-454475345 And this is a duplicate of https://github.com/apache/incubator-mxnet/issues/13831

[GitHub] aaronmarkham commented on issue #13833: Website build is flaky

2019-01-15 Thread GitBox
aaronmarkham commented on issue #13833: Website build is flaky URL: https://github.com/apache/incubator-mxnet/issues/13833#issuecomment-454475148 Hanging of status result on successful builds was something I reported quite a while ago... you just need to restart that job and maybe it'll

[GitHub] larroy commented on issue #13887: Temporarily disable website builds

2019-01-15 Thread GitBox
larroy commented on issue #13887: Temporarily disable website builds URL: https://github.com/apache/incubator-mxnet/pull/13887#issuecomment-454474441 https://github.com/apache/incubator-mxnet/pull/13841 This is an automated

[GitHub] larroy commented on issue #13887: Temporarily disable website builds

2019-01-15 Thread GitBox
larroy commented on issue #13887: Temporarily disable website builds URL: https://github.com/apache/incubator-mxnet/pull/13887#issuecomment-454474256 LGTM, I tried the same and status was not updated. I guess is an orthogonal issue.

[GitHub] KellenSunderland commented on issue #13887: Temporarily disable website builds

2019-01-15 Thread GitBox
KellenSunderland commented on issue #13887: Temporarily disable website builds URL: https://github.com/apache/incubator-mxnet/pull/13887#issuecomment-454473788 Autocorrect. Project should be PR*. On Tue, Jan 15, 2019, 9:14 AM kellen sunderland I don't think the other projects

[GitHub] larroy commented on issue #13549: Runtime feature detection

2019-01-15 Thread GitBox
larroy commented on issue #13549: Runtime feature detection URL: https://github.com/apache/incubator-mxnet/pull/13549#issuecomment-454473495 Any more comments / concerns or can we merge? This is an automated message from the

[GitHub] KellenSunderland commented on issue #13887: Temporarily disable website builds

2019-01-15 Thread GitBox
KellenSunderland commented on issue #13887: Temporarily disable website builds URL: https://github.com/apache/incubator-mxnet/pull/13887#issuecomment-454473641 I don't think the other projects linked can be merged due to a required test that won't complete (hence this attempt). On

[GitHub] marcoabreu commented on a change in pull request #13884: [CI][URGENT] Fixes for docker cache generation

2019-01-15 Thread GitBox
marcoabreu commented on a change in pull request #13884: [CI][URGENT] Fixes for docker cache generation URL: https://github.com/apache/incubator-mxnet/pull/13884#discussion_r247983625 ## File path: ci/docker_cache.py ## @@ -240,16 +234,36 @@ def script_name() -> str:

[GitHub] stu1130 commented on issue #13886: Fixes for wine detection tutorial

2019-01-15 Thread GitBox
stu1130 commented on issue #13886: Fixes for wine detection tutorial URL: https://github.com/apache/incubator-mxnet/pull/13886#issuecomment-454473031 @mxnet-label-bot add [pr-awaiting-review] Thanks for your great work @larroy

[GitHub] stu1130 commented on issue #13884: [CI][URGENT] Fixes for docker cache generation

2019-01-15 Thread GitBox
stu1130 commented on issue #13884: [CI][URGENT] Fixes for docker cache generation URL: https://github.com/apache/incubator-mxnet/pull/13884#issuecomment-454472661 @mxnet-label-bot add [pr-awaiting-review] Thanks for your contribution @larroy

[GitHub] marcoabreu commented on a change in pull request #13884: [CI][URGENT] Fixes for docker cache generation

2019-01-15 Thread GitBox
marcoabreu commented on a change in pull request #13884: [CI][URGENT] Fixes for docker cache generation URL: https://github.com/apache/incubator-mxnet/pull/13884#discussion_r247983064 ## File path: ci/docker_cache.py ## @@ -34,11 +32,11 @@ import build as build_util

[GitHub] stu1130 commented on issue #13882: [v1.4.x] fix ssd quantization script error (#13843)

2019-01-15 Thread GitBox
stu1130 commented on issue #13882: [v1.4.x] fix ssd quantization script error (#13843) URL: https://github.com/apache/incubator-mxnet/pull/13882#issuecomment-454472389 Thanks for your contribution @TaoLv @mxnet-label-bot add [pr-awaiting-review]

[GitHub] marcoabreu commented on issue #13887: Temporarily disable website builds

2019-01-15 Thread GitBox
marcoabreu commented on issue #13887: Temporarily disable website builds URL: https://github.com/apache/incubator-mxnet/pull/13887#issuecomment-454471775 Possible duplicate of https://github.com/apache/incubator-mxnet/pull/13841 @aaronmarkham for approval

[GitHub] KellenSunderland commented on issue #13887: Temporarily disable website builds

2019-01-15 Thread GitBox
KellenSunderland commented on issue #13887: Temporarily disable website builds URL: https://github.com/apache/incubator-mxnet/pull/13887#issuecomment-454459496 @larroy This looks like it properly disables flaky website builds. Does it look ok to you?

[GitHub] KellenSunderland commented on issue #13310: [MXNET-703] Update to TensorRT 5, ONNX IR 3. Fix inference bugs.

2019-01-15 Thread GitBox
KellenSunderland commented on issue #13310: [MXNET-703] Update to TensorRT 5, ONNX IR 3. Fix inference bugs. URL: https://github.com/apache/incubator-mxnet/pull/13310#issuecomment-454458963 @NRauschmayr Totally agree with your comments here. Would it be ok if we make those changes in a

[GitHub] marcoabreu commented on a change in pull request #13310: [MXNET-703] Update to TensorRT 5, ONNX IR 3. Fix inference bugs.

2019-01-15 Thread GitBox
marcoabreu commented on a change in pull request #13310: [MXNET-703] Update to TensorRT 5, ONNX IR 3. Fix inference bugs. URL: https://github.com/apache/incubator-mxnet/pull/13310#discussion_r247965631 ## File path: src/operator/contrib/nnvm_to_onnx.cc ## @@ -248,8

[GitHub] KellenSunderland opened a new pull request #13887: Temporarily disable website builds

2019-01-15 Thread GitBox
KellenSunderland opened a new pull request #13887: Temporarily disable website builds URL: https://github.com/apache/incubator-mxnet/pull/13887 ## Description ## Temporarily disable website builds ## Checklist ## ### Essentials ### Please feel free to remove inapplicable

[GitHub] KellenSunderland commented on issue #13855: Fix symbolic example for rand_zipfian function

2019-01-15 Thread GitBox
KellenSunderland commented on issue #13855: Fix symbolic example for rand_zipfian function URL: https://github.com/apache/incubator-mxnet/pull/13855#issuecomment-454447159 LGTM but I can't tell what's fixed. Were the values reported in the docs wrong?

[GitHub] KellenSunderland commented on issue #13840: [CI][URGENT] Fix permissions of ci/docker/install/ubuntu_publish.sh

2019-01-15 Thread GitBox
KellenSunderland commented on issue #13840: [CI][URGENT] Fix permissions of ci/docker/install/ubuntu_publish.sh URL: https://github.com/apache/incubator-mxnet/pull/13840#issuecomment-45944 Can you give some more detail about what this fixes?

[GitHub] larroy edited a comment on issue #13833: Website build is flaky

2019-01-15 Thread GitBox
larroy edited a comment on issue #13833: Website build is flaky URL: https://github.com/apache/incubator-mxnet/issues/13833#issuecomment-454440498 I have tried, but the status is not being reported: https://github.com/apache/incubator-mxnet/pull/13841 I expected it to report

[GitHub] larroy commented on issue #13833: Website build is flaky

2019-01-15 Thread GitBox
larroy commented on issue #13833: Website build is flaky URL: https://github.com/apache/incubator-mxnet/issues/13833#issuecomment-454440498 I have tried, but the status is not being reported: https://github.com/apache/incubator-mxnet/pull/13841

[GitHub] KellenSunderland commented on issue #13833: Website build is flaky

2019-01-15 Thread GitBox
KellenSunderland commented on issue #13833: Website build is flaky URL: https://github.com/apache/incubator-mxnet/issues/13833#issuecomment-454438238 Can we disable this until it's fixed? It's failing the majority of the builds https://github.com/apache/incubator-mxnet/commits/master

[GitHub] KellenSunderland commented on issue #13873: fix compile error in debug mode

2019-01-15 Thread GitBox
KellenSunderland commented on issue #13873: fix compile error in debug mode URL: https://github.com/apache/incubator-mxnet/pull/13873#issuecomment-454437840 Thanks for the catch. Would you be able to either add a test that shows when this fails, or to give some minimal steps to reproduce

[GitHub] KellenSunderland commented on issue #13878: [WIP] statically link mkldnn

2019-01-15 Thread GitBox
KellenSunderland commented on issue #13878: [WIP] statically link mkldnn URL: https://github.com/apache/incubator-mxnet/pull/13878#issuecomment-454436949 This would be great for users if we can do it. This is an automated

[GitHub] KellenSunderland closed pull request #13874: [WIP] Trt 5 fix remove non interactive

2019-01-15 Thread GitBox
KellenSunderland closed pull request #13874: [WIP] Trt 5 fix remove non interactive URL: https://github.com/apache/incubator-mxnet/pull/13874 As this is a foreign pull request (from a fork), the diff has been sent to your commit mailing list, comm...@mxnet.apache.org

[incubator-mxnet] Diff for: [GitHub] KellenSunderland closed pull request #13874: [WIP] Trt 5 fix remove non interactive

2019-01-15 Thread GitBox
diff --git a/3rdparty/onnx-tensorrt b/3rdparty/onnx-tensorrt index 3d8ee049970..f1c7aa63d88 16 --- a/3rdparty/onnx-tensorrt +++ b/3rdparty/onnx-tensorrt @@ -1 +1 @@ -Subproject commit 3d8ee049970e81ff4935cc7f36b653c0b27bcbbc +Subproject commit f1c7aa63d88d8d8ef70490f2ebb6b33f7450218b diff

[GitHub] larroy opened a new pull request #13886: Fixes for wine detection tutorial

2019-01-15 Thread GitBox
larroy opened a new pull request #13886: Fixes for wine detection tutorial URL: https://github.com/apache/incubator-mxnet/pull/13886 ## Description ## see tittle ## Checklist ## ### Essentials ### Please feel free to remove inapplicable items for your PR. - [x] The PR

[GitHub] dabraude edited a comment on issue #13875: importing mxnet causing subprocess to crash

2019-01-15 Thread GitBox
dabraude edited a comment on issue #13875: importing mxnet causing subprocess to crash URL: https://github.com/apache/incubator-mxnet/issues/13875#issuecomment-454425372 It still crashes with the --pre We have found that it only happens with the MKL version:

[GitHub] larroy commented on issue #9507: Segmentation Fault

2019-01-15 Thread GitBox
larroy commented on issue #9507: Segmentation Fault URL: https://github.com/apache/incubator-mxnet/issues/9507#issuecomment-454428962 Can you post a contained example for reproduction? I have used qemu as described in:

[GitHub] dabraude commented on issue #13875: importing mxnet causing subprocess to crash

2019-01-15 Thread GitBox
dabraude commented on issue #13875: importing mxnet causing subprocess to crash URL: https://github.com/apache/incubator-mxnet/issues/13875#issuecomment-454425372 It still crashes with the --pre We have found that it only happens with the MKL version

[GitHub] mariussoutier commented on issue #12553: [Scala] Training differences between macOS and Ubuntu?

2019-01-15 Thread GitBox
mariussoutier commented on issue #12553: [Scala] Training differences between macOS and Ubuntu? URL: https://github.com/apache/incubator-mxnet/issues/12553#issuecomment-454420119 @piyushghai I gave up trying to train in Scala, am only using it for inference now.

[GitHub] kirk86 opened a new issue #13885: imagenet example failure to run properly

2019-01-15 Thread GitBox
kirk86 opened a new issue #13885: imagenet example failure to run properly URL: https://github.com/apache/incubator-mxnet/issues/13885 ## Description I am following this example https://mxnet.incubator.apache.org/tutorials/vision/large_scale_classification.html to run imagenet training

[GitHub] pengzhao-intel edited a comment on issue #13881: Cmake : could not find mkl (missing MKLDNN_LIBRARY)

2019-01-15 Thread GitBox
pengzhao-intel edited a comment on issue #13881: Cmake : could not find mkl (missing MKLDNN_LIBRARY) URL: https://github.com/apache/incubator-mxnet/issues/13881#issuecomment-454382366 @lebeg related your cmake PR #13576

[GitHub] larroy opened a new pull request #13884: [CI] Fixes for docker cache generation

2019-01-15 Thread GitBox
larroy opened a new pull request #13884: [CI] Fixes for docker cache generation URL: https://github.com/apache/incubator-mxnet/pull/13884 ## Description ## * extract arguments from environment * control constants via environment * add commandline options * Fix executable

[incubator-mxnet-site] branch asf-site updated: Bump the publish timestamp.

2019-01-15 Thread zhasheng
This is an automated email from the ASF dual-hosted git repository. zhasheng pushed a commit to branch asf-site in repository https://gitbox.apache.org/repos/asf/incubator-mxnet-site.git The following commit(s) were added to refs/heads/asf-site by this push: new 8999170 Bump the publish

[GitHub] pengzhao-intel commented on issue #13881: Cmake : could not find mkl (missing MKLDNN_LIBRARY)

2019-01-15 Thread GitBox
pengzhao-intel commented on issue #13881: Cmake : could not find mkl (missing MKLDNN_LIBRARY) URL: https://github.com/apache/incubator-mxnet/issues/13881#issuecomment-454382366 @lebeg related your cmake PR This is an

[GitHub] larroy commented on a change in pull request #13549: Runtime feature detection

2019-01-15 Thread GitBox
larroy commented on a change in pull request #13549: Runtime feature detection URL: https://github.com/apache/incubator-mxnet/pull/13549#discussion_r247856232 ## File path: python/mxnet/gluon/utils.py ## @@ -222,7 +222,7 @@ def _replace_atomic(src, dst):

[GitHub] ZhennanQin commented on issue #13697: [MKLDNN] Enable signed int8 support for convolution.

2019-01-15 Thread GitBox
ZhennanQin commented on issue #13697: [MKLDNN] Enable signed int8 support for convolution. URL: https://github.com/apache/incubator-mxnet/pull/13697#issuecomment-454359907 @KellenSunderland @szha API is refactored to be backward compatible. please review again. Thanks.

[GitHub] larroy commented on a change in pull request #13549: Runtime feature detection

2019-01-15 Thread GitBox
larroy commented on a change in pull request #13549: Runtime feature detection URL: https://github.com/apache/incubator-mxnet/pull/13549#discussion_r247854348 ## File path: src/mxfeatures.cc ## @@ -0,0 +1,115 @@ +/* + * Licensed to the Apache Software Foundation (ASF)

[GitHub] zheng-da commented on issue #12502: [NGRAPH] MXNet - nGraph initial integration

2019-01-15 Thread GitBox
zheng-da commented on issue #12502: [NGRAPH] MXNet - nGraph initial integration URL: https://github.com/apache/incubator-mxnet/pull/12502#issuecomment-454328828 I compiled mxnet with ngraph and mkldnn: `make -j64 USE_OPENCV=1 USE_BLAS=openblas USE_MKLDNN=1 USE_NGRAPH=1`. It seems

[GitHub] Rehtt opened a new issue #13883: Raspberrypi 3 Failed to build docker image

2019-01-15 Thread GitBox
Rehtt opened a new issue #13883: Raspberrypi 3 Failed to build docker image URL: https://github.com/apache/incubator-mxnet/issues/13883 Step 1/15 : FROM mxnetci/dockcross-linux-armv7:09182018 pull access denied for mxnetci/dockcross-linux-armv7, repository does not exist or may require

[GitHub] dabraude commented on issue #13875: importing mxnet causing subprocess to crash

2019-01-15 Thread GitBox
dabraude commented on issue #13875: importing mxnet causing subprocess to crash URL: https://github.com/apache/incubator-mxnet/issues/13875#issuecomment-454326478 Ok we will try running that script and get back to you. We are running CUDA 10

[GitHub] TaoLv commented on issue #13843: fix ssd quantization script error

2019-01-15 Thread GitBox
TaoLv commented on issue #13843: fix ssd quantization script error URL: https://github.com/apache/incubator-mxnet/pull/13843#issuecomment-454313835 @lifu-wang You may be interested. We will have nightly build with this fix soon.

[incubator-mxnet] branch ib/julia-codeowner deleted (was 5150956)

2019-01-15 Thread iblis
This is an automated email from the ASF dual-hosted git repository. iblis pushed a change to branch ib/julia-codeowner in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. was 5150956 Add CODEOWNERS for Julia package The revisions that were on this branch are still

[GitHub] TaoLv edited a comment on issue #13882: [v1.4.x] fix ssd quantization script error (#13843)

2019-01-15 Thread GitBox
TaoLv edited a comment on issue #13882: [v1.4.x] fix ssd quantization script error (#13843) URL: https://github.com/apache/incubator-mxnet/pull/13882#issuecomment-454303997 #13843 fixed a crash issue of quantized SSD example on master branch. Since quantization is an important feature of

[GitHub] wshuail commented on issue #13074: Bug in bipartite_matching

2019-01-15 Thread GitBox
wshuail commented on issue #13074: Bug in bipartite_matching URL: https://github.com/apache/incubator-mxnet/issues/13074#issuecomment-454304056 #https://github.com/apache/incubator-mxnet/pull/13727 #https://github.com/dmlc/gluon-cv/issues/529 This was fixed already. Thx.

[GitHub] TaoLv commented on issue #13882: [v1.4.x] fix ssd quantization script error (#13843)

2019-01-15 Thread GitBox
TaoLv commented on issue #13882: [v1.4.x] fix ssd quantization script error (#13843) URL: https://github.com/apache/incubator-mxnet/pull/13882#issuecomment-454303997 #13843 fixed a crash issue of quantized SSD example on master branch. Since quantization is an important feature of MKL-DNN

[GitHub] wshuail closed issue #13074: Bug in bipartite_matching

2019-01-15 Thread GitBox
wshuail closed issue #13074: Bug in bipartite_matching URL: https://github.com/apache/incubator-mxnet/issues/13074 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and

[GitHub] TaoLv opened a new pull request #13882: [v1.4.x] fix ssd quantization script error (#13843)

2019-01-15 Thread GitBox
TaoLv opened a new pull request #13882: [v1.4.x] fix ssd quantization script error (#13843) URL: https://github.com/apache/incubator-mxnet/pull/13882 * fix ssd quantization script error * update readme for ssd * move quantized SSD instructions from quantization/README.md to

<    1   2   3