[GitHub] yzhliu commented on issue #11123: [MXNET-319][WIP] Javadoc fix

2018-06-05 Thread GitBox
yzhliu commented on issue #11123: [MXNET-319][WIP] Javadoc fix URL: https://github.com/apache/incubator-mxnet/pull/11123#issuecomment-394601946 Is it duplicated with #11143 ? This is an automated message from the Apache Git

[GitHub] yzhliu commented on a change in pull request #11126: [MXNET-386] ongoing maintenance on NDArray

2018-06-05 Thread GitBox
yzhliu commented on a change in pull request #11126: [MXNET-386] ongoing maintenance on NDArray URL: https://github.com/apache/incubator-mxnet/pull/11126#discussion_r192959577 ## File path: scala-package/core/src/main/scala/org/apache/mxnet/ExperimentalTag.scala ## @@

[GitHub] yzhliu commented on a change in pull request #11126: [MXNET-386] ongoing maintenance on NDArray

2018-06-05 Thread GitBox
yzhliu commented on a change in pull request #11126: [MXNET-386] ongoing maintenance on NDArray URL: https://github.com/apache/incubator-mxnet/pull/11126#discussion_r192964239 ## File path: scala-package/core/src/main/scala/org/apache/mxnet/NDArray.scala ## @@ -105,6

[GitHub] yzhliu commented on issue #11123: [MXNET-319][WIP] Javadoc fix

2018-06-05 Thread GitBox
yzhliu commented on issue #11123: [MXNET-319][WIP] Javadoc fix URL: https://github.com/apache/incubator-mxnet/pull/11123#issuecomment-394601946 Is it duplicated with #11143 ? This is an automated message from the Apache Git

[GitHub] lanking520 commented on a change in pull request #11126: [MXNET-386] ongoing maintenance on NDArray

2018-06-05 Thread GitBox
lanking520 commented on a change in pull request #11126: [MXNET-386] ongoing maintenance on NDArray URL: https://github.com/apache/incubator-mxnet/pull/11126#discussion_r192968127 ## File path: scala-package/core/src/main/scala/org/apache/mxnet/NDArray.scala ## @@ -537,6

[GitHub] lanking520 commented on a change in pull request #11126: [MXNET-386] ongoing maintenance on NDArray

2018-06-05 Thread GitBox
lanking520 commented on a change in pull request #11126: [MXNET-386] ongoing maintenance on NDArray URL: https://github.com/apache/incubator-mxnet/pull/11126#discussion_r192968334 ## File path: scala-package/core/src/main/scala/org/apache/mxnet/NDArray.scala ## @@ -64,22

[GitHub] lanking520 commented on a change in pull request #11126: [MXNET-386] ongoing maintenance on NDArray

2018-06-05 Thread GitBox
lanking520 commented on a change in pull request #11126: [MXNET-386] ongoing maintenance on NDArray URL: https://github.com/apache/incubator-mxnet/pull/11126#discussion_r192968711 ## File path: scala-package/macros/src/main/scala/org/apache/mxnet/NDArrayMacro.scala ## @@

[GitHub] lanking520 commented on a change in pull request #11126: [MXNET-386] ongoing maintenance on NDArray

2018-06-05 Thread GitBox
lanking520 commented on a change in pull request #11126: [MXNET-386] ongoing maintenance on NDArray URL: https://github.com/apache/incubator-mxnet/pull/11126#discussion_r192968485 ## File path: scala-package/core/src/main/scala/org/apache/mxnet/ExperimentalTag.scala ##

[GitHub] yzhliu commented on a change in pull request #11143: [MXNET-319] Javadoc fix (Approach 2)

2018-06-05 Thread GitBox
yzhliu commented on a change in pull request #11143: [MXNET-319] Javadoc fix (Approach 2) URL: https://github.com/apache/incubator-mxnet/pull/11143#discussion_r192967638 ## File path: scala-package/macros/src/main/scala/org/apache/mxnet/APIDocGenerator.scala ## @@ -60,8

[GitHub] lanking520 commented on a change in pull request #11126: [MXNET-386] ongoing maintenance on NDArray

2018-06-05 Thread GitBox
lanking520 commented on a change in pull request #11126: [MXNET-386] ongoing maintenance on NDArray URL: https://github.com/apache/incubator-mxnet/pull/11126#discussion_r192967726 ## File path: scala-package/core/src/main/scala/org/apache/mxnet/NDArray.scala ## @@ -105,6

[GitHub] lanking520 commented on a change in pull request #11126: [MXNET-386] ongoing maintenance on NDArray

2018-06-05 Thread GitBox
lanking520 commented on a change in pull request #11126: [MXNET-386] ongoing maintenance on NDArray URL: https://github.com/apache/incubator-mxnet/pull/11126#discussion_r192967931 ## File path: scala-package/macros/src/main/scala/org/apache/mxnet/APIDocGenerator.scala ##

[GitHub] nswamy opened a new pull request #11147: changes made to publish to maven

2018-06-05 Thread GitBox
nswamy opened a new pull request #11147: changes made to publish to maven URL: https://github.com/apache/incubator-mxnet/pull/11147 ## Description ## Changes that were made to publish the packages to Maven, some of the changes were already committed by the Maven Release plugin on my

[GitHub] Pablitov commented on issue #6015: mxnet binary installation error - Mac (10.12.4) - R (3.4.0).

