[GitHub] anirudh2290 commented on a change in pull request #11210: [MXNET-532] Clarify documentation of save_params(), load_params(), export()

2018-06-12 Thread GitBox
anirudh2290 commented on a change in pull request #11210: [MXNET-532] Clarify documentation of save_params(), load_params(), export() URL: https://github.com/apache/incubator-mxnet/pull/11210#discussion_r194876492 ## File path: python/mxnet/gluon/block.py ## @@ -309,6

[GitHub] ThomasDelteil commented on issue #11210: [MXNET-532] Clarify documentation of save_params(), load_params(), export()

2018-06-12 Thread GitBox
ThomasDelteil commented on issue #11210: [MXNET-532] Clarify documentation of save_params(), load_params(), export() URL: https://github.com/apache/incubator-mxnet/pull/11210#issuecomment-396719823 I think these docs should be ported to `save_parameters` and `load_parameters`

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

2018-06-12 Thread GitBox
azai91 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_r194874083 ## File path: src/operator/nn/mkldnn/mkldnn_base.cc ## @@ -304,18 +383,26 @@ void

[GitHub] piiswrong commented on issue #11210: [MXNET-532] Clarify documentation of save_params(), load_params(), export()

2018-06-12 Thread GitBox
piiswrong commented on issue #11210: [MXNET-532] Clarify documentation of save_params(), load_params(), export() URL: https://github.com/apache/incubator-mxnet/pull/11210#issuecomment-396718977 These functions are going to be deprecated. I don't think we should add so much doc to them. A

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

2018-06-12 Thread GitBox
eric-haibin-lin 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_r194872573 ## File path: src/common/exec_utils.h ## @@ -98,10 +98,13 @@ inline bool

[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] azai91 commented on a change in pull request #11129: [MXNET-497]Test kAddTo request for mkldnn operators

2018-06-12 Thread GitBox
azai91 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_r194869335 ## File path: src/operator/nn/mkldnn/mkldnn_base.cc ## @@ -128,12 +201,18 @@ void

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

2018-06-12 Thread GitBox
azai91 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_r194870537 ## File path: tests/cpp/operator/mkldnn.cc ## @@ -571,9 +613,8 @@ void

[GitHub] anirudh2290 commented on a change in pull request #11210: [MXNET-532] Clarify documentation of save_params(), load_params(), export()

2018-06-12 Thread GitBox
anirudh2290 commented on a change in pull request #11210: [MXNET-532] Clarify documentation of save_params(), load_params(), export() URL: https://github.com/apache/incubator-mxnet/pull/11210#discussion_r194870834 ## File path: python/mxnet/gluon/block.py ## @@ -309,6

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

2018-06-12 Thread GitBox
marcoabreu commented on issue #11200: Training Imagenet on AlexNet failed with corrupted JPEG format images URL: https://github.com/apache/incubator-mxnet/issues/11200#issuecomment-396714979 Usually, if websites provide a download, they also add the hash to it to validate that no

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

2018-06-12 Thread GitBox
spidyDev commented on a change in pull request #11213: [MXNET-533] MXNet-ONNX export URL: https://github.com/apache/incubator-mxnet/pull/11213#discussion_r194869698 ## File path: python/mxnet/contrib/onnx/_export/op_translations.py ## @@ -0,0 +1,1667 @@ +# 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 `debug_gluon.py`: ``` import mxnet as mx from mxnet import nd,

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

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

