[GitHub] Davdi commented on issue #13526: distributed training van.cc Check failed

2018-12-05 Thread GitBox
Davdi commented on issue #13526: distributed training van.cc Check failed URL: https://github.com/apache/incubator-mxnet/issues/13526#issuecomment-10347 > Are you running in containers? > You said "the env is 1 ps and 1 worker and should i follow the instruction on all the 2

[GitHub] jlcontreras opened a new pull request #13546: Disable flaky test: test_op_roi_align

2018-12-05 Thread GitBox
jlcontreras opened a new pull request #13546: Disable flaky test: test_op_roi_align URL: https://github.com/apache/incubator-mxnet/pull/13546 Disabling the test, as it has shown flaky behaviour https://github.com/apache/incubator-mxnet/issues/11064#issuecomment-22079

[GitHub] Joke09 commented on issue #7899: Need help: numpy array to mxnet ndarray is too slow.

2018-12-05 Thread GitBox
Joke09 commented on issue #7899: Need help: numpy array to mxnet ndarray is too slow. URL: https://github.com/apache/incubator-mxnet/issues/7899#issuecomment-15247 For inference, I have the same problem. The client send jpg to server, then the server use cv2 to do resize. When put

[GitHub] Joke09 opened a new issue #13545: For inference, I have the same problem. The client send jpg to server, then the server use cv2 to do resize. When put the image data into the mx.nd.array, it

2018-12-05 Thread GitBox
Joke09 opened a new issue #13545: For inference, I have the same problem. The client send jpg to server, then the server use cv2 to do resize. When put the image data into the mx.nd.array, it's very slow. And the Utilization of GPU is low too. How to solve it? Thank you! URL:

[GitHub] TaoLv commented on issue #13362: Add NHWC layout support to Pooling (cuDNN only)

2018-12-05 Thread GitBox
TaoLv commented on issue #13362: Add NHWC layout support to Pooling (cuDNN only) URL: https://github.com/apache/incubator-mxnet/pull/13362#issuecomment-444394597 Do we have any other operator exposing layout information to front user? Does NDArray know the layout information of itself?

[GitHub] TaoLv commented on a change in pull request #12980: Add reshape op supported by MKL-DNN

2018-12-05 Thread GitBox
TaoLv commented on a change in pull request #12980: Add reshape op supported by MKL-DNN URL: https://github.com/apache/incubator-mxnet/pull/12980#discussion_r238973876 ## File path: src/operator/tensor/matrix_op.cc ## @@ -914,7 +972,7 @@ NNVM_REGISTER_OP(depth_to_space)

[GitHub] jlcontreras commented on issue #11064: Flaky test: test_operator.test_op_roi_align

2018-12-05 Thread GitBox
jlcontreras commented on issue #11064: Flaky test: test_operator.test_op_roi_align URL: https://github.com/apache/incubator-mxnet/issues/11064#issuecomment-22079 Seems to still happen: ``` == FAIL:

[GitHub] huangzhiyuan commented on issue #12980: Add reshape op supported by MKL-DNN

2018-12-05 Thread GitBox
huangzhiyuan commented on issue #12980: Add reshape op supported by MKL-DNN URL: https://github.com/apache/incubator-mxnet/pull/12980#issuecomment-444397788 @azai91 @mseth10 could you help review the PR? This has been out for review for a long long time...

[GitHub] TaoLv commented on a change in pull request #12980: Add reshape op supported by MKL-DNN

2018-12-05 Thread GitBox
TaoLv commented on a change in pull request #12980: Add reshape op supported by MKL-DNN URL: https://github.com/apache/incubator-mxnet/pull/12980#discussion_r238972572 ## File path: src/operator/tensor/matrix_op.cc ## @@ -210,24 +272,18 @@ static void FlattenEx(const

[GitHub] Davdi edited a comment on issue #13526: distributed training van.cc Check failed

2018-12-05 Thread GitBox
Davdi edited a comment on issue #13526: distributed training van.cc Check failed URL: https://github.com/apache/incubator-mxnet/issues/13526#issuecomment-10347 > Are you running in containers? > You said "the env is 1 ps and 1 worker and should i follow the instruction on all the

[incubator-mxnet-site] branch asf-site updated: Bump the publish timestamp.

2018-12-05 Thread zhasheng
This is an automated email from the ASF dual-hosted git repository. zhasheng pushed a commit to branch asf-site in repository https://gitbox.apache.org/repos/asf/incubator-mxnet-site.git The following commit(s) were added to refs/heads/asf-site by this push: new 22629fe Bump the publish

[GitHub] marcoabreu commented on a change in pull request #13531: Use MXNET_HOME in a tempdir in windows to prevent access denied due t…

2018-12-05 Thread GitBox
marcoabreu commented on a change in pull request #13531: Use MXNET_HOME in a tempdir in windows to prevent access denied due t… URL: https://github.com/apache/incubator-mxnet/pull/13531#discussion_r239024126 ## File path: ci/windows/test_py2_cpu.ps1 ## @@ -16,11 +16,18 @@

[GitHub] juliusshufan closed issue #13524: MXNET can't build from source on MKLDNN enabled