2018-06-05 Thread GitBox
Pablitov commented on issue #6015: mxnet binary installation error - Mac (10.12.4) - R (3.4.0). URL: https://github.com/apache/incubator-mxnet/issues/6015#issuecomment-394635505 I have the same problem. Should I downgrade to R 3.something?

[GitHub] Pablitov commented on issue #6015: mxnet binary installation error - Mac (10.12.4) - R (3.4.0).

2018-06-05 Thread GitBox
Pablitov commented on issue #6015: mxnet binary installation error - Mac (10.12.4) - R (3.4.0). URL: https://github.com/apache/incubator-mxnet/issues/6015#issuecomment-394635505 I have the same problem. I have R3.5, Should I downgrade to R 3.something?

[GitHub] mjmg commented on issue #8974: MXNET compatibility with MKL libraries bundled in Microsoft R Open

2018-06-05 Thread GitBox
mjmg commented on issue #8974: MXNET compatibility with MKL libraries bundled in Microsoft R Open URL: https://github.com/apache/incubator-mxnet/issues/8974#issuecomment-394599934 @xinyu-intel Thanks for clarifying my issue/question. I have now used Intel MKL yum repo to fine tune

[GitHub] nswamy commented on a change in pull request #11126: [MXNET-386] ongoing maintenance on NDArray

2018-06-05 Thread GitBox
nswamy commented on a change in pull request #11126: [MXNET-386] ongoing maintenance on NDArray URL: https://github.com/apache/incubator-mxnet/pull/11126#discussion_r192958252 ## File path: scala-package/core/src/main/scala/org/apache/mxnet/NDArray.scala ## @@ -64,22

[GitHub] nswamy commented on a change in pull request #11126: [MXNET-386] ongoing maintenance on NDArray

2018-06-05 Thread GitBox
nswamy commented on a change in pull request #11126: [MXNET-386] ongoing maintenance on NDArray URL: https://github.com/apache/incubator-mxnet/pull/11126#discussion_r192964251 ## File path: scala-package/macros/src/main/scala/org/apache/mxnet/NDArrayMacro.scala ## @@

[GitHub] nswamy commented on a change in pull request #11126: [MXNET-386] ongoing maintenance on NDArray

2018-06-05 Thread GitBox
nswamy commented on a change in pull request #11126: [MXNET-386] ongoing maintenance on NDArray URL: https://github.com/apache/incubator-mxnet/pull/11126#discussion_r192954865 ## File path: scala-package/core/src/main/scala/org/apache/mxnet/ExperimentalTag.scala ## @@

[GitHub] nswamy commented on a change in pull request #11126: [MXNET-386] ongoing maintenance on NDArray

2018-06-05 Thread GitBox
nswamy commented on a change in pull request #11126: [MXNET-386] ongoing maintenance on NDArray URL: https://github.com/apache/incubator-mxnet/pull/11126#discussion_r192962876 ## File path: scala-package/macros/src/main/scala/org/apache/mxnet/APIDocGenerator.scala ## @@

[GitHub] nswamy commented on a change in pull request #11126: [MXNET-386] ongoing maintenance on NDArray

2018-06-05 Thread GitBox
nswamy commented on a change in pull request #11126: [MXNET-386] ongoing maintenance on NDArray URL: https://github.com/apache/incubator-mxnet/pull/11126#discussion_r192962798 ## File path: scala-package/core/src/main/scala/org/apache/mxnet/NDArray.scala ## @@ -537,6

[GitHub] marcoabreu commented on issue #10921: [MXNET-500]Test cases improvement for MKLDNN on Gluon

2018-06-05 Thread GitBox
marcoabreu commented on issue #10921: [MXNET-500]Test cases improvement for MKLDNN on Gluon URL: https://github.com/apache/incubator-mxnet/pull/10921#issuecomment-394631319 Hmm, I'm a bit on the fence here. On one hand you're only introducing new tests, so it's not that bad. On the other

[GitHub] HuichuanLiu opened a new issue #11149: Unreasonable performance of resnext models provided in model_zoo, evaluated by score.py

2018-06-05 Thread GitBox
HuichuanLiu opened a new issue #11149: Unreasonable performance of resnext models provided in model_zoo, evaluated by score.py URL: https://github.com/apache/incubator-mxnet/issues/11149 Note: Providing complete information in the most concise form is the best way to get help. This issue

[GitHub] marcoabreu commented on a change in pull request #10827: [MXNET-405][WIP] Add 2 new pipelines to the Official CI and run nightly tests.

2018-06-05 Thread GitBox
marcoabreu commented on a change in pull request #10827: [MXNET-405][WIP] Add 2 new pipelines to the Official CI and run nightly tests. URL: https://github.com/apache/incubator-mxnet/pull/10827#discussion_r193044127 ## File path: tests/nightly/JenkinsfileForBinaries ##

[GitHub] marcoabreu commented on a change in pull request #10827: [MXNET-405][WIP] Add 2 new pipelines to the Official CI and run nightly tests.

2018-06-05 Thread GitBox
marcoabreu commented on a change in pull request #10827: [MXNET-405][WIP] Add 2 new pipelines to the Official CI and run nightly tests. URL: https://github.com/apache/incubator-mxnet/pull/10827#discussion_r193043881 ## File path: tests/nightly/Jenkinsfile ## @@ -0,0

[GitHub] marcoabreu commented on a change in pull request #11096: [MXNET-472] ccache for docker builds

