[GitHub] lebeg commented on a change in pull request #11244: Fix #11214 ccache compilation

2018-06-12 Thread GitBox
lebeg commented on a change in pull request #11244: Fix #11214 ccache compilation URL: https://github.com/apache/incubator-mxnet/pull/11244#discussion_r194703498 ## File path: ci/docker/install/ubuntu_ccache.sh ## @@ -24,6 +24,12 @@ set -ex pushd . apt update +apt

[GitHub] ThomasDelteil commented on issue #11202: Flaky test Python 3: Win: test_operator.test_gru_bidirectional

2018-06-12 Thread GitBox
ThomasDelteil commented on issue #11202: Flaky test Python 3: Win: test_operator.test_gru_bidirectional URL: https://github.com/apache/incubator-mxnet/issues/11202#issuecomment-396478673

[GitHub] lihaofd commented on issue #10311: [MXNET-107]Fused GRU implementation for CPU

2018-06-12 Thread GitBox
lihaofd commented on issue #10311: [MXNET-107]Fused GRU implementation for CPU URL: https://github.com/apache/incubator-mxnet/pull/10311#issuecomment-396482725 @ThomasDelteil , we will look into the issue. Thanks! This is

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

2018-06-12 Thread GitBox
lebeg commented on a change in pull request #11041: gpu mem pool strategy URL: https://github.com/apache/incubator-mxnet/pull/11041#discussion_r194660923 ## File path: src/storage/pooled_storage_manager.h ## @@ -129,13 +139,172 @@ void GPUPooledStorageManager::ReleaseAll()

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

2018-06-12 Thread GitBox
lebeg commented on a change in pull request #11041: gpu mem pool strategy URL: https://github.com/apache/incubator-mxnet/pull/11041#discussion_r194661450 ## File path: src/storage/pooled_storage_manager.h ## @@ -129,13 +139,172 @@ void GPUPooledStorageManager::ReleaseAll()

[GitHub] larroy commented on issue #11055: [MXNET-57] Added support android64

2018-06-12 Thread GitBox
larroy commented on issue #11055: [MXNET-57] Added support android64 URL: https://github.com/apache/incubator-mxnet/pull/11055#issuecomment-396540125 @marcoabreu @piiswrong please merge This is an automated message from

[GitHub] larroy commented on issue #11055: [MXNET-57] Added support android64

2018-06-12 Thread GitBox
larroy commented on issue #11055: [MXNET-57] Added support android64 URL: https://github.com/apache/incubator-mxnet/pull/11055#issuecomment-396540125 @marcoabreu @piiswrong @szha please merge This is an automated message

[GitHub] larroy opened a new pull request #11244: Fix #11214 ccache compilation

2018-06-12 Thread GitBox
larroy opened a new pull request #11244: Fix #11214 ccache compilation URL: https://github.com/apache/incubator-mxnet/pull/11244 ## Checklist ## ### Essentials ### Please feel free to remove inapplicable items for your PR. - [x] The PR title starts with [MXNET-$JIRA_ID], where

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

2018-06-12 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_r194704368 ## File path: scala-package/macros/src/main/scala/org/apache/mxnet/APIDocGenerator.scala ## @@

[GitHub] pengzhao-intel commented on issue #11047: [MXNET-33] Enhance mkldnn pooling to support full convention

2018-06-12 Thread GitBox
pengzhao-intel commented on issue #11047: [MXNET-33] Enhance mkldnn pooling to support full convention URL: https://github.com/apache/incubator-mxnet/pull/11047#issuecomment-396573612 @piiswrong Could help merge this PR? thanks

[GitHub] larroy commented on a change in pull request #11055: [MXNET-57] Added support android64

2018-06-12 Thread GitBox
larroy commented on a change in pull request #11055: [MXNET-57] Added support android64 URL: https://github.com/apache/incubator-mxnet/pull/11055#discussion_r194704386 ## File path: ci/docker/Dockerfile.build.android_arm64 ## @@ -18,23 +18,20 @@ # # Dockerfile to build

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

2018-06-12 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-396562429 @anirudh2290 Totally understand it will be a risk. The reason is MXNET used the specified CI of MKL-DNN. When we updated

[GitHub] zhreshold commented on a change in pull request #11213: [MXNET-533] MXNet-ONNX export

2018-06-12 Thread GitBox
zhreshold commented on a change in pull request #11213: [MXNET-533] MXNet-ONNX export URL: https://github.com/apache/incubator-mxnet/pull/11213#discussion_r194634374 ## File path: python/mxnet/contrib/onnx/_export/export_onnx.py ## @@ -0,0 +1,270 @@ +# Licensed to the

[GitHub] zhreshold commented on a change in pull request #11213: [MXNET-533] MXNet-ONNX export

