[GitHub] [incubator-mxnet] JiangZhaoh opened a new pull request #17255: Set np default dtype( float32 <-> float64)

2020-01-09 Thread GitBox
JiangZhaoh opened a new pull request #17255: Set np default dtype( float32 <-> float64) URL: https://github.com/apache/incubator-mxnet/pull/17255 ## Description ## (Brief description on what this PR is about) ## Checklist ## ### Essentials ### Please feel free to remove inap

[GitHub] [incubator-mxnet] haojin2 commented on a change in pull request #17254: [numpy] change unary infer type

2020-01-09 Thread GitBox
haojin2 commented on a change in pull request #17254: [numpy] change unary infer type URL: https://github.com/apache/incubator-mxnet/pull/17254#discussion_r364600022 ## File path: src/operator/mxnet_op.h ## @@ -759,6 +759,17 @@ struct backward_grad { } }; +template +

[GitHub] [incubator-mxnet] haojin2 commented on a change in pull request #17254: [numpy] change unary infer type

2020-01-09 Thread GitBox
haojin2 commented on a change in pull request #17254: [numpy] change unary infer type URL: https://github.com/apache/incubator-mxnet/pull/17254#discussion_r364600122 ## File path: src/operator/mxnet_op.h ## @@ -759,6 +759,17 @@ struct backward_grad { } }; +template +

[GitHub] [incubator-mxnet] haojin2 commented on a change in pull request #17254: [numpy] change unary infer type

2020-01-09 Thread GitBox
haojin2 commented on a change in pull request #17254: [numpy] change unary infer type URL: https://github.com/apache/incubator-mxnet/pull/17254#discussion_r364600688 ## File path: src/operator/numpy/np_elemwise_unary_op_basic.cc ## @@ -82,6 +82,39 @@ NNVM_REGISTER_OP(_np_c

[GitHub] [incubator-mxnet] haojin2 commented on a change in pull request #17254: [numpy] change unary infer type

2020-01-09 Thread GitBox
haojin2 commented on a change in pull request #17254: [numpy] change unary infer type URL: https://github.com/apache/incubator-mxnet/pull/17254#discussion_r364603602 ## File path: src/operator/numpy/np_elemwise_unary_op_basic.cu ## @@ -39,6 +39,10 @@ NNVM_REGISTER_OP(_np_c

[GitHub] [incubator-mxnet] haojin2 commented on a change in pull request #17254: [numpy] change unary infer type

2020-01-09 Thread GitBox
haojin2 commented on a change in pull request #17254: [numpy] change unary infer type URL: https://github.com/apache/incubator-mxnet/pull/17254#discussion_r364603714 ## File path: src/operator/tensor/elemwise_binary_op.h ## @@ -525,6 +525,66 @@ class ElemwiseBinaryOp : pub

[GitHub] [incubator-mxnet] haojin2 commented on a change in pull request #17254: [numpy] change unary infer type

2020-01-09 Thread GitBox
haojin2 commented on a change in pull request #17254: [numpy] change unary infer type URL: https://github.com/apache/incubator-mxnet/pull/17254#discussion_r364604506 ## File path: tests/python/unittest/test_numpy_op.py ## @@ -1832,6 +1832,35 @@ def hybrid_forward(self, F,

[GitHub] [incubator-mxnet] haojin2 commented on a change in pull request #17254: [numpy] change unary infer type

2020-01-09 Thread GitBox
haojin2 commented on a change in pull request #17254: [numpy] change unary infer type URL: https://github.com/apache/incubator-mxnet/pull/17254#discussion_r364604329 ## File path: src/operator/tensor/elemwise_unary_op_trig.cc ## @@ -387,6 +409,28 @@ MXNET_OPERATOR_REGISTE

[GitHub] [incubator-mxnet] haojin2 commented on a change in pull request #17254: [numpy] change unary infer type

2020-01-09 Thread GitBox
haojin2 commented on a change in pull request #17254: [numpy] change unary infer type URL: https://github.com/apache/incubator-mxnet/pull/17254#discussion_r364604576 ## File path: tests/python/unittest/test_numpy_op.py ## @@ -1865,15 +1889,54 @@ def hybrid_forward(self, F,

[GitHub] [incubator-mxnet] haojin2 commented on a change in pull request #16152: [Numpy] Random.gamma() implemented

2020-01-09 Thread GitBox
haojin2 commented on a change in pull request #16152: [Numpy] Random.gamma() implemented URL: https://github.com/apache/incubator-mxnet/pull/16152#discussion_r364605387 ## File path: src/operator/numpy/random/np_gamma_op.h ## @@ -0,0 +1,350 @@ +/* + * Licensed to the Apach

[GitHub] [incubator-mxnet] marcoabreu commented on a change in pull request #17255: Set np default dtype( float32 <-> float64)

2020-01-09 Thread GitBox
marcoabreu commented on a change in pull request #17255: Set np default dtype( float32 <-> float64) URL: https://github.com/apache/incubator-mxnet/pull/17255#discussion_r364606241 ## File path: include/mxnet/imperative.h ## @@ -136,6 +137,34 @@ class Imperative { }

[GitHub] [incubator-mxnet] marcoabreu commented on a change in pull request #17255: Set np default dtype( float32 <-> float64)

2020-01-09 Thread GitBox
marcoabreu commented on a change in pull request #17255: Set np default dtype( float32 <-> float64) URL: https://github.com/apache/incubator-mxnet/pull/17255#discussion_r364606683 ## File path: include/mxnet/imperative.h ## @@ -136,6 +137,34 @@ class Imperative { }

[GitHub] [incubator-mxnet] marcoabreu commented on a change in pull request #17255: Set np default dtype( float32 <-> float64)