2018-06-05 Thread GitBox
marcoabreu commented on a change in pull request #11096: [MXNET-472] ccache for docker builds URL: https://github.com/apache/incubator-mxnet/pull/11096#discussion_r193008296 ## File path: ci/docker/Dockerfile.build.jetson ## @@ -22,13 +22,21 @@ FROM

[GitHub] marcoabreu commented on a change in pull request #10827: [MXNET-405][WIP] Add 2 new pipelines to the Official CI and run nightly tests.

2018-06-05 Thread GitBox
marcoabreu commented on a change in pull request #10827: [MXNET-405][WIP] Add 2 new pipelines to the Official CI and run nightly tests. URL: https://github.com/apache/incubator-mxnet/pull/10827#discussion_r193043648 ## File path: tests/jenkins/run_test_installation_docs.sh

[GitHub] lebeg commented on a change in pull request #11096: [MXNET-472] ccache for docker builds

2018-06-05 Thread GitBox
lebeg commented on a change in pull request #11096: [MXNET-472] ccache for docker builds URL: https://github.com/apache/incubator-mxnet/pull/11096#discussion_r193002257 ## File path: ci/docker/install/centos7_install_ccache.sh ## @@ -0,0 +1,53 @@ +#!/bin/bash + +#

[GitHub] lebeg commented on a change in pull request #11096: [MXNET-472] ccache for docker builds

2018-06-05 Thread GitBox
lebeg commented on a change in pull request #11096: [MXNET-472] ccache for docker builds URL: https://github.com/apache/incubator-mxnet/pull/11096#discussion_r193002404 ## File path: ci/docker/install/centos7_install_ccache.sh ## @@ -0,0 +1,53 @@ +#!/bin/bash + +#

[GitHub] lebeg commented on a change in pull request #11096: [MXNET-472] ccache for docker builds

2018-06-05 Thread GitBox
lebeg commented on a change in pull request #11096: [MXNET-472] ccache for docker builds URL: https://github.com/apache/incubator-mxnet/pull/11096#discussion_r193002043 ## File path: ci/docker/Dockerfile.build.jetson ## @@ -39,6 +47,9 @@ RUN git clone --recursive -b

[GitHub] marcoabreu commented on a change in pull request #11096: [MXNET-472] ccache for docker builds

2018-06-05 Thread GitBox
marcoabreu commented on a change in pull request #11096: [MXNET-472] ccache for docker builds URL: https://github.com/apache/incubator-mxnet/pull/11096#discussion_r193008698 ## File path: ci/docker/install/centos7_install_ccache.sh ## @@ -0,0 +1,53 @@ +#!/bin/bash + +#

[GitHub] marcoabreu commented on a change in pull request #11096: [MXNET-472] ccache for docker builds

2018-06-05 Thread GitBox
marcoabreu commented on a change in pull request #11096: [MXNET-472] ccache for docker builds URL: https://github.com/apache/incubator-mxnet/pull/11096#discussion_r193008844 ## File path: ci/docker/install/centos7_install_ccache.sh ## @@ -0,0 +1,53 @@ +#!/bin/bash + +#

[GitHub] marcoabreu commented on a change in pull request #10827: [MXNET-405][WIP] Add 2 new pipelines to the Official CI and run nightly tests.

2018-06-05 Thread GitBox
marcoabreu commented on a change in pull request #10827: [MXNET-405][WIP] Add 2 new pipelines to the Official CI and run nightly tests. URL: https://github.com/apache/incubator-mxnet/pull/10827#discussion_r193042844 ## File path: ci/docker/runtime_functions.sh ## @@

[GitHub] marcoabreu commented on a change in pull request #10827: [MXNET-405][WIP] Add 2 new pipelines to the Official CI and run nightly tests.

2018-06-05 Thread GitBox
marcoabreu commented on a change in pull request #10827: [MXNET-405][WIP] Add 2 new pipelines to the Official CI and run nightly tests. URL: https://github.com/apache/incubator-mxnet/pull/10827#discussion_r193042600 ## File path: ci/docker/install/ubuntu_nightly_tests.sh

[GitHub] HuichuanLiu commented on issue #11149: Unreasonable performance of resnext models provided in model_zoo, evaluated by score.py

2018-06-05 Thread GitBox
HuichuanLiu commented on issue #11149: Unreasonable performance of resnext models provided in model_zoo, evaluated by score.py URL: https://github.com/apache/incubator-mxnet/issues/11149#issuecomment-394693892 I take a look at the symbol.json file from

[GitHub] lebeg commented on a change in pull request #11096: [MXNET-472] ccache for docker builds

2018-06-05 Thread GitBox
lebeg commented on a change in pull request #11096: [MXNET-472] ccache for docker builds URL: https://github.com/apache/incubator-mxnet/pull/11096#discussion_r193003358 ## File path: ci/docker/runtime_functions.sh ## @@ -31,14 +31,49 @@ clean_repo() { git submodule

[GitHub] lebeg commented on a change in pull request #11096: [MXNET-472] ccache for docker builds

2018-06-05 Thread GitBox
lebeg commented on a change in pull request #11096: [MXNET-472] ccache for docker builds URL: https://github.com/apache/incubator-mxnet/pull/11096#discussion_r193003886 ## File path: ci/docker/runtime_functions.sh ## @@ -178,6 +225,8 @@ build_centos7_cpu() {

[GitHub] marcoabreu commented on a change in pull request #11096: [MXNET-472] ccache for docker builds

