[GitHub] reminisce commented on issue #10341: Deadlock during ThreadedEnginePerDevice destructor after CuDNNConvolutionOp::SelectAlgo called.

2018-04-04 Thread GitBox
reminisce commented on issue #10341: Deadlock during ThreadedEnginePerDevice destructor after CuDNNConvolutionOp::SelectAlgo called. URL: https://github.com/apache/incubator-mxnet/issues/10341#issuecomment-378504188 Added some prints. It seems the `on_complete()` can never finish while `W

[GitHub] reminisce opened a new pull request #10392: Fix cudnn_conv and cudnn_deconv deadlock

2018-04-04 Thread GitBox
reminisce opened a new pull request #10392: Fix cudnn_conv and cudnn_deconv deadlock URL: https://github.com/apache/incubator-mxnet/pull/10392 ## Description ## This PR is expected to address the deadlock issue https://github.com/apache/incubator-mxnet/issues/10341 introduced by https:

[GitHub] gautamkmr commented on issue #9949: Mac build from source script validation fails

2018-04-04 Thread GitBox
gautamkmr commented on issue #9949: Mac build from source script validation fails URL: https://github.com/apache/incubator-mxnet/issues/9949#issuecomment-378508239 I tried using the install-mxnet-osx-python.sh script, it did work well with python3.6 however with python-2.7.10 I guess I ha

[GitHub] haojin2 commented on a change in pull request #10208: [MXNET-117] Sparse operator broadcast_mul/div(csr, dense) = csr

2018-04-04 Thread GitBox
haojin2 commented on a change in pull request #10208: [MXNET-117] Sparse operator broadcast_mul/div(csr, dense) = csr URL: https://github.com/apache/incubator-mxnet/pull/10208#discussion_r179052324 ## File path: src/operator/tensor/elemwise_binary_broadcast_op.h ## @@ -185

[GitHub] haojin2 commented on a change in pull request #10208: [MXNET-117] Sparse operator broadcast_mul/div(csr, dense) = csr

2018-04-04 Thread GitBox
haojin2 commented on a change in pull request #10208: [MXNET-117] Sparse operator broadcast_mul/div(csr, dense) = csr URL: https://github.com/apache/incubator-mxnet/pull/10208#discussion_r179052455 ## File path: src/operator/tensor/elemwise_binary_broadcast_op.h ## @@ -185

[GitHub] haojin2 commented on a change in pull request #10208: [MXNET-117] Sparse operator broadcast_mul/div(csr, dense) = csr

2018-04-04 Thread GitBox
haojin2 commented on a change in pull request #10208: [MXNET-117] Sparse operator broadcast_mul/div(csr, dense) = csr URL: https://github.com/apache/incubator-mxnet/pull/10208#discussion_r179052299 ## File path: src/operator/tensor/elemwise_binary_broadcast_op.h ## @@ -185

[GitHub] haojin2 commented on a change in pull request #10208: [MXNET-117] Sparse operator broadcast_mul/div(csr, dense) = csr

2018-04-04 Thread GitBox
haojin2 commented on a change in pull request #10208: [MXNET-117] Sparse operator broadcast_mul/div(csr, dense) = csr URL: https://github.com/apache/incubator-mxnet/pull/10208#discussion_r179052586 ## File path: tests/python/unittest/test_sparse_operator.py ## @@ -1675,6 +

[GitHub] haojin2 commented on a change in pull request #10208: [MXNET-117] Sparse operator broadcast_mul/div(csr, dense) = csr

2018-04-04 Thread GitBox
haojin2 commented on a change in pull request #10208: [MXNET-117] Sparse operator broadcast_mul/div(csr, dense) = csr URL: https://github.com/apache/incubator-mxnet/pull/10208#discussion_r179052500 ## File path: tests/python/unittest/test_sparse_operator.py ## @@ -1675,6 +

[GitHub] Allloginsbusy opened a new issue #10393: ImageRecordIOParser2 Assertion failed

2018-04-04 Thread GitBox
Allloginsbusy opened a new issue #10393: ImageRecordIOParser2 Assertion failed URL: https://github.com/apache/incubator-mxnet/issues/10393 I use python3 and try to get images by .rec file. I use ImageRecordIter. After 300 images I have got, I see OpenCV Error: Assertion failed .

[GitHub] chr5tphr opened a new issue #10394: [Feature Request] Operator for Im2Col

2018-04-04 Thread GitBox
chr5tphr opened a new issue #10394: [Feature Request] Operator for Im2Col URL: https://github.com/apache/incubator-mxnet/issues/10394 We need an operator for im2col. Since the operation itself is already implemented in [im2col.h](https://github.com/apache/incubator-mxnet/blob/master/src

[GitHub] larroy commented on a change in pull request #9400: Fixed memory leak

2018-04-04 Thread GitBox
larroy commented on a change in pull request #9400: Fixed memory leak URL: https://github.com/apache/incubator-mxnet/pull/9400#discussion_r179082235 ## File path: src/operator/operator_tune-inl.h ## @@ -616,7 +616,7 @@ class UnaryOpTune : public OperatorTune { */ tem

[GitHub] larroy commented on a change in pull request #9400: Fixed memory leak

2018-04-04 Thread GitBox
larroy commented on a change in pull request #9400: Fixed memory leak URL: https://github.com/apache/incubator-mxnet/pull/9400#discussion_r179082235 ## File path: src/operator/operator_tune-inl.h ## @@ -616,7 +616,7 @@ class UnaryOpTune : public OperatorTune { */ tem