2018-06-12 Thread GitBox
zhreshold commented on a change in pull request #11213: [MXNET-533] MXNet-ONNX export URL: https://github.com/apache/incubator-mxnet/pull/11213#discussion_r194635311 ## File path: tests/python-pytest/onnx/export/backend.py ## @@ -0,0 +1,98 @@ +# Licensed to the Apache

[GitHub] zhreshold commented on a change in pull request #11213: [MXNET-533] MXNet-ONNX export

2018-06-12 Thread GitBox
zhreshold commented on a change in pull request #11213: [MXNET-533] MXNet-ONNX export URL: https://github.com/apache/incubator-mxnet/pull/11213#discussion_r194635931 ## File path: tests/python-pytest/onnx/export/backend.py ## @@ -0,0 +1,98 @@ +# Licensed to the Apache

[GitHub] zhreshold commented on a change in pull request #11213: [MXNET-533] MXNet-ONNX export

2018-06-12 Thread GitBox
zhreshold commented on a change in pull request #11213: [MXNET-533] MXNet-ONNX export URL: https://github.com/apache/incubator-mxnet/pull/11213#discussion_r194632794 ## File path: python/mxnet/contrib/onnx/_export/export_model.py ## @@ -0,0 +1,89 @@ +# Licensed to the

[GitHub] zhreshold commented on a change in pull request #11213: [MXNET-533] MXNet-ONNX export

2018-06-12 Thread GitBox
zhreshold commented on a change in pull request #11213: [MXNET-533] MXNet-ONNX export URL: https://github.com/apache/incubator-mxnet/pull/11213#discussion_r194632621 ## File path: python/mxnet/contrib/onnx/__init__.py ## @@ -18,3 +18,4 @@ from ._import.import_model

[GitHub] zhreshold commented on a change in pull request #11213: [MXNET-533] MXNet-ONNX export

2018-06-12 Thread GitBox
zhreshold commented on a change in pull request #11213: [MXNET-533] MXNet-ONNX export URL: https://github.com/apache/incubator-mxnet/pull/11213#discussion_r194633857 ## File path: python/mxnet/contrib/onnx/_export/export_onnx.py ## @@ -0,0 +1,270 @@ +# Licensed to the

[GitHub] zhreshold commented on a change in pull request #11213: [MXNET-533] MXNet-ONNX export

2018-06-12 Thread GitBox
zhreshold commented on a change in pull request #11213: [MXNET-533] MXNet-ONNX export URL: https://github.com/apache/incubator-mxnet/pull/11213#discussion_r194631417 ## File path: python/mxnet/contrib/onnx/_export/export_onnx.py ## @@ -0,0 +1,270 @@ +# Licensed to the

[GitHub] zhreshold commented on a change in pull request #11213: [MXNET-533] MXNet-ONNX export

2018-06-12 Thread GitBox
zhreshold commented on a change in pull request #11213: [MXNET-533] MXNet-ONNX export URL: https://github.com/apache/incubator-mxnet/pull/11213#discussion_r194636243 ## File path: tests/python-pytest/onnx/export/mxnet_export_test.py ## @@ -0,0 +1,186 @@ +# Licensed to the

[GitHub] zhreshold commented on a change in pull request #11213: [MXNET-533] MXNet-ONNX export

2018-06-12 Thread GitBox
zhreshold commented on a change in pull request #11213: [MXNET-533] MXNet-ONNX export URL: https://github.com/apache/incubator-mxnet/pull/11213#discussion_r194633522 ## File path: python/mxnet/contrib/onnx/_export/export_onnx.py ## @@ -0,0 +1,270 @@ +# Licensed to the

[GitHub] zhreshold commented on a change in pull request #11213: [MXNET-533] MXNet-ONNX export

2018-06-12 Thread GitBox
zhreshold commented on a change in pull request #11213: [MXNET-533] MXNet-ONNX export URL: https://github.com/apache/incubator-mxnet/pull/11213#discussion_r194633150 ## File path: python/mxnet/contrib/onnx/_export/export_onnx.py ## @@ -0,0 +1,270 @@ +# Licensed to the

[GitHub] liuzx32 opened a new issue #11221: How to store a large distributed training model with KVStore?

2018-06-12 Thread GitBox
liuzx32 opened a new issue #11221: How to store a large distributed training model with KVStore? URL: https://github.com/apache/incubator-mxnet/issues/11221 If the training model is large, and one single machine can‘t store the model,how do you solve it at present?The model is the result.

[GitHub] ThomasDelteil commented on issue #11219: Flaky test: test_gru_bidirectional

2018-06-12 Thread GitBox
ThomasDelteil commented on issue #11219: Flaky test: test_gru_bidirectional URL: https://github.com/apache/incubator-mxnet/issues/11219#issuecomment-396481327 @lihaofd can you take a look please? thanks! This is an automated

