[GitHub] [incubator-mxnet] ckt624 commented on a change in pull request #15349: Numpy Tensordot Operator

2019-07-11 Thread GitBox
ckt624 commented on a change in pull request #15349: Numpy Tensordot Operator URL: https://github.com/apache/incubator-mxnet/pull/15349#discussion_r302849369 ## File path: src/operator/numpy/np_tensordot_op-inl.h ## @@ -0,0 +1,575 @@ +/* + * Licensed to the Apache Software

[GitHub] [incubator-mxnet] ckt624 commented on a change in pull request #15349: Numpy Tensordot Operator

2019-07-11 Thread GitBox
ckt624 commented on a change in pull request #15349: Numpy Tensordot Operator URL: https://github.com/apache/incubator-mxnet/pull/15349#discussion_r302849525 ## File path: src/operator/numpy/np_tensordot_op-inl.h ## @@ -0,0 +1,575 @@ +/* + * Licensed to the Apache Software

[GitHub] [incubator-mxnet] ckt624 commented on a change in pull request #15349: Numpy Tensordot Operator

2019-07-11 Thread GitBox
ckt624 commented on a change in pull request #15349: Numpy Tensordot Operator URL: https://github.com/apache/incubator-mxnet/pull/15349#discussion_r302849247 ## File path: src/operator/numpy/np_tensordot_op-inl.h ## @@ -0,0 +1,575 @@ +/* + * Licensed to the Apache Software

[GitHub] [incubator-mxnet] ckt624 commented on a change in pull request #15349: Numpy Tensordot Operator

2019-07-11 Thread GitBox
ckt624 commented on a change in pull request #15349: Numpy Tensordot Operator URL: https://github.com/apache/incubator-mxnet/pull/15349#discussion_r302844970 ## File path: src/operator/numpy/np_tensordot_op.cc ## @@ -0,0 +1,344 @@ +/* + * Licensed to the Apache Software Fo

[GitHub] [incubator-mxnet] yzhliu commented on issue #15465: [RFC] Integrate TVM into Apache MXNet

2019-07-11 Thread GitBox
yzhliu commented on issue #15465: [RFC] Integrate TVM into Apache MXNet URL: https://github.com/apache/incubator-mxnet/issues/15465#issuecomment-510756109 @larroy In this proposal it cannot. It requires more deeply integrated with tvm/relay to run graph-level optimization, which @junrushao

[GitHub] [incubator-mxnet] sandeep-krishnamurthy commented on a change in pull request #15245: FP16 Support for C Predict API

2019-07-11 Thread GitBox
sandeep-krishnamurthy commented on a change in pull request #15245: FP16 Support for C Predict API URL: https://github.com/apache/incubator-mxnet/pull/15245#discussion_r302831853 ## File path: amalgamation/python/mxnet_predict.py ## @@ -160,10 +249,18 @@ def forward(self,

[GitHub] [incubator-mxnet] apeforest commented on a change in pull request #15169: Softmax with length

2019-07-11 Thread GitBox
apeforest commented on a change in pull request #15169: Softmax with length URL: https://github.com/apache/incubator-mxnet/pull/15169#discussion_r302823990 ## File path: src/operator/nn/softmax-inl.h ## @@ -464,35 +755,56 @@ void SoftmaxGradCompute(const nnvm::NodeAttrs& at

[GitHub] [incubator-mxnet] apeforest commented on a change in pull request #15169: Softmax with length

2019-07-11 Thread GitBox
apeforest commented on a change in pull request #15169: Softmax with length URL: https://github.com/apache/incubator-mxnet/pull/15169#discussion_r302822638 ## File path: src/operator/nn/softmax-inl.h ## @@ -464,35 +755,56 @@ void SoftmaxGradCompute(const nnvm::NodeAttrs& at

[GitHub] [incubator-mxnet] apeforest commented on a change in pull request #15169: Softmax with length

2019-07-11 Thread GitBox
apeforest commented on a change in pull request #15169: Softmax with length URL: https://github.com/apache/incubator-mxnet/pull/15169#discussion_r302822454 ## File path: tests/python/unittest/test_operator.py ## @@ -7838,6 +7871,7 @@ def get_output_names_callback(name, arr)

[GitHub] [incubator-mxnet] apeforest commented on a change in pull request #15169: Softmax with length

2019-07-11 Thread GitBox
apeforest commented on a change in pull request #15169: Softmax with length URL: https://github.com/apache/incubator-mxnet/pull/15169#discussion_r302822362 ## File path: src/operator/nn/softmax-inl.h ## @@ -113,6 +113,60 @@ inline void Softmax(Stream *s, DType *in, OType *o

[GitHub] [incubator-mxnet] pengzhao-intel commented on issue #15518: [MKLDNN] Enable subgraph backend mkldnn by default.

2019-07-11 Thread GitBox
pengzhao-intel commented on issue #15518: [MKLDNN] Enable subgraph backend mkldnn by default. URL: https://github.com/apache/incubator-mxnet/pull/15518#issuecomment-510737400 @reminisce @zheng-da please help take a review and let us know if you have any concerns. -

[GitHub] [incubator-mxnet] haojin2 commented on a change in pull request #15461: Numpy logspace

2019-07-11 Thread GitBox
haojin2 commented on a change in pull request #15461: Numpy logspace URL: https://github.com/apache/incubator-mxnet/pull/15461#discussion_r302819106 ## File path: tests/python/unittest/test_numpy_op.py ## @@ -1162,6 +1162,70 @@ def test_np_broadcast_arrays(): pass