2020-01-09 Thread GitBox
marcoabreu commented on a change in pull request #17255: Set np default dtype( float32 <-> float64) URL: https://github.com/apache/incubator-mxnet/pull/17255#discussion_r364606558 ## File path: include/mxnet/imperative.h ## @@ -136,6 +137,34 @@ class Imperative { }

[GitHub] [incubator-mxnet] JiangZhaoh commented on a change in pull request #17255: Set np default dtype( float32 <-> float64)

2020-01-09 Thread GitBox
JiangZhaoh commented on a change in pull request #17255: Set np default dtype( float32 <-> float64) URL: https://github.com/apache/incubator-mxnet/pull/17255#discussion_r364612206 ## File path: include/mxnet/imperative.h ## @@ -136,6 +137,34 @@ class Imperative { }

[GitHub] [incubator-mxnet] JiangZhaoh closed pull request #17255: [DO NOT REVIEW]Set np default dtype( float32 <-> float64)

2020-01-09 Thread GitBox
JiangZhaoh closed pull request #17255: [DO NOT REVIEW]Set np default dtype( float32 <-> float64) URL: https://github.com/apache/incubator-mxnet/pull/17255 This is an automated message from the Apache Git Service. To respond

[GitHub] [incubator-mxnet] JiangZhaoh commented on issue #17255: [DO NOT REVIEW]Set np default dtype( float32 <-> float64)

2020-01-09 Thread GitBox
JiangZhaoh commented on issue #17255: [DO NOT REVIEW]Set np default dtype( float32 <-> float64) URL: https://github.com/apache/incubator-mxnet/pull/17255#issuecomment-572460479 > This PR changes so many things and it feels odd. I'm afraid it will cause quite some confusion when people expl

[GitHub] [incubator-mxnet] JiangZhaoh commented on issue #17255: [DO NOT REVIEW]Set np default dtype( float32 <-> float64)

2020-01-09 Thread GitBox
JiangZhaoh commented on issue #17255: [DO NOT REVIEW]Set np default dtype( float32 <-> float64) URL: https://github.com/apache/incubator-mxnet/pull/17255#issuecomment-572460667 > I don't know... Somehow this change feels wrong to me - a default is a default and people can always specify th

[GitHub] [incubator-mxnet] JiangZhaoh removed a comment on issue #17255: [DO NOT REVIEW]Set np default dtype( float32 <-> float64)

2020-01-09 Thread GitBox
JiangZhaoh removed a comment on issue #17255: [DO NOT REVIEW]Set np default dtype( float32 <-> float64) URL: https://github.com/apache/incubator-mxnet/pull/17255#issuecomment-572460479 > This PR changes so many things and it feels odd. I'm afraid it will cause quite some confusion when peo

[GitHub] [incubator-mxnet] marcoabreu commented on issue #17255: [DO NOT REVIEW]Set np default dtype( float32 <-> float64)

2020-01-09 Thread GitBox
marcoabreu commented on issue #17255: [DO NOT REVIEW]Set np default dtype( float32 <-> float64) URL: https://github.com/apache/incubator-mxnet/pull/17255#issuecomment-572464020 My question is rather towards whether the functionality itself will provide a good user experience - the implemen

[GitHub] [incubator-mxnet] haojin2 commented on issue #17255: [DO NOT REVIEW]Set np default dtype( float32 <-> float64)

2020-01-09 Thread GitBox
haojin2 commented on issue #17255: [DO NOT REVIEW]Set np default dtype( float32 <-> float64) URL: https://github.com/apache/incubator-mxnet/pull/17255#issuecomment-572465096 @marcoabreu If you explicitly specified some dtype in an op with a `dtype` option, it means that you're not expe

[GitHub] [incubator-mxnet] marcoabreu commented on issue #17255: [DO NOT REVIEW]Set np default dtype( float32 <-> float64)

2020-01-09 Thread GitBox
marcoabreu commented on issue #17255: [DO NOT REVIEW]Set np default dtype( float32 <-> float64) URL: https://github.com/apache/incubator-mxnet/pull/17255#issuecomment-572466288 Okay, so there's a functionality in numpy to set the default dtype or is this approach chosen as a compromise to

[GitHub] [incubator-mxnet] haojin2 commented on issue #17255: [DO NOT REVIEW]Set np default dtype( float32 <-> float64)

2020-01-09 Thread GitBox
haojin2 commented on issue #17255: [DO NOT REVIEW]Set np default dtype( float32 <-> float64) URL: https://github.com/apache/incubator-mxnet/pull/17255#issuecomment-572472159 @marcoabreu Let's probably take a look at this example to help you understand: ```python import numpy as np

[GitHub] [incubator-mxnet] haojin2 edited a comment on issue #17255: [DO NOT REVIEW]Set np default dtype( float32 <-> float64)

2020-01-09 Thread GitBox
haojin2 edited a comment on issue #17255: [DO NOT REVIEW]Set np default dtype( float32 <-> float64) URL: https://github.com/apache/incubator-mxnet/pull/17255#issuecomment-572472159 @marcoabreu Let's probably take a look at this example to help you understand: ```python import numpy

[GitHub] [incubator-mxnet] haojin2 edited a comment on issue #17255: [DO NOT REVIEW]Set np default dtype( float32 <-> float64)

2020-01-09 Thread GitBox
haojin2 edited a comment on issue #17255: [DO NOT REVIEW]Set np default dtype( float32 <-> float64) URL: https://github.com/apache/incubator-mxnet/pull/17255#issuecomment-572472159 @marcoabreu Let's probably take a look at this example to help you understand: ```python import numpy

