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

[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] 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] 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] 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] 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_r194633150 ## 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_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_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] 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] 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] 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] 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] 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] 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 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] 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 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] 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] 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.

<    1   2   3