2018-06-05 Thread GitBox
marcoabreu commented on a change in pull request #11096: [MXNET-472] ccache for docker builds URL: https://github.com/apache/incubator-mxnet/pull/11096#discussion_r193009305 ## File path: ci/docker/runtime_functions.sh ## @@ -31,14 +31,49 @@ clean_repo() { git

[GitHub] marcoabreu commented on issue #10921: [MXNET-500]Test cases improvement for MKLDNN on Gluon

2018-06-05 Thread GitBox
marcoabreu commented on issue #10921: [MXNET-500]Test cases improvement for MKLDNN on Gluon URL: https://github.com/apache/incubator-mxnet/pull/10921#issuecomment-394656584 I'm on my phone right now. How many tests would be active after the merge? Tao Lv schrieb am Di., 5. Juni

[GitHub] marcoabreu commented on a change in pull request #10827: [MXNET-405][WIP] Add 2 new pipelines to the Official CI and run nightly tests.

2018-06-05 Thread GitBox
marcoabreu commented on a change in pull request #10827: [MXNET-405][WIP] Add 2 new pipelines to the Official CI and run nightly tests. URL: https://github.com/apache/incubator-mxnet/pull/10827#discussion_r193043157 ## File path: ci/docker/Dockerfile.build.ubuntu_emscripten

[GitHub] BiranLi commented on issue #11076: [MXNET-491] Use depthwise convolution by cuDNNv7 if available, updated version

2018-06-05 Thread GitBox
BiranLi commented on issue #11076: [MXNET-491] Use depthwise convolution by cuDNNv7 if available, updated version URL: https://github.com/apache/incubator-mxnet/pull/11076#issuecomment-394693005 @austingg Yes, I have tested the same case with batchsize 128. > model: mobilenet-v2(only

[GitHub] lebeg commented on a change in pull request #11096: [MXNET-472] ccache for docker builds

2018-06-05 Thread GitBox
lebeg commented on a change in pull request #11096: [MXNET-472] ccache for docker builds URL: https://github.com/apache/incubator-mxnet/pull/11096#discussion_r193002705 ## File path: ci/docker/runtime_functions.sh ## @@ -31,14 +31,49 @@ clean_repo() { git submodule

[GitHub] lebeg commented on a change in pull request #11096: [MXNET-472] ccache for docker builds

2018-06-05 Thread GitBox
lebeg commented on a change in pull request #11096: [MXNET-472] ccache for docker builds URL: https://github.com/apache/incubator-mxnet/pull/11096#discussion_r193015749 ## File path: ci/docker/install/centos7_install_ccache.sh ## @@ -0,0 +1,53 @@ +#!/bin/bash + +#

[GitHub] lebeg commented on a change in pull request #11096: [MXNET-472] ccache for docker builds

2018-06-05 Thread GitBox
lebeg commented on a change in pull request #11096: [MXNET-472] ccache for docker builds URL: https://github.com/apache/incubator-mxnet/pull/11096#discussion_r193015875 ## File path: ci/docker/install/centos7_install_ccache.sh ## @@ -0,0 +1,53 @@ +#!/bin/bash + +#

[GitHub] lebeg commented on a change in pull request #11096: [MXNET-472] ccache for docker builds

2018-06-05 Thread GitBox
lebeg commented on a change in pull request #11096: [MXNET-472] ccache for docker builds URL: https://github.com/apache/incubator-mxnet/pull/11096#discussion_r193015521 ## File path: ci/docker/install/centos7_install_ccache.sh ## @@ -0,0 +1,53 @@ +#!/bin/bash + +#

[GitHub] WalterMa commented on issue #1356: override global initialization method in layer configuration

2018-06-05 Thread GitBox
WalterMa commented on issue #1356: override global initialization method in layer configuration URL: https://github.com/apache/incubator-mxnet/issues/1356#issuecomment-394673623 @RedShift51 @ThomasDelteil I also meet **AssertionError: initializer must be of string type** when I try

[GitHub] marcoabreu commented on a change in pull request #10827: [MXNET-405][WIP] Add 2 new pipelines to the Official CI and run nightly tests.

2018-06-05 Thread GitBox
marcoabreu commented on a change in pull request #10827: [MXNET-405][WIP] Add 2 new pipelines to the Official CI and run nightly tests. URL: https://github.com/apache/incubator-mxnet/pull/10827#discussion_r193044553 ## File path:

[GitHub] HuichuanLiu commented on issue #11149: Unreasonable performance of resnext models provided in model_zoo, evaluated by score.py

2018-06-05 Thread GitBox
HuichuanLiu commented on issue #11149: Unreasonable performance of resnext models provided in model_zoo, evaluated by score.py URL: https://github.com/apache/incubator-mxnet/issues/11149#issuecomment-394693892 I take a look at the symbol.json file from

[GitHub] marcoabreu commented on a change in pull request #10827: [MXNET-405][WIP] Add 2 new pipelines to the Official CI and run nightly tests.

2018-06-05 Thread GitBox
marcoabreu commented on a change in pull request #10827: [MXNET-405][WIP] Add 2 new pipelines to the Official CI and run nightly tests. URL: https://github.com/apache/incubator-mxnet/pull/10827#discussion_r193041704 ## File path: ci/docker/Dockerfile.build.ubuntu_emscripten

[GitHub] lebeg commented on a change in pull request #11096: [MXNET-472] ccache for docker builds

2018-06-05 Thread GitBox
lebeg commented on a change in pull request #11096: [MXNET-472] ccache for docker builds URL: https://github.com/apache/incubator-mxnet/pull/11096#discussion_r193000917 ## File path: ci/build.py ## @@ -127,12 +140,16 @@ def container_run(platform: str,