[GitHub] ThomasDelteil commented on a change in pull request #11213: [MXNET-533] MXNet-ONNX export

2018-06-12 Thread GitBox
ThomasDelteil commented on a change in pull request #11213: [MXNET-533] MXNet-ONNX export URL: https://github.com/apache/incubator-mxnet/pull/11213#discussion_r194631489 ## File path: python/mxnet/contrib/onnx/_export/export_helper.py ## @@ -0,0 +1,67 @@ +# Licensed to

[GitHub] ThomasDelteil commented on a change in pull request #11213: [MXNET-533] MXNet-ONNX export

2018-06-12 Thread GitBox
ThomasDelteil commented on a change in pull request #11213: [MXNET-533] MXNet-ONNX export URL: https://github.com/apache/incubator-mxnet/pull/11213#discussion_r194633141 ## File path: python/mxnet/contrib/onnx/_export/export_model.py ## @@ -0,0 +1,89 @@ +# Licensed to the

[GitHub] ThomasDelteil commented on issue #10311: [MXNET-107]Fused GRU implementation for CPU

2018-06-12 Thread GitBox
ThomasDelteil commented on issue #10311: [MXNET-107]Fused GRU implementation for CPU URL: https://github.com/apache/incubator-mxnet/pull/10311#issuecomment-396480918 This seems to have generated flakiness on this test: https://github.com/apache/incubator-mxnet/issues/11202 and here

[GitHub] ThomasDelteil commented on issue #10311: [MXNET-107]Fused GRU implementation for CPU

2018-06-12 Thread GitBox
ThomasDelteil commented on issue #10311: [MXNET-107]Fused GRU implementation for CPU URL: https://github.com/apache/incubator-mxnet/pull/10311#issuecomment-396480918 This seems to have generated flakiness on this test: https://github.com/apache/incubator-mxnet/issues/11202

[GitHub] ThomasDelteil commented on issue #11219: Flaky test: test_gru_bidirectional

2018-06-12 Thread GitBox
ThomasDelteil commented on issue #11219: Flaky test: test_gru_bidirectional URL: https://github.com/apache/incubator-mxnet/issues/11219#issuecomment-396483700 http://jenkins.mxnet-ci.amazon-ml.com/blue/organizations/jenkins/incubator-mxnet/detail/PR-11236/1/pipeline

[GitHub] eric-haibin-lin commented on issue #11241: Conv1D throws CUDNN_STATUS_EXECUTION_FAILED

2018-06-12 Thread GitBox
eric-haibin-lin commented on issue #11241: Conv1D throws CUDNN_STATUS_EXECUTION_FAILED URL: https://github.com/apache/incubator-mxnet/issues/11241#issuecomment-396484475 Can be also reproduced by the following code: ``` import mxnet as mx from mxnet import nd, sym, autograd

[GitHub] yjcn opened a new issue #11242: Same model with c_predict_api gets an incorrect result but it is right in python.

2018-06-12 Thread GitBox
yjcn opened a new issue #11242: Same model with c_predict_api gets an incorrect result but it is right in python. URL: https://github.com/apache/incubator-mxnet/issues/11242 I use my own dataset( 3 classes) to train a vgg16_reduced model. And it works well in [mxnet-ssd

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

2018-06-12 Thread GitBox
kurman commented on issue #11205: Clojure Contrib Package URL: https://github.com/apache/incubator-mxnet/pull/11205#issuecomment-396498227 @gigasquid thanks for splitting the commit, I am already finding that it is easer going through the PR.

[GitHub] lebeg 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-12 Thread GitBox
lebeg 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_r194652560 ## File path:

[GitHub] lebeg 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-12 Thread GitBox
lebeg 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_r194650475 ## File path: tests/nightly/Jenkinsfile ## @@ -0,0 +1,180 @@

[GitHub] lebeg 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-12 Thread GitBox
lebeg 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_r194652196 ## File path: tests/nightly/broken_link_checker_test/JenkinsfileForBLC

[GitHub] lebeg 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-12 Thread GitBox
lebeg 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_r194648900 ## File path: tests/jenkins/run_test_installation_docs.sh ##

[GitHub] lebeg 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-12 Thread GitBox
lebeg 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_r194647428 ## File path: docs/install/index.md ## @@ -84,7 +84,7 @@ $

[GitHub] lebeg commented on issue #10564: Simplified CUDA language detection in cmake

2018-06-12 Thread GitBox
lebeg commented on issue #10564: Simplified CUDA language detection in cmake URL: https://github.com/apache/incubator-mxnet/pull/10564#issuecomment-396509117 I will look into it as soon as I get some time. This is an

[GitHub] lebeg 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-12 Thread GitBox
lebeg 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_r194651047 ## File path: tests/nightly/Jenkinsfile ## @@ -0,0 +1,180 @@