2018-12-05 Thread GitBox
juliusshufan closed issue #13524: MXNET can't build from source on MKLDNN enabled URL: https://github.com/apache/incubator-mxnet/issues/13524 This is an automated message from the Apache Git Service. To respond to the

[GitHub] juliusshufan commented on issue #13524: MXNET can't build from source on MKLDNN enabled

2018-12-05 Thread GitBox
juliusshufan commented on issue #13524: MXNET can't build from source on MKLDNN enabled URL: https://github.com/apache/incubator-mxnet/issues/13524#issuecomment-77218 @azai91 Thanks for prompt fix, the compilation is okay now. I close the issue accordingly.

[GitHub] larroy opened a new pull request #13547: Fix Jetson compilation

2018-12-05 Thread GitBox
larroy opened a new pull request #13547: Fix Jetson compilation URL: https://github.com/apache/incubator-mxnet/pull/13547 ## Description ## Port #13532 to the release branch ## Checklist ## ### Essentials ### Please feel free to remove inapplicable items for your PR. - [

[GitHub] eric-haibin-lin opened a new issue #11064: Flaky test: test_operator.test_op_roi_align

2018-12-05 Thread GitBox
eric-haibin-lin opened a new issue #11064: Flaky test: test_operator.test_op_roi_align URL: https://github.com/apache/incubator-mxnet/issues/11064 ``` == FAIL: test_operator.test_op_roi_align

[GitHub] larroy commented on a change in pull request #13532: [MXNET-43] Fix Jetson compilation

2018-12-05 Thread GitBox
larroy commented on a change in pull request #13532: [MXNET-43] Fix Jetson compilation URL: https://github.com/apache/incubator-mxnet/pull/13532#discussion_r239069506 ## File path: ci/docker/Dockerfile.build.jetson ## @@ -77,6 +77,7 @@ RUN

[GitHub] larroy commented on a change in pull request #13532: [MXNET-43] Fix Jetson compilation

2018-12-05 Thread GitBox
larroy commented on a change in pull request #13532: [MXNET-43] Fix Jetson compilation URL: https://github.com/apache/incubator-mxnet/pull/13532#discussion_r239069506 ## File path: ci/docker/Dockerfile.build.jetson ## @@ -77,6 +77,7 @@ RUN

[GitHub] gigasquid commented on issue #13523: #13441 [Clojure] Add Spec Validations for the Random namespace

2018-12-05 Thread GitBox
gigasquid commented on issue #13523: #13441 [Clojure] Add Spec Validations for the Random namespace URL: https://github.com/apache/incubator-mxnet/pull/13523#issuecomment-444505339 It's definitely fine to change the example to make it work correctly :) I'm ok with either changing

[GitHub] larroy commented on a change in pull request #13532: [MXNET-43] Fix Jetson compilation

2018-12-05 Thread GitBox
larroy commented on a change in pull request #13532: [MXNET-43] Fix Jetson compilation URL: https://github.com/apache/incubator-mxnet/pull/13532#discussion_r239098417 ## File path: make/crosscompile.jetson.mk ## @@ -57,10 +57,10 @@ DEBUG = 0 USE_SIGNAL_HANDLER = 1 #

[GitHub] mdespriee commented on issue #13039: [MXNET-918] Random module

2018-12-05 Thread GitBox
mdespriee commented on issue #13039: [MXNET-918] Random module URL: https://github.com/apache/incubator-mxnet/pull/13039#issuecomment-444523531 I'm a bit confused with these failing tests, as I can't reproduce the failure locally. I also don't get why it is environment dependant. I'll have

[GitHub] lebeg commented on a change in pull request #13532: [MXNET-43] Fix Jetson compilation

2018-12-05 Thread GitBox
lebeg commented on a change in pull request #13532: [MXNET-43] Fix Jetson compilation URL: https://github.com/apache/incubator-mxnet/pull/13532#discussion_r239085211 ## File path: make/crosscompile.jetson.mk ## @@ -57,10 +57,10 @@ DEBUG = 0 USE_SIGNAL_HANDLER = 1 #

[GitHub] lebeg commented on a change in pull request #13532: [MXNET-43] Fix Jetson compilation

2018-12-05 Thread GitBox
lebeg commented on a change in pull request #13532: [MXNET-43] Fix Jetson compilation URL: https://github.com/apache/incubator-mxnet/pull/13532#discussion_r239099950 ## File path: make/crosscompile.jetson.mk ## @@ -57,10 +57,10 @@ DEBUG = 0 USE_SIGNAL_HANDLER = 1 #

[GitHub] larroy commented on a change in pull request #13532: [MXNET-43] Fix Jetson compilation

2018-12-05 Thread GitBox
larroy commented on a change in pull request #13532: [MXNET-43] Fix Jetson compilation URL: https://github.com/apache/incubator-mxnet/pull/13532#discussion_r239135041 ## File path: make/crosscompile.jetson.mk ## @@ -57,10 +57,10 @@ DEBUG = 0 USE_SIGNAL_HANDLER = 1 #