[GitHub] lebeg commented on a change in pull request #11096: [MXNET-472] ccache for docker builds

2018-06-05 Thread GitBox
lebeg commented on a change in pull request #11096: [MXNET-472] ccache for docker builds URL: https://github.com/apache/incubator-mxnet/pull/11096#discussion_r193000782 ## File path: ci/build.py ## @@ -116,9 +119,19 @@ def buildir() -> str: return

[GitHub] lebeg commented on a change in pull request #11096: [MXNET-472] ccache for docker builds

2018-06-05 Thread GitBox
lebeg commented on a change in pull request #11096: [MXNET-472] ccache for docker builds URL: https://github.com/apache/incubator-mxnet/pull/11096#discussion_r193004422 ## File path: ci/docker/runtime_functions.sh ## @@ -190,6 +239,8 @@ build_centos7_mkldnn() {

[GitHub] BiranLi commented on issue #11076: [MXNET-491] Use depthwise convolution by cuDNNv7 if available, updated version

2018-06-05 Thread GitBox
BiranLi commented on issue #11076: [MXNET-491] Use depthwise convolution by cuDNNv7 if available, updated version URL: https://github.com/apache/incubator-mxnet/pull/11076#issuecomment-394646557 I have tested mobilenetv2 in V100. > hardware: tesla-v100 32G > system: centos-7.2

[GitHub] lebeg opened a new pull request #11148: [WIP] [MXNET-115] USE_LAPACK is forced on all platforms

2018-06-05 Thread GitBox
lebeg opened a new pull request #11148: [WIP] [MXNET-115] USE_LAPACK is forced on all platforms URL: https://github.com/apache/incubator-mxnet/pull/11148 ## Description ## **[WIP]** This is a fix and general improvement of BLAS libraries handling with cmake. ## Checklist ##

[GitHub] lebeg commented on a change in pull request #11096: [MXNET-472] ccache for docker builds

2018-06-05 Thread GitBox
lebeg commented on a change in pull request #11096: [MXNET-472] ccache for docker builds URL: https://github.com/apache/incubator-mxnet/pull/11096#discussion_r193003169 ## File path: ci/docker/runtime_functions.sh ## @@ -31,14 +31,49 @@ clean_repo() { git submodule

[GitHub] marcoabreu commented on a change in pull request #11096: [MXNET-472] ccache for docker builds

2018-06-05 Thread GitBox
marcoabreu commented on a change in pull request #11096: [MXNET-472] ccache for docker builds URL: https://github.com/apache/incubator-mxnet/pull/11096#discussion_r193007561 ## File path: ci/docker/install/centos7_install_ccache.sh ## @@ -0,0 +1,53 @@ +#!/bin/bash + +#

[GitHub] marcoabreu commented on a change in pull request #11096: [MXNET-472] ccache for docker builds

2018-06-05 Thread GitBox
marcoabreu commented on a change in pull request #11096: [MXNET-472] ccache for docker builds URL: https://github.com/apache/incubator-mxnet/pull/11096#discussion_r193007288 ## File path: ci/docker/install/centos7_install_ccache.sh ## @@ -0,0 +1,53 @@ +#!/bin/bash + +#

[GitHub] TaoLv commented on issue #10921: [MXNET-500]Test cases improvement for MKLDNN on Gluon

2018-06-05 Thread GitBox
TaoLv commented on issue #10921: [MXNET-500]Test cases improvement for MKLDNN on Gluon URL: https://github.com/apache/incubator-mxnet/pull/10921#issuecomment-394655979 Thanks, @marcoabreu . I totally understand your concern on that. There is no other PR pending on this. I just want those

[GitHub] lebeg commented on a change in pull request #11096: [MXNET-472] ccache for docker builds

2018-06-05 Thread GitBox
lebeg commented on a change in pull request #11096: [MXNET-472] ccache for docker builds URL: https://github.com/apache/incubator-mxnet/pull/11096#discussion_r193000447 ## File path: ci/docker/install/centos7_install_ccache.sh ## @@ -0,0 +1,53 @@ +#!/bin/bash + +#

[GitHub] lebeg commented on a change in pull request #11096: [MXNET-472] ccache for docker builds

2018-06-05 Thread GitBox
lebeg commented on a change in pull request #11096: [MXNET-472] ccache for docker builds URL: https://github.com/apache/incubator-mxnet/pull/11096#discussion_r193001428 ## File path: ci/docker/Dockerfile.build.jetson ## @@ -22,13 +22,21 @@ FROM

[GitHub] austingg commented on issue #11076: [MXNET-491] Use depthwise convolution by cuDNNv7 if available, updated version

2018-06-05 Thread GitBox
austingg commented on issue #11076: [MXNET-491] Use depthwise convolution by cuDNNv7 if available, updated version URL: https://github.com/apache/incubator-mxnet/pull/11076#issuecomment-394651628 @BiranLi do you do some more benchmark, like more batch size and take backward into

[GitHub] marcoabreu commented on a change in pull request #10827: [MXNET-405][WIP] Add 2 new pipelines to the Official CI and run nightly tests.

2018-06-05 Thread GitBox
marcoabreu commented on a change in pull request #10827: [MXNET-405][WIP] Add 2 new pipelines to the Official CI and run nightly tests. URL: https://github.com/apache/incubator-mxnet/pull/10827#discussion_r193042398 ## File path: ci/docker/install/ubuntu_emscripten.sh ##