[GitHub] larroy commented on a change in pull request #9400: Fixed memory leak

2018-04-04 Thread GitBox
larroy commented on a change in pull request #9400: Fixed memory leak URL: https://github.com/apache/incubator-mxnet/pull/9400#discussion_r179082235 ## File path: src/operator/operator_tune-inl.h ## @@ -616,7 +616,7 @@ class UnaryOpTune : public OperatorTune { */ tem

[GitHub] dabraude opened a new issue #10395: Android build documentation

2018-04-04 Thread GitBox
dabraude opened a new issue #10395: Android build documentation URL: https://github.com/apache/incubator-mxnet/issues/10395 It would be really useful to have documentation about building android from source. This is an a

[GitHub] larroy commented on issue #8729: Build amalgamation using a docker

2018-04-04 Thread GitBox
larroy commented on issue #8729: Build amalgamation using a docker URL: https://github.com/apache/incubator-mxnet/issues/8729#issuecomment-378564236 Hi, you can use the new scripts in the ci folder: ci/build.py right now it should build for android but the build is untested. --

[GitHub] larroy opened a new pull request #10396: [WIP] Use static openblas

2018-04-04 Thread GitBox
larroy opened a new pull request #10396: [WIP] Use static openblas URL: https://github.com/apache/incubator-mxnet/pull/10396 ## Description ## (Brief description on what this PR is about) ## Checklist ## ### Essentials ### Please feel free to remove inapplicable items for you

[GitHub] larroy commented on a change in pull request #9808: Ssd camera demo

2018-04-04 Thread GitBox
larroy commented on a change in pull request #9808: Ssd camera demo URL: https://github.com/apache/incubator-mxnet/pull/9808#discussion_r179115561 ## File path: example/ssd/dataset/cv2Iterator.py ## @@ -0,0 +1,56 @@ +# Licensed to the Apache Software Foundation (ASF) under

[GitHub] marcoabreu commented on a change in pull request #10390: [WIP] Initial PR for RAT License Check

2018-04-04 Thread GitBox
marcoabreu commented on a change in pull request #10390: [WIP] Initial PR for RAT License Check URL: https://github.com/apache/incubator-mxnet/pull/10390#discussion_r179118180 ## File path: tests/nightly/apache_rat_license_check/README.md ## @@ -0,0 +1,36 @@ +# Apache RAT

[GitHub] marcoabreu commented on issue #10375: [MXNET-187] GPU fake shuffle functions for Fermi architecture

2018-04-04 Thread GitBox
marcoabreu commented on issue #10375: [MXNET-187] GPU fake shuffle functions for Fermi architecture URL: https://github.com/apache/incubator-mxnet/pull/10375#issuecomment-378579698 What do you think about printing a warning in make/cmake if a

[GitHub] marcoabreu commented on issue #10375: [MXNET-187] GPU fake shuffle functions for Fermi architecture

2018-04-04 Thread GitBox
marcoabreu commented on issue #10375: [MXNET-187] GPU fake shuffle functions for Fermi architecture URL: https://github.com/apache/incubator-mxnet/pull/10375#issuecomment-378579698 What do you think about printing a warning in make/cmake if a Kepler GPU has been detected?

[GitHub] marcoabreu commented on issue #10387: Flaky test(scala): test_arange

2018-04-04 Thread GitBox
marcoabreu commented on issue #10387: Flaky test(scala): test_arange URL: https://github.com/apache/incubator-mxnet/issues/10387#issuecomment-378580564 See https://github.com/apache/incubator-mxnet/issues/8383 for reference T

[GitHub] marcoabreu commented on issue #10012: [Bug] There are two broken links on the top level README.md file which point to the old CI

2018-04-04 Thread GitBox
marcoabreu commented on issue #10012: [Bug] There are two broken links on the top level README.md file which point to the old CI URL: https://github.com/apache/incubator-mxnet/issues/10012#issuecomment-378596021 Thanks a lot! ---

[GitHub] marcoabreu commented on issue #10183: [MXNET-120] Float16 support for distributed training

2018-04-04 Thread GitBox
marcoabreu commented on issue #10183: [MXNET-120] Float16 support for distributed training URL: https://github.com/apache/incubator-mxnet/pull/10183#issuecomment-378596808 To pick up the topic from earlier last week. We're not running any tests on KVStore until https://github.com/apache/in

[GitHub] marcoabreu commented on issue #10232: [MXNET-136] Enabling USE_DIST_KVSTORE flag for CI

2018-04-04 Thread GitBox
marcoabreu commented on issue #10232: [MXNET-136] Enabling USE_DIST_KVSTORE flag for CI URL: https://github.com/apache/incubator-mxnet/pull/10232#issuecomment-378596922 @rahul003 have you been able to work on this? This is an

[GitHub] marcoabreu commented on issue #10365: [MXNET-261]Update MKLDNN & Add CPP Test

2018-04-04 Thread GitBox
marcoabreu commented on issue #10365: [MXNET-261]Update MKLDNN & Add CPP Test URL: https://github.com/apache/incubator-mxnet/pull/10365#issuecomment-378599212 > @marcoabreu Regarding MKL-DNN, there's release version we can use. But the MXNET development progress is very fast so more new fea

[GitHub] marcoabreu commented on issue #10232: [MXNET-136] Enabling USE_DIST_KVSTORE flag for CI

