[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 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_r194633141 ## 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_r194631895 ## 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_r194631417 ## File path: python/mxnet/contrib/onnx/_export/export_onnx.py ## @@ -0,0 +1,270 @@ +# Licensed to the Apac

[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 Apac

[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 Apac

[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 Soft

[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 Soft

[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 Apac

[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 impo

[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 Apac

[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 Apac

[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 python](https://gi

[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] 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 at

[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] 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: tests/nightly/broken_link_checker_test/broken_link_checke

[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 @@ $ wg

[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 automated

[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 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] pwn1 commented on issue #11098: Installation instructions MacOS/R/CPU don't work.

2018-06-12 Thread GitBox
pwn1 commented on issue #11098: Installation instructions MacOS/R/CPU don't work. URL: https://github.com/apache/incubator-mxnet/issues/11098#issuecomment-396518164 I also followed your instructions @anirudhacharya and got the same opencv error. I installed opencv using homebrew, then I

[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 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 th

[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 $JIRA

[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 inst

[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] 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] 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 Apach

[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 M

[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 opened a new pull request #11245: [WIP][ARM] improvements to ARMv7 based builds.

2018-06-12 Thread GitBox
larroy opened a new pull request #11245: [WIP][ARM] improvements to ARMv7 based builds. URL: https://github.com/apache/incubator-mxnet/pull/11245 Fix build with OpenCV 2. Native RPi build. Openblas compilation fixes and version pinning. (@lebeg) Disabled bundled OpenMP for cross compi

[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_r194709196 ## File path: scala-package/core/src/main/scala/org/apache/mxnet/annotation/Experimental.scala ##

[GitHub] yasserzamani opened a new pull request #63: update python gpu build from source instructions on windows

2018-06-12 Thread GitBox
yasserzamani opened a new pull request #63: update python gpu build from source instructions on windows URL: https://github.com/apache/incubator-mxnet-site/pull/63 Recently I built MXNet from source on windows (gpu, python). During the process I encountered some issues which I've added fix

[GitHub] larroy closed pull request #10797: Better cleaning of git repo, signal handlers

2018-06-12 Thread GitBox
larroy closed pull request #10797: Better cleaning of git repo, signal handlers URL: https://github.com/apache/incubator-mxnet/pull/10797 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

[GitHub] nswamy closed pull request #11244: Fix #11214 ccache compilation

2018-06-12 Thread GitBox
nswamy closed pull request #11244: Fix #11214 ccache compilation URL: https://github.com/apache/incubator-mxnet/pull/11244 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

[incubator-mxnet] branch master updated: Fix #11214 ccache compilation (#11244)

2018-06-12 Thread nswamy
This is an automated email from the ASF dual-hosted git repository. nswamy 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 0254f9e Fix #11214 ccache compilation

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

2018-06-12 Thread GitBox
nswamy commented on issue #11123: [MXNET-319][DO NOT MERGE][Deprecated] Javadoc fix URL: https://github.com/apache/incubator-mxnet/pull/11123#issuecomment-396611127 @lanking520 Can we close this, as discussed in another PR we agreed not to change the internal APIs to use new APIs? ---

[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_r194761222 ## File path: CMakeLists.txt ## @@ -15,9 +15,9 @@ mxnet_option(USE_NCCL "Use NVidia NCCL wit

[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_r194763223 ## File path: CMakeLists.txt ## @@ -15,9 +15,9 @@ mxnet_option(USE_NCCL "Use NVidia NCCL with

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

2018-06-12 Thread GitBox
lebeg commented on issue #10778: Make android error URL: https://github.com/apache/incubator-mxnet/issues/10778#issuecomment-396614975 In case you are using make based build you should try as suggested by @rahul003: ``` USE_F16C=0 make -j$(nproc) ``` or ``` cp make/confi

[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 automate

[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 bu

[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 re

[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 changes

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

2018-06-12 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_r194804382 ## File path: scala-package/core/src/main/scala/org/apache/mxnet/annotation/Experimental.scala ##

[GitHub] nswamy commented on issue #11126: [MXNET-386] ongoing maintenance on NDArray

2018-06-12 Thread GitBox
nswamy commented on issue #11126: [MXNET-386] ongoing maintenance on NDArray URL: https://github.com/apache/incubator-mxnet/pull/11126#issuecomment-396652353 Does it work? > On Jun 12, 2018, at 6:23 PM, Lanking wrote: > > @lanking520 commented on this pull request. > >

[GitHub] marcoabreu commented on issue #11226: Fix build.py when CCACHE_DIR is set.

2018-06-12 Thread GitBox
marcoabreu commented on issue #11226: Fix build.py when CCACHE_DIR is set. URL: https://github.com/apache/incubator-mxnet/pull/11226#issuecomment-396655637 Well I don't know what you expect. You modify the ccache shellscript and then expect the cache to not be rebuilt? http://jenkins.mxnet

[GitHub] kalyc commented on issue #11221: How to store a large distributed training model with KVStore?

2018-06-12 Thread GitBox
kalyc commented on issue #11221: How to store a large distributed training model with KVStore? URL: https://github.com/apache/incubator-mxnet/issues/11221#issuecomment-396655948 @liuzx32 could you please close this GitHub issue? ---

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

2018-06-12 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-396656573 I'm tempted to say that your tests might be revealing a memory leak/corruption in MXNet. Could you please try

[GitHub] sandeep-krishnamurthy commented on a change in pull request #11140: [MXNET-344] [ONNX-MXNet] Add new Operator Translations for ONNX import module

2018-06-12 Thread GitBox
sandeep-krishnamurthy commented on a change in pull request #11140: [MXNET-344] [ONNX-MXNet] Add new Operator Translations for ONNX import module URL: https://github.com/apache/incubator-mxnet/pull/11140#discussion_r194813510 ## File path: python/mxnet/contrib/onnx/_import/op_transl

[GitHub] sandeep-krishnamurthy commented on a change in pull request #11140: [MXNET-344] [ONNX-MXNet] Add new Operator Translations for ONNX import module

2018-06-12 Thread GitBox
sandeep-krishnamurthy commented on a change in pull request #11140: [MXNET-344] [ONNX-MXNet] Add new Operator Translations for ONNX import module URL: https://github.com/apache/incubator-mxnet/pull/11140#discussion_r194813105 ## File path: python/mxnet/contrib/onnx/_import/op_transl

[GitHub] zheng-da commented on a change in pull request #11129: [MXNET-497]Test kAddTo request for mkldnn operators

2018-06-12 Thread GitBox
zheng-da commented on a change in pull request #11129: [MXNET-497]Test kAddTo request for mkldnn operators URL: https://github.com/apache/incubator-mxnet/pull/11129#discussion_r194813020 ## File path: src/operator/nn/mkldnn/mkldnn_base.cc ## @@ -304,18 +383,26 @@ void Fall

[GitHub] zheng-da commented on a change in pull request #11129: [MXNET-497]Test kAddTo request for mkldnn operators

2018-06-12 Thread GitBox
zheng-da commented on a change in pull request #11129: [MXNET-497]Test kAddTo request for mkldnn operators URL: https://github.com/apache/incubator-mxnet/pull/11129#discussion_r194813504 ## File path: src/operator/nn/mkldnn/mkldnn_sum.cc ## @@ -58,46 +58,54 @@ void MKLDNNS

[GitHub] zheng-da commented on a change in pull request #11129: [MXNET-497]Test kAddTo request for mkldnn operators

2018-06-12 Thread GitBox
zheng-da commented on a change in pull request #11129: [MXNET-497]Test kAddTo request for mkldnn operators URL: https://github.com/apache/incubator-mxnet/pull/11129#discussion_r194814080 ## File path: tests/cpp/operator/mkldnn.cc ## @@ -582,12 +623,20 @@ void VerifySumResu

[GitHub] zheng-da commented on a change in pull request #11129: [MXNET-497]Test kAddTo request for mkldnn operators

2018-06-12 Thread GitBox
zheng-da commented on a change in pull request #11129: [MXNET-497]Test kAddTo request for mkldnn operators URL: https://github.com/apache/incubator-mxnet/pull/11129#discussion_r194814020 ## File path: tests/cpp/operator/mkldnn.cc ## @@ -571,9 +613,8 @@ void VerifyActResult

[GitHub] zheng-da commented on a change in pull request #11129: [MXNET-497]Test kAddTo request for mkldnn operators

2018-06-12 Thread GitBox
zheng-da commented on a change in pull request #11129: [MXNET-497]Test kAddTo request for mkldnn operators URL: https://github.com/apache/incubator-mxnet/pull/11129#discussion_r194808938 ## File path: src/common/exec_utils.h ## @@ -98,10 +98,13 @@ inline bool SetupDefaultB

[GitHub] zheng-da commented on a change in pull request #11129: [MXNET-497]Test kAddTo request for mkldnn operators

2018-06-12 Thread GitBox
zheng-da commented on a change in pull request #11129: [MXNET-497]Test kAddTo request for mkldnn operators URL: https://github.com/apache/incubator-mxnet/pull/11129#discussion_r194812866 ## File path: src/operator/nn/mkldnn/mkldnn_base.cc ## @@ -304,18 +383,26 @@ void Fall

[GitHub] zheng-da commented on a change in pull request #11129: [MXNET-497]Test kAddTo request for mkldnn operators

2018-06-12 Thread GitBox
zheng-da commented on a change in pull request #11129: [MXNET-497]Test kAddTo request for mkldnn operators URL: https://github.com/apache/incubator-mxnet/pull/11129#discussion_r194810836 ## File path: src/operator/nn/mkldnn/mkldnn_base.cc ## @@ -128,12 +201,18 @@ void Comm

[GitHub] zheng-da commented on a change in pull request #11129: [MXNET-497]Test kAddTo request for mkldnn operators

2018-06-12 Thread GitBox
zheng-da commented on a change in pull request #11129: [MXNET-497]Test kAddTo request for mkldnn operators URL: https://github.com/apache/incubator-mxnet/pull/11129#discussion_r194811728 ## File path: src/operator/nn/mkldnn/mkldnn_base.cc ## @@ -304,18 +383,26 @@ void Fall

[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_r194815085 ## File path: python/mxnet/contrib/onnx/_export/export_helper.py ## @@ -0,0 +1,67 @@ +# Licensed to the Apac

[GitHub] kalyc commented on issue #11243: weird gpu memory usage

2018-06-12 Thread GitBox
kalyc commented on issue #11243: weird gpu memory usage URL: https://github.com/apache/incubator-mxnet/issues/11243#issuecomment-396664174 Hello @dwSun, thanks for submitting your issue. There is an open discussion forum about mxnet on discuss.mxnet.io - you could post your query there to

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

2018-06-12 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_r194818676 ## File path: scala-package/core/src/main/scala/org/apache/mxnet/annotation/Experimental.scala ##

[GitHub] piiswrong closed pull request #11235: fix loading params if ignore_extra is set

2018-06-12 Thread GitBox
piiswrong closed pull request #11235: fix loading params if ignore_extra is set URL: https://github.com/apache/incubator-mxnet/pull/11235 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

[incubator-mxnet] branch master updated: fix loading params if ignore_extra is set (#11235)

2018-06-12 Thread jxie
This is an automated email from the ASF dual-hosted git repository. jxie 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 2d761c9 fix loading params if ignore_ext

[GitHub] piiswrong closed pull request #11233: [MXNET-491] Use depthwise convolution by cuDNNv7 if available, updated version #11076

2018-06-12 Thread GitBox
piiswrong closed pull request #11233: [MXNET-491] Use depthwise convolution by cuDNNv7 if available, updated version #11076 URL: https://github.com/apache/incubator-mxnet/pull/11233 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed b

[incubator-mxnet] branch v1.2.0 updated: [MXNET-491] Use depthwise convolution by cuDNNv7 if available, updated version (#11076) (#11233)

2018-06-12 Thread jxie
This is an automated email from the ASF dual-hosted git repository. jxie pushed a commit to branch v1.2.0 in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git The following commit(s) were added to refs/heads/v1.2.0 by this push: new 546a233 [MXNET-491] Use depthwise convol

[GitHub] zheng-da commented on a change in pull request #11047: [MXNET-33] Enhance mkldnn pooling to support full convention

2018-06-12 Thread GitBox
zheng-da commented on a change in pull request #11047: [MXNET-33] Enhance mkldnn pooling to support full convention URL: https://github.com/apache/incubator-mxnet/pull/11047#discussion_r194822168 ## File path: tests/python/gpu/test_operator_gpu.py ## @@ -920,6 +920,35 @@ d

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

2018-06-12 Thread GitBox
piiswrong commented on issue #11237: Bring back MXNET_GPU_COPY_NTHREADS env variable URL: https://github.com/apache/incubator-mxnet/pull/11237#issuecomment-396668011 Why use 2 by default? This is an automated message from the

[GitHub] zheng-da commented on a change in pull request #11047: [MXNET-33] Enhance mkldnn pooling to support full convention

2018-06-12 Thread GitBox
zheng-da commented on a change in pull request #11047: [MXNET-33] Enhance mkldnn pooling to support full convention URL: https://github.com/apache/incubator-mxnet/pull/11047#discussion_r194823882 ## File path: tests/python/gpu/test_operator_gpu.py ## @@ -920,6 +920,35 @@ d

[GitHub] anirudhacharya commented on a change in pull request #11140: [MXNET-344] [ONNX-MXNet] Add new Operator Translations for ONNX import module

2018-06-12 Thread GitBox
anirudhacharya commented on a change in pull request #11140: [MXNET-344] [ONNX-MXNet] Add new Operator Translations for ONNX import module URL: https://github.com/apache/incubator-mxnet/pull/11140#discussion_r194824905 ## File path: python/mxnet/contrib/onnx/_import/op_translations.

[GitHub] DickJC123 commented on issue #11241: Conv1D throws CUDNN_STATUS_EXECUTION_FAILED

2018-06-12 Thread GitBox
DickJC123 commented on issue #11241: Conv1D throws CUDNN_STATUS_EXECUTION_FAILED URL: https://github.com/apache/incubator-mxnet/issues/11241#issuecomment-396671647 What GPU are you trying to run on? What were the nvcc args used to build your libmxnet.so? -

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

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

[GitHub] anirudhacharya commented on a change in pull request #11140: [MXNET-344] [ONNX-MXNet] Add new Operator Translations for ONNX import module

2018-06-12 Thread GitBox
anirudhacharya commented on a change in pull request #11140: [MXNET-344] [ONNX-MXNet] Add new Operator Translations for ONNX import module URL: https://github.com/apache/incubator-mxnet/pull/11140#discussion_r194826453 ## File path: python/mxnet/contrib/onnx/_import/op_translations.

[GitHub] anirudhacharya commented on a change in pull request #11140: [MXNET-344] [ONNX-MXNet] Add new Operator Translations for ONNX import module

2018-06-12 Thread GitBox
anirudhacharya commented on a change in pull request #11140: [MXNET-344] [ONNX-MXNet] Add new Operator Translations for ONNX import module URL: https://github.com/apache/incubator-mxnet/pull/11140#discussion_r194827030 ## File path: python/mxnet/contrib/onnx/_import/op_translations.

[GitHub] fye881 commented on issue #8014: cross compile mxnet for android, without using Amalgamation?

2018-06-12 Thread GitBox
fye881 commented on issue #8014: cross compile mxnet for android, without using Amalgamation? URL: https://github.com/apache/incubator-mxnet/issues/8014#issuecomment-396673584 Yes I think I get it compiled afterwards. On Monday, June 11, 2018, 4:08:19 PM PDT, Kalyanee Chendke

[GitHub] kalyc commented on issue #8014: cross compile mxnet for android, without using Amalgamation?

2018-06-12 Thread GitBox
kalyc commented on issue #8014: cross compile mxnet for android, without using Amalgamation? URL: https://github.com/apache/incubator-mxnet/issues/8014#issuecomment-396674403 @fye881 could you please resolve this issue in that case?

[GitHub] onlyphantom commented on issue #11098: Installation instructions MacOS/R/CPU don't work.

2018-06-12 Thread GitBox
onlyphantom commented on issue #11098: Installation instructions MacOS/R/CPU don't work. URL: https://github.com/apache/incubator-mxnet/issues/11098#issuecomment-396677745 Thanks for the reply @anirudh2290 ; No luck as the two other users above stated; For now I switched back to R 3.4 (re

[GitHub] fye881 closed issue #8014: cross compile mxnet for android, without using Amalgamation?

2018-06-12 Thread GitBox
fye881 closed issue #8014: cross compile mxnet for android, without using Amalgamation? URL: https://github.com/apache/incubator-mxnet/issues/8014 This is an automated message from the Apache Git Service. To respond to the m

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

2018-06-12 Thread GitBox
anirudh2290 commented on issue #11212: cherry-pick bug fixes in MKLDNN for v1.2.0 URL: https://github.com/apache/incubator-mxnet/pull/11212#issuecomment-396680308 @pengzhao-intel There are still a lot of non CI-specific code changes. Since MKLDNN is marked as experimental for the release,

[GitHub] zheng-da commented on a change in pull request #11047: [MXNET-33] Enhance mkldnn pooling to support full convention

2018-06-12 Thread GitBox
zheng-da commented on a change in pull request #11047: [MXNET-33] Enhance mkldnn pooling to support full convention URL: https://github.com/apache/incubator-mxnet/pull/11047#discussion_r194835024 ## File path: tests/python/gpu/test_operator_gpu.py ## @@ -920,6 +920,35 @@ d

[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-396680740 I set it to use 2 by default, because it has been set to 2 by default since v0.1.0, so I didn't want to cause performance degr

[GitHub] kalyc commented on issue #11242: Same model with c_predict_api gets an incorrect result but it is right in python.

2018-06-12 Thread GitBox
kalyc commented on 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#issuecomment-396681153 Hello @yjcn - Thanks for submitting the issue. Could you paste a log of your results here?

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

2018-06-12 Thread GitBox
leleamol commented on issue #62: Seed files for broken link checker job URL: https://github.com/apache/incubator-mxnet-site/pull/62#issuecomment-396681419 The blc_output.txt file was not needed. I have removed it. The url_list.txt contains the list of URLs that are publicly accessible.

[GitHub] kalyc commented on issue #11238: UX for ONNX Documentation is broken

2018-06-12 Thread GitBox
kalyc commented on issue #11238: UX for ONNX Documentation is broken URL: https://github.com/apache/incubator-mxnet/issues/11238#issuecomment-396682315 @sandeep-krishnamurthy Could you add label "Doc", "ONNX" to this issue? T

[GitHub] ThomasDelteil commented on a change in pull request #11127: add import_ for SymbolBlock

2018-06-12 Thread GitBox
ThomasDelteil commented on a change in pull request #11127: add import_ for SymbolBlock URL: https://github.com/apache/incubator-mxnet/pull/11127#discussion_r194838415 ## File path: docs/tutorials/gluon/save_load_params.md ## @@ -61,7 +61,7 @@ def build_lenet(net):

[GitHub] kalyc commented on issue #11224: ‘make lint’ is broken under python2

2018-06-12 Thread GitBox
kalyc commented on issue #11224: ‘make lint’ is broken under python2 URL: https://github.com/apache/incubator-mxnet/issues/11224#issuecomment-396684720 @TaoLv thanks for submitting this issue. @sandeep-krishnamurthy could you add label "Bug" & "Python" to this issue? --

  1   2   3   >