[GitHub] lebeg 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-12 Thread GitBox
lebeg 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_r194647662 ## File path: tests/jenkins/run_test_installation_docs.sh ##

[GitHub] lebeg 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-12 Thread GitBox
lebeg 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_r194651963 ## File path: tests/nightly/broken_link_checker_test/JenkinsfileForBLC

[GitHub] lebeg 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-12 Thread GitBox
lebeg 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_r194650363 ## File path: tests/nightly/Jenkinsfile ## @@ -0,0 +1,180 @@

[GitHub] lebeg 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-12 Thread GitBox
lebeg 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_r194651162 ## File path: tests/nightly/Jenkinsfile ## @@ -0,0 +1,180 @@

[GitHub] lebeg 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-12 Thread GitBox
lebeg 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_r194649621 ## File path: tests/nightly/Jenkinsfile ## @@ -0,0 +1,180 @@

[GitHub] ThomasDelteil commented on a change in pull request #11236: Improve hybridblock doc

2018-06-12 Thread GitBox
ThomasDelteil commented on a change in pull request #11236: Improve hybridblock doc URL: https://github.com/apache/incubator-mxnet/pull/11236#discussion_r194623975 ## File path: python/mxnet/gluon/block.py ## @@ -600,6 +605,28 @@ def _summary_hook(block, _, outputs):

[GitHub] ThomasDelteil commented on a change in pull request #11236: Improve hybridblock doc

2018-06-12 Thread GitBox
ThomasDelteil commented on a change in pull request #11236: Improve hybridblock doc URL: https://github.com/apache/incubator-mxnet/pull/11236#discussion_r194623493 ## File path: docs/tutorials/gluon/hybrid.md ## @@ -105,23 +105,35 @@ Hybridize will speed up execution and

[GitHub] ThomasDelteil commented on a change in pull request #11236: Improve hybridblock doc

2018-06-12 Thread GitBox
ThomasDelteil commented on a change in pull request #11236: Improve hybridblock doc URL: https://github.com/apache/incubator-mxnet/pull/11236#discussion_r194623975 ## File path: python/mxnet/gluon/block.py ## @@ -600,6 +605,28 @@ def _summary_hook(block, _, outputs):

[GitHub] ThomasDelteil commented on a change in pull request #11236: Improve hybridblock doc

2018-06-12 Thread GitBox
ThomasDelteil commented on a change in pull request #11236: Improve hybridblock doc URL: https://github.com/apache/incubator-mxnet/pull/11236#discussion_r194623778 ## File path: python/mxnet/gluon/block.py ## @@ -600,6 +605,28 @@ def _summary_hook(block, _, outputs):

[GitHub] eric-haibin-lin opened a new issue #11241: Conv1D throws CUDNN_STATUS_EXECUTION_FAILED

2018-06-12 Thread GitBox
eric-haibin-lin opened a new issue #11241: Conv1D throws CUDNN_STATUS_EXECUTION_FAILED URL: https://github.com/apache/incubator-mxnet/issues/11241 Setup: ``` $ pip install mxnet-cu90==1.1.0 $ nvcc --version nvcc: NVIDIA (R) Cuda compiler driver Copyright (c) 2005-2017

[GitHub] ThomasDelteil commented on a change in pull request #11213: [MXNET-533] MXNet-ONNX export

2018-06-12 Thread GitBox
ThomasDelteil commented on a change in pull request #11213: [MXNET-533] MXNet-ONNX export URL: https://github.com/apache/incubator-mxnet/pull/11213#discussion_r194631895 ## File path: python/mxnet/contrib/onnx/_export/export_model.py ## @@ -0,0 +1,89 @@ +# Licensed to the

[GitHub] dwSun opened a new issue #11243: weird gpu memory usage

2018-06-12 Thread GitBox
dwSun opened a new issue #11243: weird gpu memory usage URL: https://github.com/apache/incubator-mxnet/issues/11243 ## Description The script below will eat all gpu memory gradually in the first epoch. Sometimes it will cause OOM error and crash. Sometimes it just keep running, but

[GitHub] I3orn2FLY commented on issue #10778: Make android error

2018-06-12 Thread GitBox
I3orn2FLY commented on issue #10778: Make android error URL: https://github.com/apache/incubator-mxnet/issues/10778#issuecomment-396565617 I have the same problem! How to solve it? This is an automated message from the

[GitHub] lebeg commented on a change in pull request #11055: [MXNET-57] Added support android64

2018-06-12 Thread GitBox
lebeg commented on a change in pull request #11055: [MXNET-57] Added support android64 URL: https://github.com/apache/incubator-mxnet/pull/11055#discussion_r194713319 ## File path: ci/docker/Dockerfile.build.android_arm64 ## @@ -18,23 +18,20 @@ # # Dockerfile to build