[GitHub] marcoabreu commented on issue #10827: [MXNET-405][WIP] Add 2 new pipelines to the Official CI and run nightly tests.

2018-06-05 Thread GitBox
marcoabreu commented on issue #10827: [MXNET-405][WIP] Add 2 new pipelines to the Official CI and run nightly tests. URL: https://github.com/apache/incubator-mxnet/pull/10827#issuecomment-394682620 I think we should separate the Dockerfiles from nightly from the regular ones. Otherwise,

[GitHub] HuichuanLiu commented on issue #11149: Unreasonable performance of resnext models provided in model_zoo, evaluated by score.py

2018-06-05 Thread GitBox
HuichuanLiu commented on issue #11149: Unreasonable performance of resnext models provided in model_zoo, evaluated by score.py URL: https://github.com/apache/incubator-mxnet/issues/11149#issuecomment-394693892 I take a look at the symbol.json file from

[GitHub] lebeg commented on a change in pull request #11107: [WIP][PoC] mxnet-lite

2018-06-05 Thread GitBox
lebeg commented on a change in pull request #11107: [WIP][PoC] mxnet-lite URL: https://github.com/apache/incubator-mxnet/pull/11107#discussion_r193062870 ## File path: cpp-lite/include/mxnet-lite/ndarray.h ## @@ -0,0 +1,284 @@ +/* Review comment: Header only is

[GitHub] lebeg commented on a change in pull request #11107: [WIP][PoC] mxnet-lite

2018-06-05 Thread GitBox
lebeg commented on a change in pull request #11107: [WIP][PoC] mxnet-lite URL: https://github.com/apache/incubator-mxnet/pull/11107#discussion_r193069098 ## File path: cpp-lite/include/mxnet-lite/ndarray.h ## @@ -0,0 +1,284 @@ +/* + * Licensed to the Apache Software

[GitHub] lebeg commented on a change in pull request #11107: [WIP][PoC] mxnet-lite

2018-06-05 Thread GitBox
lebeg commented on a change in pull request #11107: [WIP][PoC] mxnet-lite URL: https://github.com/apache/incubator-mxnet/pull/11107#discussion_r193070348 ## File path: cpp-lite/include/mxnet-lite/symbol_block.h ## @@ -0,0 +1,151 @@ +/* + * Licensed to the Apache Software

[GitHub] lebeg commented on a change in pull request #11107: [WIP][PoC] mxnet-lite

2018-06-05 Thread GitBox
lebeg commented on a change in pull request #11107: [WIP][PoC] mxnet-lite URL: https://github.com/apache/incubator-mxnet/pull/11107#discussion_r193066956 ## File path: cpp-lite/include/mxnet-lite/ndarray.h ## @@ -0,0 +1,284 @@ +/* + * Licensed to the Apache Software

[GitHub] marcoabreu commented on issue #10921: [MXNET-500]Test cases improvement for MKLDNN on Gluon

2018-06-05 Thread GitBox
marcoabreu commented on issue #10921: [MXNET-500]Test cases improvement for MKLDNN on Gluon URL: https://github.com/apache/incubator-mxnet/pull/10921#issuecomment-394772551 Alright, lets create GitHub issues for the disabled tests and merge. Please put the link into the message that's

[GitHub] larroy commented on issue #10945: Flaky test: test_operator_gpu.test_sgd

2018-06-05 Thread GitBox
larroy commented on issue #10945: Flaky test: test_operator_gpu.test_sgd URL: https://github.com/apache/incubator-mxnet/issues/10945#issuecomment-394753655 http://jenkins.mxnet-ci.amazon-ml.com/blue/organizations/jenkins/incubator-mxnet/detail/PR-11138/1/pipeline

[GitHub] marcoabreu opened a new pull request #11151: [MXNET-472] ccache for docker builds

2018-06-05 Thread GitBox
marcoabreu opened a new pull request #11151: [MXNET-472] ccache for docker builds URL: https://github.com/apache/incubator-mxnet/pull/11151 ## Description ## Introduce instance-local ccache to unix-based builds ## Checklist ## ### Essentials ### - [x] The PR title starts

[GitHub] Feywell opened a new issue #11150: Hope for adding sample about mx.rnn.ConvGRUCell()

2018-06-05 Thread GitBox
Feywell opened a new issue #11150: Hope for adding sample about mx.rnn.ConvGRUCell() URL: https://github.com/apache/incubator-mxnet/issues/11150 I need to use `ConvGRU `(symbol interface) to complete a task. But I just learn to use `contrib.rnn.Conv2DGRUCell` . How can I use the

[GitHub] TaoLv commented on issue #10921: [MXNET-500]Test cases improvement for MKLDNN on Gluon

2018-06-05 Thread GitBox
TaoLv commented on issue #10921: [MXNET-500]Test cases improvement for MKLDNN on Gluon URL: https://github.com/apache/incubator-mxnet/pull/10921#issuecomment-394743165 About 30 test cases would be active there. This is an

[GitHub] aaronmarkham opened a new pull request #11152: updating Scala IntelliJ tutorial

2018-06-05 Thread GitBox
aaronmarkham opened a new pull request #11152: updating Scala IntelliJ tutorial URL: https://github.com/apache/incubator-mxnet/pull/11152 Updated for new Maven package. This is an automated message from the Apache Git

[GitHub] Ishitori opened a new pull request #11153: Add tqdm to support The Straight Dope CI

