[GitHub] pengzhao-intel commented on a change in pull request #11024: [MXNET-394] concat of CSR NDArrays on first dimension

2018-06-09 Thread GitBox
pengzhao-intel commented on a change in pull request #11024: [MXNET-394] concat of CSR NDArrays on first dimension URL: https://github.com/apache/incubator-mxnet/pull/11024#discussion_r194247442 ## File path: src/operator/nn/concat.cc ## @@ -112,18 +112,30 @@ inline static

[GitHub] ZiyueHuang commented on issue #11182: fix propagation of cpu shared context, issue #11160

2018-06-09 Thread GitBox
ZiyueHuang commented on issue #11182: fix propagation of cpu shared context, issue #11160 URL: https://github.com/apache/incubator-mxnet/pull/11182#issuecomment-396023182 good to merge? @piiswrong This is an automated messag

[GitHub] ZiyueHuang commented on issue #11024: [MXNET-394] concat of CSR NDArrays on first dimension

2018-06-09 Thread GitBox
ZiyueHuang commented on issue #11024: [MXNET-394] concat of CSR NDArrays on first dimension URL: https://github.com/apache/incubator-mxnet/pull/11024#issuecomment-396023197 good to merge? @eric-haibin-lin This is an automate

[GitHub] chinakook commented on issue #11203: ConvTranspose2d giving incorrect output

2018-06-09 Thread GitBox
chinakook commented on issue #11203: ConvTranspose2d giving incorrect output URL: https://github.com/apache/incubator-mxnet/issues/11203#issuecomment-396020279 As the convtranspose is the backward of conv, It may also has a bug in the conv's backward with this settings. --

[GitHub] haojin2 commented on a change in pull request #11179: [MXNET-404] elemwise_add/sub between rsp and rsp on GPU

2018-06-09 Thread GitBox
haojin2 commented on a change in pull request #11179: [MXNET-404] elemwise_add/sub between rsp and rsp on GPU URL: https://github.com/apache/incubator-mxnet/pull/11179#discussion_r194243373 ## File path: src/operator/tensor/elemwise_binary_op_basic.cu ## @@ -22,12 +22,141

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

2018-06-09 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 737bcd8 Bump the publish ti

[GitHub] marcoabreu closed issue #11214: Flaky Docker build due to ccache

2018-06-09 Thread GitBox
marcoabreu closed issue #11214: Flaky Docker build due to ccache URL: https://github.com/apache/incubator-mxnet/issues/11214 This is an automated message from the Apache Git Service. To respond to the message, please log on G

[GitHub] marcoabreu commented on issue #11214: Flaky Docker build due to ccache

2018-06-09 Thread GitBox
marcoabreu commented on issue #11214: Flaky Docker build due to ccache URL: https://github.com/apache/incubator-mxnet/issues/11214#issuecomment-396009934 Resolved by #11217 This is an automated message from the Apache Git Se

[GitHub] marcoabreu closed pull request #11217: Fixes for CI #11214

2018-06-09 Thread GitBox
marcoabreu closed pull request #11217: Fixes for CI #11214 URL: https://github.com/apache/incubator-mxnet/pull/11217 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 this is a foreign pull reques

[incubator-mxnet] branch master updated: Fixes for CI #11214 (#11217)

2018-06-09 Thread marcoabreu
This is an automated email from the ASF dual-hosted git repository. marcoabreu 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 462c3c3 Fixes for CI #11214 (#1121

[GitHub] larroy commented on issue #11217: Fixes for CI #11214

2018-06-09 Thread GitBox
larroy commented on issue #11217: Fixes for CI #11214 URL: https://github.com/apache/incubator-mxnet/pull/11217#issuecomment-396007514 CI is passing There's some comments on a script, other than that should fix the docker creation, up to you to merge or have another round of CI. I'm fine

[GitHub] larroy commented on issue #11217: Fixes for CI #11214

2018-06-09 Thread GitBox
larroy commented on issue #11217: Fixes for CI #11214 URL: https://github.com/apache/incubator-mxnet/pull/11217#issuecomment-396007514 CI is passing This is an automated message from the Apache Git Service. To respond to t

[GitHub] marcoabreu commented on issue #11216: Scala inference memory leak fix #11204