[GitHub] [incubator-mxnet] marcoabreu commented on issue #17255: [DO NOT REVIEW]Set np default dtype( float32 <-> float64)

2020-01-09 Thread GitBox
marcoabreu commented on issue #17255: [DO NOT REVIEW]Set np default dtype( float32 <-> float64) URL: https://github.com/apache/incubator-mxnet/pull/17255#issuecomment-572474784 Thanks for elaborating, makes sense. Merging it into set_np sounds like a good addition, indeed. I k

[GitHub] [incubator-mxnet] haojin2 commented on issue #17255: [DO NOT REVIEW]Set np default dtype( float32 <-> float64)

2020-01-09 Thread GitBox
haojin2 commented on issue #17255: [DO NOT REVIEW]Set np default dtype( float32 <-> float64) URL: https://github.com/apache/incubator-mxnet/pull/17255#issuecomment-572477944 @marcoabreu It's really case by case for each op, it should only affect array creation ops, and this is not the fina

[GitHub] [incubator-mxnet] haojin2 edited a comment on issue #17255: [DO NOT REVIEW]Set np default dtype( float32 <-> float64)

2020-01-09 Thread GitBox
haojin2 edited a comment on issue #17255: [DO NOT REVIEW]Set np default dtype( float32 <-> float64) URL: https://github.com/apache/incubator-mxnet/pull/17255#issuecomment-572477944 @marcoabreu It's really case by case for each op, it should only affect array creation ops, and this is not t

[GitHub] [incubator-mxnet] haojin2 commented on a change in pull request #17234: Op Quantile/Percentile [Numpy]

2020-01-09 Thread GitBox
haojin2 commented on a change in pull request #17234: Op Quantile/Percentile [Numpy] URL: https://github.com/apache/incubator-mxnet/pull/17234#discussion_r364656365 ## File path: src/operator/numpy/np_percentile_op-inl.h ## @@ -0,0 +1,307 @@ +/* + * Licensed to the Apache

[GitHub] [incubator-mxnet] haojin2 commented on a change in pull request #17234: Op Quantile/Percentile [Numpy]

2020-01-09 Thread GitBox
haojin2 commented on a change in pull request #17234: Op Quantile/Percentile [Numpy] URL: https://github.com/apache/incubator-mxnet/pull/17234#discussion_r364657942 ## File path: src/operator/numpy/np_percentile_op-inl.h ## @@ -0,0 +1,307 @@ +/* + * Licensed to the Apache

[GitHub] [incubator-mxnet] haojin2 commented on a change in pull request #17234: Op Quantile/Percentile [Numpy]

2020-01-09 Thread GitBox
haojin2 commented on a change in pull request #17234: Op Quantile/Percentile [Numpy] URL: https://github.com/apache/incubator-mxnet/pull/17234#discussion_r364658344 ## File path: src/operator/numpy/np_percentile_op-inl.h ## @@ -0,0 +1,307 @@ +/* + * Licensed to the Apache

[GitHub] [incubator-mxnet] haojin2 commented on a change in pull request #17234: Op Quantile/Percentile [Numpy]

2020-01-09 Thread GitBox
haojin2 commented on a change in pull request #17234: Op Quantile/Percentile [Numpy] URL: https://github.com/apache/incubator-mxnet/pull/17234#discussion_r364658195 ## File path: src/operator/numpy/np_percentile_op-inl.h ## @@ -0,0 +1,307 @@ +/* + * Licensed to the Apache

[GitHub] [incubator-mxnet] haojin2 commented on a change in pull request #17234: Op Quantile/Percentile [Numpy]

2020-01-09 Thread GitBox
haojin2 commented on a change in pull request #17234: Op Quantile/Percentile [Numpy] URL: https://github.com/apache/incubator-mxnet/pull/17234#discussion_r364658691 ## File path: src/operator/numpy/np_percentile_op-inl.h ## @@ -0,0 +1,307 @@ +/* + * Licensed to the Apache

[GitHub] [incubator-mxnet] haojin2 commented on a change in pull request #17234: Op Quantile/Percentile [Numpy]

2020-01-09 Thread GitBox
haojin2 commented on a change in pull request #17234: Op Quantile/Percentile [Numpy] URL: https://github.com/apache/incubator-mxnet/pull/17234#discussion_r364659372 ## File path: src/operator/numpy/np_percentile_op-inl.h ## @@ -0,0 +1,307 @@ +/* + * Licensed to the Apache

[GitHub] [incubator-mxnet] haojin2 commented on a change in pull request #17234: Op Quantile/Percentile [Numpy]

2020-01-09 Thread GitBox
haojin2 commented on a change in pull request #17234: Op Quantile/Percentile [Numpy] URL: https://github.com/apache/incubator-mxnet/pull/17234#discussion_r364660488 ## File path: src/operator/tensor/ordering_op-inl.h ## @@ -558,6 +558,202 @@ void TopKImpl(const RunContext

[GitHub] [incubator-mxnet] haojin2 commented on a change in pull request #17234: Op Quantile/Percentile [Numpy]

2020-01-09 Thread GitBox
haojin2 commented on a change in pull request #17234: Op Quantile/Percentile [Numpy] URL: https://github.com/apache/incubator-mxnet/pull/17234#discussion_r364660646 ## File path: src/operator/tensor/ordering_op-inl.h ## @@ -558,6 +558,202 @@ void TopKImpl(const RunContext

[GitHub] [incubator-mxnet] haojin2 commented on a change in pull request #17234: Op Quantile/Percentile [Numpy]

2020-01-09 Thread GitBox
haojin2 commented on a change in pull request #17234: Op Quantile/Percentile [Numpy] URL: https://github.com/apache/incubator-mxnet/pull/17234#discussion_r364660946 ## File path: tests/python/unittest/test_numpy_interoperability.py ## @@ -156,6 +156,20 @@ def _add_workload