[GitHub] [incubator-mxnet] haojin2 commented on a change in pull request #15461: Numpy logspace

2019-07-11 Thread GitBox
haojin2 commented on a change in pull request #15461: Numpy logspace URL: https://github.com/apache/incubator-mxnet/pull/15461#discussion_r302818965 ## File path: python/mxnet/ndarray/numpy/_op.py ## @@ -1691,3 +1692,101 @@ def arcsin(x, out=None, **kwargs): http://www

[GitHub] [incubator-mxnet] haojin2 commented on a change in pull request #15461: Numpy logspace

2019-07-11 Thread GitBox
haojin2 commented on a change in pull request #15461: Numpy logspace URL: https://github.com/apache/incubator-mxnet/pull/15461#discussion_r302818876 ## File path: python/mxnet/ndarray/numpy/_op.py ## @@ -1691,3 +1692,101 @@ def arcsin(x, out=None, **kwargs): http://www

[GitHub] [incubator-mxnet] haojin2 commented on a change in pull request #15456: [Numpy] numpy compatible invert

2019-07-11 Thread GitBox
haojin2 commented on a change in pull request #15456: [Numpy] numpy compatible invert URL: https://github.com/apache/incubator-mxnet/pull/15456#discussion_r302818755 ## File path: python/mxnet/ndarray/numpy/_op.py ## @@ -1414,6 +1413,48 @@ def rint(x, out=None, **kwargs):

[GitHub] [incubator-mxnet] haojin2 commented on a change in pull request #15456: [Numpy] numpy compatible invert

2019-07-11 Thread GitBox
haojin2 commented on a change in pull request #15456: [Numpy] numpy compatible invert URL: https://github.com/apache/incubator-mxnet/pull/15456#discussion_r302818678 ## File path: python/mxnet/ndarray/numpy/_op.py ## @@ -1414,6 +1413,48 @@ def rint(x, out=None, **kwargs):

[GitHub] [incubator-mxnet] haojin2 commented on a change in pull request #15456: [Numpy] numpy compatible invert

2019-07-11 Thread GitBox
haojin2 commented on a change in pull request #15456: [Numpy] numpy compatible invert URL: https://github.com/apache/incubator-mxnet/pull/15456#discussion_r302818708 ## File path: python/mxnet/ndarray/numpy/_op.py ## @@ -1414,6 +1413,48 @@ def rint(x, out=None, **kwargs):

[incubator-mxnet] branch numpy updated (1bff19b -> 19c7c6e)