2018-06-09 Thread GitBox
marcoabreu commented on issue #11216: Scala inference memory leak fix #11204 URL: https://github.com/apache/incubator-mxnet/pull/11216#issuecomment-396005932 Great, thank you. Could you file this against master please and we then cherry-pick it into 1.2? ---

[GitHub] marcoabreu commented on a change in pull request #11211: docs build update for v1.2.1

2018-06-09 Thread GitBox
marcoabreu commented on a change in pull request #11211: docs build update for v1.2.1 URL: https://github.com/apache/incubator-mxnet/pull/11211#discussion_r194223364 ## File path: docs/build_version_doc/artifacts/0.12.0.index.html ## @@ -0,0 +1,1241 @@ + Review comment:

[GitHub] marcoabreu commented on issue #11217: Fixes for CI #11214

2018-06-09 Thread GitBox
marcoabreu commented on issue #11217: Fixes for CI #11214 URL: https://github.com/apache/incubator-mxnet/pull/11217#issuecomment-396005710 Hey @larroy, is this PR done? This is an automated message from the Apache Git Service.

[GitHub] eric-haibin-lin opened a new issue #11218: Feature request: randint operator

2018-06-09 Thread GitBox
eric-haibin-lin opened a new issue #11218: Feature request: randint operator URL: https://github.com/apache/incubator-mxnet/issues/11218 reference: https://docs.scipy.org/doc/numpy/reference/generated/numpy.random.randint.html --

[GitHub] EsraaRagaa closed issue #8562: Number of output from the mx.sym.SoftmaxOutput

2018-06-09 Thread GitBox
EsraaRagaa closed issue #8562: Number of output from the mx.sym.SoftmaxOutput URL: https://github.com/apache/incubator-mxnet/issues/8562 This is an automated message from the Apache Git Service. To respond to the message, ple

[GitHub] EsraaRagaa commented on issue #8562: Number of output from the mx.sym.SoftmaxOutput

2018-06-09 Thread GitBox
EsraaRagaa commented on issue #8562: Number of output from the mx.sym.SoftmaxOutput URL: https://github.com/apache/incubator-mxnet/issues/8562#issuecomment-395998824 Thanks, I found that I should edit my network. This is an

[GitHub] lanking520 commented on issue #11216: Scala inference memory leak fix #11204

2018-06-09 Thread GitBox
lanking520 commented on issue #11216: Scala inference memory leak fix #11204 URL: https://github.com/apache/incubator-mxnet/pull/11216#issuecomment-395984615 Hi @marcoabreu , this issue has been found by a customer and we feel it really important to the Scala package. It will crash users in

[GitHub] gigasquid commented on issue #11205: Clojure Contrib Package

2018-06-09 Thread GitBox
gigasquid commented on issue #11205: Clojure Contrib Package URL: https://github.com/apache/incubator-mxnet/pull/11205#issuecomment-395978081 @kurman - I rewrote the commits into 2 parts - one with the main package and tests and another with the examples. Please let me know if I can do anyt

[GitHub] larroy opened a new pull request #11217: Fixes for CI #11214

2018-06-09 Thread GitBox
larroy opened a new pull request #11217: Fixes for CI #11214 URL: https://github.com/apache/incubator-mxnet/pull/11217 ## Description ## (Brief description on what this PR is about) ## Checklist ## ### Essentials ### Please feel free to remove inapplicable items for your PR.

[GitHub] larroy commented on issue #11214: Flaky Docker build due to ccache

2018-06-09 Thread GitBox
larroy commented on issue #11214: Flaky Docker build due to ccache URL: https://github.com/apache/incubator-mxnet/issues/11214#issuecomment-395966737 looking into this This is an automated message from the Apache Git Service.

[GitHub] marcoabreu commented on issue #11200: Training Imagenet on AlexNet failed with corrupted JPEG format images

2018-06-09 Thread GitBox
marcoabreu commented on issue #11200: Training Imagenet on AlexNet failed with corrupted JPEG format images URL: https://github.com/apache/incubator-mxnet/issues/11200#issuecomment-395961637 Hello, thank you for reporting this! Would you mind providing a few example image- and rec-