[GitHub] nswamy commented on a change in pull request #11204: Scala inference memory leak fix

2018-06-12 Thread GitBox
nswamy commented on a change in pull request #11204: Scala inference memory leak fix URL: https://github.com/apache/incubator-mxnet/pull/11204#discussion_r194773876 ## File path: scala-package/core/src/main/scala/org/apache/mxnet/FeedForward.scala ## @@ -224,13 +224,24 @@

[GitHub] szha commented on issue #11127: add import_ for SymbolBlock

2018-06-12 Thread GitBox
szha commented on issue #11127: add import_ for SymbolBlock URL: https://github.com/apache/incubator-mxnet/pull/11127#issuecomment-396641831 @anirudh2290, no, such change would break users who are on 1.2.0. This is an

[GitHub] szha commented on issue #63: update python gpu build from source instructions on windows

2018-06-12 Thread GitBox
szha commented on issue #63: update python gpu build from source instructions on windows URL: https://github.com/apache/incubator-mxnet-site/pull/63#issuecomment-396647313 @yasserzamani thanks a lot for the contribution. This repo is a staging place for the API doc deployment. The doc to

[GitHub] marcoabreu commented on a change in pull request #11055: [MXNET-57] Added support android64

2018-06-12 Thread GitBox
marcoabreu commented on a change in pull request #11055: [MXNET-57] Added support android64 URL: https://github.com/apache/incubator-mxnet/pull/11055#discussion_r194802029 ## File path: ci/docker/Dockerfile.build.android_arm64 ## @@ -18,23 +18,20 @@ # # Dockerfile to

[GitHub] lanking520 closed pull request #11143: [MXNET-319] Javadoc fix

2018-06-12 Thread GitBox
lanking520 closed pull request #11143: [MXNET-319] Javadoc fix URL: https://github.com/apache/incubator-mxnet/pull/11143 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

[GitHub] lanking520 commented on issue #11143: [MXNET-319] Javadoc fix

2018-06-12 Thread GitBox
lanking520 commented on issue #11143: [MXNET-319] Javadoc fix URL: https://github.com/apache/incubator-mxnet/pull/11143#issuecomment-396649842 This PR is screwed by the submodule update. Abandon this one and create an identical one here: https://github.com/apache/incubator-mxnet/pull/11239

[GitHub] haojin2 opened a new pull request #11246: [MXNET-453] Fix flaky test_sgd

2018-06-12 Thread GitBox
haojin2 opened a new pull request #11246: [MXNET-453] Fix flaky test_sgd URL: https://github.com/apache/incubator-mxnet/pull/11246 ## Description ## As title ## Checklist ## ### Essentials ### - [x] Changes are complete (i.e. I finished coding on this PR) - [x] All

[GitHub] larroy commented on a change in pull request #11055: [MXNET-57] Added support android64

2018-06-12 Thread GitBox
larroy commented on a change in pull request #11055: [MXNET-57] Added support android64 URL: https://github.com/apache/incubator-mxnet/pull/11055#discussion_r194849319 ## File path: ci/docker/Dockerfile.build.android_arm64 ## @@ -18,23 +18,20 @@ # # Dockerfile to build

[GitHub] haojin2 commented on a change in pull request #11229: [MXNET-379] L1 Normalization

2018-06-12 Thread GitBox
haojin2 commented on a change in pull request #11229: [MXNET-379] L1 Normalization URL: https://github.com/apache/incubator-mxnet/pull/11229#discussion_r194871298 ## File path: tests/python/unittest/test_ndarray.py ## @@ -1304,6 +1304,34 @@ def

[GitHub] marcoabreu commented on issue #62: Seed files for broken link checker job

2018-06-12 Thread GitBox
marcoabreu commented on issue #62: Seed files for broken link checker job URL: https://github.com/apache/incubator-mxnet-site/pull/62#issuecomment-396734859 Thank you. Please work together with Aaron to get this merged. His script automatically purges all content of this

[GitHub] lanking520 commented on issue #11198: [MXNET-531][Work In Progress] MNIST Examples for Scala new API

2018-06-12 Thread GitBox
lanking520 commented on issue #11198: [MXNET-531][Work In Progress] MNIST Examples for Scala new API URL: https://github.com/apache/incubator-mxnet/pull/11198#issuecomment-396758571 @nswamy Done that, this one MNIST specifically

[GitHub] szha closed issue #10406: Flaky test - test_gluon_data.test_datasets

2018-06-12 Thread GitBox
szha closed issue #10406: Flaky test - test_gluon_data.test_datasets URL: https://github.com/apache/incubator-mxnet/issues/10406 This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] szha commented on issue #10406: Flaky test - test_gluon_data.test_datasets