2018-04-04 Thread GitBox
marcoabreu commented on issue #10232: [MXNET-136] Enabling USE_DIST_KVSTORE flag for CI URL: https://github.com/apache/incubator-mxnet/pull/10232#issuecomment-378596922 @rahul003 have you been able to work on this? I have seen you made a few commits, let me know if I can be of any help her

[GitHub] slyforce opened a new issue #10397: grad_req in multi-task example

2018-04-04 Thread GitBox
slyforce opened a new issue #10397: grad_req in multi-task example URL: https://github.com/apache/incubator-mxnet/issues/10397 I'm referring to the python script in example/multi-task/example_multi_task.py As far as I understood gradients need to be accumulated between two loss fu

[GitHub] larroy commented on issue #10202: [MXNET-131] Refine build.py, misc fixes, tweak commandline, add help

2018-04-04 Thread GitBox
larroy commented on issue #10202: [MXNET-131] Refine build.py, misc fixes, tweak commandline, add help URL: https://github.com/apache/incubator-mxnet/pull/10202#issuecomment-378611775 @marcoabreu can we merge this? This is a

[GitHub] larroy commented on issue #10268: Update 3rdparty/dmlc-core

2018-04-04 Thread GitBox
larroy commented on issue #10268: Update 3rdparty/dmlc-core URL: https://github.com/apache/incubator-mxnet/pull/10268#issuecomment-378612228 @marcoabreu @cjolivier01 This is an automated message from the Apache Git Service. T

[GitHub] larroy commented on issue #10268: Update 3rdparty/dmlc-core

2018-04-04 Thread GitBox
larroy commented on issue #10268: Update 3rdparty/dmlc-core URL: https://github.com/apache/incubator-mxnet/pull/10268#issuecomment-378612228 @marcoabreu @cjolivier01 @eric-haibin-lin This is an automated message from the Apac

[GitHub] lebeg opened a new issue #10398: ARMv6 build with 8-10 times bigger file size

2018-04-04 Thread GitBox
lebeg opened a new issue #10398: ARMv6 build with 8-10 times bigger file size URL: https://github.com/apache/incubator-mxnet/issues/10398 The ARMv6 generated wheel grew by 8-10 times in terms of file size according to https://github.com/apache/incubator-mxnet/commit/85215b6176ef3612d198f59

[GitHub] marcoabreu commented on a change in pull request #10202: [MXNET-131] Refine build.py, misc fixes, tweak commandline, add help

2018-04-04 Thread GitBox
marcoabreu commented on a change in pull request #10202: [MXNET-131] Refine build.py, misc fixes, tweak commandline, add help URL: https://github.com/apache/incubator-mxnet/pull/10202#discussion_r179156040 ## File path: ci/build.py ## @@ -138,6 +161,14 @@ def script_name()

[GitHub] marcoabreu commented on a change in pull request #10202: [MXNET-131] Refine build.py, misc fixes, tweak commandline, add help

2018-04-04 Thread GitBox
marcoabreu commented on a change in pull request #10202: [MXNET-131] Refine build.py, misc fixes, tweak commandline, add help URL: https://github.com/apache/incubator-mxnet/pull/10202#discussion_r179156249 ## File path: ci/build.py ## @@ -147,30 +178,70 @@ def script_name(

[GitHub] marcoabreu commented on a change in pull request #10202: [MXNET-131] Refine build.py, misc fixes, tweak commandline, add help

2018-04-04 Thread GitBox
marcoabreu commented on a change in pull request #10202: [MXNET-131] Refine build.py, misc fixes, tweak commandline, add help URL: https://github.com/apache/incubator-mxnet/pull/10202#discussion_r179155974 ## File path: ci/build.py ## @@ -121,13 +139,18 @@ def script_name(

[GitHub] marcoabreu commented on a change in pull request #10202: [MXNET-131] Refine build.py, misc fixes, tweak commandline, add help

2018-04-04 Thread GitBox
marcoabreu commented on a change in pull request #10202: [MXNET-131] Refine build.py, misc fixes, tweak commandline, add help URL: https://github.com/apache/incubator-mxnet/pull/10202#discussion_r179156081 ## File path: ci/build.py ## @@ -138,6 +161,14 @@ def script_name()

[GitHub] marcoabreu commented on a change in pull request #10202: [MXNET-131] Refine build.py, misc fixes, tweak commandline, add help

2018-04-04 Thread GitBox
marcoabreu commented on a change in pull request #10202: [MXNET-131] Refine build.py, misc fixes, tweak commandline, add help URL: https://github.com/apache/incubator-mxnet/pull/10202#discussion_r179156298 ## File path: ci/build.py ## @@ -147,30 +178,70 @@ def script_name(

[GitHub] marcoabreu commented on a change in pull request #10202: [MXNET-131] Refine build.py, misc fixes, tweak commandline, add help

2018-04-04 Thread GitBox
marcoabreu commented on a change in pull request #10202: [MXNET-131] Refine build.py, misc fixes, tweak commandline, add help URL: https://github.com/apache/incubator-mxnet/pull/10202#discussion_r179158068 ## File path: ci/build.py ## @@ -147,30 +178,70 @@ def script_name(

[GitHub] marcoabreu commented on issue #10268: Update 3rdparty/dmlc-core

2018-04-04 Thread GitBox
marcoabreu commented on issue #10268: Update 3rdparty/dmlc-core URL: https://github.com/apache/incubator-mxnet/pull/10268#issuecomment-378617892 Your commit is empty, isn't it? This is an automated message from the Apache Git