[GitHub] marcoabreu commented on issue #11216: Scala inference memory leak fix #11204

2018-06-09 Thread GitBox
marcoabreu commented on issue #11216: Scala inference memory leak fix #11204 URL: https://github.com/apache/incubator-mxnet/pull/11216#issuecomment-395961456 Hi @andrewfayres, thanks a lot for your first contribution! Are there tools to detect these memory leaks or is it testable afterwar

[GitHub] marcoabreu commented on issue #11216: Scala inference memory leak fix #11204

2018-06-09 Thread GitBox
marcoabreu commented on issue #11216: Scala inference memory leak fix #11204 URL: https://github.com/apache/incubator-mxnet/pull/11216#issuecomment-395961456 Are there tools to detect these memory leaks or is it testable afterwards in any way? --

[GitHub] marcoabreu commented on issue #11216: Scala inference memory leak fix #11204

2018-06-09 Thread GitBox
marcoabreu commented on issue #11216: Scala inference memory leak fix #11204 URL: https://github.com/apache/incubator-mxnet/pull/11216#issuecomment-395961456 Hi @andrewfayres, thanks a lot for your first contribution! Are there tools to detect these memory leaks or is it testable afterwar

[GitHub] marcoabreu commented on issue #11214: Flaky build due to ccache

2018-06-09 Thread GitBox
marcoabreu commented on issue #11214: Flaky build due to ccache URL: https://github.com/apache/incubator-mxnet/issues/11214#issuecomment-395961063 Related log: ``` Step 6/26 : RUN /work/ubuntu_ccache.sh ---> Running in 1c1280d9d2f7 + pushd . + apt update / /

[GitHub] marcoabreu commented on issue #11214: Flaky build due to ccache

2018-06-09 Thread GitBox
marcoabreu commented on issue #11214: Flaky build due to ccache URL: https://github.com/apache/incubator-mxnet/issues/11214#issuecomment-395960944 This is caused by the compilation of CCache of our Docker images and not related to our actual execution logic. It seems like our docker

[GitHub] marcoabreu commented on a change in pull request #11041: gpu mem pool strategy

2018-06-09 Thread GitBox
marcoabreu commented on a change in pull request #11041: gpu mem pool strategy URL: https://github.com/apache/incubator-mxnet/pull/11041#discussion_r194218366 ## File path: tests/cpp/storage/storage_test.cc ## @@ -22,11 +21,13 @@ * \file storage_test.cc * \brief cpu/gpu

[GitHub] marcoabreu commented on a change in pull request #11211: docs build update for v1.2.1

2018-06-09 Thread GitBox
marcoabreu commented on a change in pull request #11211: docs build update for v1.2.1 URL: https://github.com/apache/incubator-mxnet/pull/11211#discussion_r194223364 ## File path: docs/build_version_doc/artifacts/0.12.0.index.html ## @@ -0,0 +1,1241 @@ + Review comment:

[GitHub] pengzhao-intel commented on issue #11212: cherry-pick bug fixes in MKLDNN for v1.2.0

2018-06-09 Thread GitBox
pengzhao-intel commented on issue #11212: cherry-pick bug fixes in MKLDNN for v1.2.0 URL: https://github.com/apache/incubator-mxnet/pull/11212#issuecomment-395957308 @zheng-da please try if this issue (#11028) is gone in your branch. ---

[GitHub] marcoabreu commented on a change in pull request #11041: gpu mem pool strategy

2018-06-09 Thread GitBox
marcoabreu commented on a change in pull request #11041: gpu mem pool strategy URL: https://github.com/apache/incubator-mxnet/pull/11041#discussion_r194218366 ## File path: tests/cpp/storage/storage_test.cc ## @@ -22,11 +21,13 @@ * \file storage_test.cc * \brief cpu/gpu

[GitHub] marcoabreu commented on a change in pull request #11041: gpu mem pool strategy

2018-06-09 Thread GitBox
marcoabreu commented on a change in pull request #11041: gpu mem pool strategy URL: https://github.com/apache/incubator-mxnet/pull/11041#discussion_r194218296 ## File path: tests/cpp/storage/storage_test.cc ## @@ -60,5 +61,39 @@ TEST(Storage, Basic_GPU) { storage->Free