[GitHub] [incubator-mxnet] ChaiBapchya commented on issue #17483: Tests failed when I try to build scala-package from source

2020-02-27 Thread GitBox
ChaiBapchya commented on issue #17483: Tests failed when I try to build scala-package from source URL: https://github.com/apache/incubator-mxnet/issues/17483#issuecomment-592375627 @zachgk @lanking520 gentle reminder This

[GitHub] [incubator-mxnet] ChaiBapchya commented on issue #15925: [CI] illegal memory access

2020-02-27 Thread GitBox
ChaiBapchya commented on issue #15925: [CI] illegal memory access URL: https://github.com/apache/incubator-mxnet/issues/15925#issuecomment-592369423 G4 instance with cuda10.0 that is? This is an automated message from the

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

2020-02-27 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 18d526d Bump the

[GitHub] [incubator-mxnet] sxjscience commented on issue #17665: No speedup from using FP16 (4 times slower than PyTorch)

2020-02-27 Thread GitBox
sxjscience commented on issue #17665: No speedup from using FP16 (4 times slower than PyTorch) URL: https://github.com/apache/incubator-mxnet/issues/17665#issuecomment-592326750 @ptrendx Do you have any idea? This is an

[GitHub] [incubator-mxnet] Tommliu opened a new pull request #17717: [Numpy] FFI Bincount

2020-02-27 Thread GitBox
Tommliu opened a new pull request #17717: [Numpy] FFI Bincount URL: https://github.com/apache/incubator-mxnet/pull/17717 ## Description ## Change Operator Registration of Bincount to FFI ## Checklist ## ### Essentials ### Please feel free to remove inapplicable items for

[GitHub] [incubator-mxnet] leezu commented on issue #17713: test_operator_gpu.test_embedding_with_type 'an illegal memory access was encountered'

2020-02-27 Thread GitBox
leezu commented on issue #17713: test_operator_gpu.test_embedding_with_type 'an illegal memory access was encountered' URL: https://github.com/apache/incubator-mxnet/issues/17713#issuecomment-592308134 May be a bug in Cuda 10.0. Can't reproduce on 10.1. However,

[GitHub] [incubator-mxnet] zixuanweeei commented on issue #17702: Support projection feature for LSTM on CPU (Only Inference)

2020-02-27 Thread GitBox
zixuanweeei commented on issue #17702: Support projection feature for LSTM on CPU (Only Inference) URL: https://github.com/apache/incubator-mxnet/pull/17702#issuecomment-592304640 CI has passed last time. The latest commit just added some documents for the projection feature. Accordingly,

[GitHub] [incubator-mxnet] zixuanweeei commented on a change in pull request #17702: Support projection feature for LSTM on CPU (Only Inference)

2020-02-27 Thread GitBox
zixuanweeei commented on a change in pull request #17702: Support projection feature for LSTM on CPU (Only Inference) URL: https://github.com/apache/incubator-mxnet/pull/17702#discussion_r385498649 ## File path: src/operator/rnn.cc ## @@ -385,7 +382,9 @@ The definition of

[incubator-mxnet] branch master updated (0e6ab21 -> 1af06d9)

2020-02-27 Thread haoj
This is an automated email from the ASF dual-hosted git repository. haoj pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. from 0e6ab21 Move cpp-package/include/mxnet-cpp/.gitignore to avoid copying it on installation add 1af06d9

[GitHub] [incubator-mxnet] haojin2 merged pull request #17618: [Numpy] Rewrite sample_n

2020-02-27 Thread GitBox
haojin2 merged pull request #17618: [Numpy] Rewrite sample_n URL: https://github.com/apache/incubator-mxnet/pull/17618 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [incubator-mxnet] connorgoggins commented on issue #17716: [Large Tensor] linalg ops fail w/input dim >= 2**32

2020-02-27 Thread GitBox
connorgoggins commented on issue #17716: [Large Tensor] linalg ops fail w/input dim >= 2**32 URL: https://github.com/apache/incubator-mxnet/issues/17716#issuecomment-592291579 @mxnet-label-bot add [MKL] This is an automated

[GitHub] [incubator-mxnet] connorgoggins removed a comment on issue #17716: [Large Tensor] linalg ops fail w/input dim >= 2**32

2020-02-27 Thread GitBox
connorgoggins removed a comment on issue #17716: [Large Tensor] linalg ops fail w/input dim >= 2**32 URL: https://github.com/apache/incubator-mxnet/issues/17716#issuecomment-592290187 @mxnet-label-bot remove [MKL] This is

[GitHub] [incubator-mxnet] connorgoggins removed a comment on issue #17716: [Large Tensor] linalg ops fail w/input dim >= 2**32

2020-02-27 Thread GitBox
connorgoggins removed a comment on issue #17716: [Large Tensor] linalg ops fail w/input dim >= 2**32 URL: https://github.com/apache/incubator-mxnet/issues/17716#issuecomment-592290007 @mxnet-label-bot add [MKL] This is an