2018-06-05 Thread GitBox
Ishitori opened a new pull request #11153: Add tqdm to support The Straight Dope CI URL: https://github.com/apache/incubator-mxnet/pull/11153 ## Description ## Some tutorials of The Straight Dope relies on tqdm to show progress. Add it to Docker, so it wouldn't fail the execution

[GitHub] nswamy commented on a change in pull request #11152: updating Scala IntelliJ tutorial & installation instructions [WIP]

2018-06-05 Thread GitBox
nswamy commented on a change in pull request #11152: updating Scala IntelliJ tutorial & installation instructions [WIP] URL: https://github.com/apache/incubator-mxnet/pull/11152#discussion_r193147587 ## File path: docs/tutorials/scala/mxnet_scala_on_intellij.md ## @@

[GitHub] lanking520 commented on issue #11093: Some wrong with mxnet on spark: params.jars = jars.split(", |:")

2018-06-05 Thread GitBox
lanking520 commented on issue #11093: Some wrong with mxnet on spark: params.jars = jars.split(",|:") URL: https://github.com/apache/incubator-mxnet/issues/11093#issuecomment-394793603 @yzhliu @nswamy @@andrewfayres .I think we need to build regex to identify these different formats,

[GitHub] szha commented on a change in pull request #11151: [MXNET-472] ccache for docker builds

2018-06-05 Thread GitBox
szha commented on a change in pull request #11151: [MXNET-472] ccache for docker builds URL: https://github.com/apache/incubator-mxnet/pull/11151#discussion_r193162037 ## File path: make/crosscompile.jetson.mk ## @@ -57,10 +57,10 @@ DEBUG = 0 USE_SIGNAL_HANDLER = 1 #

[GitHub] eric-haibin-lin closed pull request #11037: [MXNET-503] Website landing page for MMS

2018-06-05 Thread GitBox
eric-haibin-lin closed pull request #11037: [MXNET-503] Website landing page for MMS URL: https://github.com/apache/incubator-mxnet/pull/11037 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As

[incubator-mxnet] branch master updated (a87de58 -> d552640)