[GitHub] marcoabreu commented on a change in pull request #13532: [MXNET-43] Fix Jetson compilation

2018-12-05 Thread GitBox
marcoabreu commented on a change in pull request #13532: [MXNET-43] Fix Jetson compilation URL: https://github.com/apache/incubator-mxnet/pull/13532#discussion_r239135331 ## File path: make/crosscompile.jetson.mk ## @@ -57,10 +57,10 @@ DEBUG = 0 USE_SIGNAL_HANDLER = 1

[GitHub] lebeg commented on issue #11707: test_nccl.test_nccl_pushpull was disabled

2018-12-05 Thread GitBox
lebeg commented on issue #11707: test_nccl.test_nccl_pushpull was disabled URL: https://github.com/apache/incubator-mxnet/issues/11707#issuecomment-444560236 @perdasilva https://github.com/apache/incubator-mxnet/pull/12477 wasn't merged, so it should stay open.

[incubator-mxnet] branch v1.4.x updated: Fix Jetson compilation (#13547)

2018-12-05 Thread marcoabreu
This is an automated email from the ASF dual-hosted git repository. marcoabreu pushed a commit to branch v1.4.x in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git The following commit(s) were added to refs/heads/v1.4.x by this push: new d793feb Fix Jetson compilation

[GitHub] marcoabreu closed pull request #13547: [MXNET-43] Fix Jetson compilation (v1.4.x branch)

2018-12-05 Thread GitBox
marcoabreu closed pull request #13547: [MXNET-43] Fix Jetson compilation (v1.4.x branch) URL: https://github.com/apache/incubator-mxnet/pull/13547 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance:

[GitHub] larroy commented on a change in pull request #13532: [MXNET-43] Fix Jetson compilation

2018-12-05 Thread GitBox
larroy commented on a change in pull request #13532: [MXNET-43] Fix Jetson compilation URL: https://github.com/apache/incubator-mxnet/pull/13532#discussion_r239134775 ## File path: make/crosscompile.jetson.mk ## @@ -57,10 +57,10 @@ DEBUG = 0 USE_SIGNAL_HANDLER = 1 #

[GitHub] kohr-h opened a new issue #13548: [Feature Request / Proposal] Pixel shuffle layer

2018-12-05 Thread GitBox
kohr-h opened a new issue #13548: [Feature Request / Proposal] Pixel shuffle layer URL: https://github.com/apache/incubator-mxnet/issues/13548 Upsampling based on pixel shuffling has been proposed in the paper [Real-Time Single Image and Video Super-Resolution Using an Efficient Sub-Pixel

[GitHub] larroy commented on a change in pull request #13532: [MXNET-43] Fix Jetson compilation

2018-12-05 Thread GitBox
larroy commented on a change in pull request #13532: [MXNET-43] Fix Jetson compilation URL: https://github.com/apache/incubator-mxnet/pull/13532#discussion_r239135041 ## File path: make/crosscompile.jetson.mk ## @@ -57,10 +57,10 @@ DEBUG = 0 USE_SIGNAL_HANDLER = 1 #

[GitHub] lebeg commented on a change in pull request #13532: [MXNET-43] Fix Jetson compilation

2018-12-05 Thread GitBox
lebeg commented on a change in pull request #13532: [MXNET-43] Fix Jetson compilation URL: https://github.com/apache/incubator-mxnet/pull/13532#discussion_r239153481 ## File path: make/crosscompile.jetson.mk ## @@ -57,10 +57,10 @@ DEBUG = 0 USE_SIGNAL_HANDLER = 1 #

[GitHub] marcoabreu commented on a change in pull request #13532: [MXNET-43] Fix Jetson compilation

2018-12-05 Thread GitBox
marcoabreu commented on a change in pull request #13532: [MXNET-43] Fix Jetson compilation URL: https://github.com/apache/incubator-mxnet/pull/13532#discussion_r239117119 ## File path: make/crosscompile.jetson.mk ## @@ -57,10 +57,10 @@ DEBUG = 0 USE_SIGNAL_HANDLER = 1

[GitHub] relaxli00 commented on issue #13482: [Feature Request] Release gpu memory by API.

2018-12-05 Thread GitBox
relaxli00 commented on issue #13482: [Feature Request] Release gpu memory by API. URL: https://github.com/apache/incubator-mxnet/issues/13482#issuecomment-444545427 +1, yes, this is a function needed badly This is an

[GitHub] zhreshold commented on issue #13537: Fix #13521

2018-12-05 Thread GitBox
zhreshold commented on issue #13537: Fix #13521 URL: https://github.com/apache/incubator-mxnet/pull/13537#issuecomment-444624026 @eric-haibin-lin Could you help review and merge? Thx This is an automated message from the

[GitHub] samskalicky edited a comment on issue #13438: libc getenv is not threadsafe

2018-12-05 Thread GitBox
samskalicky edited a comment on issue #13438: libc getenv is not threadsafe URL: https://github.com/apache/incubator-mxnet/issues/13438#issuecomment-444604976 @larroy We were already using locks inside the handlers. setenv uses a lock in libc around modifying the environment. This doesnt

[incubator-mxnet] branch v1.4.x updated: Fix #13521 (#13537)

2018-12-05 Thread zhreshold
This is an automated email from the ASF dual-hosted git repository. zhreshold pushed a commit to branch v1.4.x in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git The following commit(s) were added to refs/heads/v1.4.x by this push: new d63fdfd Fix #13521 (#13537)

[GitHub] leleamol commented on a change in pull request #13294: [MXNET-1083] Add the example to demonstrate the inference workflow using C++ API

2018-12-05 Thread GitBox
leleamol commented on a change in pull request #13294: [MXNET-1083] Add the example to demonstrate the inference workflow using C++ API URL: https://github.com/apache/incubator-mxnet/pull/13294#discussion_r239274909 ## File path:

[GitHub] leleamol commented on a change in pull request #13294: [MXNET-1083] Add the example to demonstrate the inference workflow using C++ API

2018-12-05 Thread GitBox
leleamol commented on a change in pull request #13294: [MXNET-1083] Add the example to demonstrate the inference workflow using C++ API URL: https://github.com/apache/incubator-mxnet/pull/13294#discussion_r239274844 ## File path:

[GitHub] leleamol commented on a change in pull request #13294: [MXNET-1083] Add the example to demonstrate the inference workflow using C++ API

2018-12-05 Thread GitBox
leleamol commented on a change in pull request #13294: [MXNET-1083] Add the example to demonstrate the inference workflow using C++ API URL: https://github.com/apache/incubator-mxnet/pull/13294#discussion_r239274873 ## File path:

[GitHub] leleamol commented on a change in pull request #13294: [MXNET-1083] Add the example to demonstrate the inference workflow using C++ API

2018-12-05 Thread GitBox
leleamol commented on a change in pull request #13294: [MXNET-1083] Add the example to demonstrate the inference workflow using C++ API URL: https://github.com/apache/incubator-mxnet/pull/13294#discussion_r239275057 ## File path:

[GitHub] vandanavk opened a new pull request #13553: [WIP] ONNX test code cleanup

2018-12-05 Thread GitBox
vandanavk opened a new pull request #13553: [WIP] ONNX test code cleanup URL: https://github.com/apache/incubator-mxnet/pull/13553 ## Description ## (Brief description on what this PR is about) ## Checklist ## ### Essentials ### Please feel free to remove inapplicable items

[incubator-mxnet-site] branch asf-site updated: Bump the publish timestamp.

2018-12-05 Thread zhasheng
This is an automated email from the ASF dual-hosted git repository. zhasheng pushed a commit to branch asf-site in repository https://gitbox.apache.org/repos/asf/incubator-mxnet-site.git The following commit(s) were added to refs/heads/asf-site by this push: new cd215ef Bump the publish

[GitHub] gigasquid commented on issue #13523: #13441 [Clojure] Add Spec Validations for the Random namespace

2018-12-05 Thread GitBox
gigasquid commented on issue #13523: #13441 [Clojure] Add Spec Validations for the Random namespace URL: https://github.com/apache/incubator-mxnet/pull/13523#issuecomment-444709860 Thanks @hellonico for your work in improving this. I ran through the examples and everything looks good. It

[GitHub] kohr-h commented on a change in pull request #13094: Simplifications and some fun stuff for the MNIST Gluon tutorial

2018-12-05 Thread GitBox
kohr-h commented on a change in pull request #13094: Simplifications and some fun stuff for the MNIST Gluon tutorial URL: https://github.com/apache/incubator-mxnet/pull/13094#discussion_r239217521 ## File path: docs/tutorials/gluon/mnist.md ## @@ -1,333 +1,443 @@ -#

[GitHub] DickJC123 commented on issue #13362: Add NHWC layout support to Pooling (cuDNN only)

2018-12-05 Thread GitBox
DickJC123 commented on issue #13362: Add NHWC layout support to Pooling (cuDNN only) URL: https://github.com/apache/incubator-mxnet/pull/13362#issuecomment-444698761 @apeforest Re: bool return of Forward(), Backward(), etc. As you know, MXNet has been moving to a more imperative

[GitHub] kohr-h commented on issue #13094: Simplifications and some fun stuff for the MNIST Gluon tutorial

2018-12-05 Thread GitBox
kohr-h commented on issue #13094: Simplifications and some fun stuff for the MNIST Gluon tutorial URL: https://github.com/apache/incubator-mxnet/pull/13094#issuecomment-444630151 Okay, now we're in the right ballpark with the results. Thanks @ThomasDelteil for the catch.

[GitHub] mdespriee commented on issue #13039: [MXNET-918] Random module

2018-12-05 Thread GitBox
mdespriee commented on issue #13039: [MXNET-918] Random module URL: https://github.com/apache/incubator-mxnet/pull/13039#issuecomment-444653841 > That's strange. It looks like the code-gen doesn't work somehow on Linux Actually the code gen fails this way when the

[GitHub] apeforest commented on issue #13362: Add NHWC layout support to Pooling (cuDNN only)

2018-12-05 Thread GitBox
apeforest commented on issue #13362: Add NHWC layout support to Pooling (cuDNN only) URL: https://github.com/apache/incubator-mxnet/pull/13362#issuecomment-444663652 @ptrendx It would be great if you can add in the PR description why we need this option. Thanks!

[GitHub] Vikas89 commented on issue #13526: distributed training van.cc Check failed

2018-12-05 Thread GitBox
Vikas89 commented on issue #13526: distributed training van.cc Check failed URL: https://github.com/apache/incubator-mxnet/issues/13526#issuecomment-444668438 As I see there are 3 different issues here: ``` File "/usr/local/lib/python3.6/dist-packages/mxnet/base.py", line 252, in

[GitHub] ChaiBapchya commented on a change in pull request #13543: Chi_square_check for discrete distribution fix

2018-12-05 Thread GitBox
ChaiBapchya commented on a change in pull request #13543: Chi_square_check for discrete distribution fix URL: https://github.com/apache/incubator-mxnet/pull/13543#discussion_r239260946 ## File path: python/mxnet/test_utils.py ## @@ -1911,12 +1911,15 @@ def

[GitHub] leleamol commented on a change in pull request #13294: [MXNET-1083] Add the example to demonstrate the inference workflow using C++ API

2018-12-05 Thread GitBox
leleamol commented on a change in pull request #13294: [MXNET-1083] Add the example to demonstrate the inference workflow using C++ API URL: https://github.com/apache/incubator-mxnet/pull/13294#discussion_r239276098 ## File path:

[GitHub] leleamol commented on a change in pull request #13294: [MXNET-1083] Add the example to demonstrate the inference workflow using C++ API

2018-12-05 Thread GitBox
leleamol commented on a change in pull request #13294: [MXNET-1083] Add the example to demonstrate the inference workflow using C++ API URL: https://github.com/apache/incubator-mxnet/pull/13294#discussion_r239275908 ## File path:

[GitHub] ptrendx commented on issue #13362: Add NHWC layout support to Pooling (cuDNN only)

2018-12-05 Thread GitBox
ptrendx commented on issue #13362: Add NHWC layout support to Pooling (cuDNN only) URL: https://github.com/apache/incubator-mxnet/pull/13362#issuecomment-444689260 @Vikas89 I took a stab at modifying PR description to include motivation and testing that we did.

[GitHub] leleamol commented on a change in pull request #13294: [MXNET-1083] Add the example to demonstrate the inference workflow using C++ API

2018-12-05 Thread GitBox
leleamol commented on a change in pull request #13294: [MXNET-1083] Add the example to demonstrate the inference workflow using C++ API URL: https://github.com/apache/incubator-mxnet/pull/13294#discussion_r239275736 ## File path:

[GitHub] leleamol commented on a change in pull request #13294: [MXNET-1083] Add the example to demonstrate the inference workflow using C++ API

2018-12-05 Thread GitBox
leleamol commented on a change in pull request #13294: [MXNET-1083] Add the example to demonstrate the inference workflow using C++ API URL: https://github.com/apache/incubator-mxnet/pull/13294#discussion_r239275884 ## File path:

[GitHub] leleamol commented on a change in pull request #13294: [MXNET-1083] Add the example to demonstrate the inference workflow using C++ API

2018-12-05 Thread GitBox
leleamol commented on a change in pull request #13294: [MXNET-1083] Add the example to demonstrate the inference workflow using C++ API URL: https://github.com/apache/incubator-mxnet/pull/13294#discussion_r239276048 ## File path:

[GitHub] leleamol commented on a change in pull request #13294: [MXNET-1083] Add the example to demonstrate the inference workflow using C++ API

2018-12-05 Thread GitBox
leleamol commented on a change in pull request #13294: [MXNET-1083] Add the example to demonstrate the inference workflow using C++ API URL: https://github.com/apache/incubator-mxnet/pull/13294#discussion_r239276259 ## File path: cpp-package/example/inference/README.md ##

[GitHub] zhreshold commented on issue #13550: Fixes infinite loop using imagedetiter

2018-12-05 Thread GitBox
zhreshold commented on issue #13550: Fixes infinite loop using imagedetiter URL: https://github.com/apache/incubator-mxnet/pull/13550#issuecomment-444700527 LGTM, suggest to cherry-pick into v1.4.x as well. This is an

[incubator-mxnet] branch master updated: Simplifications and some fun stuff for the MNIST Gluon tutorial (#13094)

2018-12-05 Thread thomasdelteil
This is an automated email from the ASF dual-hosted git repository. thomasdelteil 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 8bbac82 Simplifications and

[GitHub] larroy edited a comment on issue #13528: Scala jar contains the scala standard library

2018-12-05 Thread GitBox
larroy edited a comment on issue #13528: Scala jar contains the scala standard library URL: https://github.com/apache/incubator-mxnet/issues/13528#issuecomment-444626117 While I think is ok to provide a fat jar for people trying MXNet in the console as you point out, we should provide a

[GitHub] larroy edited a comment on issue #13528: Scala jar contains the scala standard library

2018-12-05 Thread GitBox
larroy edited a comment on issue #13528: Scala jar contains the scala standard library URL: https://github.com/apache/incubator-mxnet/issues/13528#issuecomment-444626117 While I think is ok to provide a fat jar for people trying MXNet in the console as you point out, we should provide a

[GitHub] larroy edited a comment on issue #13528: Scala jar contains the scala standard library

2018-12-05 Thread GitBox
larroy edited a comment on issue #13528: Scala jar contains the scala standard library URL: https://github.com/apache/incubator-mxnet/issues/13528#issuecomment-444626117 While I think is ok to provide a fat jar for people trying MXNet in the console as you point out, we should provide a

[GitHub] larroy edited a comment on issue #13528: Scala jar contains the scala standard library

2018-12-05 Thread GitBox
larroy edited a comment on issue #13528: Scala jar contains the scala standard library URL: https://github.com/apache/incubator-mxnet/issues/13528#issuecomment-444626117 While I think is ok to provide a fat jar for people trying MXNet in the console as you point out, we should provide a

[GitHub] larroy edited a comment on issue #13528: Scala jar contains the scala standard library

2018-12-05 Thread GitBox
larroy edited a comment on issue #13528: Scala jar contains the scala standard library URL: https://github.com/apache/incubator-mxnet/issues/13528#issuecomment-444626117 While I think is ok to provide a fat jar for people trying MXNet in the console as you point out, we should provide a

[GitHub] aaronmarkham opened a new issue #13552: can't restart CI website test

2018-12-05 Thread GitBox
aaronmarkham opened a new issue #13552: can't restart CI website test URL: https://github.com/apache/incubator-mxnet/issues/13552 ## Description This pipeline test failed because it couldn't reach github. Not only is this a transient issue that we should probably have at least one or

[GitHub] ptrendx commented on a change in pull request #13362: Add NHWC layout support to Pooling (cuDNN only)

2018-12-05 Thread GitBox
ptrendx commented on a change in pull request #13362: Add NHWC layout support to Pooling (cuDNN only) URL: https://github.com/apache/incubator-mxnet/pull/13362#discussion_r239253297 ## File path: python/mxnet/gluon/nn/conv_layers.py ## @@ -673,7 +673,7 @@ def

[GitHub] ptrendx commented on a change in pull request #13362: Add NHWC layout support to Pooling (cuDNN only)

2018-12-05 Thread GitBox
ptrendx commented on a change in pull request #13362: Add NHWC layout support to Pooling (cuDNN only) URL: https://github.com/apache/incubator-mxnet/pull/13362#discussion_r239280157 ## File path: tests/python/gpu/test_operator_gpu.py ## @@ -608,6 +608,52 @@ def

[GitHub] zachgk commented on a change in pull request #13493: [MXNET-1224]: improve scala maven jni build.

2018-12-05 Thread GitBox
zachgk commented on a change in pull request #13493: [MXNET-1224]: improve scala maven jni build. URL: https://github.com/apache/incubator-mxnet/pull/13493#discussion_r239286858 ## File path: scala-package/pom.xml ## @@ -39,6 +39,8 @@ 2.11.8 2.11 +g++

[GitHub] aaronmarkham commented on issue #13544: turn on Sphinx warnings as errors

2018-12-05 Thread GitBox
aaronmarkham commented on issue #13544: turn on Sphinx warnings as errors URL: https://github.com/apache/incubator-mxnet/pull/13544#issuecomment-444708562 This is time sensitive. As it ages, we'll have another round of cleaning up bugs that are introduced to get a clean start.

[GitHub] lanking520 commented on issue #13528: Scala jar contains the scala standard library

2018-12-05 Thread GitBox
lanking520 commented on issue #13528: Scala jar contains the scala standard library URL: https://github.com/apache/incubator-mxnet/issues/13528#issuecomment-444633928 @larroy I think your explanation does make sense in this case. For Scala users essentially this is ok since they should

[GitHub] sandeep-krishnamurthy commented on a change in pull request #13362: Add NHWC layout support to Pooling (cuDNN only)

2018-12-05 Thread GitBox
sandeep-krishnamurthy commented on a change in pull request #13362: Add NHWC layout support to Pooling (cuDNN only) URL: https://github.com/apache/incubator-mxnet/pull/13362#discussion_r239271822 ## File path: src/operator/nn/pooling.cu ## @@ -116,10 +118,12 @@ void

[GitHub] sandeep-krishnamurthy commented on a change in pull request #13362: Add NHWC layout support to Pooling (cuDNN only)

2018-12-05 Thread GitBox
sandeep-krishnamurthy commented on a change in pull request #13362: Add NHWC layout support to Pooling (cuDNN only) URL: https://github.com/apache/incubator-mxnet/pull/13362#discussion_r239272144 ## File path: tests/python/gpu/test_operator_gpu.py ## @@ -608,6 +608,52 @@

[GitHub] sandeep-krishnamurthy commented on a change in pull request #13362: Add NHWC layout support to Pooling (cuDNN only)

2018-12-05 Thread GitBox
sandeep-krishnamurthy commented on a change in pull request #13362: Add NHWC layout support to Pooling (cuDNN only) URL: https://github.com/apache/incubator-mxnet/pull/13362#discussion_r239259258 ## File path: python/mxnet/gluon/nn/conv_layers.py ## @@ -738,12 +741,13 @@

[GitHub] DickJC123 commented on issue #13362: Add NHWC layout support to Pooling (cuDNN only)

2018-12-05 Thread GitBox
DickJC123 commented on issue #13362: Add NHWC layout support to Pooling (cuDNN only) URL: https://github.com/apache/incubator-mxnet/pull/13362#issuecomment-444690325 @TaoLv , providing details and motivation of the PR (partial duplicate of info just added by @ptrendx ): In MXNet,

[GitHub] ptrendx commented on a change in pull request #13362: Add NHWC layout support to Pooling (cuDNN only)

2018-12-05 Thread GitBox
ptrendx commented on a change in pull request #13362: Add NHWC layout support to Pooling (cuDNN only) URL: https://github.com/apache/incubator-mxnet/pull/13362#discussion_r239277149 ## File path: src/operator/nn/cudnn/cudnn_pooling-inl.h ## @@ -165,55 +170,78 @@ class

[GitHub] larroy commented on issue #13528: Scala jar contains the scala standard library

2018-12-05 Thread GitBox
larroy commented on issue #13528: Scala jar contains the scala standard library URL: https://github.com/apache/incubator-mxnet/issues/13528#issuecomment-444626117 While I think is ok to provide a fat jar for people trying MXNet in the console as you point out, we should provide a standard

[incubator-mxnet-site] branch asf-site updated: Bump the publish timestamp.

2018-12-05 Thread zhasheng
This is an automated email from the ASF dual-hosted git repository. zhasheng pushed a commit to branch asf-site in repository https://gitbox.apache.org/repos/asf/incubator-mxnet-site.git The following commit(s) were added to refs/heads/asf-site by this push: new 294ea65 Bump the publish

[GitHub] kohr-h commented on issue #13548: [Feature Request / Proposal] Pixel shuffle layer

2018-12-05 Thread GitBox
kohr-h commented on issue #13548: [Feature Request / Proposal] Pixel shuffle layer URL: https://github.com/apache/incubator-mxnet/issues/13548#issuecomment-444646007 True, it's just reshaping in the end, although not entirely trivial to make work as hybrid block. Regarding placement,

[GitHub] lebeg commented on issue #13364: [MXNET-1225] Always use config.mk in make install instructions

2018-12-05 Thread GitBox
lebeg commented on issue #13364: [MXNET-1225] Always use config.mk in make install instructions URL: https://github.com/apache/incubator-mxnet/pull/13364#issuecomment-444645994 I would like to mention that, unfortunately, the current option for algebra packages in CMake is

[incubator-mxnet] branch master updated: Fix #13521 (#13537)

2018-12-05 Thread zhreshold
This is an automated email from the ASF dual-hosted git repository. zhreshold 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 f6b4665 Fix #13521 (#13537)

[GitHub] zhreshold closed pull request #13537: Fix #13521

2018-12-05 Thread GitBox
zhreshold closed pull request #13537: Fix #13521 URL: https://github.com/apache/incubator-mxnet/pull/13537 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 request (from a

[GitHub] zhreshold closed issue #13521: Gluon DataLoader cannot release the processes in the pool

2018-12-05 Thread GitBox
zhreshold closed issue #13521: Gluon DataLoader cannot release the processes in the pool URL: https://github.com/apache/incubator-mxnet/issues/13521 This is an automated message from the Apache Git Service. To respond to

[GitHub] ptrendx commented on issue #13362: Add NHWC layout support to Pooling (cuDNN only)

2018-12-05 Thread GitBox
ptrendx commented on issue #13362: Add NHWC layout support to Pooling (cuDNN only) URL: https://github.com/apache/incubator-mxnet/pull/13362#issuecomment-444669921 @TaoLv Re MKLDNN: Per title of this PR this is enabling NHWC support only for cuDNN path. I don't know whether MKLDNN

[GitHub] leleamol commented on a change in pull request #13294: [MXNET-1083] Add the example to demonstrate the inference workflow using C++ API

2018-12-05 Thread GitBox
leleamol commented on a change in pull request #13294: [MXNET-1083] Add the example to demonstrate the inference workflow using C++ API URL: https://github.com/apache/incubator-mxnet/pull/13294#discussion_r239274813 ## File path: cpp-package/example/inference/README.md ##

[GitHub] vandanavk commented on issue #13553: [WIP] ONNX test code cleanup

2018-12-05 Thread GitBox
vandanavk commented on issue #13553: [WIP] ONNX test code cleanup URL: https://github.com/apache/incubator-mxnet/pull/13553#issuecomment-444704079 @mxnet-label-bot add [ONNX, pr-work-in-progress] This is an automated message

[GitHub] vandanavk removed a comment on issue #13553: [WIP] ONNX test code cleanup

2018-12-05 Thread GitBox
vandanavk removed a comment on issue #13553: [WIP] ONNX test code cleanup URL: https://github.com/apache/incubator-mxnet/pull/13553#issuecomment-444704079 @mxnet-label-bot add [ONNX, pr-work-in-progress] This is an automated

[GitHub] larroy edited a comment on issue #13528: Scala jar contains the scala standard library

2018-12-05 Thread GitBox
larroy edited a comment on issue #13528: Scala jar contains the scala standard library URL: https://github.com/apache/incubator-mxnet/issues/13528#issuecomment-444626117 While I think is ok to provide a fat jar for people trying MXNet in the console as you point out, we should provide a

[GitHub] yuxihu commented on a change in pull request #13543: Chi_square_check for discrete distribution fix

2018-12-05 Thread GitBox
yuxihu commented on a change in pull request #13543: Chi_square_check for discrete distribution fix URL: https://github.com/apache/incubator-mxnet/pull/13543#discussion_r239248853 ## File path: python/mxnet/test_utils.py ## @@ -1911,12 +1911,15 @@ def

[GitHub] ptrendx commented on a change in pull request #13362: Add NHWC layout support to Pooling (cuDNN only)

2018-12-05 Thread GitBox
ptrendx commented on a change in pull request #13362: Add NHWC layout support to Pooling (cuDNN only) URL: https://github.com/apache/incubator-mxnet/pull/13362#discussion_r239254371 ## File path: src/operator/nn/cudnn/cudnn_pooling-inl.h ## @@ -165,55 +170,78 @@ class

[GitHub] ptrendx commented on a change in pull request #13362: Add NHWC layout support to Pooling (cuDNN only)

2018-12-05 Thread GitBox
ptrendx commented on a change in pull request #13362: Add NHWC layout support to Pooling (cuDNN only) URL: https://github.com/apache/incubator-mxnet/pull/13362#discussion_r239254766 ## File path: src/operator/nn/pooling-inl.h ## @@ -53,6 +53,7 @@ struct PoolingParam :

[GitHub] ptrendx commented on issue #13362: Add NHWC layout support to Pooling (cuDNN only)

2018-12-05 Thread GitBox
ptrendx commented on issue #13362: Add NHWC layout support to Pooling (cuDNN only) URL: https://github.com/apache/incubator-mxnet/pull/13362#issuecomment-444670966 @apeforest @Vikas89 Not sure I understand the question. Are you asking why do we need NHWC support for pooling?

[GitHub] apeforest commented on a change in pull request #13362: Add NHWC layout support to Pooling (cuDNN only)

2018-12-05 Thread GitBox
apeforest commented on a change in pull request #13362: Add NHWC layout support to Pooling (cuDNN only) URL: https://github.com/apache/incubator-mxnet/pull/13362#discussion_r239265544 ## File path: src/operator/nn/cudnn/cudnn_pooling-inl.h ## @@ -165,55 +170,78 @@ class

[GitHub] larroy edited a comment on issue #13528: Scala jar contains the scala standard library

2018-12-05 Thread GitBox
larroy edited a comment on issue #13528: Scala jar contains the scala standard library URL: https://github.com/apache/incubator-mxnet/issues/13528#issuecomment-444626117 While I think is ok to provide a fat jar for people trying MXNet in the console as you point out, we should provide a

[GitHub] larroy edited a comment on issue #13528: Scala jar contains the scala standard library

2018-12-05 Thread GitBox
larroy edited a comment on issue #13528: Scala jar contains the scala standard library URL: https://github.com/apache/incubator-mxnet/issues/13528#issuecomment-444626117 While I think is ok to provide a fat jar for people trying MXNet in the console as you point out, we should provide a

[GitHub] larroy commented on issue #13528: Scala jar contains the scala standard library

2018-12-05 Thread GitBox
larroy commented on issue #13528: Scala jar contains the scala standard library URL: https://github.com/apache/incubator-mxnet/issues/13528#issuecomment-444638931 That's why maven / SBT etc. have the concept of transitive dependencies:

[incubator-mxnet] branch master updated (bd8e0f8 -> f6f8401)

2018-12-05 Thread marcoabreu
This is an automated email from the ASF dual-hosted git repository. marcoabreu pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. from bd8e0f8 [MXNET-769] Use MXNET_HOME in a tempdir in windows to prevent access denied due t… (#13531)

[GitHub] marcoabreu closed issue #13504: flaky test ARMv7 QEMU

2018-12-05 Thread GitBox
marcoabreu closed issue #13504: flaky test ARMv7 QEMU URL: https://github.com/apache/incubator-mxnet/issues/13504 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and

[GitHub] marcoabreu closed pull request #13551: Add a retry to qemu_provision

2018-12-05 Thread GitBox
marcoabreu closed pull request #13551: Add a retry to qemu_provision URL: https://github.com/apache/incubator-mxnet/pull/13551 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

[GitHub] apeforest commented on a change in pull request #13362: Add NHWC layout support to Pooling (cuDNN only)

2018-12-05 Thread GitBox
apeforest commented on a change in pull request #13362: Add NHWC layout support to Pooling (cuDNN only) URL: https://github.com/apache/incubator-mxnet/pull/13362#discussion_r239250805 ## File path: src/operator/nn/pooling-inl.h ## @@ -53,6 +53,7 @@ struct PoolingParam :

  1   2   3   >