[GitHub] [incubator-mxnet] connorgoggins commented on issue #17716: [Large Tensor] linalg ops fail w/input dim >= 2**32

2020-02-27 Thread GitBox
connorgoggins commented on issue #17716: [Large Tensor] linalg ops fail w/input dim >= 2**32 URL: https://github.com/apache/incubator-mxnet/issues/17716#issuecomment-592290187 @mxnet-label-bot remove [MKL] This is an

[GitHub] [incubator-mxnet] connorgoggins commented on issue #17716: [Large Tensor] linalg ops fail w/input dim >= 2**32

2020-02-27 Thread GitBox
connorgoggins commented on issue #17716: [Large Tensor] linalg ops fail w/input dim >= 2**32 URL: https://github.com/apache/incubator-mxnet/issues/17716#issuecomment-592290007 @mxnet-label-bot add [MKL] This is an automated

[GitHub] [incubator-mxnet] connorgoggins opened a new issue #17716: [Large Tensor] linalg ops fail w/input dim >= 2**32

2020-02-27 Thread GitBox
connorgoggins opened a new issue #17716: [Large Tensor] linalg ops fail w/input dim >= 2**32 URL: https://github.com/apache/incubator-mxnet/issues/17716 ## Description While testing the `linalg_*` ops on large tensor (dimension >= 2**32) data, I found that all of these ops fail with a

[GitHub] [incubator-mxnet] TaoLv edited a comment on issue #17715: MXNet nightly cpu build (w/o MKLDNN) does not work with Horovod

2020-02-27 Thread GitBox
TaoLv edited a comment on issue #17715: MXNet nightly cpu build (w/o MKLDNN) does not work with Horovod URL: https://github.com/apache/incubator-mxnet/issues/17715#issuecomment-592278212 Hmm if you're using the latest nightly build, I'm afraid MKL-DNN is already enabled in it. Could you

[GitHub] [incubator-mxnet] xinyu-intel commented on issue #17679: [MKL-DNN] BatchNormRelu Fusion

2020-02-27 Thread GitBox
xinyu-intel commented on issue #17679: [MKL-DNN] BatchNormRelu Fusion URL: https://github.com/apache/incubator-mxnet/pull/17679#issuecomment-592278870 @zhreshold @hetong007 Can you help take a review? This is an automated

[GitHub] [incubator-mxnet] TaoLv commented on issue #17715: MXNet nightly cpu build (w/o MKLDNN) does not work with Horovod

2020-02-27 Thread GitBox
TaoLv commented on issue #17715: MXNet nightly cpu build (w/o MKLDNN) does not work with Horovod URL: https://github.com/apache/incubator-mxnet/issues/17715#issuecomment-592278212 Hmm if you're using the latest nightly build, I'm afraid MKL-DNN is already enabled in it. Could please

[GitHub] [incubator-mxnet] apeforest opened a new issue #17715: MXNet nightly cpu build (w/o MKLDNN) does not work with Horovod

2020-02-27 Thread GitBox
apeforest opened a new issue #17715: MXNet nightly cpu build (w/o MKLDNN) does not work with Horovod URL: https://github.com/apache/incubator-mxnet/issues/17715 ## Description Installing Horovod with MXNet nightly CPU (w/o MKLDNN) failed. GPU build and CPU mkldnn build are both okay.

[GitHub] [incubator-mxnet] zixuanweeei commented on a change in pull request #17702: Support projection feature for LSTM on CPU (Only Inference)

2020-02-27 Thread GitBox
zixuanweeei commented on a change in pull request #17702: Support projection feature for LSTM on CPU (Only Inference) URL: https://github.com/apache/incubator-mxnet/pull/17702#discussion_r385467427 ## File path: src/operator/rnn.cc ## @@ -385,7 +382,9 @@ The definition of

[incubator-mxnet] branch leezu-patch-1 updated (cd562c4 -> 3f1f22c)

2020-02-27 Thread lausen
This is an automated email from the ASF dual-hosted git repository. lausen pushed a change to branch leezu-patch-1 in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. discard cd562c4 Fix lapack detection on Ubuntu 18.04 in Makefile new 3f1f22c Fix lapack detection on

[incubator-mxnet] 01/01: Fix lapack detection on Ubuntu 18.04 in Makefile

2020-02-27 Thread lausen
This is an automated email from the ASF dual-hosted git repository. lausen pushed a commit to branch leezu-patch-1 in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git commit 3f1f22cea36d49ec58a7f6d02fdc70b544713222 Author: Leonard Lausen AuthorDate: Thu Feb 27 17:07:29 2020

[GitHub] [incubator-mxnet] leezu opened a new pull request #17714: Fix lapack detection on Ubuntu 18.04 in Makefile

2020-02-27 Thread GitBox
leezu opened a new pull request #17714: Fix lapack detection on Ubuntu 18.04 in Makefile URL: https://github.com/apache/incubator-mxnet/pull/17714 This is an automated message from the Apache Git Service. To respond to the