2018-06-05 Thread haibin
This is an automated email from the ASF dual-hosted git repository. haibin pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. from a87de58 [MXNET-506] Fix missing titles in certain pages (#11132) add d552640 [MXNET-503] Website

[GitHub] marcoabreu commented on a change in pull request #11151: [MXNET-472] ccache for docker builds

2018-06-05 Thread GitBox
marcoabreu commented on a change in pull request #11151: [MXNET-472] ccache for docker builds URL: https://github.com/apache/incubator-mxnet/pull/11151#discussion_r193178715 ## File path: make/crosscompile.jetson.mk ## @@ -57,10 +57,10 @@ DEBUG = 0 USE_SIGNAL_HANDLER = 1

[GitHub] anirudhacharya commented on issue #11145: [MXNET-517] add sample ratio for ROI Align

2018-06-05 Thread GitBox
anirudhacharya commented on issue #11145: [MXNET-517] add sample ratio for ROI Align URL: https://github.com/apache/incubator-mxnet/pull/11145#issuecomment-394798612 could you please add tests for this. This is an automated

[GitHub] kpmurali opened a new pull request #11155: [MXNET-521] Add Facebook open-graph tag integration

2018-06-05 Thread GitBox
kpmurali opened a new pull request #11155: [MXNET-521] Add Facebook open-graph tag integration URL: https://github.com/apache/incubator-mxnet/pull/11155 ## Description ## Adding Facebook Open-Graph tags ## Checklist ## ### Changes ### - [ x ] Adding meta og tags for title,

[GitHub] lebeg commented on a change in pull request #11053: [MXNET-244] Fixed armv7 wheel

2018-06-05 Thread GitBox
lebeg commented on a change in pull request #11053: [MXNET-244] Fixed armv7 wheel URL: https://github.com/apache/incubator-mxnet/pull/11053#discussion_r193142451 ## File path: ci/docker/install/install_openblas.sh ## @@ -0,0 +1,30 @@ +#!/usr/bin/env bash Review

[GitHub] szha commented on issue #11154: Revert "[MXNET-503] Website landing page for MMS (#11037)"

2018-06-05 Thread GitBox
szha commented on issue #11154: Revert "[MXNET-503] Website landing page for MMS (#11037)" URL: https://github.com/apache/incubator-mxnet/pull/11154#issuecomment-394814739 @aaronmarkham please open another PR for the change after revert. Sorry for the inconvenience.

[GitHub] nswamy commented on a change in pull request #11152: updating Scala IntelliJ tutorial & installation instructions [WIP]

2018-06-05 Thread GitBox
nswamy commented on a change in pull request #11152: updating Scala IntelliJ tutorial & installation instructions [WIP] URL: https://github.com/apache/incubator-mxnet/pull/11152#discussion_r193146538 ## File path: docs/tutorials/scala/mxnet_scala_on_intellij.md ## @@

[GitHub] nswamy commented on a change in pull request #11152: updating Scala IntelliJ tutorial & installation instructions [WIP]

2018-06-05 Thread GitBox
nswamy commented on a change in pull request #11152: updating Scala IntelliJ tutorial & installation instructions [WIP] URL: https://github.com/apache/incubator-mxnet/pull/11152#discussion_r193147235 ## File path: docs/tutorials/scala/README.md ## @@ -1,6 +1,34 @@ #

[GitHub] nswamy commented on a change in pull request #11152: updating Scala IntelliJ tutorial & installation instructions [WIP]

2018-06-05 Thread GitBox
nswamy commented on a change in pull request #11152: updating Scala IntelliJ tutorial & installation instructions [WIP] URL: https://github.com/apache/incubator-mxnet/pull/11152#discussion_r193145907 ## File path: docs/tutorials/scala/mxnet_scala_on_intellij.md ## @@

[GitHub] szha commented on a change in pull request #11151: [MXNET-472] ccache for docker builds

2018-06-05 Thread GitBox
szha commented on a change in pull request #11151: [MXNET-472] ccache for docker builds URL: https://github.com/apache/incubator-mxnet/pull/11151#discussion_r193161092 ## File path: ci/docker/runtime_functions.sh ## @@ -31,14 +31,49 @@ clean_repo() { git submodule

[GitHub] eric-haibin-lin closed pull request #11132: [MXNET-506] Fix missing titles in certain pages

2018-06-05 Thread GitBox
eric-haibin-lin closed pull request #11132: [MXNET-506] Fix missing titles in certain pages URL: https://github.com/apache/incubator-mxnet/pull/11132 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of

[incubator-mxnet] branch master updated: [MXNET-506] Fix missing titles in certain pages (#11132)

2018-06-05 Thread haibin
This is an automated email from the ASF dual-hosted git repository. haibin 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 a87de58 [MXNET-506] Fix missing

[GitHub] eric-haibin-lin closed pull request #10656: [MXNET-356] Fix for the search results issues

2018-06-05 Thread GitBox
eric-haibin-lin closed pull request #10656: [MXNET-356] Fix for the search results issues URL: https://github.com/apache/incubator-mxnet/pull/10656 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance:

[GitHub] szha opened a new pull request #11154: Revert "[MXNET-503] Website landing page for MMS (#11037)"

2018-06-05 Thread GitBox
szha opened a new pull request #11154: Revert "[MXNET-503] Website landing page for MMS (#11037)" URL: https://github.com/apache/incubator-mxnet/pull/11154 This reverts commit d5526400a5752f286f879632ad905c08c9c7aff5. ## Description ## #11037 was merged prematurely. cc'd

[GitHub] marcoabreu commented on a change in pull request #11151: [MXNET-472] ccache for docker builds

2018-06-05 Thread GitBox
marcoabreu commented on a change in pull request #11151: [MXNET-472] ccache for docker builds URL: https://github.com/apache/incubator-mxnet/pull/11151#discussion_r193178300 ## File path: ci/docker/runtime_functions.sh ## @@ -31,14 +31,49 @@ clean_repo() { git

[GitHub] olga-gorun commented on issue #11139: MXNet prediction on docker image mxnet/python is much slower than outside docker container

2018-06-05 Thread GitBox
olga-gorun commented on issue #11139: MXNet prediction on docker image mxnet/python is much slower than outside docker container URL: https://github.com/apache/incubator-mxnet/issues/11139#issuecomment-394830915 This issue was discussed on the forum, see

[GitHub] olga-gorun closed issue #11139: MXNet prediction on docker image mxnet/python is much slower than outside docker container

2018-06-05 Thread GitBox
olga-gorun closed issue #11139: MXNet prediction on docker image mxnet/python is much slower than outside docker container URL: https://github.com/apache/incubator-mxnet/issues/11139 This is an automated message from the

[GitHub] zhreshold opened a new issue #11160: mx.nd.zeros/ones don't recognize cpu_shared as context

2018-06-05 Thread GitBox
zhreshold opened a new issue #11160: mx.nd.zeros/ones don't recognize cpu_shared as context URL: https://github.com/apache/incubator-mxnet/issues/11160 Note: Providing complete information in the most concise form is the best way to get help. This issue template serves as the checklist

[GitHub] ashokei commented on issue #11090: Define build target for mkldnn lib build to fix 'make clean USE_MKMLDNN=1' issue

2018-06-05 Thread GitBox
ashokei commented on issue #11090: Define build target for mkldnn lib build to fix 'make clean USE_MKMLDNN=1' issue URL: https://github.com/apache/incubator-mxnet/pull/11090#issuecomment-394855285 @jinhuang415 most of the changes look ok for mkldnn, but i notice you are removing

[GitHub] zhreshold commented on issue #11138: minor fixes to example/ssd

2018-06-05 Thread GitBox
zhreshold commented on issue #11138: minor fixes to example/ssd URL: https://github.com/apache/incubator-mxnet/pull/11138#issuecomment-394863849 Same for the CI  This is an automated message from the Apache Git Service. To

[GitHub] kpmurali commented on a change in pull request #11128: [MXNET-504] Add version select + queryString capabilities + C++ instructions to install page

2018-06-05 Thread GitBox
kpmurali commented on a change in pull request #11128: [MXNET-504] Add version select + queryString capabilities + C++ instructions to install page URL: https://github.com/apache/incubator-mxnet/pull/11128#discussion_r193225889 ## File path: docs/install/index.md ## @@

[GitHub] haojin2 commented on issue #11021: [MXNET-380] count_include_pad argument for Avg Pooling

2018-06-05 Thread GitBox
haojin2 commented on issue #11021: [MXNET-380] count_include_pad argument for Avg Pooling URL: https://github.com/apache/incubator-mxnet/pull/11021#issuecomment-394866016 @piiswrong Addressed all the reviews so far, please take another look when you’ve got time, thanks!

  1   2   >