[GitHub] [incubator-mxnet] haojin2 commented on a change in pull request #17234: Op Quantile/Percentile [Numpy]

2020-01-09 Thread GitBox
haojin2 commented on a change in pull request #17234: Op Quantile/Percentile [Numpy] URL: https://github.com/apache/incubator-mxnet/pull/17234#discussion_r364661074 ## File path: tests/python/unittest/test_numpy_op.py ## @@ -5680,6 +5680,49 @@ def test_np_share_memory():

[GitHub] [incubator-mxnet] Wallart opened a new issue #17256: Sparse compression causes errors

2020-01-09 Thread GitBox
Wallart opened a new issue #17256: Sparse compression causes errors URL: https://github.com/apache/incubator-mxnet/issues/17256 Hello everyone, I am trying to use sparse tensors to save memory in my Transformer architecture and I'm applying F.sparse.cast_storage on an attention weights

[GitHub] [incubator-mxnet] leezu merged pull request #17253: Remove the straight dope from nightly test

2020-01-09 Thread GitBox
leezu merged pull request #17253: Remove the straight dope from nightly test URL: https://github.com/apache/incubator-mxnet/pull/17253 This is an automated message from the Apache Git Service. To respond to the message, pleas

[incubator-mxnet] branch master updated (4ff1c67 -> 83578b9)