[incubator-mxnet] 01/01: Fix lapack detection on Ubuntu 18.04 in Makefile

2020-02-27 Thread lausen
This is an automated email from the ASF dual-hosted git repository. lausen pushed a commit to branch leezu-patch-1 in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git commit 3f1f22cea36d49ec58a7f6d02fdc70b544713222 Author: Leonard Lausen AuthorDate: Thu Feb 27 17:07:29 2020

[incubator-mxnet] branch leezu-patch-1 updated (cd562c4 -> 3f1f22c)

2020-02-27 Thread lausen
This is an automated email from the ASF dual-hosted git repository. lausen pushed a change to branch leezu-patch-1 in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. discard cd562c4 Fix lapack detection on Ubuntu 18.04 in Makefile new 3f1f22c Fix lapack detection on

[incubator-mxnet] branch leezu-patch-1 updated (cd562c4 -> 3f1f22c)

2020-02-27 Thread lausen
This is an automated email from the ASF dual-hosted git repository. lausen pushed a change to branch leezu-patch-1 in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. discard cd562c4 Fix lapack detection on Ubuntu 18.04 in Makefile new 3f1f22c Fix lapack detection on

[incubator-mxnet] 01/01: Fix lapack detection on Ubuntu 18.04 in Makefile

2020-02-27 Thread lausen
This is an automated email from the ASF dual-hosted git repository. lausen pushed a commit to branch leezu-patch-1 in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git commit 3f1f22cea36d49ec58a7f6d02fdc70b544713222 Author: Leonard Lausen AuthorDate: Thu Feb 27 17:07:29 2020

[incubator-mxnet] branch leezu-patch-1 created (now cd562c4)

2020-02-27 Thread lausen
This is an automated email from the ASF dual-hosted git repository. lausen pushed a change to branch leezu-patch-1 in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. at cd562c4 Fix lapack detection on Ubuntu 18.04 in Makefile This branch includes the following new

[incubator-mxnet] 01/01: Fix lapack detection on Ubuntu 18.04 in Makefile

2020-02-27 Thread lausen
This is an automated email from the ASF dual-hosted git repository. lausen pushed a commit to branch leezu-patch-1 in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git commit cd562c44a7332838f80a3a05e44768f29c4e2348 Author: Leonard Lausen AuthorDate: Thu Feb 27 17:07:29 2020

[incubator-mxnet] 01/01: Fix lapack detection on Ubuntu 18.04 in Makefile

2020-02-27 Thread lausen
This is an automated email from the ASF dual-hosted git repository. lausen pushed a commit to branch leezu-patch-1 in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git commit 3f1f22cea36d49ec58a7f6d02fdc70b544713222 Author: Leonard Lausen AuthorDate: Thu Feb 27 17:07:29 2020

[incubator-mxnet] branch leezu-patch-1 updated (cd562c4 -> 3f1f22c)

2020-02-27 Thread lausen
This is an automated email from the ASF dual-hosted git repository. lausen pushed a change to branch leezu-patch-1 in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. discard cd562c4 Fix lapack detection on Ubuntu 18.04 in Makefile new 3f1f22c Fix lapack detection on

[incubator-mxnet] branch leezu-patch-1 created (now cd562c4)

2020-02-27 Thread lausen
This is an automated email from the ASF dual-hosted git repository. lausen pushed a change to branch leezu-patch-1 in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. at cd562c4 Fix lapack detection on Ubuntu 18.04 in Makefile This branch includes the following new

[incubator-mxnet] 01/01: Fix lapack detection on Ubuntu 18.04 in Makefile

2020-02-27 Thread lausen
This is an automated email from the ASF dual-hosted git repository. lausen pushed a commit to branch leezu-patch-1 in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git commit cd562c44a7332838f80a3a05e44768f29c4e2348 Author: Leonard Lausen AuthorDate: Thu Feb 27 17:07:29 2020

[incubator-mxnet] branch leezu-patch-1 created (now cd562c4)

2020-02-27 Thread lausen
This is an automated email from the ASF dual-hosted git repository. lausen pushed a change to branch leezu-patch-1 in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. at cd562c4 Fix lapack detection on Ubuntu 18.04 in Makefile This branch includes the following new

[incubator-mxnet] 01/01: Fix lapack detection on Ubuntu 18.04 in Makefile

2020-02-27 Thread lausen
This is an automated email from the ASF dual-hosted git repository. lausen pushed a commit to branch leezu-patch-1 in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git commit cd562c44a7332838f80a3a05e44768f29c4e2348 Author: Leonard Lausen AuthorDate: Thu Feb 27 17:07:29 2020

[incubator-mxnet] branch master updated (55e6987 -> 0e6ab21)