2018-06-12 Thread GitBox
szha commented on issue #10406: Flaky test - test_gluon_data.test_datasets URL: https://github.com/apache/incubator-mxnet/issues/10406#issuecomment-396760237 This test has been stable since #11181 thanks to @ThomasDelteil

[GitHub] szha commented on issue #10973: Flaky test_deconvolution

2018-06-12 Thread GitBox
szha commented on issue #10973: Flaky test_deconvolution URL: https://github.com/apache/incubator-mxnet/issues/10973#issuecomment-396760496 Got a couple of this too http://jenkins.mxnet-ci.amazon-ml.com/blue/organizations/jenkins/incubator-mxnet/detail/PR-11041/82/pipeline/724

[GitHub] Roshrini commented on a change in pull request #11213: [MXNET-533] MXNet-ONNX export

2018-06-12 Thread GitBox
Roshrini commented on a change in pull request #11213: [MXNET-533] MXNet-ONNX export URL: https://github.com/apache/incubator-mxnet/pull/11213#discussion_r194918505 ## File path: python/mxnet/contrib/onnx/_export/export_onnx.py ## @@ -0,0 +1,270 @@ +# Licensed to the

[GitHub] marcoabreu commented on a change in pull request #11198: [MXNET-531][Work In Progress] MNIST Examples for Scala new API

2018-06-12 Thread GitBox
marcoabreu commented on a change in pull request #11198: [MXNET-531][Work In Progress] MNIST Examples for Scala new API URL: https://github.com/apache/incubator-mxnet/pull/11198#discussion_r194921289 ## File path:

[GitHub] marcoabreu commented on a change in pull request #11198: [MXNET-531][Work In Progress] MNIST Examples for Scala new API

2018-06-12 Thread GitBox
marcoabreu commented on a change in pull request #11198: [MXNET-531][Work In Progress] MNIST Examples for Scala new API URL: https://github.com/apache/incubator-mxnet/pull/11198#discussion_r194921345 ## File path:

[GitHub] ctcyang commented on issue #11237: Bring back MXNET_GPU_COPY_NTHREADS env variable

2018-06-12 Thread GitBox
ctcyang commented on issue #11237: Bring back MXNET_GPU_COPY_NTHREADS env variable URL: https://github.com/apache/incubator-mxnet/pull/11237#issuecomment-396778091 When it's set to 1, copies between GPUs are serialized. Even between 2 GPUs, if GPU 0 is currently sending something to 1 on

[GitHub] lanking520 closed pull request #11123: [MXNET-319][DO NOT MERGE][Deprecated] Javadoc fix

2018-06-12 Thread GitBox
lanking520 closed pull request #11123: [MXNET-319][DO NOT MERGE][Deprecated] Javadoc fix URL: https://github.com/apache/incubator-mxnet/pull/11123 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] ctcyang commented on issue #11237: Bring back MXNET_GPU_COPY_NTHREADS env variable

2018-06-12 Thread GitBox
ctcyang commented on issue #11237: Bring back MXNET_GPU_COPY_NTHREADS env variable URL: https://github.com/apache/incubator-mxnet/pull/11237#issuecomment-396778091 When it's set to 1, copies between GPUs are serialized. Even between 2 GPUs, if GPU 0 is currently sending something to 1 on

[GitHub] reminisce opened a new pull request #11251: [WIP] Subgraph op

2018-06-12 Thread GitBox
reminisce opened a new pull request #11251: [WIP] Subgraph op URL: https://github.com/apache/incubator-mxnet/pull/11251 ## Description ## (Brief description on what this PR is about) ## Checklist ## ### Essentials ### Please feel free to remove inapplicable items for your

[GitHub] azai91 commented on issue #11129: [MXNET-497]Test kAddTo request for mkldnn operators

2018-06-12 Thread GitBox
azai91 commented on issue #11129: [MXNET-497]Test kAddTo request for mkldnn operators URL: https://github.com/apache/incubator-mxnet/pull/11129#issuecomment-396790290 @pengzhao-intel is anyone working on a PR to revert the regression or should I work on?

[GitHub] azai91 commented on issue #11129: [MXNET-497]Test kAddTo request for mkldnn operators

2018-06-12 Thread GitBox
azai91 commented on issue #11129: [MXNET-497]Test kAddTo request for mkldnn operators URL: https://github.com/apache/incubator-mxnet/pull/11129#issuecomment-396790290 @pengzhao-intel is anyone working on a PR to revert the regression or is someone already reverting?

[GitHub] Erdos001 opened a new issue #11255: P100 + MXNet is slow

2018-06-12 Thread GitBox
Erdos001 opened a new issue #11255: P100 + MXNet is slow URL: https://github.com/apache/incubator-mxnet/issues/11255 Note: Providing complete information in the most concise form is the best way to get help. This issue template serves as the checklist for essential information to most of