[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_r194868524 ## File path: src/operator/tensor/broadcast_reduce_op.h ## @@ -984,13 +986,13 @@ void SqRootForL2(const

[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_r194868019 ## File path: src/operator/tensor/broadcast_reduce_op.h ## @@ -880,30 +880,32 @@ inline bool

[GitHub] ThomasDelteil commented on issue #10817: Static memory allocation for cached_op

2018-06-12 Thread GitBox
ThomasDelteil commented on issue #10817: Static memory allocation for cached_op URL: https://github.com/apache/incubator-mxnet/pull/10817#issuecomment-396701926 @piiswrong I have mixed results with this PR. Did you benchmark your code? I am running this notebook:

[GitHub] ThomasDelteil commented on issue #10817: Static memory allocation for cached_op

2018-06-12 Thread GitBox
ThomasDelteil commented on issue #10817: Static memory allocation for cached_op URL: https://github.com/apache/incubator-mxnet/pull/10817#issuecomment-396701926 @piiswrong I have mixed results with this PR. Did you benchmark your code? I am running this notebook:

[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 @marcoabreu The blc_output.txt file was not needed. I have removed it. The url_list.txt contains the list of URLs that are publicly

[GitHub] piiswrong opened a new pull request #11248: improve memory allocation error

2018-06-12 Thread GitBox
piiswrong opened a new pull request #11248: improve memory allocation error URL: https://github.com/apache/incubator-mxnet/pull/11248 ## Description ## (Brief description on what this PR is about) ## Checklist ## ### Essentials ### Please feel free to remove inapplicable

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

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

[GitHub] wenyangchu opened a new pull request #11247: add seed_aug parameter for ImageRecordItr to fix random seed for augm…

2018-06-12 Thread GitBox
wenyangchu opened a new pull request #11247: add seed_aug parameter for ImageRecordItr to fix random seed for augm… URL: https://github.com/apache/incubator-mxnet/pull/11247 ## Description ## Add random seed control to augmentations in io.ImageRecordIter. Motivation: I am doing a

[GitHub] ThomasDelteil commented on issue #10817: Static memory allocation for cached_op

2018-06-12 Thread GitBox
ThomasDelteil commented on issue #10817: Static memory allocation for cached_op URL: https://github.com/apache/incubator-mxnet/pull/10817#issuecomment-396701926 @piiswrong I have mixed results with this PR. Did you benchmark your code? I am running this notebook:

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

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

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

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

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

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

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

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

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

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

[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] 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-396694032 Tesla V100. ``` git checkout 5b99b25e5f6ab3a20c7bcf4821a6af0a1a95f823 git submodule update --init

[GitHub] larroy commented on issue #11245: [MXNET-244][MXNET-523][ARM] improvements to ARMv7 based builds.

2018-06-12 Thread GitBox
larroy commented on issue #11245: [MXNET-244][MXNET-523][ARM] improvements to ARMv7 based builds. URL: https://github.com/apache/incubator-mxnet/pull/11245#issuecomment-396693768 Ready to merge, see also https://github.com/apache/incubator-mxnet/pull/11220 This has been cleaned due to

[GitHub] lanking520 commented on a change in pull request #11239: [MXNET-319] Javadoc fix

2018-06-12 Thread GitBox
lanking520 commented on a change in pull request #11239: [MXNET-319] Javadoc fix URL: https://github.com/apache/incubator-mxnet/pull/11239#discussion_r194848777 ## File path: scala-package/native/osx-x86_64-cpu/pom.xml ## @@ -67,7 +67,7 @@ -Wl,-x

[GitHub] larroy closed pull request #11220: [MXNET-244][MXNET-523][ARM] improvements to ARMv7 based builds.

2018-06-12 Thread GitBox
larroy closed pull request #11220: [MXNET-244][MXNET-523][ARM] improvements to ARMv7 based builds. URL: https://github.com/apache/incubator-mxnet/pull/11220 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] haojin2 commented on issue #11021: [MXNET-380] count_include_pad argument for Avg Pooling

2018-06-12 Thread GitBox
haojin2 commented on issue #11021: [MXNET-380] count_include_pad argument for Avg Pooling URL: https://github.com/apache/incubator-mxnet/pull/11021#issuecomment-396691275 @piiswrong Is this PR good to be merged? This is an

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

2018-06-12 Thread GitBox
larroy commented on a change in pull request #11245: [WIP][MXNET-244][MXNET-523][ARM] improvements to ARMv7 based builds. URL: https://github.com/apache/incubator-mxnet/pull/11245#discussion_r194845622 ## File path: ci/docker/runtime_functions.sh ## @@ -120,33 +134,40 @@

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

2018-06-12 Thread GitBox
larroy commented on a change in pull request #11245: [WIP][MXNET-244][MXNET-523][ARM] improvements to ARMv7 based builds. URL: https://github.com/apache/incubator-mxnet/pull/11245#discussion_r194845622 ## File path: ci/docker/runtime_functions.sh ## @@ -120,33 +134,40 @@

[GitHub] anirudh2290 commented on a change in pull request #11210: [MXNET-532] Clarify documentation of save_params(), load_params(), export()

2018-06-12 Thread GitBox
anirudh2290 commented on a change in pull request #11210: [MXNET-532] Clarify documentation of save_params(), load_params(), export() URL: https://github.com/apache/incubator-mxnet/pull/11210#discussion_r194845052 ## File path: python/mxnet/gluon/block.py ## @@ -309,6

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

2018-06-12 Thread GitBox
marcoabreu commented on a change in pull request #11245: [WIP][MXNET-244][MXNET-523][ARM] improvements to ARMv7 based builds. URL: https://github.com/apache/incubator-mxnet/pull/11245#discussion_r194844402 ## File path: ci/docker/install/ubuntu_ccache.sh ## @@ -24,6

[GitHub] nswamy commented on a change in pull request #11239: [MXNET-319] Javadoc fix

2018-06-12 Thread GitBox
nswamy commented on a change in pull request #11239: [MXNET-319] Javadoc fix URL: https://github.com/apache/incubator-mxnet/pull/11239#discussion_r194842540 ## File path: scala-package/native/osx-x86_64-cpu/pom.xml ## @@ -67,7 +67,7 @@ -Wl,-x

[GitHub] anirudh2290 commented on issue #11054: [MXNET-244] Fixed armv7 wheel (1.2.0 release)

2018-06-12 Thread GitBox
anirudh2290 commented on issue #11054: [MXNET-244] Fixed armv7 wheel (1.2.0 release) URL: https://github.com/apache/incubator-mxnet/pull/11054#issuecomment-396686426 ping @lebeg . Is this ready ? This is an automated message

[GitHub] kalyc commented on issue #11240: rcnn example throws CUDNN_STATUS_BAD_PARAM when running under cudnn 6.0

2018-06-12 Thread GitBox
kalyc commented on issue #11240: rcnn example throws CUDNN_STATUS_BAD_PARAM when running under cudnn 6.0 URL: https://github.com/apache/incubator-mxnet/issues/11240#issuecomment-396685593 Thanks for submitting this issue @xioryu You could post this on discuss.mxnet.io for further details

[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_r194839899 ## File path: scala-package/core/src/main/scala/org/apache/mxnet/FeedForward.scala ## @@ -224,13 +224,24 @@

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

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

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

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

[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

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

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

[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

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

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

[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

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

[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

[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

[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

[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

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

[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

[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

[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

[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

[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

[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

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

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

[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

[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 #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?

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

[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

[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

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

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

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

<    1   2   3   >