2019-07-11 Thread haoj
This is an automated email from the ASF dual-hosted git repository. haoj pushed a change to branch numpy in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. from 1bff19b [Numpy] Numpy compatible argsort (#15501) add 19c7c6e fix memory override bug in multinomial (#1

[GitHub] [incubator-mxnet] haojin2 merged pull request #15397: fix memory overwrite bug in numpy multinomial

2019-07-11 Thread GitBox
haojin2 merged pull request #15397: fix memory overwrite bug in numpy multinomial URL: https://github.com/apache/incubator-mxnet/pull/15397 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [incubator-mxnet] haojin2 commented on a change in pull request #15434: [numpy] numpy copysign

2019-07-11 Thread GitBox
haojin2 commented on a change in pull request #15434: [numpy] numpy copysign URL: https://github.com/apache/incubator-mxnet/pull/15434#discussion_r302818417 ## File path: src/operator/numpy/np_elemwise_broadcast_op.cc ## @@ -198,5 +218,25 @@ MXNET_OPERATOR_REGISTER_NP_BINA

[GitHub] [incubator-mxnet] haojin2 commented on a change in pull request #15434: [numpy] numpy copysign

2019-07-11 Thread GitBox
haojin2 commented on a change in pull request #15434: [numpy] numpy copysign URL: https://github.com/apache/incubator-mxnet/pull/15434#discussion_r302818417 ## File path: src/operator/numpy/np_elemwise_broadcast_op.cc ## @@ -198,5 +218,25 @@ MXNET_OPERATOR_REGISTER_NP_BINA

[GitHub] [incubator-mxnet] haojin2 commented on a change in pull request #15371: numpy compatible dsplit operator

2019-07-11 Thread GitBox
haojin2 commented on a change in pull request #15371: numpy compatible dsplit operator URL: https://github.com/apache/incubator-mxnet/pull/15371#discussion_r302817980 ## File path: tests/python/unittest/test_numpy_op.py ## @@ -485,6 +485,55 @@ def hybrid_forward(self, F, a

[GitHub] [incubator-mxnet] haojin2 commented on a change in pull request #15371: numpy compatible dsplit operator

2019-07-11 Thread GitBox
haojin2 commented on a change in pull request #15371: numpy compatible dsplit operator URL: https://github.com/apache/incubator-mxnet/pull/15371#discussion_r302818005 ## File path: tests/python/unittest/test_numpy_op.py ## @@ -485,6 +485,55 @@ def hybrid_forward(self, F, a

[GitHub] [incubator-mxnet] haojin2 commented on issue #15350: Add operator arctan2 with its test code

2019-07-11 Thread GitBox
haojin2 commented on issue #15350: Add operator arctan2 with its test code URL: https://github.com/apache/incubator-mxnet/pull/15350#issuecomment-510733604 @tingying2020 Your rebase has messed up your commit history, please clean it up so that this PR only contains your own commits. --

[GitHub] [incubator-mxnet] haojin2 commented on a change in pull request #15346: numpy_split support param type fix; numpy_vsplit added

2019-07-11 Thread GitBox
haojin2 commented on a change in pull request #15346: numpy_split support param type fix; numpy_vsplit added URL: https://github.com/apache/incubator-mxnet/pull/15346#discussion_r302817747 ## File path: python/mxnet/numpy/multiarray.py ## @@ -1835,6 +1835,83 @@ def split(a

[GitHub] [incubator-mxnet] haojin2 commented on a change in pull request #15325: [Numpy] Numpy vstack

2019-07-11 Thread GitBox
haojin2 commented on a change in pull request #15325: [Numpy] Numpy vstack URL: https://github.com/apache/incubator-mxnet/pull/15325#discussion_r302817626 ## File path: tests/python/unittest/test_numpy_op.py ## @@ -1162,6 +1162,63 @@ def test_np_broadcast_arrays(): pas

[GitHub] [incubator-mxnet] haojin2 commented on a change in pull request #15325: [Numpy] Numpy vstack

2019-07-11 Thread GitBox
haojin2 commented on a change in pull request #15325: [Numpy] Numpy vstack URL: https://github.com/apache/incubator-mxnet/pull/15325#discussion_r302817626 ## File path: tests/python/unittest/test_numpy_op.py ## @@ -1162,6 +1162,63 @@ def test_np_broadcast_arrays(): pas

[GitHub] [incubator-mxnet] haojin2 commented on a change in pull request #15314: [Numpy] Numpy dstack

2019-07-11 Thread GitBox
haojin2 commented on a change in pull request #15314: [Numpy] Numpy dstack URL: https://github.com/apache/incubator-mxnet/pull/15314#discussion_r302817068 ## File path: src/operator/numpy/np_matrix_op.cc ## @@ -55,6 +55,67 @@ bool NumpyTransposeShape(const nnvm::NodeAttrs&

[GitHub] [incubator-mxnet] haojin2 commented on a change in pull request #15314: [Numpy] Numpy dstack

2019-07-11 Thread GitBox
haojin2 commented on a change in pull request #15314: [Numpy] Numpy dstack URL: https://github.com/apache/incubator-mxnet/pull/15314#discussion_r302817044 ## File path: src/operator/numpy/np_matrix_op.cc ## @@ -55,6 +55,67 @@ bool NumpyTransposeShape(const nnvm::NodeAttrs&

[GitHub] [incubator-mxnet] haojin2 commented on a change in pull request #15314: [Numpy] Numpy dstack

2019-07-11 Thread GitBox
haojin2 commented on a change in pull request #15314: [Numpy] Numpy dstack URL: https://github.com/apache/incubator-mxnet/pull/15314#discussion_r302816983 ## File path: src/operator/nn/concat-inl.h ## @@ -141,6 +141,37 @@ void ConcatCompute(const nnvm::NodeAttrs& attrs, con

[GitHub] [incubator-mxnet] haojin2 commented on a change in pull request #15312: Numpy bitwise_xor operator

2019-07-11 Thread GitBox
haojin2 commented on a change in pull request #15312: Numpy bitwise_xor operator URL: https://github.com/apache/incubator-mxnet/pull/15312#discussion_r302816832 ## File path: src/operator/numpy/np_elemwise_binary_op.cc ## @@ -0,0 +1,51 @@ +/* + * Licensed to the Apache Soft

[GitHub] [incubator-mxnet] haojin2 commented on a change in pull request #15312: Numpy bitwise_xor operator

2019-07-11 Thread GitBox
haojin2 commented on a change in pull request #15312: Numpy bitwise_xor operator URL: https://github.com/apache/incubator-mxnet/pull/15312#discussion_r302816645 ## File path: src/operator/numpy/np_elemwise_binary_op.cc ## @@ -0,0 +1,51 @@ +/* + * Licensed to the Apache Soft

[GitHub] [incubator-mxnet] haojin2 commented on a change in pull request #15302: [Numpy] Numpy hstack

2019-07-11 Thread GitBox
haojin2 commented on a change in pull request #15302: [Numpy] Numpy hstack URL: https://github.com/apache/incubator-mxnet/pull/15302#discussion_r302816499 ## File path: src/operator/numpy/np_matrix_op.cc ## @@ -55,6 +55,60 @@ bool NumpyTransposeShape(const nnvm::NodeAttrs&

[GitHub] [incubator-mxnet] haojin2 commented on a change in pull request #15302: [Numpy] Numpy hstack

2019-07-11 Thread GitBox
haojin2 commented on a change in pull request #15302: [Numpy] Numpy hstack URL: https://github.com/apache/incubator-mxnet/pull/15302#discussion_r302816555 ## File path: tests/python/unittest/test_numpy_op.py ## @@ -860,6 +860,72 @@ def get_new_shape(shape, axis):

[GitHub] [incubator-mxnet] haojin2 commented on issue #15293: Numpy bitwise OR operator

2019-07-11 Thread GitBox
haojin2 commented on issue #15293: Numpy bitwise OR operator URL: https://github.com/apache/incubator-mxnet/pull/15293#issuecomment-510730018 @endvroy Your rebase messed up your commit history, please clean up so that your PR only contains your own commits.

[GitHub] [incubator-mxnet] haojin2 commented on a change in pull request #15349: Numpy Tensordot Operator

2019-07-11 Thread GitBox
haojin2 commented on a change in pull request #15349: Numpy Tensordot Operator URL: https://github.com/apache/incubator-mxnet/pull/15349#discussion_r302813785 ## File path: tests/python/unittest/test_numpy_op.py ## @@ -26,6 +26,151 @@ from mxnet.test_utils import check_nu

[GitHub] [incubator-mxnet] haojin2 commented on a change in pull request #15349: Numpy Tensordot Operator

2019-07-11 Thread GitBox
haojin2 commented on a change in pull request #15349: Numpy Tensordot Operator URL: https://github.com/apache/incubator-mxnet/pull/15349#discussion_r302812085 ## File path: src/operator/numpy/np_tensordot_op.cc ## @@ -0,0 +1,344 @@ +/* + * Licensed to the Apache Software F

[GitHub] [incubator-mxnet] haojin2 commented on issue #15169: Softmax with length

2019-07-11 Thread GitBox
haojin2 commented on issue #15169: Softmax with length URL: https://github.com/apache/incubator-mxnet/pull/15169#issuecomment-510725925 @szha @eric-haibin-lin Finally the CI passed... Please give a review when you have time.

[GitHub] [incubator-mxnet] haojin2 commented on a change in pull request #15349: Numpy Tensordot Operator

2019-07-11 Thread GitBox
haojin2 commented on a change in pull request #15349: Numpy Tensordot Operator URL: https://github.com/apache/incubator-mxnet/pull/15349#discussion_r302804904 ## File path: src/operator/numpy/np_tensordot_op-inl.h ## @@ -0,0 +1,575 @@ +/* + * Licensed to the Apache Softwar

[GitHub] [incubator-mxnet] haojin2 commented on a change in pull request #15349: Numpy Tensordot Operator

2019-07-11 Thread GitBox
haojin2 commented on a change in pull request #15349: Numpy Tensordot Operator URL: https://github.com/apache/incubator-mxnet/pull/15349#discussion_r302804904 ## File path: src/operator/numpy/np_tensordot_op-inl.h ## @@ -0,0 +1,575 @@ +/* + * Licensed to the Apache Softwar

[GitHub] [incubator-mxnet] haojin2 commented on a change in pull request #15349: Numpy Tensordot Operator

2019-07-11 Thread GitBox
haojin2 commented on a change in pull request #15349: Numpy Tensordot Operator URL: https://github.com/apache/incubator-mxnet/pull/15349#discussion_r302804045 ## File path: src/operator/numpy/np_tensordot_op-inl.h ## @@ -0,0 +1,575 @@ +/* + * Licensed to the Apache Softwar

[GitHub] [incubator-mxnet] haojin2 commented on a change in pull request #15349: Numpy Tensordot Operator

2019-07-11 Thread GitBox
haojin2 commented on a change in pull request #15349: Numpy Tensordot Operator URL: https://github.com/apache/incubator-mxnet/pull/15349#discussion_r302804104 ## File path: src/operator/numpy/np_tensordot_op-inl.h ## @@ -0,0 +1,575 @@ +/* + * Licensed to the Apache Softwar

[GitHub] [incubator-mxnet] haojin2 commented on a change in pull request #15349: Numpy Tensordot Operator

2019-07-11 Thread GitBox
haojin2 commented on a change in pull request #15349: Numpy Tensordot Operator URL: https://github.com/apache/incubator-mxnet/pull/15349#discussion_r302803909 ## File path: src/operator/numpy/np_tensordot_op-inl.h ## @@ -0,0 +1,575 @@ +/* + * Licensed to the Apache Softwar

[GitHub] [incubator-mxnet] braindotai removed a comment on issue #15429: Operator Performance Regression on CPU

2019-07-11 Thread GitBox
braindotai removed a comment on issue #15429: Operator Performance Regression on CPU URL: https://github.com/apache/incubator-mxnet/issues/15429#issuecomment-509612532 According to the above mentioned benchmark results, it looks like v1.5 is gonna be more faster than v1.4. Am I right? --

[GitHub] [incubator-mxnet] ChaiBapchya removed a comment on issue #15475: Add transpose_conv, sorting and searching operator benchmarks to Opperf

2019-07-11 Thread GitBox
ChaiBapchya removed a comment on issue #15475: Add transpose_conv, sorting and searching operator benchmarks to Opperf URL: https://github.com/apache/incubator-mxnet/pull/15475#issuecomment-510713389 Apart from that weird Travis build error (which I guess will go by retriggering), LGTM! -

[GitHub] [incubator-mxnet] ChaiBapchya commented on issue #15475: Add transpose_conv, sorting and searching operator benchmarks to Opperf

2019-07-11 Thread GitBox
ChaiBapchya commented on issue #15475: Add transpose_conv, sorting and searching operator benchmarks to Opperf URL: https://github.com/apache/incubator-mxnet/pull/15475#issuecomment-510713389 Apart from that weird Travis build error (which I guess will go by retriggering), LGTM! -

[GitHub] [incubator-mxnet] ChaiBapchya commented on issue #15516: Fix memory leak reported by ASAN in NNVM to ONNX conversion

2019-07-11 Thread GitBox
ChaiBapchya commented on issue #15516: Fix memory leak reported by ASAN in NNVM to ONNX conversion URL: https://github.com/apache/incubator-mxnet/pull/15516#issuecomment-510711867 @larroy isn't it consistent with the existing naming system? That is, to call the instance of that class a

[incubator-mxnet] branch ib/bump-jl10 deleted (was 907f066)

2019-07-11 Thread iblis
This is an automated email from the ASF dual-hosted git repository. iblis pushed a change to branch ib/bump-jl10 in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. was 907f066 CI: upgrade Julia version from 1.0.3 to 1.0.4 The revisions that were on this branch are stil

[GitHub] [incubator-mxnet] zixuanweeei commented on issue #15497: [MKLDNN] Independent gradients requests check with respect to weights and bias of convolution

2019-07-11 Thread GitBox
zixuanweeei commented on issue #15497: [MKLDNN] Independent gradients requests check with respect to weights and bias of convolution URL: https://github.com/apache/incubator-mxnet/pull/15497#issuecomment-510705077 @matteosal I tested your example with commit 9ca042, and it ran successfully

[GitHub] [incubator-mxnet] zixuanweeei edited a comment on issue #15497: [MKLDNN] Independent gradients requests check with respect to weights and bias of convolution

2019-07-11 Thread GitBox
zixuanweeei edited a comment on issue #15497: [MKLDNN] Independent gradients requests check with respect to weights and bias of convolution URL: https://github.com/apache/incubator-mxnet/pull/15497#issuecomment-510705077 @matteosal I tested your example with commit 9ca0428, and it ran succ

[GitHub] [incubator-mxnet] Zha0q1 edited a comment on issue #15490: Utility to help developers debug operators: Tensor Inspector

2019-07-11 Thread GitBox
Zha0q1 edited a comment on issue #15490: Utility to help developers debug operators: Tensor Inspector URL: https://github.com/apache/incubator-mxnet/pull/15490#issuecomment-510702857 Created a separate PR for a new tutorial on usage. https://github.com/apache/incubator-mxnet/pull/15517

[GitHub] [incubator-mxnet] Zha0q1 commented on issue #15490: Utility to help developers debug operators: Tensor Inspector

2019-07-11 Thread GitBox
Zha0q1 commented on issue #15490: Utility to help developers debug operators: Tensor Inspector URL: https://github.com/apache/incubator-mxnet/pull/15490#issuecomment-510702857 Created a separate PR for a new tutorial on how to usage. https://github.com/apache/incubator-mxnet/pull/15517

[GitHub] [incubator-mxnet] Zha0q1 opened a new pull request #15517: Tensor Inspector Tutorial

2019-07-11 Thread GitBox
Zha0q1 opened a new pull request #15517: Tensor Inspector Tutorial URL: https://github.com/apache/incubator-mxnet/pull/15517 ## Description ## Tutorial for this PR: https://github.com/apache/incubator-mxnet/pull/15490 ## Checklist ## ### Essentials ### Please feel free to rem

[GitHub] [incubator-mxnet] larroy commented on issue #14779: [WIP] Fully connected, higher order grad

2019-07-11 Thread GitBox
larroy commented on issue #14779: [WIP] Fully connected, higher order grad URL: https://github.com/apache/incubator-mxnet/pull/14779#issuecomment-510700193 Let me know if you have questions about notation or the development or see some mistakes.

[GitHub] [incubator-mxnet] larroy commented on a change in pull request #15449: cuda/cuDNN lib version checking. Force cuDNN v7 usage.

2019-07-11 Thread GitBox
larroy commented on a change in pull request #15449: cuda/cuDNN lib version checking. Force cuDNN v7 usage. URL: https://github.com/apache/incubator-mxnet/pull/15449#discussion_r302783900 ## File path: src/common/cuda_utils.cc ## @@ -0,0 +1,108 @@ +/* + * Licensed to the

[GitHub] [incubator-mxnet] larroy commented on issue #15465: [RFC] Integrate TVM into Apache MXNet

2019-07-11 Thread GitBox
larroy commented on issue #15465: [RFC] Integrate TVM into Apache MXNet URL: https://github.com/apache/incubator-mxnet/issues/15465#issuecomment-510688231 Can it fuse different operators in the graph? I think this would be badly needed for us.

[GitHub] [incubator-mxnet] DickJC123 commented on a change in pull request #15449: cuda/cuDNN lib version checking. Force cuDNN v7 usage.

2019-07-11 Thread GitBox
DickJC123 commented on a change in pull request #15449: cuda/cuDNN lib version checking. Force cuDNN v7 usage. URL: https://github.com/apache/incubator-mxnet/pull/15449#discussion_r302772718 ## File path: src/common/cuda_utils.cc ## @@ -0,0 +1,108 @@ +/* + * Licensed to t

[incubator-mxnet] branch master updated: Rebase #13757 to master (#15189)

2019-07-11 Thread wkcn
This is an automated email from the ASF dual-hosted git repository. wkcn 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 554b196 Rebase #13757 to master (#15189)

[GitHub] [incubator-mxnet] wkcn commented on issue #15189: Rebase #13757 to master

2019-07-11 Thread GitBox
wkcn commented on issue #15189: Rebase #13757 to master URL: https://github.com/apache/incubator-mxnet/pull/15189#issuecomment-510674853 Thanks for your contribution! This is an automated message from the Apache Git Service. T

[GitHub] [incubator-mxnet] wkcn merged pull request #15189: Rebase #13757 to master

2019-07-11 Thread GitBox
wkcn merged pull request #15189: Rebase #13757 to master URL: https://github.com/apache/incubator-mxnet/pull/15189 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub a

[GitHub] [incubator-mxnet] Zha0q1 commented on a change in pull request #15490: Utility to help developers debug operators: Tensor Inspector

2019-07-11 Thread GitBox
Zha0q1 commented on a change in pull request #15490: Utility to help developers debug operators: Tensor Inspector URL: https://github.com/apache/incubator-mxnet/pull/15490#discussion_r302753473 ## File path: src/common/tensor_inspector.h ## @@ -0,0 +1,717 @@ +/* + * Licens

[GitHub] [incubator-mxnet] rondogency commented on a change in pull request #15490: Utility to help developers debug operators: Tensor Inspector

2019-07-11 Thread GitBox
rondogency commented on a change in pull request #15490: Utility to help developers debug operators: Tensor Inspector URL: https://github.com/apache/incubator-mxnet/pull/15490#discussion_r302749772 ## File path: src/common/tensor_inspector.h ## @@ -0,0 +1,717 @@ +/* + * Li

[incubator-mxnet] branch master updated (68460a9 -> 1f3195f)

2019-07-11 Thread aaronmarkham
This is an automated email from the ASF dual-hosted git repository. aaronmarkham pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. from 68460a9 [Doc] Add MKL install method apt/yum into tutorial (#15491) add 1f3195f Julia docs (#15

[incubator-mxnet] branch master updated (1f3195f -> 5ffd598)

2019-07-11 Thread aaronmarkham
This is an automated email from the ASF dual-hosted git repository. aaronmarkham pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. from 1f3195f Julia docs (#15454) add 5ffd598 CI: upgrade Julia version from 1.0.3 to 1.0.4 (#15502)

[GitHub] [incubator-mxnet] aaronmarkham closed issue #15415: documentation link is broken, goes to spam site

2019-07-11 Thread GitBox
aaronmarkham closed issue #15415: documentation link is broken, goes to spam site URL: https://github.com/apache/incubator-mxnet/issues/15415 This is an automated message from the Apache Git Service. To respond to the messag

[GitHub] [incubator-mxnet] aaronmarkham merged pull request #15502: CI: upgrade Julia version from 1.0.3 to 1.0.4

2019-07-11 Thread GitBox
aaronmarkham merged pull request #15502: CI: upgrade Julia version from 1.0.3 to 1.0.4 URL: https://github.com/apache/incubator-mxnet/pull/15502 This is an automated message from the Apache Git Service. To respond to the mes

[GitHub] [incubator-mxnet] aaronmarkham merged pull request #15454: Julia docs

2019-07-11 Thread GitBox
aaronmarkham merged pull request #15454: Julia docs URL: https://github.com/apache/incubator-mxnet/pull/15454 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and us

[GitHub] [incubator-mxnet] Zha0q1 edited a comment on issue #15490: Utility to help developers debug operators: Tensor Inspector

2019-07-11 Thread GitBox
Zha0q1 edited a comment on issue #15490: Utility to help developers debug operators: Tensor Inspector URL: https://github.com/apache/incubator-mxnet/pull/15490#issuecomment-510654624 Just realized there were more than one Tensor structs. Added `mshadow::` before `Tensor` so compiler would

[GitHub] [incubator-mxnet] Zha0q1 edited a comment on issue #15490: Utility to help developers debug operators: Tensor Inspector

2019-07-11 Thread GitBox
Zha0q1 edited a comment on issue #15490: Utility to help developers debug operators: Tensor Inspector URL: https://github.com/apache/incubator-mxnet/pull/15490#issuecomment-510654624 Just realized there were more than one Tensor structs. Added `mshadow::` before `Tensor` so compiler would

[GitHub] [incubator-mxnet] Zha0q1 commented on issue #15490: Utility to help developers debug operators: Tensor Inspector

2019-07-11 Thread GitBox
Zha0q1 commented on issue #15490: Utility to help developers debug operators: Tensor Inspector URL: https://github.com/apache/incubator-mxnet/pull/15490#issuecomment-510654624 Just realized there were more than one Tensor structs. Added `shadow::` before `Tensor` so compiler would not be c

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

2019-07-11 Thread marcoabreu
This is an automated email from the ASF dual-hosted git repository. marcoabreu 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 02913a4 Bump the publish

[GitHub] [incubator-mxnet] vandanavk commented on issue #13627: [MXNET-895] ONNX import/export: TopK

2019-07-11 Thread GitBox
vandanavk commented on issue #13627: [MXNET-895] ONNX import/export: TopK URL: https://github.com/apache/incubator-mxnet/pull/13627#issuecomment-510633603 @zhreshold @anirudhacharya for review This is an automated message fro

[GitHub] [incubator-mxnet] vandanavk commented on issue #13627: [MXNET-895] ONNX import/export: TopK

2019-07-11 Thread GitBox
vandanavk commented on issue #13627: [MXNET-895] ONNX import/export: TopK URL: https://github.com/apache/incubator-mxnet/pull/13627#issuecomment-510630308 @mxnet-label-bot remove [pr-work-in-progress] This is an automated mess

[GitHub] [incubator-mxnet] larroy edited a comment on issue #14779: [WIP] Fully connected, higher order grad

2019-07-11 Thread GitBox
larroy edited a comment on issue #14779: [WIP] Fully connected, higher order grad URL: https://github.com/apache/incubator-mxnet/pull/14779#issuecomment-510624357 The development is only for flattened inputs. What I call Pi is the product of additional dimensions which are flattened, as s

[GitHub] [incubator-mxnet] larroy commented on issue #14779: [WIP] Fully connected, higher order grad

2019-07-11 Thread GitBox
larroy commented on issue #14779: [WIP] Fully connected, higher order grad URL: https://github.com/apache/incubator-mxnet/pull/14779#issuecomment-510624357 The development is only for flattened inputs. This is an automated mes

[GitHub] [incubator-mxnet] larroy commented on issue #14779: [WIP] Fully connected, higher order grad

2019-07-11 Thread GitBox
larroy commented on issue #14779: [WIP] Fully connected, higher order grad URL: https://github.com/apache/incubator-mxnet/pull/14779#issuecomment-510624220 Adding development for review: ![IMG_20190711_123355__01](https://user-images.githubusercontent.com/928489/61079741-4eb10c00-a3d8-11

[GitHub] [incubator-mxnet] larroy commented on a change in pull request #15253: [MXNET-978] Add higher order gradient support `tan`, `tanh`

2019-07-11 Thread GitBox
larroy commented on a change in pull request #15253: [MXNET-978] Add higher order gradient support `tan`, `tanh` URL: https://github.com/apache/incubator-mxnet/pull/15253#discussion_r302708484 ## File path: src/operator/tensor/elemwise_unary_op_trig.cc ## @@ -139,7 +139,31

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

2019-07-11 Thread marcoabreu
This is an automated email from the ASF dual-hosted git repository. marcoabreu 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 2feec54 Bump the publish

[GitHub] [incubator-mxnet] larroy edited a comment on issue #15285: Graph dumper

2019-07-11 Thread GitBox
larroy edited a comment on issue #15285: Graph dumper URL: https://github.com/apache/incubator-mxnet/pull/15285#issuecomment-509858209 @ptrendx Thanks I will try to use your suggestion in the next iteration. This is an auto

[GitHub] [incubator-mxnet] larroy commented on a change in pull request #15449: cuda/cuDNN lib version checking. Force cuDNN v7 usage.

2019-07-11 Thread GitBox
larroy commented on a change in pull request #15449: cuda/cuDNN lib version checking. Force cuDNN v7 usage. URL: https://github.com/apache/incubator-mxnet/pull/15449#discussion_r302695533 ## File path: src/common/cuda_utils.cc ## @@ -0,0 +1,108 @@ +/* + * Licensed to the

[GitHub] [incubator-mxnet] kevinzh92 opened a new pull request #15516: Fix memory leak reported by ASAN in NNVM to ONNX conversion

2019-07-11 Thread GitBox
kevinzh92 opened a new pull request #15516: Fix memory leak reported by ASAN in NNVM to ONNX conversion URL: https://github.com/apache/incubator-mxnet/pull/15516 ## Description ## Fix memory leak reported by ASAN in NNVM to ONNX conversion of a constant. ## Checklist ## ### Ess

[GitHub] [incubator-mxnet] ChaiBapchya edited a comment on issue #15506: Improving error message

2019-07-11 Thread GitBox
ChaiBapchya edited a comment on issue #15506: Improving error message URL: https://github.com/apache/incubator-mxnet/issues/15506#issuecomment-510581504 So you're right. Root cause was related to disk space. (Though it is weird how even after mounting 100gig of EBS volume on /data director

[GitHub] [incubator-mxnet] ChaiBapchya commented on issue #15506: Improving error message

2019-07-11 Thread GitBox
ChaiBapchya commented on issue #15506: Improving error message URL: https://github.com/apache/incubator-mxnet/issues/15506#issuecomment-510581504 Issue resolved by 1. Switching to Ubuntu AMI (instead of DL AMI that comes with loads of other packages and libraries I don't need) 2. Inc

[GitHub] [incubator-mxnet] DickJC123 commented on a change in pull request #15449: cuda/cuDNN lib version checking. Force cuDNN v7 usage.

2019-07-11 Thread GitBox
DickJC123 commented on a change in pull request #15449: cuda/cuDNN lib version checking. Force cuDNN v7 usage. URL: https://github.com/apache/incubator-mxnet/pull/15449#discussion_r302657800 ## File path: src/common/cuda_utils.cc ## @@ -0,0 +1,108 @@ +/* + * Licensed to t

[GitHub] [incubator-mxnet] ChaiBapchya commented on issue #15506: Improving error message

2019-07-11 Thread GitBox
ChaiBapchya commented on issue #15506: Improving error message URL: https://github.com/apache/incubator-mxnet/issues/15506#issuecomment-510574285 I had an issue with this part of that error ` when writing output to : ` And I am running this on p2.8x large GPU-optimized AWS EC2 ins

[GitHub] [incubator-mxnet] frankfliu commented on issue #15514: dmlc.lib is not generated

2019-07-11 Thread GitBox
frankfliu commented on issue #15514: dmlc.lib is not generated URL: https://github.com/apache/incubator-mxnet/issues/15514#issuecomment-510568547 @mxnet-label-bot add [build] This is an automated message from the Apache Git S

[GitHub] [incubator-mxnet] frankfliu commented on issue #15512: possible bug in nd.gather_nd

2019-07-11 Thread GitBox
frankfliu commented on issue #15512: possible bug in nd.gather_nd URL: https://github.com/apache/incubator-mxnet/issues/15512#issuecomment-510568295 @mxnet-label-bot add [backend, cuda, question] This is an automated message

[GitHub] [incubator-mxnet] frankfliu commented on issue #15509: Train models under the director '/mxnet/example/gluon/'

2019-07-11 Thread GitBox
frankfliu commented on issue #15509: Train models under the director '/mxnet/example/gluon/' URL: https://github.com/apache/incubator-mxnet/issues/15509#issuecomment-510566851 @mxnet-label-bot add [Question, distributed, training] --

[GitHub] [incubator-mxnet] kshitij12345 commented on a change in pull request #15253: [MXNET-978] Add higher order gradient support `tan`, `tanh`

2019-07-11 Thread GitBox
kshitij12345 commented on a change in pull request #15253: [MXNET-978] Add higher order gradient support `tan`, `tanh` URL: https://github.com/apache/incubator-mxnet/pull/15253#discussion_r30264 ## File path: src/operator/tensor/elemwise_unary_op_trig.cc ## @@ -139,7 +

[GitHub] [incubator-mxnet] matteosal commented on issue #15497: [MKLDNN] Independent gradients requests check with respect to weights and bias of convolution

2019-07-11 Thread GitBox
matteosal commented on issue #15497: [MKLDNN] Independent gradients requests check with respect to weights and bias of convolution URL: https://github.com/apache/incubator-mxnet/pull/15497#issuecomment-510563332 The example of https://github.com/apache/incubator-mxnet/issues/15464 is fixed

[GitHub] [incubator-mxnet] access2rohit commented on issue #15490: Utility to help developers debug operators: Tensor Inspector

2019-07-11 Thread GitBox
access2rohit commented on issue #15490: Utility to help developers debug operators: Tensor Inspector URL: https://github.com/apache/incubator-mxnet/pull/15490#issuecomment-510561397 Can you split the PR into 2 separate PRs. One for CPU and one for GPU ?

[GitHub] [incubator-mxnet] iblis17 commented on issue #15454: Julia docs

2019-07-11 Thread GitBox
iblis17 commented on issue #15454: Julia docs URL: https://github.com/apache/incubator-mxnet/pull/15454#issuecomment-510545691 Nice work! @aaronmarkham This is an automated message from the Apache Git Service. To respond to t

[GitHub] [incubator-mxnet] iblis17 commented on a change in pull request #15454: Julia docs

2019-07-11 Thread GitBox
iblis17 commented on a change in pull request #15454: Julia docs URL: https://github.com/apache/incubator-mxnet/pull/15454#discussion_r302620627 ## File path: docs/install/ubuntu_setup.md ## @@ -310,25 +310,93 @@ Refer to the [Clojure setup guide](https://github.com/apache

[GitHub] [incubator-mxnet] aaronmarkham commented on a change in pull request #15454: Julia docs

2019-07-11 Thread GitBox
aaronmarkham commented on a change in pull request #15454: Julia docs URL: https://github.com/apache/incubator-mxnet/pull/15454#discussion_r302619949 ## File path: docs/install/ubuntu_setup.md ## @@ -310,25 +310,93 @@ Refer to the [Clojure setup guide](https://github.com/a

[GitHub] [incubator-mxnet] aaronmarkham commented on a change in pull request #15454: Julia docs

2019-07-11 Thread GitBox
aaronmarkham commented on a change in pull request #15454: Julia docs URL: https://github.com/apache/incubator-mxnet/pull/15454#discussion_r302618951 ## File path: docs/install/ubuntu_setup.md ## @@ -310,25 +310,93 @@ Refer to the [Clojure setup guide](https://github.com/a

[GitHub] [incubator-mxnet] qqaatw edited a comment on issue #15484: Binding Model fails with simple_bind error

2019-07-11 Thread GitBox
qqaatw edited a comment on issue #15484: Binding Model fails with simple_bind error URL: https://github.com/apache/incubator-mxnet/issues/15484#issuecomment-510480630 @Maicus I have the same problem as you, could you tell more in detail about the Target and the TargetCode Section? thanks.

[GitHub] [incubator-mxnet] kshitij12345 opened a new pull request #15515: [MXNET-978] Higher Order Gradient Support `arcsin`, `arccos`.

2019-07-11 Thread GitBox
kshitij12345 opened a new pull request #15515: [MXNET-978] Higher Order Gradient Support `arcsin`, `arccos`. URL: https://github.com/apache/incubator-mxnet/pull/15515 ## Description ## PR intends to add support for higher order gradient for `arcsin`, `arccos`. ## Checklist ## #

[GitHub] [incubator-mxnet] smissan opened a new issue #15514: dmlc.lib is not generated

2019-07-11 Thread GitBox
smissan opened a new issue #15514: dmlc.lib is not generated URL: https://github.com/apache/incubator-mxnet/issues/15514 when trying to build latest head, dmlc.dll is being generated, but not dmlc.lib (using ms visual studio 2015). Therefore, building mxnet dll fails with 1>-- Bu

[GitHub] [incubator-mxnet] mxnet-label-bot commented on issue #15514: dmlc.lib is not generated

2019-07-11 Thread GitBox
mxnet-label-bot commented on issue #15514: dmlc.lib is not generated URL: https://github.com/apache/incubator-mxnet/issues/15514#issuecomment-510484008 Hey, this is the MXNet Label Bot. Thank you for submitting the issue! I will try and suggest some labels so that the appropriate MXNe

[GitHub] [incubator-mxnet] marcoabreu commented on issue #15506: Improving error message

2019-07-11 Thread GitBox
marcoabreu commented on issue #15506: Improving error message URL: https://github.com/apache/incubator-mxnet/issues/15506#issuecomment-510483367 ```No space left on device``` Sounds pretty clear to me. Is this not talking about disk space? What was the root cause? -

  1   2   >