[GitHub] Roshrini commented on a change in pull request #11213: [MXNET-533] MXNet-ONNX export

2018-06-12 Thread GitBox
Roshrini commented on a change in pull request #11213: [MXNET-533] MXNet-ONNX export URL: https://github.com/apache/incubator-mxnet/pull/11213#discussion_r194912618 ## File path: python/mxnet/contrib/onnx/_export/export_helper.py ## @@ -0,0 +1,64 @@ +# Licensed to the

[GitHub] Roshrini commented on a change in pull request #11213: [MXNET-533] MXNet-ONNX export

2018-06-12 Thread GitBox
Roshrini commented on a change in pull request #11213: [MXNET-533] MXNet-ONNX export URL: https://github.com/apache/incubator-mxnet/pull/11213#discussion_r194912635 ## File path: python/mxnet/contrib/onnx/_export/export_helper.py ## @@ -0,0 +1,64 @@ +# Licensed to the

[GitHub] szha commented on issue #11202: Flaky test Python 3: Win: test_operator.test_gru_bidirectional

2018-06-12 Thread GitBox
szha commented on issue #11202: Flaky test Python 3: Win: test_operator.test_gru_bidirectional URL: https://github.com/apache/incubator-mxnet/issues/11202#issuecomment-396759995 duplicate #11219 This is an automated

[GitHub] szha closed issue #11202: Flaky test Python 3: Win: test_operator.test_gru_bidirectional

2018-06-12 Thread GitBox
szha closed issue #11202: Flaky test Python 3: Win: test_operator.test_gru_bidirectional URL: https://github.com/apache/incubator-mxnet/issues/11202 This is an automated message from the Apache Git Service. To respond to

[GitHub] marcoabreu commented on issue #11247: add seed_aug parameter for ImageRecordItr to fix random seed for augm…

2018-06-12 Thread GitBox
marcoabreu commented on issue #11247: add seed_aug parameter for ImageRecordItr to fix random seed for augm… URL: https://github.com/apache/incubator-mxnet/pull/11247#issuecomment-396767163 Hi @wenyangchu , thanks a lot for your contribution! Would you mind adding a test to ensure

[GitHub] marcoabreu commented on issue #11237: Bring back MXNET_GPU_COPY_NTHREADS env variable

2018-06-12 Thread GitBox
marcoabreu commented on issue #11237: Bring back MXNET_GPU_COPY_NTHREADS env variable URL: https://github.com/apache/incubator-mxnet/pull/11237#issuecomment-396767385 Would it make sense to choose this value dependent on the number of GPUs available?

[GitHub] szha commented on a change in pull request #11254: add blocklist

2018-06-12 Thread GitBox
szha commented on a change in pull request #11254: add blocklist URL: https://github.com/apache/incubator-mxnet/pull/11254#discussion_r194935377 ## File path: docs/api/python/gluon/gluon.md ## @@ -116,6 +116,8 @@ net.hybridize() Block HybridBlock +BlockList

[GitHub] uran209 commented on issue #5432: Predicting with mx.io.ImageRecordIter object returns different size vector each time is run

2018-06-12 Thread GitBox
uran209 commented on issue #5432: Predicting with mx.io.ImageRecordIter object returns different size vector each time is run URL: https://github.com/apache/incubator-mxnet/issues/5432#issuecomment-396798033 I have the same problem of "Predicting with mx.io.ImageRecordIter object returns

[GitHub] marcoabreu commented on a change in pull request #11245: [MXNET-244][MXNET-523][ARM] improvements to ARMv7 based builds.

2018-06-12 Thread GitBox
marcoabreu commented on a change in pull request #11245: [MXNET-244][MXNET-523][ARM] improvements to ARMv7 based builds. URL: https://github.com/apache/incubator-mxnet/pull/11245#discussion_r194950646 ## File path: tests/python/unittest/test_ndarray.py ## @@ -711,9 +711,8

[GitHub] szha closed issue #10974: Cpp CI may be broken

2018-06-12 Thread GitBox
szha closed issue #10974: Cpp CI may be broken URL: https://github.com/apache/incubator-mxnet/issues/10974 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the

[GitHub] szha commented on issue #10974: Cpp CI may be broken

2018-06-12 Thread GitBox
szha commented on issue #10974: Cpp CI may be broken URL: https://github.com/apache/incubator-mxnet/issues/10974#issuecomment-396760757 This has been fixed by #11065 thanks to @haojin2 This is an automated message from the

[GitHub] Roshrini commented on a change in pull request #11213: [MXNET-533] MXNet-ONNX export