2020-02-27 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 55e6987 cmake: remove -mf16c flag for android build (#17523) add 0e6ab21 Move

[incubator-mxnet] branch master updated (55e6987 -> 0e6ab21)

2020-02-27 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 55e6987 cmake: remove -mf16c flag for android build (#17523) add 0e6ab21 Move

[incubator-mxnet] branch master updated (55e6987 -> 0e6ab21)

2020-02-27 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 55e6987 cmake: remove -mf16c flag for android build (#17523) add 0e6ab21 Move

[incubator-mxnet] branch master updated: Move cpp-package/include/mxnet-cpp/.gitignore to avoid copying it on installation

2020-02-27 Thread lausen
This is an automated email from the ASF dual-hosted git repository. lausen 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 0e6ab21 Move

[GitHub] [incubator-mxnet] leezu closed issue #17704: cpp-package: .gitignore in wrong path

2020-02-27 Thread GitBox
leezu closed issue #17704: cpp-package: .gitignore in wrong path URL: https://github.com/apache/incubator-mxnet/issues/17704 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [incubator-mxnet] leezu merged pull request #17709: Fix #17704: move the .gitignore file to rigth path

2020-02-27 Thread GitBox
leezu merged pull request #17709: Fix #17704: move the .gitignore file to rigth path URL: https://github.com/apache/incubator-mxnet/pull/17709 This is an automated message from the Apache Git Service. To respond to the

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

2020-02-27 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 552825e Bump the

[GitHub] [incubator-mxnet] eric-haibin-lin commented on a change in pull request #17702: Support projection feature for LSTM on CPU (Only Inference)

2020-02-27 Thread GitBox
eric-haibin-lin commented on a change in pull request #17702: Support projection feature for LSTM on CPU (Only Inference) URL: https://github.com/apache/incubator-mxnet/pull/17702#discussion_r385442120 ## File path: src/operator/rnn.cc ## @@ -385,7 +382,9 @@ The

[GitHub] [incubator-mxnet] ptrendx commented on issue #17658: Update website, README and NEWS with 1.6.0

2020-02-27 Thread GitBox
ptrendx commented on issue #17658: Update website, README and NEWS with 1.6.0 URL: https://github.com/apache/incubator-mxnet/pull/17658#issuecomment-592233655 Ok, I believe it should be ready for review. I updated: - README, - NEWS.md (including fixing multiple not working

[GitHub] [incubator-mxnet] leezu commented on issue #15925: [CI] illegal memory access

2020-02-27 Thread GitBox
leezu commented on issue #15925: [CI] illegal memory access URL: https://github.com/apache/incubator-mxnet/issues/15925#issuecomment-592233431 Could the CI issue be related to https://github.com/apache/incubator-mxnet/issues/17713 ? That can be reproduced deterministically on G4 instance

[GitHub] [incubator-mxnet] leezu opened a new issue #17713: test_operator_gpu.test_embedding_with_type 'an illegal memory access was encountered'

2020-02-27 Thread GitBox
leezu opened a new issue #17713: test_operator_gpu.test_embedding_with_type 'an illegal memory access was encountered' URL: https://github.com/apache/incubator-mxnet/issues/17713 ## Description Embedding operator in `test_operator_gpu.test_embedding_with_type` triggers illegal memory

[GitHub] [incubator-mxnet] ChaiBapchya commented on issue #14994: Flaky test: test_lstm_clip

2020-02-27 Thread GitBox
ChaiBapchya commented on issue #14994: Flaky test: test_lstm_clip URL: https://github.com/apache/incubator-mxnet/issues/14994#issuecomment-592214753 For unrelated PR #17487

[GitHub] [incubator-mxnet] sxjscience commented on issue #17665: No speedup from using FP16 (4 times slower than PyTorch)

2020-02-27 Thread GitBox
sxjscience commented on issue #17665: No speedup from using FP16 (4 times slower than PyTorch) URL: https://github.com/apache/incubator-mxnet/issues/17665#issuecomment-592204581 I tried with `nvprof` and find that MXNet and PyTorch uses different kernels: For MXNet, it's

[GitHub] [incubator-mxnet] access2rohit commented on a change in pull request #17632: [Large Tensor] Fixed RNN op

2020-02-27 Thread GitBox
access2rohit commented on a change in pull request #17632: [Large Tensor] Fixed RNN op URL: https://github.com/apache/incubator-mxnet/pull/17632#discussion_r385399026 ## File path: src/operator/rnn_impl.h ## @@ -127,9 +127,9 @@ void LstmForwardTraining(DType* ws,

[GitHub] [incubator-mxnet] access2rohit commented on a change in pull request #17632: [Large Tensor] Fixed RNN op

2020-02-27 Thread GitBox
access2rohit commented on a change in pull request #17632: [Large Tensor] Fixed RNN op URL: https://github.com/apache/incubator-mxnet/pull/17632#discussion_r385399105 ## File path: src/operator/rnn_impl.h ## @@ -146,15 +146,15 @@ void LstmForwardTraining(DType* ws,

[GitHub] [incubator-mxnet] access2rohit commented on a change in pull request #17632: [Large Tensor] Fixed RNN op

2020-02-27 Thread GitBox
access2rohit commented on a change in pull request #17632: [Large Tensor] Fixed RNN op URL: https://github.com/apache/incubator-mxnet/pull/17632#discussion_r385398150 ## File path: src/operator/rnn-inl.h ## @@ -361,9 +361,9 @@ void RNNBackward(DType* ws,

[GitHub] [incubator-mxnet] access2rohit commented on a change in pull request #17632: [Large Tensor] Fixed RNN op

2020-02-27 Thread GitBox
access2rohit commented on a change in pull request #17632: [Large Tensor] Fixed RNN op URL: https://github.com/apache/incubator-mxnet/pull/17632#discussion_r385397998 ## File path: src/operator/rnn-inl.h ## @@ -320,9 +320,9 @@ void RNNForwardInference(DType* ws,

[GitHub] [incubator-mxnet] access2rohit commented on a change in pull request #17632: [Large Tensor] Fixed RNN op

2020-02-27 Thread GitBox
access2rohit commented on a change in pull request #17632: [Large Tensor] Fixed RNN op URL: https://github.com/apache/incubator-mxnet/pull/17632#discussion_r385397900 ## File path: src/operator/rnn-inl.h ## @@ -278,9 +278,9 @@ void RNNForwardTraining(DType* ws,

[GitHub] [incubator-mxnet] access2rohit commented on a change in pull request #17632: [Large Tensor] Fixed RNN op

2020-02-27 Thread GitBox
access2rohit commented on a change in pull request #17632: [Large Tensor] Fixed RNN op URL: https://github.com/apache/incubator-mxnet/pull/17632#discussion_r385397647 ## File path: src/operator/rnn-inl.h ## @@ -213,8 +213,8 @@ inline size_t GetRNNWorkspaceSize(int

[GitHub] [incubator-mxnet] access2rohit commented on a change in pull request #17632: [Large Tensor] Fixed RNN op

2020-02-27 Thread GitBox
access2rohit commented on a change in pull request #17632: [Large Tensor] Fixed RNN op URL: https://github.com/apache/incubator-mxnet/pull/17632#discussion_r385397242 ## File path: src/operator/rnn-inl.h ## @@ -123,7 +123,7 @@ struct RNNParam : public dmlc::Parameter {

[GitHub] [incubator-mxnet] access2rohit commented on a change in pull request #17632: [Large Tensor] Fixed RNN op

2020-02-27 Thread GitBox
access2rohit commented on a change in pull request #17632: [Large Tensor] Fixed RNN op URL: https://github.com/apache/incubator-mxnet/pull/17632#discussion_r385396554 ## File path: src/operator/rnn-inl.h ## @@ -213,8 +213,8 @@ inline size_t GetRNNWorkspaceSize(int

[GitHub] [incubator-mxnet] access2rohit commented on a change in pull request #17632: [Large Tensor] Fixed RNN op

2020-02-27 Thread GitBox
access2rohit commented on a change in pull request #17632: [Large Tensor] Fixed RNN op URL: https://github.com/apache/incubator-mxnet/pull/17632#discussion_r385396355 ## File path: src/operator/rnn-inl.h ## @@ -123,7 +123,7 @@ struct RNNParam : public dmlc::Parameter {

[GitHub] [incubator-mxnet] access2rohit commented on a change in pull request #17632: [Large Tensor] Fixed RNN op

2020-02-27 Thread GitBox
access2rohit commented on a change in pull request #17632: [Large Tensor] Fixed RNN op URL: https://github.com/apache/incubator-mxnet/pull/17632#discussion_r385396554 ## File path: src/operator/rnn-inl.h ## @@ -213,8 +213,8 @@ inline size_t GetRNNWorkspaceSize(int

[GitHub] [incubator-mxnet] access2rohit commented on a change in pull request #17632: [Large Tensor] Fixed RNN op

2020-02-27 Thread GitBox
access2rohit commented on a change in pull request #17632: [Large Tensor] Fixed RNN op URL: https://github.com/apache/incubator-mxnet/pull/17632#discussion_r385396355 ## File path: src/operator/rnn-inl.h ## @@ -123,7 +123,7 @@ struct RNNParam : public dmlc::Parameter {

[GitHub] [incubator-mxnet] access2rohit commented on a change in pull request #17632: [Large Tensor] Fixed RNN op

2020-02-27 Thread GitBox
access2rohit commented on a change in pull request #17632: [Large Tensor] Fixed RNN op URL: https://github.com/apache/incubator-mxnet/pull/17632#discussion_r385393949 ## File path: src/operator/rnn-inl.h ## @@ -140,14 +140,14 @@ inline int GetRnnParamSize(int num_layer,

[GitHub] [incubator-mxnet] access2rohit commented on a change in pull request #17632: [Large Tensor] Fixed RNN op

2020-02-27 Thread GitBox
access2rohit commented on a change in pull request #17632: [Large Tensor] Fixed RNN op URL: https://github.com/apache/incubator-mxnet/pull/17632#discussion_r385394072 ## File path: src/operator/rnn-inl.h ## @@ -182,8 +182,8 @@ inline int GetRnnBiasSize(int num_layer, *

[GitHub] [incubator-mxnet] leezu commented on issue #17712: Gluon Block hooks reference counting bug

2020-02-27 Thread GitBox
leezu commented on issue #17712: Gluon Block hooks reference counting bug URL: https://github.com/apache/incubator-mxnet/issues/17712#issuecomment-592195594 Seems to be an upstream bug, so I opened https://bugs.python.org/issue39778

[GitHub] [incubator-mxnet] leezu closed issue #17712: Gluon Block hooks reference counting bug

2020-02-27 Thread GitBox
leezu closed issue #17712: Gluon Block hooks reference counting bug URL: https://github.com/apache/incubator-mxnet/issues/17712 This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [incubator-mxnet] leezu opened a new issue #17712: Gluon Block hooks reference counting bug

2020-02-27 Thread GitBox
leezu opened a new issue #17712: Gluon Block hooks reference counting bug URL: https://github.com/apache/incubator-mxnet/issues/17712 ## Description Garbage collection can trigger `_PyObject_AssertFailed` if Gluon Block hooks are used. ### Error Message ``` bash

[GitHub] [incubator-mxnet] snnn commented on issue #17711: [ONNX export] Fixing spatial export for batchnorm

2020-02-27 Thread GitBox
snnn commented on issue #17711: [ONNX export] Fixing spatial export for batchnorm URL: https://github.com/apache/incubator-mxnet/pull/17711#issuecomment-592158271 LGTM. This is an automated message from the Apache Git

[GitHub] [incubator-mxnet] vinitra opened a new pull request #17711: [ONNX export] Fixing spatial export for batchnorm

2020-02-27 Thread GitBox
vinitra opened a new pull request #17711: [ONNX export] Fixing spatial export for batchnorm URL: https://github.com/apache/incubator-mxnet/pull/17711 ## Description ## In the ONNX model zoo, we noticed that models like ArcFace and DUC that have been exported from mxnet with batchnorm

[GitHub] [incubator-mxnet] gromovadarya90 opened a new issue #17710: WANT TO SEE HOT PICS WITH ME?   

2020-02-27 Thread GitBox
gromovadarya90 opened a new issue #17710: WANT TO SEE HOT PICS WITH ME?    URL: https://github.com/apache/incubator-mxnet/issues/17710 WANT TO SEE HOT PICS WITH ME?       https://is.gd/ms4xm3 This is an

[GitHub] [incubator-mxnet] ChaiBapchya commented on a change in pull request #17487: [OpPerf] Consolidate array manipulation related operators

2020-02-27 Thread GitBox
ChaiBapchya commented on a change in pull request #17487: [OpPerf] Consolidate array manipulation related operators URL: https://github.com/apache/incubator-mxnet/pull/17487#discussion_r385342488 ## File path: benchmark/opperf/README.md ## @@ -72,6 +73,8 @@ python

[GitHub] [incubator-mxnet] ChaiBapchya commented on a change in pull request #17487: [OpPerf] Consolidate array manipulation related operators

2020-02-27 Thread GitBox
ChaiBapchya commented on a change in pull request #17487: [OpPerf] Consolidate array manipulation related operators URL: https://github.com/apache/incubator-mxnet/pull/17487#discussion_r379225052 ## File path: benchmark/opperf/utils/op_registry_utils.py ## @@ -137,26

[GitHub] [incubator-mxnet] sxjscience edited a comment on issue #17665: No speedup from using FP16 (4 times slower than PyTorch)

2020-02-27 Thread GitBox
sxjscience edited a comment on issue #17665: No speedup from using FP16 (4 times slower than PyTorch) URL: https://github.com/apache/incubator-mxnet/issues/17665#issuecomment-591832250 I can replicate the performance gap. Also, I added `mx.nd.waitall()` in the first script:

[GitHub] [incubator-mxnet] sxjscience edited a comment on issue #17665: No speedup from using FP16 (4 times slower than PyTorch)

2020-02-27 Thread GitBox
sxjscience edited a comment on issue #17665: No speedup from using FP16 (4 times slower than PyTorch) URL: https://github.com/apache/incubator-mxnet/issues/17665#issuecomment-591832250 I can replicate the performance gap. Also, I added `mx.nd.waitall()` in the first script:

[GitHub] [incubator-mxnet] ChaiBapchya commented on a change in pull request #17487: [OpPerf] Consolidate array manipulation related operators

2020-02-27 Thread GitBox
ChaiBapchya commented on a change in pull request #17487: [OpPerf] Consolidate array manipulation related operators URL: https://github.com/apache/incubator-mxnet/pull/17487#discussion_r385322042 ## File path: benchmark/opperf/utils/op_registry_utils.py ## @@ -137,26

[GitHub] [incubator-mxnet] ChaiBapchya commented on issue #17487: [OpPerf] Consolidate array manipulation related operators

2020-02-27 Thread GitBox
ChaiBapchya commented on issue #17487: [OpPerf] Consolidate array manipulation related operators URL: https://github.com/apache/incubator-mxnet/pull/17487#issuecomment-592133902 > Can you please this list -

[GitHub] [incubator-mxnet] sl1pkn07 opened a new pull request #17709: Fix #17704: move the .gitignore file to rigth path

2020-02-27 Thread GitBox
sl1pkn07 opened a new pull request #17709: Fix #17704: move the .gitignore file to rigth path URL: https://github.com/apache/incubator-mxnet/pull/17709 …(this avoid to install it when run make install) ## Description ## Fix #17704 ## Checklist ## ### Essentials ###

[GitHub] [incubator-mxnet] sl1pkn07 commented on issue #17704: cpp-package: .gitignore in wrong path

2020-02-27 Thread GitBox
sl1pkn07 commented on issue #17704: cpp-package: .gitignore in wrong path URL: https://github.com/apache/incubator-mxnet/issues/17704#issuecomment-592124328 done This is an automated message from the Apache Git Service. To

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

2020-02-27 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 25caa25 Bump the

[GitHub] [incubator-mxnet] eric-haibin-lin commented on issue #17651: Distributed training with kvstore crashes if worker has different number of data batches

2020-02-27 Thread GitBox
eric-haibin-lin commented on issue #17651: Distributed training with kvstore crashes if worker has different number of data batches URL: https://github.com/apache/incubator-mxnet/issues/17651#issuecomment-592112039 I think that's correct. Please let me know if you come across issues using

[GitHub] [incubator-mxnet] connorgoggins commented on a change in pull request #17677: [Large Tensor] Fix cumsum op

2020-02-27 Thread GitBox
connorgoggins commented on a change in pull request #17677: [Large Tensor] Fix cumsum op URL: https://github.com/apache/incubator-mxnet/pull/17677#discussion_r385289205 ## File path: src/operator/numpy/np_cumsum-inl.h ## @@ -98,11 +98,11 @@ void CumsumForwardImpl(const

[GitHub] [incubator-mxnet] connorgoggins commented on a change in pull request #17677: [Large Tensor] Fix cumsum op

2020-02-27 Thread GitBox
connorgoggins commented on a change in pull request #17677: [Large Tensor] Fix cumsum op URL: https://github.com/apache/incubator-mxnet/pull/17677#discussion_r385289319 ## File path: src/operator/numpy/np_cumsum-inl.h ## @@ -157,10 +157,10 @@ void CumsumBackwardImpl(const

[GitHub] [incubator-mxnet] connorgoggins commented on a change in pull request #17677: [Large Tensor] Fix cumsum op

2020-02-27 Thread GitBox
connorgoggins commented on a change in pull request #17677: [Large Tensor] Fix cumsum op URL: https://github.com/apache/incubator-mxnet/pull/17677#discussion_r385289461 ## File path: src/operator/numpy/np_cumsum-inl.h ## @@ -157,10 +157,10 @@ void CumsumBackwardImpl(const

[GitHub] [incubator-mxnet] apeforest commented on a change in pull request #17677: [Large Tensor] Fix cumsum op

2020-02-27 Thread GitBox
apeforest commented on a change in pull request #17677: [Large Tensor] Fix cumsum op URL: https://github.com/apache/incubator-mxnet/pull/17677#discussion_r385286229 ## File path: src/operator/numpy/np_cumsum-inl.h ## @@ -157,10 +157,10 @@ void CumsumBackwardImpl(const

[GitHub] [incubator-mxnet] access2rohit commented on a change in pull request #17677: [Large Tensor] Fix cumsum op

2020-02-27 Thread GitBox
access2rohit commented on a change in pull request #17677: [Large Tensor] Fix cumsum op URL: https://github.com/apache/incubator-mxnet/pull/17677#discussion_r385285544 ## File path: src/operator/numpy/np_cumsum-inl.h ## @@ -157,10 +157,10 @@ void CumsumBackwardImpl(const

[GitHub] [incubator-mxnet] access2rohit commented on a change in pull request #17677: [Large Tensor] Fix cumsum op

2020-02-27 Thread GitBox
access2rohit commented on a change in pull request #17677: [Large Tensor] Fix cumsum op URL: https://github.com/apache/incubator-mxnet/pull/17677#discussion_r385285641 ## File path: src/operator/numpy/np_cumsum-inl.h ## @@ -157,10 +157,10 @@ void CumsumBackwardImpl(const

[GitHub] [incubator-mxnet] apeforest commented on a change in pull request #17677: [Large Tensor] Fix cumsum op

2020-02-27 Thread GitBox
apeforest commented on a change in pull request #17677: [Large Tensor] Fix cumsum op URL: https://github.com/apache/incubator-mxnet/pull/17677#discussion_r385284895 ## File path: src/operator/numpy/np_cumsum-inl.h ## @@ -98,11 +98,11 @@ void CumsumForwardImpl(const

[GitHub] [incubator-mxnet] leezu commented on issue #17704: cpp-package: .gitignore in wrong path

2020-02-27 Thread GitBox
leezu commented on issue #17704: cpp-package: .gitignore in wrong path URL: https://github.com/apache/incubator-mxnet/issues/17704#issuecomment-592091298 It's valid to have .gitignore in subdirectories, so it's not necessary to change all. `cpp-package/include/mxnet-cpp/.gitignore` is

[GitHub] [incubator-mxnet] leezu commented on issue #17708: Silence all compiler warnings when build

2020-02-27 Thread GitBox
leezu commented on issue #17708: Silence all compiler warnings when build URL: https://github.com/apache/incubator-mxnet/issues/17708#issuecomment-592087728 @hzfan could you fix the warnings in ffi that you introduced. CI already tests with `-WError` to prevent introducing more

[GitHub] [incubator-mxnet] sl1pkn07 opened a new issue #17708: Silence all compiler warnings when build

2020-02-27 Thread GitBox
sl1pkn07 opened a new issue #17708: Silence all compiler warnings when build URL: https://github.com/apache/incubator-mxnet/issues/17708 ## Description silence all warnings when build the project ## References when build the project, a tons of warnings like ~~~

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

2020-02-27 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 431890f Bump the

[GitHub] [incubator-mxnet] EmilPi edited a comment on issue #17454: IdentityAttachKLSparseReg - operator argument error

2020-02-27 Thread GitBox
EmilPi edited a comment on issue #17454: IdentityAttachKLSparseReg - operator argument error URL: https://github.com/apache/incubator-mxnet/issues/17454#issuecomment-591908426 Any news on this? I have the same issue with mxnet==1.5.0 and mxnet-cu100==1.5.0 installed with pip3 .

[GitHub] [incubator-mxnet] EmilPi commented on issue #17454: IdentityAttachKLSparseReg - operator argument error

2020-02-27 Thread GitBox
EmilPi commented on issue #17454: IdentityAttachKLSparseReg - operator argument error URL: https://github.com/apache/incubator-mxnet/issues/17454#issuecomment-591908426 Any news on this? This is an automated message from

[GitHub] [incubator-mxnet] ciyongch commented on issue #17705: mkldnn quantized FC is slow

2020-02-27 Thread GitBox
ciyongch commented on issue #17705: mkldnn quantized FC is slow URL: https://github.com/apache/incubator-mxnet/issues/17705#issuecomment-591868495 @eric-haibin-lin I just created a PR https://github.com/apache/incubator-mxnet/pull/17707 to address this issue, please take a review.

[GitHub] [incubator-mxnet] ciyongch opened a new pull request #17707: [MKLDNN] Remove overhead of sg_mkldnn_fullyconnected op

2020-02-27 Thread GitBox
ciyongch opened a new pull request #17707: [MKLDNN] Remove overhead of sg_mkldnn_fullyconnected op URL: https://github.com/apache/incubator-mxnet/pull/17707 ## Description ## This PR is mainly focus on removing the overhead of sg_mkldnn_fullyconnected especially in the case of

[GitHub] [incubator-mxnet] sl1pkn07 commented on issue #17704: cpp-package: .gitignore in wrong path

2020-02-27 Thread GitBox
sl1pkn07 commented on issue #17704: cpp-package: .gitignore in wrong path URL: https://github.com/apache/incubator-mxnet/issues/17704#issuecomment-591866571 ~~~ ┌─┤[$]|[sl1pkn07]|[sL1pKn07]|[~/aplicaciones/incubator-mxnet]| └───╼ find . -name .gitignore ./python/.gitignore

[GitHub] [incubator-mxnet] sl1pkn07 closed pull request #17580: fix #17579

2020-02-27 Thread GitBox
sl1pkn07 closed pull request #17580: fix #17579 URL: https://github.com/apache/incubator-mxnet/pull/17580 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[GitHub] [incubator-mxnet] hkvision commented on issue #17651: Distributed training with kvstore crashes if worker has different number of data batches

2020-02-27 Thread GitBox
hkvision commented on issue #17651: Distributed training with kvstore crashes if worker has different number of data batches URL: https://github.com/apache/incubator-mxnet/issues/17651#issuecomment-591834890 @eric-haibin-lin Hi, I'm using NDArrayIter. I checked ResizeIter, so in order

[GitHub] [incubator-mxnet] hkvision edited a comment on issue #17651: Distributed training with kvstore crashes if worker has different number of data batches

2020-02-27 Thread GitBox
hkvision edited a comment on issue #17651: Distributed training with kvstore crashes if worker has different number of data batches URL: https://github.com/apache/incubator-mxnet/issues/17651#issuecomment-591834890 @eric-haibin-lin Hi, I'm using NDArrayIter. I checked ResizeIter and it