2020-01-09 Thread lausen
This is an automated email from the ASF dual-hosted git repository. lausen pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. from 4ff1c67 fix latency calculation and print issue (#17217) add 83578b9 remove the straight dope from ni

[GitHub] [incubator-mxnet] leezu commented on issue #17246: R-package install.packages("mxnet") broken

2020-01-09 Thread GitBox
leezu commented on issue #17246: R-package install.packages("mxnet") broken URL: https://github.com/apache/incubator-mxnet/issues/17246#issuecomment-572513164 You can try following the compile-from source guide instead: https://mxnet.apache.org/get_started/ubuntu_setup.html#install-the-mxn

[incubator-mxnet] branch master updated (4ff1c67 -> 83578b9)

2020-01-09 Thread lausen
This is an automated email from the ASF dual-hosted git repository. lausen pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. from 4ff1c67 fix latency calculation and print issue (#17217) add 83578b9 remove the straight dope from ni

[GitHub] [incubator-mxnet] leezu edited a comment on issue #17246: R-package install.packages("mxnet") broken

2020-01-09 Thread GitBox
leezu edited a comment on issue #17246: R-package install.packages("mxnet") broken URL: https://github.com/apache/incubator-mxnet/issues/17246#issuecomment-572513164 Until this is fixed, you can try following the compile-from source guide instead: https://mxnet.apache.org/get_started/ubu

[GitHub] [incubator-mxnet] ronnac opened a new issue #17257: armv81

2020-01-09 Thread GitBox
ronnac opened a new issue #17257: armv81 URL: https://github.com/apache/incubator-mxnet/issues/17257 ## Description The make/config.mk file doesn't account for the armv81 architecture. It should be treated the same as armv71 or armv61 with regards to USE_SSE=0 and USE_F16C=0. ###

[GitHub] [incubator-mxnet] leezu commented on issue #17257: armv81

2020-01-09 Thread GitBox
leezu commented on issue #17257: armv81 URL: https://github.com/apache/incubator-mxnet/issues/17257#issuecomment-572542740 `make/config.mk` is used for the deprecated Makefile build. Based on your steps to reproduce, you are using the CMake build. You could look into the `CMakeLists

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

2020-01-09 Thread aaronmarkham
This is an automated email from the ASF dual-hosted git repository. aaronmarkham 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 e191e9e Bump the publis

[GitHub] [incubator-mxnet] Justobe opened a new issue #17258: mxnet.base.MXNetError: [12:37:13] src/executor/../common/exec_utils.h:391: InferShape pass cannot decide shapes for the following argument

2020-01-09 Thread GitBox
Justobe opened a new issue #17258: mxnet.base.MXNetError: [12:37:13] src/executor/../common/exec_utils.h:391: InferShape pass cannot decide shapes for the following arguments URL: https://github.com/apache/incubator-mxnet/issues/17258 ## Description I find that MXNET behaves differently

[GitHub] [incubator-mxnet] szha opened a new pull request #17259: [CD] fix CD pipeline

2020-01-09 Thread GitBox
szha opened a new pull request #17259: [CD] fix CD pipeline URL: https://github.com/apache/incubator-mxnet/pull/17259 ## Description ## fix CD pipeline ## Checklist ## ### Essentials ### Please feel free to remove inapplicable items for your PR. - [ ] Changes are complete

[GitHub] [incubator-mxnet] Justobe opened a new issue #17260: ValueError: Layer #1 (named "batch_normalization_1" in the current model) was found to correspond to layer batch_normalization_1 in the sa

2020-01-09 Thread GitBox
Justobe opened a new issue #17260: ValueError: Layer #1 (named "batch_normalization_1" in the current model) was found to correspond to layer batch_normalization_1 in the save file. However the new layer batch_normalization_1 expects 4 weights, but the saved weights have 3 elements. URL: https:/

[GitHub] [incubator-mxnet] ronnac commented on issue #17257: armv81

2020-01-09 Thread GitBox
ronnac commented on issue #17257: armv81 URL: https://github.com/apache/incubator-mxnet/issues/17257#issuecomment-572555770 Thank yoou! Maybe the instructions on https://mxnet.apache.org/get_started/build_from_source need to be updated. It states there " For building with Java/Sc

[GitHub] [incubator-mxnet] leezu commented on issue #17257: armv81

2020-01-09 Thread GitBox
leezu commented on issue #17257: armv81 URL: https://github.com/apache/incubator-mxnet/issues/17257#issuecomment-572560350 With respect to the armv8 build, take a look at https://github.com/apache/incubator-mxnet/blob/83578b9a61d5ddb5ed54576fac17e24f97f35e52/ci/docker/runtime_functi

[GitHub] [incubator-mxnet] marcoabreu commented on a change in pull request #17259: [CD] fix CD pipeline

2020-01-09 Thread GitBox
marcoabreu commented on a change in pull request #17259: [CD] fix CD pipeline URL: https://github.com/apache/incubator-mxnet/pull/17259#discussion_r364771567 ## File path: tests/python/mkl/test_mkldnn.py ## @@ -95,7 +95,7 @@ def __getitem__(self, key): for _ in loader:

[GitHub] [incubator-mxnet] apkuhar commented on issue #15974: USE_NNPACK build flag not honored.

2020-01-09 Thread GitBox
apkuhar commented on issue #15974: USE_NNPACK build flag not honored. URL: https://github.com/apache/incubator-mxnet/issues/15974#issuecomment-572620375 I managed to build an older version of mxnet with nnpack, but it was a couple of times slower than the binary build ob 1.5.1 on raspberr

[GitHub] [incubator-mxnet] apkuhar edited a comment on issue #15974: USE_NNPACK build flag not honored.

2020-01-09 Thread GitBox
apkuhar edited a comment on issue #15974: USE_NNPACK build flag not honored. URL: https://github.com/apache/incubator-mxnet/issues/15974#issuecomment-572620375 I managed to build an older version of mxnet with nnpack, but it was a couple of times slower than the binary build ob 1.5.1 on r

[GitHub] [incubator-mxnet] codecov-io commented on issue #17259: [CD] fix CD pipeline

2020-01-09 Thread GitBox
codecov-io commented on issue #17259: [CD] fix CD pipeline URL: https://github.com/apache/incubator-mxnet/pull/17259#issuecomment-572623348 # [Codecov](https://codecov.io/gh/apache/incubator-mxnet/pull/17259?src=pr&el=h1) Report > Merging [#17259](https://codecov.io/gh/apache/incubator

[GitHub] [incubator-mxnet] codecov-io edited a comment on issue #17259: [CD] fix CD pipeline

2020-01-09 Thread GitBox
codecov-io edited a comment on issue #17259: [CD] fix CD pipeline URL: https://github.com/apache/incubator-mxnet/pull/17259#issuecomment-572623348 # [Codecov](https://codecov.io/gh/apache/incubator-mxnet/pull/17259?src=pr&el=h1) Report > Merging [#17259](https://codecov.io/gh/apache/in

[GitHub] [incubator-mxnet] ptrendx commented on issue #17209: Remove dtype from Variable created from Gluon Parameter

2020-01-09 Thread GitBox
ptrendx commented on issue #17209: Remove dtype from Variable created from Gluon Parameter URL: https://github.com/apache/incubator-mxnet/pull/17209#issuecomment-572663516 @MoisesHer has an example of a model where removing dtype actually breaks the transition to symbolic for some reason.

[GitHub] [incubator-mxnet] codecov-io removed a comment on issue #17259: [CD] fix CD pipeline

2020-01-09 Thread GitBox
codecov-io removed a comment on issue #17259: [CD] fix CD pipeline URL: https://github.com/apache/incubator-mxnet/pull/17259#issuecomment-572623348 # [Codecov](https://codecov.io/gh/apache/incubator-mxnet/pull/17259?src=pr&el=h1) Report > Merging [#17259](https://codecov.io/gh/apache/i

[GitHub] [incubator-mxnet] stu1130 closed pull request #17248: Run the nightly test against S3 pip wheel

2020-01-09 Thread GitBox
stu1130 closed pull request #17248: Run the nightly test against S3 pip wheel URL: https://github.com/apache/incubator-mxnet/pull/17248 This is an automated message from the Apache Git Service. To respond to the message, plea

[GitHub] [incubator-mxnet] guanxinq commented on a change in pull request #17242: add RandomApply in gluon's transforms

2020-01-09 Thread GitBox
guanxinq commented on a change in pull request #17242: add RandomApply in gluon's transforms URL: https://github.com/apache/incubator-mxnet/pull/17242#discussion_r364893422 ## File path: python/mxnet/gluon/data/vision/transforms.py ## @@ -581,3 +582,28 @@ def hybrid_forwar

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

2020-01-09 Thread aaronmarkham
This is an automated email from the ASF dual-hosted git repository. aaronmarkham 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 bc4b360 Bump the publis

[GitHub] [incubator-mxnet] apeforest merged pull request #16755: Enabling large tensor support for binary broadcast operators

2020-01-09 Thread GitBox
apeforest merged pull request #16755: Enabling large tensor support for binary broadcast operators URL: https://github.com/apache/incubator-mxnet/pull/16755 This is an automated message from the Apache Git Service. To respon

[incubator-mxnet] branch master updated (83578b9 -> 6ba9aad)

2020-01-09 Thread apeforest
This is an automated email from the ASF dual-hosted git repository. apeforest pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. from 83578b9 remove the straight dope from nightly test (#17253) add 6ba9aad Enabling large tensor supp

[GitHub] [incubator-mxnet] djaym7 opened a new issue #17261: Gluon adding new parameters to learn -error

2020-01-09 Thread GitBox
djaym7 opened a new issue #17261: Gluon adding new parameters to learn -error URL: https://github.com/apache/incubator-mxnet/issues/17261 I am trying to replicate this pytorch's class, what am i doing wrong here ? @zhreshold help please. ![image](https://user-images.githubusercont

[GitHub] [incubator-mxnet] eric-haibin-lin merged pull request #17235: [DOC] Add a few tips for running horovod

2020-01-09 Thread GitBox
eric-haibin-lin merged pull request #17235: [DOC] Add a few tips for running horovod URL: https://github.com/apache/incubator-mxnet/pull/17235 This is an automated message from the Apache Git Service. To respond to the messa

[incubator-mxnet] branch master updated (6ba9aad -> ac88f1e)

2020-01-09 Thread haibin
This is an automated email from the ASF dual-hosted git repository. haibin pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. from 6ba9aad Enabling large tensor support for binary broadcast operators (#16755) add ac88f1e [DOC] Add

[GitHub] [incubator-mxnet] ChaiBapchya commented on issue #16898: Sparse int64 Large tensor support

2020-01-09 Thread GitBox
ChaiBapchya commented on issue #16898: Sparse int64 Large tensor support URL: https://github.com/apache/incubator-mxnet/pull/16898#issuecomment-572720920 > Does https://cwiki.apache.org/confluence/display/MXNET/Large+Tensor+Support correctly document the current practice/changes? Ye

[GitHub] [incubator-mxnet] djaym7 commented on issue #17261: Gluon adding new parameters to learn -error

2020-01-09 Thread GitBox
djaym7 commented on issue #17261: Gluon adding new parameters to learn -error URL: https://github.com/apache/incubator-mxnet/issues/17261#issuecomment-572729107 Ablation Studies: 1. Without passing p in hybrid_forward gave this error, so followed the "Dense" layer source code ![ima

[GitHub] [incubator-mxnet] MyraBaba commented on issue #17216: how to compile mxnet with cpp and gpu support in the docker ?

2020-01-09 Thread GitBox
MyraBaba commented on issue #17216: how to compile mxnet with cpp and gpu support in the docker ? URL: https://github.com/apache/incubator-mxnet/issues/17216#issuecomment-572732511 So is this include mxnet cpp support and all required include files for cpp mxnet ? --

[GitHub] [incubator-mxnet] djaym7 commented on issue #17261: Gluon adding new parameters to learn -error

2020-01-09 Thread GitBox
djaym7 commented on issue #17261: Gluon adding new parameters to learn -error URL: https://github.com/apache/incubator-mxnet/issues/17261#issuecomment-572754155 Solved This is an automated message from the Apache Git Service.

[GitHub] [incubator-mxnet] djaym7 closed issue #17261: Gluon adding new parameters to learn -error

2020-01-09 Thread GitBox
djaym7 closed issue #17261: Gluon adding new parameters to learn -error URL: https://github.com/apache/incubator-mxnet/issues/17261 This is an automated message from the Apache Git Service. To respond to the message, please l

[GitHub] [incubator-mxnet] QueensGambit commented on issue #17216: how to compile mxnet with cpp and gpu support in the docker ?

2020-01-09 Thread GitBox
QueensGambit commented on issue #17216: how to compile mxnet with cpp and gpu support in the docker ? URL: https://github.com/apache/incubator-mxnet/issues/17216#issuecomment-572763504 Yes, the docker file contains a shared object library at: * /usr/local/lib/libmxnet.so which wa

[GitHub] [incubator-mxnet] cyrusbehr opened a new issue #17262: Unable to build / link mxnet against cuda 10.2

2020-01-09 Thread GitBox
cyrusbehr opened a new issue #17262: Unable to build / link mxnet against cuda 10.2 URL: https://github.com/apache/incubator-mxnet/issues/17262 I am trying to build mxnet with cuda 10.2 in a docker container. For my build, I am using the following docker image from nvidia: `nvidia/cuda:

[GitHub] [incubator-mxnet] rondogency commented on issue #17241: Add CustomOp tutorial doc

2020-01-09 Thread GitBox
rondogency commented on issue #17241: Add CustomOp tutorial doc URL: https://github.com/apache/incubator-mxnet/pull/17241#issuecomment-572773130 @samskalicky @wkcn resolved all the comments! This is an automated message from t

[GitHub] [incubator-mxnet] zhreshold opened a new issue #17263: [mxnet 2.0][item 4.8][RFC] Gluon Data API Extension and Fixes(Part 1)

2020-01-09 Thread GitBox
zhreshold opened a new issue #17263: [mxnet 2.0][item 4.8][RFC] Gluon Data API Extension and Fixes(Part 1) URL: https://github.com/apache/incubator-mxnet/issues/17263 ## Description This is the part 1 of Gluon Data API extension and fixes, which mainly focus on cleaning up diverging

[GitHub] [incubator-mxnet] haojin2 commented on a change in pull request #17014: [NumPy] Add NumPy support for norm

2020-01-09 Thread GitBox
haojin2 commented on a change in pull request #17014: [NumPy] Add NumPy support for norm URL: https://github.com/apache/incubator-mxnet/pull/17014#discussion_r364987599 ## File path: src/operator/numpy/linalg/broadcast_reduce_customized-inl.cuh ## @@ -0,0 +1,416 @@ +/* + *

[GitHub] [incubator-mxnet] eric-haibin-lin commented on issue #16735: Use single-bit for mask in dropout operator

2020-01-09 Thread GitBox
eric-haibin-lin commented on issue #16735: Use single-bit for mask in dropout operator URL: https://github.com/apache/incubator-mxnet/pull/16735#issuecomment-572787745 For GPT-2, the memory usage goes from 30GB to 26GB. For BERT, it goes from 26GB to 23GB. I didn't notice much difference i

[GitHub] [incubator-mxnet] haojin2 commented on a change in pull request #17188: [Numpy] Add linalg.eig/eigh/eigvals/eigvalsh op

2020-01-09 Thread GitBox
haojin2 commented on a change in pull request #17188: [Numpy] Add linalg.eig/eigh/eigvals/eigvalsh op URL: https://github.com/apache/incubator-mxnet/pull/17188#discussion_r364996931 ## File path: src/operator/numpy/linalg/np_eigvals.cc ## @@ -0,0 +1,124 @@ +/* + * Licensed

[GitHub] [incubator-mxnet] haojin2 commented on a change in pull request #17188: [Numpy] Add linalg.eig/eigh/eigvals/eigvalsh op

2020-01-09 Thread GitBox
haojin2 commented on a change in pull request #17188: [Numpy] Add linalg.eig/eigh/eigvals/eigvalsh op URL: https://github.com/apache/incubator-mxnet/pull/17188#discussion_r364998303 ## File path: src/operator/numpy/linalg/np_eig.cu ## @@ -0,0 +1,61 @@ +/* + * Licensed to t

[GitHub] [incubator-mxnet] haojin2 commented on a change in pull request #17188: [Numpy] Add linalg.eig/eigh/eigvals/eigvalsh op

2020-01-09 Thread GitBox
haojin2 commented on a change in pull request #17188: [Numpy] Add linalg.eig/eigh/eigvals/eigvalsh op URL: https://github.com/apache/incubator-mxnet/pull/17188#discussion_r364998737 ## File path: python/mxnet/symbol/numpy/linalg.py ## @@ -496,3 +497,265 @@ def tensorsolve(

[GitHub] [incubator-mxnet] stu1130 opened a new pull request #17264: Image CenterCrop Op

2020-01-09 Thread GitBox
stu1130 opened a new pull request #17264: Image CenterCrop Op URL: https://github.com/apache/incubator-mxnet/pull/17264 ## Description ## 1. Add image.center_crop op which takes 3-D & 4-D image 2. Make CenterCrop hybridizable ## Checklist ## ### Essentials ### Please feel

[GitHub] [incubator-mxnet] zrsm commented on issue #2838: Cannot find -lcuda? what's wrong?

2020-01-09 Thread GitBox
zrsm commented on issue #2838: Cannot find -lcuda? what's wrong? URL: https://github.com/apache/incubator-mxnet/issues/2838#issuecomment-572795545 > libcuda.so is under /usr/local/cuda/lib64/stubs/libcuda.so, just make a soft link to /usr/local/cuda/lib64/libcuda.so. I am not sure if this

[GitHub] [incubator-mxnet] haojin2 commented on a change in pull request #17188: [Numpy] Add linalg.eig/eigh/eigvals/eigvalsh op

2020-01-09 Thread GitBox
haojin2 commented on a change in pull request #17188: [Numpy] Add linalg.eig/eigh/eigvals/eigvalsh op URL: https://github.com/apache/incubator-mxnet/pull/17188#discussion_r365001641 ## File path: src/operator/numpy/linalg/np_eigvals.cc ## @@ -0,0 +1,124 @@ +/* + * Licensed

[GitHub] [incubator-mxnet] haojin2 commented on a change in pull request #17188: [Numpy] Add linalg.eig/eigh/eigvals/eigvalsh op

2020-01-09 Thread GitBox
haojin2 commented on a change in pull request #17188: [Numpy] Add linalg.eig/eigh/eigvals/eigvalsh op URL: https://github.com/apache/incubator-mxnet/pull/17188#discussion_r365001641 ## File path: src/operator/numpy/linalg/np_eigvals.cc ## @@ -0,0 +1,124 @@ +/* + * Licensed

[GitHub] [incubator-mxnet] haojin2 commented on a change in pull request #17188: [Numpy] Add linalg.eig/eigh/eigvals/eigvalsh op

2020-01-09 Thread GitBox
haojin2 commented on a change in pull request #17188: [Numpy] Add linalg.eig/eigh/eigvals/eigvalsh op URL: https://github.com/apache/incubator-mxnet/pull/17188#discussion_r365002633 ## File path: src/operator/numpy/linalg/np_eig.cc ## @@ -0,0 +1,157 @@ +/* + * Licensed to

[GitHub] [incubator-mxnet] roywei commented on issue #17250: mxnet.base.MXNetError: Error in operator transpose176: [02:14:06] src/operator/tensor/./matrix_op-inl.h:354: Check failed: shp.ndim() == pa

2020-01-09 Thread GitBox
roywei commented on issue #17250: mxnet.base.MXNetError: Error in operator transpose176: [02:14:06] src/operator/tensor/./matrix_op-inl.h:354: Check failed: shp.ndim() == param.axes.ndim() (-1 vs. 4) URL: https://github.com/apache/incubator-mxnet/issues/17250#issuecomment-572799442 Hi @Ju

[GitHub] [incubator-mxnet] samskalicky commented on a change in pull request #17241: Add CustomOp tutorial doc

2020-01-09 Thread GitBox
samskalicky commented on a change in pull request #17241: Add CustomOp tutorial doc URL: https://github.com/apache/incubator-mxnet/pull/17241#discussion_r365006013 ## File path: example/extensions/lib_custom_op/README.md ## @@ -0,0 +1,83 @@ +CustomOp Example and Tutorial +

[GitHub] [incubator-mxnet] samskalicky commented on a change in pull request #17241: Add CustomOp tutorial doc

2020-01-09 Thread GitBox
samskalicky commented on a change in pull request #17241: Add CustomOp tutorial doc URL: https://github.com/apache/incubator-mxnet/pull/17241#discussion_r365006466 ## File path: example/extensions/lib_custom_op/README.md ## @@ -0,0 +1,83 @@ +CustomOp Example and Tutorial +

[GitHub] [incubator-mxnet] samskalicky commented on a change in pull request #17241: Add CustomOp tutorial doc

2020-01-09 Thread GitBox
samskalicky commented on a change in pull request #17241: Add CustomOp tutorial doc URL: https://github.com/apache/incubator-mxnet/pull/17241#discussion_r365007081 ## File path: example/extensions/lib_custom_op/README.md ## @@ -0,0 +1,83 @@ +CustomOp Example and Tutorial +

[GitHub] [incubator-mxnet] samskalicky commented on a change in pull request #17241: Add CustomOp tutorial doc

2020-01-09 Thread GitBox
samskalicky commented on a change in pull request #17241: Add CustomOp tutorial doc URL: https://github.com/apache/incubator-mxnet/pull/17241#discussion_r365007081 ## File path: example/extensions/lib_custom_op/README.md ## @@ -0,0 +1,83 @@ +CustomOp Example and Tutorial +

[GitHub] [incubator-mxnet] rondogency commented on issue #17241: Add CustomOp tutorial doc

2020-01-09 Thread GitBox
rondogency commented on issue #17241: Add CustomOp tutorial doc URL: https://github.com/apache/incubator-mxnet/pull/17241#issuecomment-572807610 @eric-haibin-lin @aaronmarkham can you also take a quick look at the doc, thanks! ---

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

2020-01-09 Thread aaronmarkham
This is an automated email from the ASF dual-hosted git repository. aaronmarkham 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 3170a97 Bump the publis

[GitHub] [incubator-mxnet] apeforest edited a comment on issue #16735: Use single-bit for mask in dropout operator

2020-01-09 Thread GitBox
apeforest edited a comment on issue #16735: Use single-bit for mask in dropout operator URL: https://github.com/apache/incubator-mxnet/pull/16735#issuecomment-572830100 @TaoLv Thanks for your review. I ran operator profiling using benchmark.opperf.utils.benchmark_utils.run_performance_test

[GitHub] [incubator-mxnet] apeforest commented on issue #16735: Use single-bit for mask in dropout operator

2020-01-09 Thread GitBox
apeforest commented on issue #16735: Use single-bit for mask in dropout operator URL: https://github.com/apache/incubator-mxnet/pull/16735#issuecomment-572830100 @TaoLv Thanks for your review. I ran operator profining using benchmark.opperf.utils.benchmark_utils.run_performance_test. The re

[GitHub] [incubator-mxnet] szha commented on a change in pull request #17259: [CD] fix CD pipeline

2020-01-09 Thread GitBox
szha commented on a change in pull request #17259: [CD] fix CD pipeline URL: https://github.com/apache/incubator-mxnet/pull/17259#discussion_r365044385 ## File path: tests/python/mkl/test_mkldnn.py ## @@ -95,7 +95,7 @@ def __getitem__(self, key): for _ in loader:

[GitHub] [incubator-mxnet] szha commented on a change in pull request #16408: Add MXNet Ops for fast multihead attention

2020-01-09 Thread GitBox
szha commented on a change in pull request #16408: Add MXNet Ops for fast multihead attention URL: https://github.com/apache/incubator-mxnet/pull/16408#discussion_r365049085 ## File path: tests/python/gpu/test_operator_gpu.py ## @@ -2493,13 +2493,327 @@ def test_arange_lik

[GitHub] [incubator-mxnet] Justobe commented on issue #17250: mxnet.base.MXNetError: Error in operator transpose176: [02:14:06] src/operator/tensor/./matrix_op-inl.h:354: Check failed: shp.ndim() == p

2020-01-09 Thread GitBox
Justobe commented on issue #17250: mxnet.base.MXNetError: Error in operator transpose176: [02:14:06] src/operator/tensor/./matrix_op-inl.h:354: Check failed: shp.ndim() == param.axes.ndim() (-1 vs. 4) URL: https://github.com/apache/incubator-mxnet/issues/17250#issuecomment-572847927 Hello

[GitHub] [incubator-mxnet] Justobe edited a comment on issue #17250: mxnet.base.MXNetError: Error in operator transpose176: [02:14:06] src/operator/tensor/./matrix_op-inl.h:354: Check failed: shp.ndim

2020-01-09 Thread GitBox
Justobe edited a comment on issue #17250: mxnet.base.MXNetError: Error in operator transpose176: [02:14:06] src/operator/tensor/./matrix_op-inl.h:354: Check failed: shp.ndim() == param.axes.ndim() (-1 vs. 4) URL: https://github.com/apache/incubator-mxnet/issues/17250#issuecomment-572847927

[GitHub] [incubator-mxnet] hanke580 commented on a change in pull request #17171: [Numpy] add row_stack (=vstack)

2020-01-09 Thread GitBox
hanke580 commented on a change in pull request #17171: [Numpy] add row_stack (=vstack) URL: https://github.com/apache/incubator-mxnet/pull/17171#discussion_r365056395 ## File path: python/mxnet/ndarray/numpy/_op.py ## @@ -3685,6 +3685,51 @@ def get_list(arrays): retur

[GitHub] [incubator-mxnet] ChaiBapchya commented on a change in pull request #16898: Sparse int64 Large tensor support

2020-01-09 Thread GitBox
ChaiBapchya commented on a change in pull request #16898: Sparse int64 Large tensor support URL: https://github.com/apache/incubator-mxnet/pull/16898#discussion_r365056993 ## File path: src/operator/tensor/cast_storage-inl.h ## @@ -283,14 +283,14 @@ struct CopyCsrDataToDns

  1   2   >