[GitHub] saswatac opened a new issue #10399: Memory leak while using adam optimizer

2018-04-04 Thread GitBox
saswatac opened a new issue #10399: Memory leak while using adam optimizer URL: https://github.com/apache/incubator-mxnet/issues/10399 ## Description Memory use grows with time while using adam optimizer. ## Environment info (Required) --Python Info--

[GitHub] anirudh2290 opened a new pull request #10400: Add support for cast storage on same stypes

2018-04-04 Thread GitBox
anirudh2290 opened a new pull request #10400: Add support for cast storage on same stypes URL: https://github.com/apache/incubator-mxnet/pull/10400 ## Description ## Fixes #8524 ## Checklist ## ### Essentials ### Please feel free to remove inapplicable items for your PR.

[GitHub] anirudh2290 commented on issue #8238: different behaviour of customop in latest MXNet

2018-04-04 Thread GitBox
anirudh2290 commented on issue #8238: different behaviour of customop in latest MXNet URL: https://github.com/apache/incubator-mxnet/issues/8238#issuecomment-378649387 @VikingMew can you please help provide a brief description and current status of the issue in English. I looked at the di

[GitHub] anirudh2290 commented on issue #8801: Cannot find custom operator when visualizing network.

2018-04-04 Thread GitBox
anirudh2290 commented on issue #8801: Cannot find custom operator when visualizing network. URL: https://github.com/apache/incubator-mxnet/issues/8801#issuecomment-378656272 @eric-haibin-lin can this issue be closed. The issue has been answered by @PhanTask. -

[GitHub] haojin2 commented on a change in pull request #10208: [MXNET-117] Sparse operator broadcast_mul/div(csr, dense) = csr

2018-04-04 Thread GitBox
haojin2 commented on a change in pull request #10208: [MXNET-117] Sparse operator broadcast_mul/div(csr, dense) = csr URL: https://github.com/apache/incubator-mxnet/pull/10208#discussion_r179206473 ## File path: python/mxnet/ndarray/sparse.py ## @@ -1159,6 +1186,322 @@ def

[GitHub] haojin2 commented on a change in pull request #10208: [MXNET-117] Sparse operator broadcast_mul/div(csr, dense) = csr

2018-04-04 Thread GitBox
haojin2 commented on a change in pull request #10208: [MXNET-117] Sparse operator broadcast_mul/div(csr, dense) = csr URL: https://github.com/apache/incubator-mxnet/pull/10208#discussion_r179206464 ## File path: python/mxnet/ndarray/sparse.py ## @@ -1159,6 +1186,322 @@ def

[GitHub] reminisce commented on issue #10392: Fix cudnn_conv and cudnn_deconv deadlock

2018-04-04 Thread GitBox
reminisce commented on issue #10392: Fix cudnn_conv and cudnn_deconv deadlock URL: https://github.com/apache/incubator-mxnet/pull/10392#issuecomment-378665802 @KellenSunderland @marcoabreu Thanks for the initial investigation of the root cause. Even though this should be a safe fix, I am st