2018-06-12 Thread GitBox
Roshrini commented on a change in pull request #11213: [MXNET-533] MXNet-ONNX export URL: https://github.com/apache/incubator-mxnet/pull/11213#discussion_r194914781 ## File path: python/mxnet/contrib/onnx/_export/export_helper.py ## @@ -0,0 +1,64 @@ +# Licensed to the

[GitHub] Roshrini commented on a change in pull request #11213: [MXNET-533] MXNet-ONNX export

2018-06-12 Thread GitBox
Roshrini commented on a change in pull request #11213: [MXNET-533] MXNet-ONNX export URL: https://github.com/apache/incubator-mxnet/pull/11213#discussion_r194914843 ## File path: python/mxnet/contrib/onnx/_export/export_helper.py ## @@ -0,0 +1,64 @@ +# Licensed to the

[GitHub] Roshrini commented on a change in pull request #11213: [MXNET-533] MXNet-ONNX export

2018-06-12 Thread GitBox
Roshrini commented on a change in pull request #11213: [MXNET-533] MXNet-ONNX export URL: https://github.com/apache/incubator-mxnet/pull/11213#discussion_r194918283 ## File path: python/mxnet/contrib/onnx/_export/export_onnx.py ## @@ -0,0 +1,270 @@ +# Licensed to the

[GitHub] eric-haibin-lin commented on a change in pull request #11199: [MXNET-538] Add XUnit to python tests

2018-06-12 Thread GitBox
eric-haibin-lin commented on a change in pull request #11199: [MXNET-538] Add XUnit to python tests URL: https://github.com/apache/incubator-mxnet/pull/11199#discussion_r194920022 ## File path: ci/docker/runtime_functions.sh ## @@ -500,14 +502,14 @@

[GitHub] marcoabreu closed issue #11221: How to store a large distributed training model with KVStore?

2018-06-12 Thread GitBox
marcoabreu closed issue #11221: How to store a large distributed training model with KVStore? URL: https://github.com/apache/incubator-mxnet/issues/11221 This is an automated message from the Apache Git Service. To respond

[GitHub] lanking520 commented on a change in pull request #11198: [MXNET-531][Work In Progress] MNIST Examples for Scala new API

2018-06-12 Thread GitBox
lanking520 commented on a change in pull request #11198: [MXNET-531][Work In Progress] MNIST Examples for Scala new API URL: https://github.com/apache/incubator-mxnet/pull/11198#discussion_r194921916 ## File path:

[GitHub] lanking520 commented on issue #11198: [MXNET-531][Work In Progress] MNIST Examples for Scala new API

2018-06-12 Thread GitBox
lanking520 commented on issue #11198: [MXNET-531][Work In Progress] MNIST Examples for Scala new API URL: https://github.com/apache/incubator-mxnet/pull/11198#issuecomment-396777833 Abandon this PR since it includes too much conflict with Master Branch, open one here:

[GitHub] lanking520 closed pull request #11198: [MXNET-531][Work In Progress] MNIST Examples for Scala new API

2018-06-12 Thread GitBox
lanking520 closed pull request #11198: [MXNET-531][Work In Progress] MNIST Examples for Scala new API URL: https://github.com/apache/incubator-mxnet/pull/11198 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

[GitHub] lanking520 opened a new pull request #11250: [MXNET-531][Work In Progress] MNIST Examples for Scala new API

2018-06-12 Thread GitBox
lanking520 opened a new pull request #11250: [MXNET-531][Work In Progress] MNIST Examples for Scala new API URL: https://github.com/apache/incubator-mxnet/pull/11250 ## Description ## This PR contains examples written from the new API. @nswamy @yzhliu @andrewfayres All examples

[GitHub] lanking520 commented on issue #11123: [MXNET-319][DO NOT MERGE][Deprecated] Javadoc fix

2018-06-12 Thread GitBox
lanking520 commented on issue #11123: [MXNET-319][DO NOT MERGE][Deprecated] Javadoc fix URL: https://github.com/apache/incubator-mxnet/pull/11123#issuecomment-396778128 Master branch merged, no need to go with this solution

[GitHub] pengzhao-intel commented on issue #11129: [MXNET-497]Test kAddTo request for mkldnn operators

2018-06-12 Thread GitBox
pengzhao-intel commented on issue #11129: [MXNET-497]Test kAddTo request for mkldnn operators URL: https://github.com/apache/incubator-mxnet/pull/11129#issuecomment-396792242 @azai91, You can try to figure out the root cause and fix it with a new PR. I don't think we need to revert the

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

2018-06-12 Thread GitBox
zheng-da commented on issue #11212: cherry-pick bug fixes in MKLDNN for v1.2.0 URL: https://github.com/apache/incubator-mxnet/pull/11212#issuecomment-396762289 this test was from the PR https://github.com/apache/incubator-mxnet/pull/10578/files. On Tue, Jun 12, 2018 at 4:01

  1   2   3   >