[incubator-mxnet] branch master updated: [MXNET-131] Refine build.py, misc fixes, tweak commandline, add help (#10202)

2018-04-04 Thread marcoabreu
This is an automated email from the ASF dual-hosted git repository. marcoabreu pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git The following commit(s) were added to refs/heads/master by this push: new 779faef [MXNET-131] Refine build.p

[GitHub] marcoabreu closed pull request #10202: [MXNET-131] Refine build.py, misc fixes, tweak commandline, add help

2018-04-04 Thread GitBox
marcoabreu closed pull request #10202: [MXNET-131] Refine build.py, misc fixes, tweak commandline, add help URL: https://github.com/apache/incubator-mxnet/pull/10202 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sak

[GitHub] marcoabreu commented on issue #10375: [MXNET-187] GPU fake shuffle functions for Fermi architecture

2018-04-04 Thread GitBox
marcoabreu commented on issue #10375: [MXNET-187] GPU fake shuffle functions for Fermi architecture URL: https://github.com/apache/incubator-mxnet/pull/10375#issuecomment-378579698 What do you think about printing a warning in make/cmake if a pre-kepler GPU has been detected?

[GitHub] Will0622 opened a new issue #10401: Gluon nn BatchNorm layer cannot ignore the beta

2018-04-04 Thread GitBox
Will0622 opened a new issue #10401: Gluon nn BatchNorm layer cannot ignore the beta URL: https://github.com/apache/incubator-mxnet/issues/10401 In mxnet symbol API, we cannot fix the beta to be 0. So, I tried to use the Gluon nn BatchNorm layer which allows setting center=False to ignore t

[GitHub] rahul003 commented on issue #10183: [MXNET-120] Float16 support for distributed training

2018-04-04 Thread GitBox
rahul003 commented on issue #10183: [MXNET-120] Float16 support for distributed training URL: https://github.com/apache/incubator-mxnet/pull/10183#issuecomment-378678940 I've been running into scala issues when I introduce the dist flag to the builds. With @Roshrini's help the build phase

[GitHub] rahul003 commented on issue #10232: [MXNET-136] Enabling USE_DIST_KVSTORE flag for CI

2018-04-04 Thread GitBox
rahul003 commented on issue #10232: [MXNET-136] Enabling USE_DIST_KVSTORE flag for CI URL: https://github.com/apache/incubator-mxnet/pull/10232#issuecomment-378679297 The scala build (in unit test phase) links with couple of libraries it looks like, which aren't available. I believe we nee

[GitHub] zheng-da commented on issue #10365: [MXNET-261]Update MKLDNN & Add CPP Test

2018-04-04 Thread GitBox
zheng-da commented on issue #10365: [MXNET-261]Update MKLDNN & Add CPP Test URL: https://github.com/apache/incubator-mxnet/pull/10365#issuecomment-378679790 @xinyu-intel @pengzhao-intel could you describe what is the root of this problem? It's a little weird that both MKL-DNN and CuDNN h

[GitHub] rahul003 commented on issue #10232: [MXNET-136] Enabling USE_DIST_KVSTORE flag for CI

2018-04-04 Thread GitBox
rahul003 commented on issue #10232: [MXNET-136] Enabling USE_DIST_KVSTORE flag for CI URL: https://github.com/apache/incubator-mxnet/pull/10232#issuecomment-378679933 ``` work/mxnet/3rdparty/ps-lite/build/libps.a: No such file or directory work/mxnet/deps/lib/libprotobuf-lite.a: No su

[GitHub] rahul003 commented on issue #10232: [MXNET-136] Enabling USE_DIST_KVSTORE flag for CI

2018-04-04 Thread GitBox
rahul003 commented on issue #10232: [MXNET-136] Enabling USE_DIST_KVSTORE flag for CI URL: https://github.com/apache/incubator-mxnet/pull/10232#issuecomment-378682112 I'm confused which build in the build phase, is used by the scala build ---

[GitHub] eric-haibin-lin commented on a change in pull request #10374: Sparse support for Custom Op

2018-04-04 Thread GitBox
eric-haibin-lin commented on a change in pull request #10374: Sparse support for Custom Op URL: https://github.com/apache/incubator-mxnet/pull/10374#discussion_r179219700 ## File path: example/numpy-ops/custom_sparse_sqr.py ## @@ -0,0 +1,121 @@ +# Licensed to the Apache So

[GitHub] eric-haibin-lin commented on a change in pull request #10374: Sparse support for Custom Op

2018-04-04 Thread GitBox
eric-haibin-lin commented on a change in pull request #10374: Sparse support for Custom Op URL: https://github.com/apache/incubator-mxnet/pull/10374#discussion_r179222586 ## File path: example/numpy-ops/custom_sparse_sqr.py ## @@ -0,0 +1,121 @@ +# Licensed to the Apache So

[GitHub] eric-haibin-lin commented on a change in pull request #10374: Sparse support for Custom Op

2018-04-04 Thread GitBox
eric-haibin-lin commented on a change in pull request #10374: Sparse support for Custom Op URL: https://github.com/apache/incubator-mxnet/pull/10374#discussion_r179223222 ## File path: tests/python/unittest/test_operator.py ## @@ -4012,12 +4012,23 @@ def test_rcbrt_op():

[GitHub] eric-haibin-lin commented on a change in pull request #10400: Add support for cast storage on same stypes

2018-04-04 Thread GitBox
eric-haibin-lin commented on a change in pull request #10400: Add support for cast storage on same stypes URL: https://github.com/apache/incubator-mxnet/pull/10400#discussion_r179225126 ## File path: src/operator/tensor/cast_storage-inl.h ## @@ -328,6 +329,53 @@ void CastS

[GitHub] eric-haibin-lin commented on a change in pull request #10400: Add support for cast storage on same stypes

2018-04-04 Thread GitBox
eric-haibin-lin commented on a change in pull request #10400: Add support for cast storage on same stypes URL: https://github.com/apache/incubator-mxnet/pull/10400#discussion_r179225556 ## File path: src/operator/tensor/cast_storage-inl.h ## @@ -328,6 +329,53 @@ void CastS

[GitHub] eric-haibin-lin commented on issue #10400: Add support for cast storage on same stypes

2018-04-04 Thread GitBox
eric-haibin-lin commented on issue #10400: Add support for cast storage on same stypes URL: https://github.com/apache/incubator-mxnet/pull/10400#issuecomment-378685548 Do you mind also updating the doc for the added support? https://mxnet.incubator.apache.org/versions/master/api/python/nda

[GitHub] eric-haibin-lin commented on a change in pull request #10392: [MXNET-266] Fix cudnn_conv and cudnn_deconv deadlock

2018-04-04 Thread GitBox
eric-haibin-lin commented on a change in pull request #10392: [MXNET-266] Fix cudnn_conv and cudnn_deconv deadlock URL: https://github.com/apache/incubator-mxnet/pull/10392#discussion_r179226688 ## File path: src/operator/nn/cudnn/cudnn_deconvolution-inl.h ## @@ -536,19 +5

[GitHub] rahul003 commented on issue #10232: [MXNET-136] Enabling USE_DIST_KVSTORE flag for CI

2018-04-04 Thread GitBox
rahul003 commented on issue #10232: [MXNET-136] Enabling USE_DIST_KVSTORE flag for CI URL: https://github.com/apache/incubator-mxnet/pull/10232#issuecomment-378682112 I'm confused which build in the build phase, is used by the scala build ---

[GitHub] eric-haibin-lin commented on a change in pull request #10392: [MXNET-266] Fix cudnn_conv and cudnn_deconv deadlock

2018-04-04 Thread GitBox
eric-haibin-lin commented on a change in pull request #10392: [MXNET-266] Fix cudnn_conv and cudnn_deconv deadlock URL: https://github.com/apache/incubator-mxnet/pull/10392#discussion_r179226575 ## File path: src/operator/nn/cudnn/cudnn_convolution-inl.h ## @@ -512,18 +511

[GitHub] eric-haibin-lin commented on issue #10183: [MXNET-120] Float16 support for distributed training

2018-04-04 Thread GitBox
eric-haibin-lin commented on issue #10183: [MXNET-120] Float16 support for distributed training URL: https://github.com/apache/incubator-mxnet/pull/10183#issuecomment-378687201 @marcoabreu are the nightly tests moved to apache CI already? ---

[GitHub] eric-haibin-lin commented on a change in pull request #10390: [WIP] Initial PR for RAT License Check

2018-04-04 Thread GitBox
eric-haibin-lin commented on a change in pull request #10390: [WIP] Initial PR for RAT License Check URL: https://github.com/apache/incubator-mxnet/pull/10390#discussion_r179229493 ## File path: tests/nightly/apache_rat_license_check/.rat-excludes ## @@ -0,0 +1,54 @@ +.*x

[GitHub] eric-haibin-lin commented on a change in pull request #10390: [WIP] Initial PR for RAT License Check

2018-04-04 Thread GitBox
eric-haibin-lin commented on a change in pull request #10390: [WIP] Initial PR for RAT License Check URL: https://github.com/apache/incubator-mxnet/pull/10390#discussion_r179229299 ## File path: tests/nightly/apache_rat_license_check/.rat-excludes ## @@ -0,0 +1,54 @@ +.*x

[GitHub] eric-haibin-lin commented on a change in pull request #10390: [WIP] Initial PR for RAT License Check

2018-04-04 Thread GitBox
eric-haibin-lin commented on a change in pull request #10390: [WIP] Initial PR for RAT License Check URL: https://github.com/apache/incubator-mxnet/pull/10390#discussion_r179229683 ## File path: tests/nightly/apache_rat_license_check/.rat-excludes ## @@ -0,0 +1,54 @@ +.*x

[GitHub] eric-haibin-lin closed issue #8801: Cannot find custom operator when visualizing network.

2018-04-04 Thread GitBox
eric-haibin-lin closed issue #8801: Cannot find custom operator when visualizing network. URL: https://github.com/apache/incubator-mxnet/issues/8801 This is an automated message from the Apache Git Service. To respond to the

[GitHub] lanking520 commented on issue #8344: Broken link for "MXNet R Reference Manual"?

2018-04-04 Thread GitBox
lanking520 commented on issue #8344: Broken link for "MXNet R Reference Manual"? URL: https://github.com/apache/incubator-mxnet/issues/8344#issuecomment-378691904 @chetkhatri @piiswrong Hi Eric and Chetan, do we have any way to recover the old R manual? Reference: https://github.c

[GitHub] eric-haibin-lin commented on issue #10310: MemoryError on linear classification with 400million dimension feature input

2018-04-04 Thread GitBox
eric-haibin-lin commented on issue #10310: MemoryError on linear classification with 400million dimension feature input URL: https://github.com/apache/incubator-mxnet/issues/10310#issuecomment-378692376 Hi @zhanglistar Some notes on architecture of MXNet: http://mxnet.incubator.apache

[GitHub] altosaar opened a new issue #10402: Can't press down arrow in search results dropdown on https://mxnet.incubator.apache.org/

2018-04-04 Thread GitBox
altosaar opened a new issue #10402: Can't press down arrow in search results dropdown on https://mxnet.incubator.apache.org/ URL: https://github.com/apache/incubator-mxnet/issues/10402 It would improve UX to be able to press the down arrow in the dropdown that appears when we search https:

[GitHub] reminisce commented on a change in pull request #10392: [MXNET-266] Fix cudnn_conv and cudnn_deconv deadlock

2018-04-04 Thread GitBox
reminisce commented on a change in pull request #10392: [MXNET-266] Fix cudnn_conv and cudnn_deconv deadlock URL: https://github.com/apache/incubator-mxnet/pull/10392#discussion_r179234819 ## File path: src/operator/nn/cudnn/cudnn_convolution-inl.h ## @@ -512,18 +511,15 @@

[GitHub] haojin2 commented on a change in pull request #10371: [MXNET-263] [WIP] Support for dot(dns, csr) = dns and dot(dns, csr.T) = dns on GPU

2018-04-04 Thread GitBox
haojin2 commented on a change in pull request #10371: [MXNET-263] [WIP] Support for dot(dns, csr) = dns and dot(dns, csr.T) = dns on GPU URL: https://github.com/apache/incubator-mxnet/pull/10371#discussion_r179234938 ## File path: src/operator/tensor/dot-inl.cuh ## @@ -442

[incubator-mxnet] branch master updated: [MXNET-116] Optimized functions with batch input (#10343)

2018-04-04 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 aa3f065 [MXNET-116] Optimized function

[GitHub] nswamy closed pull request #10343: [MXNET-116] Optimized functions with batch input

2018-04-04 Thread GitBox
nswamy closed pull request #10343: [MXNET-116] Optimized functions with batch input URL: https://github.com/apache/incubator-mxnet/pull/10343 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 thi

[GitHub] cjolivier01 commented on issue #10375: [MXNET-187] GPU fake shuffle functions for Fermi architecture

2018-04-04 Thread GitBox
cjolivier01 commented on issue #10375: [MXNET-187] GPU fake shuffle functions for Fermi architecture URL: https://github.com/apache/incubator-mxnet/pull/10375#issuecomment-378697030 compile error msg On Wed, Apr 4, 2018 at 5:18 AM Marco de Abreu wrote: > What do you think

[GitHub] haojin2 commented on a change in pull request #10371: [MXNET-263] [WIP] Support for dot(dns, csr) = dns and dot(dns, csr.T) = dns on GPU

2018-04-04 Thread GitBox
haojin2 commented on a change in pull request #10371: [MXNET-263] [WIP] Support for dot(dns, csr) = dns and dot(dns, csr.T) = dns on GPU URL: https://github.com/apache/incubator-mxnet/pull/10371#discussion_r179237417 ## File path: src/operator/tensor/dot.cu ## @@ -23,10 +2

[GitHub] haojin2 commented on a change in pull request #10371: [MXNET-263] [WIP] Support for dot(dns, csr) = dns and dot(dns, csr.T) = dns on GPU

2018-04-04 Thread GitBox
haojin2 commented on a change in pull request #10371: [MXNET-263] [WIP] Support for dot(dns, csr) = dns and dot(dns, csr.T) = dns on GPU URL: https://github.com/apache/incubator-mxnet/pull/10371#discussion_r179237403 ## File path: src/operator/tensor/dot.cu ## @@ -23,10 +2

[GitHub] anirudh2290 commented on issue #10400: Add support for cast storage on same stypes

2018-04-04 Thread GitBox
anirudh2290 commented on issue #10400: Add support for cast storage on same stypes URL: https://github.com/apache/incubator-mxnet/pull/10400#issuecomment-378698573 @eric-haibin-lin Addressed. This is an automated message from

[GitHub] anirudhacharya commented on issue #9971: onnx-mxnet pip package prevents gpu use

2018-04-04 Thread GitBox
anirudhacharya commented on issue #9971: onnx-mxnet pip package prevents gpu use URL: https://github.com/apache/incubator-mxnet/issues/9971#issuecomment-378699596 The onnx import functionality has now been ported into MXNet repo itself under contrib and it does not prevent GPU use. There a

[GitHub] eric-haibin-lin commented on issue #10388: [MXNET-265] Update optimizer doc to clarify wd behaviors

2018-04-04 Thread GitBox
eric-haibin-lin commented on issue #10388: [MXNET-265] Update optimizer doc to clarify wd behaviors URL: https://github.com/apache/incubator-mxnet/pull/10388#issuecomment-378700149 @ZiyueHuang @szhengac @sxjscience could you help review?

[GitHub] piiswrong commented on issue #10392: [MXNET-266] Fix cudnn_conv and cudnn_deconv deadlock

2018-04-04 Thread GitBox
piiswrong commented on issue #10392: [MXNET-266] Fix cudnn_conv and cudnn_deconv deadlock URL: https://github.com/apache/incubator-mxnet/pull/10392#issuecomment-378700289 Please fix indentation and we can merge This is an aut

[GitHub] haojin2 commented on a change in pull request #10208: [MXNET-117] Sparse operator broadcast_mul/div(csr, dense) = csr

2018-04-04 Thread GitBox
haojin2 commented on a change in pull request #10208: [MXNET-117] Sparse operator broadcast_mul/div(csr, dense) = csr URL: https://github.com/apache/incubator-mxnet/pull/10208#discussion_r179242273 ## File path: python/mxnet/ndarray/sparse.py ## @@ -1159,6 +1186,322 @@ def

[GitHub] haojin2 commented on a change in pull request #10208: [MXNET-117] Sparse operator broadcast_mul/div(csr, dense) = csr

2018-04-04 Thread GitBox
haojin2 commented on a change in pull request #10208: [MXNET-117] Sparse operator broadcast_mul/div(csr, dense) = csr URL: https://github.com/apache/incubator-mxnet/pull/10208#discussion_r179242333 ## File path: python/mxnet/ndarray/sparse.py ## @@ -1159,6 +1185,293 @@ def

[GitHub] reminisce commented on issue #10392: [MXNET-266] Fix cudnn_conv and cudnn_deconv deadlock

2018-04-04 Thread GitBox
reminisce commented on issue #10392: [MXNET-266] Fix cudnn_conv and cudnn_deconv deadlock URL: https://github.com/apache/incubator-mxnet/pull/10392#issuecomment-378703469 Indentation fixed. Let's wait till the CI passes. This

[GitHub] mbaijal commented on a change in pull request #10390: [WIP] Initial PR for RAT License Check

2018-04-04 Thread GitBox
mbaijal commented on a change in pull request #10390: [WIP] Initial PR for RAT License Check URL: https://github.com/apache/incubator-mxnet/pull/10390#discussion_r179245499 ## File path: tests/nightly/apache_rat_license_check/README.md ## @@ -0,0 +1,36 @@ +# Apache RAT Li

[GitHub] piiswrong closed pull request #10268: Update 3rdparty/dmlc-core

2018-04-04 Thread GitBox
piiswrong closed pull request #10268: Update 3rdparty/dmlc-core URL: https://github.com/apache/incubator-mxnet/pull/10268 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 r

[GitHub] szhengac commented on a change in pull request #10388: [MXNET-265] Update optimizer doc to clarify wd behaviors

2018-04-04 Thread GitBox
szhengac commented on a change in pull request #10388: [MXNET-265] Update optimizer doc to clarify wd behaviors URL: https://github.com/apache/incubator-mxnet/pull/10388#discussion_r179251532 ## File path: python/mxnet/optimizer.py ## @@ -1210,6 +1210,15 @@ class AdaDelta(

[GitHub] szhengac commented on a change in pull request #10388: [MXNET-265] Update optimizer doc to clarify wd behaviors

2018-04-04 Thread GitBox
szhengac commented on a change in pull request #10388: [MXNET-265] Update optimizer doc to clarify wd behaviors URL: https://github.com/apache/incubator-mxnet/pull/10388#discussion_r179251962 ## File path: python/mxnet/optimizer.py ## @@ -1076,6 +1076,13 @@ class AdaGrad(O

[GitHub] szhengac commented on a change in pull request #10388: [MXNET-265] Update optimizer doc to clarify wd behaviors

2018-04-04 Thread GitBox
szhengac commented on a change in pull request #10388: [MXNET-265] Update optimizer doc to clarify wd behaviors URL: https://github.com/apache/incubator-mxnet/pull/10388#discussion_r179251962 ## File path: python/mxnet/optimizer.py ## @@ -1076,6 +1076,13 @@ class AdaGrad(O

[GitHub] szhengac commented on a change in pull request #10388: [MXNET-265] Update optimizer doc to clarify wd behaviors

2018-04-04 Thread GitBox
szhengac commented on a change in pull request #10388: [MXNET-265] Update optimizer doc to clarify wd behaviors URL: https://github.com/apache/incubator-mxnet/pull/10388#discussion_r179251532 ## File path: python/mxnet/optimizer.py ## @@ -1210,6 +1210,15 @@ class AdaDelta(

[GitHub] szhengac commented on issue #10388: [MXNET-265] Update optimizer doc to clarify wd behaviors

2018-04-04 Thread GitBox
szhengac commented on issue #10388: [MXNET-265] Update optimizer doc to clarify wd behaviors URL: https://github.com/apache/incubator-mxnet/pull/10388#issuecomment-378714141 I found the following line rescales the gradient outside the clipping function. This is inconsistent with other opti

[GitHub] anirudh2290 opened a new issue #10403: Flaky test - ONNX

2018-04-04 Thread GitBox
anirudh2290 opened a new issue #10403: Flaky test - ONNX URL: https://github.com/apache/incubator-mxnet/issues/10403 ``` @contextmanager def _error_catcher(self): """ Catch low-level python exceptions, instead re-raising urllib3

[GitHub] jesterhazy opened a new pull request #10404: fix nproc arg

2018-04-04 Thread GitBox
jesterhazy opened a new pull request #10404: fix nproc arg URL: https://github.com/apache/incubator-mxnet/pull/10404 ## Description ## fix nproc arg to make current version causes error: `./prepare_mkldnn.sh: line 105: NUM_PROC: command not found` ## Checklist

[GitHub] rahul003 commented on issue #10232: [MXNET-136] Enabling USE_DIST_KVSTORE flag for CI

2018-04-04 Thread GitBox
rahul003 commented on issue #10232: [MXNET-136] Enabling USE_DIST_KVSTORE flag for CI URL: https://github.com/apache/incubator-mxnet/pull/10232#issuecomment-378726983 @marcoabreu This is ready to be merged! This is an automat

[GitHub] snflake commented on a change in pull request #7393: add depthwise convolution's gpu version optimization

2018-04-04 Thread GitBox
snflake commented on a change in pull request #7393: add depthwise convolution's gpu version optimization URL: https://github.com/apache/incubator-mxnet/pull/7393#discussion_r179268166 ## File path: src/operator/convolution.cu ## @@ -45,6 +47,18 @@ Operator* CreateOp(Convo

[GitHub] mbaijal commented on a change in pull request #10390: [WIP] Initial PR for RAT License Check

2018-04-04 Thread GitBox
mbaijal commented on a change in pull request #10390: [WIP] Initial PR for RAT License Check URL: https://github.com/apache/incubator-mxnet/pull/10390#discussion_r179268483 ## File path: tests/nightly/apache_rat_license_check/.rat-excludes ## @@ -0,0 +1,54 @@ +.*xml +\..*

[GitHub] mbaijal commented on a change in pull request #10390: [WIP] Initial PR for RAT License Check

2018-04-04 Thread GitBox
mbaijal commented on a change in pull request #10390: [WIP] Initial PR for RAT License Check URL: https://github.com/apache/incubator-mxnet/pull/10390#discussion_r179268703 ## File path: tests/nightly/apache_rat_license_check/.rat-excludes ## @@ -0,0 +1,54 @@ +.*xml +\..*

[GitHub] eric-haibin-lin opened a new pull request #10405: [MXNET-176] [WIP] Support axis=None for sum operator

2018-04-04 Thread GitBox
eric-haibin-lin opened a new pull request #10405: [MXNET-176] [WIP] Support axis=None for sum operator URL: https://github.com/apache/incubator-mxnet/pull/10405 ## Description ## (Brief description on what this PR is about) ## Checklist ## ### Essentials ### Please feel free

[GitHub] eric-haibin-lin commented on issue #10041: Reduce operators do not support axis=None

2018-04-04 Thread GitBox
eric-haibin-lin commented on issue #10041: Reduce operators do not support axis=None URL: https://github.com/apache/incubator-mxnet/issues/10041#issuecomment-378744083 https://github.com/apache/incubator-mxnet/pull/10405 Th

  1   2   >