[GitHub] eric-haibin-lin commented on issue #8590: the error:mshadow::cuda::AddTakeGrad' : ambiguous call to overloaded function indexing_op.h

2017-11-07 Thread GitBox
eric-haibin-lin commented on issue #8590: the error:mshadow::cuda::AddTakeGrad' : ambiguous call to overloaded function indexing_op.h URL: https://github.com/apache/incubator-mxnet/issues/8590#issuecomment-342734066 Is that a build failure? What's your MXNet version/commit id and output

[GitHub] reminisce commented on a change in pull request #8566: optimize broadcast

2017-11-07 Thread GitBox
reminisce commented on a change in pull request #8566: optimize broadcast URL: https://github.com/apache/incubator-mxnet/pull/8566#discussion_r149591356 ## File path: src/operator/mxnet_op.h ## @@ -345,6 +394,13 @@ __global__ void mxnet_generic_kernel(int N, Args... args)

[GitHub] szha commented on a change in pull request #8512: gluon rnn refactor

2017-11-07 Thread GitBox
szha commented on a change in pull request #8512: gluon rnn refactor URL: https://github.com/apache/incubator-mxnet/pull/8512#discussion_r149585223 ## File path: tests/python/unittest/test_gluon_rnn.py ## @@ -25,6 +25,7 @@ def test_rnn(): cell =

[GitHub] eric-haibin-lin commented on a change in pull request #8512: gluon rnn refactor

2017-11-07 Thread GitBox
eric-haibin-lin commented on a change in pull request #8512: gluon rnn refactor URL: https://github.com/apache/incubator-mxnet/pull/8512#discussion_r149585026 ## File path: tests/python/unittest/test_gluon_rnn.py ## @@ -25,6 +25,7 @@ def test_rnn(): cell =

[GitHub] tz-hmc opened a new issue #8591: How do I make a siamese network with pretrained models (esp. keeping the weights the same?)

2017-11-07 Thread GitBox
tz-hmc opened a new issue #8591: How do I make a siamese network with pretrained models (esp. keeping the weights the same?) URL: https://github.com/apache/incubator-mxnet/issues/8591 ## Description How do I ensure the weights are kept the same? Can I unpack the internal layers somehow

[GitHub] 994088895 commented on issue #8590: the error:mshadow::cuda::AddTakeGrad' : ambiguous call to overloaded function indexing_op.h

2017-11-07 Thread GitBox
994088895 commented on issue #8590: the error:mshadow::cuda::AddTakeGrad' : ambiguous call to overloaded function indexing_op.h URL: https://github.com/apache/incubator-mxnet/issues/8590#issuecomment-342721281 I have the error mshadow::cuda::AddTakeGrad' : ambiguous call to overloaded

[GitHub] 994088895 opened a new issue #8590: the error:mshadow::cuda::AddTakeGrad' : ambiguous call to overloaded function indexing_op.h

2017-11-07 Thread GitBox
994088895 opened a new issue #8590: the error:mshadow::cuda::AddTakeGrad' : ambiguous call to overloaded function indexing_op.h URL: https://github.com/apache/incubator-mxnet/issues/8590 Note: Providing complete information in the most concise form is the best way to get help. This issue

[GitHub] ZiyueHuang commented on a change in pull request #8539: expose group2ctx to module

2017-11-07 Thread GitBox
ZiyueHuang commented on a change in pull request #8539: expose group2ctx to module URL: https://github.com/apache/incubator-mxnet/pull/8539#discussion_r149582373 ## File path: tests/python/unittest/test_module.py ## @@ -70,6 +70,63 @@ def test_module_input_grads():

[GitHub] ZiyueHuang commented on a change in pull request #8539: expose group2ctx to module

2017-11-07 Thread GitBox
ZiyueHuang commented on a change in pull request #8539: expose group2ctx to module URL: https://github.com/apache/incubator-mxnet/pull/8539#discussion_r149582373 ## File path: tests/python/unittest/test_module.py ## @@ -70,6 +70,63 @@ def test_module_input_grads():

[GitHub] ZiyueHuang commented on a change in pull request #8539: expose group2ctx to module

2017-11-07 Thread GitBox
ZiyueHuang commented on a change in pull request #8539: expose group2ctx to module URL: https://github.com/apache/incubator-mxnet/pull/8539#discussion_r149582123 ## File path: python/mxnet/module/module.py ## @@ -73,6 +75,11 @@ def __init__(self, symbol,

[GitHub] ZiyueHuang commented on a change in pull request #8539: expose group2ctx to module

2017-11-07 Thread GitBox
ZiyueHuang commented on a change in pull request #8539: expose group2ctx to module URL: https://github.com/apache/incubator-mxnet/pull/8539#discussion_r149581557 ## File path: python/mxnet/module/module.py ## @@ -59,10 +59,12 @@ class Module(BaseModule): state_names

[GitHub] ZiyueHuang commented on a change in pull request #8539: expose group2ctx to module

2017-11-07 Thread GitBox
ZiyueHuang commented on a change in pull request #8539: expose group2ctx to module URL: https://github.com/apache/incubator-mxnet/pull/8539#discussion_r149581557 ## File path: python/mxnet/module/module.py ## @@ -59,10 +59,12 @@ class Module(BaseModule): state_names

[GitHub] eric-haibin-lin commented on a change in pull request #8539: expose group2ctx to module

2017-11-07 Thread GitBox
eric-haibin-lin commented on a change in pull request #8539: expose group2ctx to module URL: https://github.com/apache/incubator-mxnet/pull/8539#discussion_r149578575 ## File path: python/mxnet/module/module.py ## @@ -73,6 +75,11 @@ def __init__(self, symbol,

[GitHub] eric-haibin-lin commented on a change in pull request #8539: expose group2ctx to module

2017-11-07 Thread GitBox
eric-haibin-lin commented on a change in pull request #8539: expose group2ctx to module URL: https://github.com/apache/incubator-mxnet/pull/8539#discussion_r149578869 ## File path: python/mxnet/module/module.py ## @@ -59,10 +59,12 @@ class Module(BaseModule):

[GitHub] eric-haibin-lin commented on a change in pull request #8539: expose group2ctx to module

2017-11-07 Thread GitBox
eric-haibin-lin commented on a change in pull request #8539: expose group2ctx to module URL: https://github.com/apache/incubator-mxnet/pull/8539#discussion_r149578224 ## File path: tests/python/unittest/test_module.py ## @@ -70,6 +70,63 @@ def test_module_input_grads():

[GitHub] kevinthesun opened a new issue #8589: [Discussion] Data Pipeline Intermediate Representation in MXNet/NNVM

2017-11-07 Thread GitBox
kevinthesun opened a new issue #8589: [Discussion] Data Pipeline Intermediate Representation in MXNet/NNVM URL: https://github.com/apache/incubator-mxnet/issues/8589 Tensorflow has a transform package https://github.com/tensorflow/transform which is capable of export a data

[GitHub] zhreshold commented on issue #8562: Number of output from the mx.sym.SoftmaxOutput

2017-11-07 Thread GitBox
zhreshold commented on issue #8562: Number of output from the mx.sym.SoftmaxOutput URL: https://github.com/apache/incubator-mxnet/issues/8562#issuecomment-342693812 what you need: ``` ... fc1_act = xxx fc2 = mx.symbol.FullyConnected(data=fc1_act, num_hidden=2) lenet =

[GitHub] zhreshold commented on issue #8573: /usr/bin/ld: cannot find -lgtest ?

2017-11-07 Thread GitBox
zhreshold commented on issue #8573: /usr/bin/ld: cannot find -lgtest ? URL: https://github.com/apache/incubator-mxnet/issues/8573#issuecomment-342693279 To build from source, please follow this guide: http://mxnet.incubator.apache.org/install/index.html

[GitHub] ZiqianXie commented on issue #8585: API document needs to be updated

2017-11-07 Thread GitBox
ZiqianXie commented on issue #8585: API document needs to be updated URL: https://github.com/apache/incubator-mxnet/issues/8585#issuecomment-342691956 Sorry, I just realize that my version of mxnet is 0.11.0. This is an

[GitHub] ZiqianXie closed issue #8585: API document needs to be updated

2017-11-07 Thread GitBox
ZiqianXie closed issue #8585: API document needs to be updated URL: https://github.com/apache/incubator-mxnet/issues/8585 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] solin319 commented on issue #8584: Equivalent parameters for lr_mult and decay_mult of convolution in caffe in mxnet

2017-11-07 Thread GitBox
solin319 commented on issue #8584: Equivalent parameters for lr_mult and decay_mult of convolution in caffe in mxnet URL: https://github.com/apache/incubator-mxnet/issues/8584#issuecomment-342691070 You can use set_lr_mult, set_wd_mult in class Optimizer.

[GitHub] szha commented on issue #8378: upgrade MKL

2017-11-07 Thread GitBox
szha commented on issue #8378: upgrade MKL URL: https://github.com/apache/incubator-mxnet/pull/8378#issuecomment-339418566 Thanks, @ykim362. @eric-haibin-lin @piiswrong shall we merge this? This is an automated message from

[GitHub] zihaolucky commented on issue #8583: Improve custom cross-entropy operator example

2017-11-07 Thread GitBox
zihaolucky commented on issue #8583: Improve custom cross-entropy operator example URL: https://github.com/apache/incubator-mxnet/issues/8583#issuecomment-342690181 Add a performance test against NDArry and Numpy array would be great.

[GitHub] solin319 commented on issue #8583: Improve custom cross-entropy operator example

2017-11-07 Thread GitBox
solin319 commented on issue #8583: Improve custom cross-entropy operator example URL: https://github.com/apache/incubator-mxnet/issues/8583#issuecomment-342689116 I pull a new request in #8588. The test result were equal between use np.array and mx.NDArray. The first test result are as

[GitHub] zhreshold commented on issue #8563: Inconsistent Python calls, Python output different from C++ output

2017-11-07 Thread GitBox
zhreshold commented on issue #8563: Inconsistent Python calls, Python output different from C++ output URL: https://github.com/apache/incubator-mxnet/issues/8563#issuecomment-342681055 Okay, this turned out to be not a bug, it's the dropout behavior. Turn off is_train mode will resolve

[GitHub] solin319 opened a new pull request #8588: implement cross-entropy operator with NDArray

2017-11-07 Thread GitBox
solin319 opened a new pull request #8588: implement cross-entropy operator with NDArray URL: https://github.com/apache/incubator-mxnet/pull/8588 #8583 In the recommender example, reimplement cross-entropy operator with NDArray.

[GitHub] ShownX commented on issue #8559: How to use Prelu in gluon?

2017-11-07 Thread GitBox
ShownX commented on issue #8559: How to use Prelu in gluon? URL: https://github.com/apache/incubator-mxnet/issues/8559#issuecomment-342687345 How? This is an automated message from the Apache Git Service. To respond to the

[GitHub] anirudh2290 opened a new issue #8587: Issue with CSVIter for small datasets

2017-11-07 Thread GitBox
anirudh2290 opened a new issue #8587: Issue with CSVIter for small datasets URL: https://github.com/apache/incubator-mxnet/issues/8587 Note: Providing complete information in the most concise form is the best way to get help. This issue template serves as the checklist for essential

[GitHub] lyblsgo opened a new issue #8586: Word-level language modeling RNN, Training on multiple GPUs

2017-11-07 Thread GitBox
lyblsgo opened a new issue #8586: Word-level language modeling RNN, Training on multiple GPUs URL: https://github.com/apache/incubator-mxnet/issues/8586 How to train Word-level-language-modeling-RNN on multiple GPUs, does anyone have try this?

[incubator-mxnet] branch master updated: [sparse] slice for csr on two dimensions, cpu implementation (#8331)

2017-11-07 Thread jxie
This is an automated email from the ASF dual-hosted git repository. jxie 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 bf2336c [sparse] slice for csr on two

[GitHub] piiswrong closed pull request #8331: [sparse] slice for csr on two dimensions, cpu implementation

2017-11-07 Thread GitBox
piiswrong closed pull request #8331: [sparse] slice for csr on two dimensions, cpu implementation URL: https://github.com/apache/incubator-mxnet/pull/8331 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of

[GitHub] ZiqianXie opened a new issue #8585: API document needs to be updated

2017-11-07 Thread GitBox
ZiqianXie opened a new issue #8585: API document needs to be updated URL: https://github.com/apache/incubator-mxnet/issues/8585 [http://mxnet.incubator.apache.org/api/python/ndarray/random.html#module-mxnet.ndarray.random](Here) the API document says that the mxnet.ndarray.random is the

[GitHub] eric-haibin-lin commented on a change in pull request #8331: [sparse] slice for csr on two dimensions, cpu implementation

2017-11-07 Thread GitBox
eric-haibin-lin commented on a change in pull request #8331: [sparse] slice for csr on two dimensions, cpu implementation URL: https://github.com/apache/incubator-mxnet/pull/8331#discussion_r149505376 ## File path: src/operator/tensor/matrix_op-inl.h ## @@ -601,18

[GitHub] eric-haibin-lin commented on a change in pull request #8331: [sparse] slice for csr on two dimensions, cpu implementation

2017-11-07 Thread GitBox
eric-haibin-lin commented on a change in pull request #8331: [sparse] slice for csr on two dimensions, cpu implementation URL: https://github.com/apache/incubator-mxnet/pull/8331#discussion_r149505293 ## File path: src/operator/tensor/matrix_op-inl.h ## @@ -551,48 +550,43

[GitHub] ShownX commented on issue #8581: More examples ?

2017-11-07 Thread GitBox
ShownX commented on issue #8581: More examples ? URL: https://github.com/apache/incubator-mxnet/issues/8581#issuecomment-342679374 How about here: https://github.com/apache/incubator-mxnet/tree/master/example/gluon/word_language_model

[GitHub] solin319 commented on issue #8583: Improve custom cross-entropy operator example

2017-11-07 Thread GitBox
solin319 commented on issue #8583: Improve custom cross-entropy operator example URL: https://github.com/apache/incubator-mxnet/issues/8583#issuecomment-342678867 I can have a try to implement cross-entropy operator with NDArray.

[GitHub] zhreshold commented on issue #8423: Re-implement segnet in MXnet

2017-11-07 Thread GitBox
zhreshold commented on issue #8423: Re-implement segnet in MXnet URL: https://github.com/apache/incubator-mxnet/pull/8423#issuecomment-342675988 @solin319 Have you finished? Can you rebase to fix the CI? This is an automated

[incubator-mxnet] branch master updated: Update mshadow to 2d7780c3f2eefe4453fa419862d1b2089bedb8d5 (#8516)

2017-11-07 Thread zhasheng
This is an automated email from the ASF dual-hosted git repository. zhasheng 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 bbf8b3e Update mshadow to

svn commit: r23001 - /dev/incubator/mxnet/0.12.1.rc0/

2017-11-07 Thread cjolivier01
Author: cjolivier01 Date: Wed Nov 8 00:44:48 2017 New Revision: 23001 Log: Added mxnet-0.12.1.rc0 Added: dev/incubator/mxnet/0.12.1.rc0/ dev/incubator/mxnet/0.12.1.rc0/apache-mxnet-src-0.12.1.rc0-incubating.tar.gz (with props)

[GitHub] larroy commented on issue #8578: Add development instructions and update installation instructions

2017-11-07 Thread GitBox
larroy commented on issue #8578: Add development instructions and update installation instructions URL: https://github.com/apache/incubator-mxnet/pull/8578#issuecomment-34266 I think is good to have an initial reference on how to build & test in the root folder, the additional

[GitHub] larroy commented on a change in pull request #8578: Add development instructions and update installation instructions

2017-11-07 Thread GitBox
larroy commented on a change in pull request #8578: Add development instructions and update installation instructions URL: https://github.com/apache/incubator-mxnet/pull/8578#discussion_r149545383 ## File path: docs/install/index.md ## @@ -65,7 +65,7 @@ Indicate your

[GitHub] piiswrong closed pull request #8264: Operators for mean(csr, axis=0) and mean(csr, axis=1)

2017-11-07 Thread GitBox
piiswrong closed pull request #8264: Operators for mean(csr, axis=0) and mean(csr, axis=1) URL: https://github.com/apache/incubator-mxnet/pull/8264 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance:

[incubator-mxnet] branch master updated: Operators for mean(csr, axis=0) and mean(csr, axis=1) (#8264)

2017-11-07 Thread jxie
This is an automated email from the ASF dual-hosted git repository. jxie 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 c2358f9 Operators for mean(csr, axis=0)

[GitHub] yajiedesign commented on a change in pull request #8572: use first class cuda with cmake 3.9 and cuda9.0 support

2017-11-07 Thread GitBox
yajiedesign commented on a change in pull request #8572: use first class cuda with cmake 3.9 and cuda9.0 support URL: https://github.com/apache/incubator-mxnet/pull/8572#discussion_r149538454 ## File path: cmake/Blas.cmake ## @@ -0,0 +1,43 @@ +set(mshadow_LINKER_LIBS "")

[GitHub] cjolivier01 commented on issue #8579: Automatic OMP operator tuning based upon kernel operation workload

2017-11-07 Thread GitBox
cjolivier01 commented on issue #8579: Automatic OMP operator tuning based upon kernel operation workload URL: https://github.com/apache/incubator-mxnet/pull/8579#issuecomment-342663369 Closed during changes in order to not trigger CI (CI is having problems)

[GitHub] yajiedesign commented on a change in pull request #8572: use first class cuda with cmake 3.9 and cuda9.0 support

2017-11-07 Thread GitBox
yajiedesign commented on a change in pull request #8572: use first class cuda with cmake 3.9 and cuda9.0 support URL: https://github.com/apache/incubator-mxnet/pull/8572#discussion_r149538371 ## File path: CMakeLists.txt ## @@ -1,17 +1,41 @@

[GitHub] yajiedesign commented on a change in pull request #8572: use first class cuda with cmake 3.9 and cuda9.0 support

2017-11-07 Thread GitBox
yajiedesign commented on a change in pull request #8572: use first class cuda with cmake 3.9 and cuda9.0 support URL: https://github.com/apache/incubator-mxnet/pull/8572#discussion_r149539798 ## File path: CMakeLists.txt ## @@ -1,17 +1,41 @@

[GitHub] yajiedesign commented on a change in pull request #8572: use first class cuda with cmake 3.9 and cuda9.0 support

2017-11-07 Thread GitBox
yajiedesign commented on a change in pull request #8572: use first class cuda with cmake 3.9 and cuda9.0 support URL: https://github.com/apache/incubator-mxnet/pull/8572#discussion_r149539745 ## File path: CMakeLists.txt ## @@ -1,17 +1,41 @@

[GitHub] yajiedesign commented on a change in pull request #8572: use first class cuda with cmake 3.9 and cuda9.0 support

2017-11-07 Thread GitBox
yajiedesign commented on a change in pull request #8572: use first class cuda with cmake 3.9 and cuda9.0 support URL: https://github.com/apache/incubator-mxnet/pull/8572#discussion_r149539611 ## File path: cmake/FirstCuda.cmake ## @@ -0,0 +1,233 @@ +# Licensed to the

[GitHub] yajiedesign commented on a change in pull request #8572: use first class cuda with cmake 3.9 and cuda9.0 support

2017-11-07 Thread GitBox
yajiedesign commented on a change in pull request #8572: use first class cuda with cmake 3.9 and cuda9.0 support URL: https://github.com/apache/incubator-mxnet/pull/8572#discussion_r149538454 ## File path: cmake/Blas.cmake ## @@ -0,0 +1,43 @@ +set(mshadow_LINKER_LIBS "")

[GitHub] yajiedesign commented on a change in pull request #8572: use first class cuda with cmake 3.9 and cuda9.0 support

2017-11-07 Thread GitBox
yajiedesign commented on a change in pull request #8572: use first class cuda with cmake 3.9 and cuda9.0 support URL: https://github.com/apache/incubator-mxnet/pull/8572#discussion_r149538371 ## File path: CMakeLists.txt ## @@ -1,17 +1,41 @@

[GitHub] yajiedesign commented on a change in pull request #8572: use first class cuda with cmake 3.9 and cuda9.0 support

2017-11-07 Thread GitBox
yajiedesign commented on a change in pull request #8572: use first class cuda with cmake 3.9 and cuda9.0 support URL: https://github.com/apache/incubator-mxnet/pull/8572#discussion_r149537797 ## File path: CMakeLists.txt ## @@ -128,14 +152,20 @@ endif()

[GitHub] yajiedesign commented on issue #8572: use first class cuda with cmake 3.9 and cuda9.0 support

2017-11-07 Thread GitBox
yajiedesign commented on issue #8572: use first class cuda with cmake 3.9 and cuda9.0 support URL: https://github.com/apache/incubator-mxnet/pull/8572#issuecomment-342659887 @cjolivier01 if you want use cuda9.0,the new cmake is must. and the new cmake can parallel compiling with nvcc.

[GitHub] yajiedesign commented on issue #8572: use first class cuda with cmake 3.9 and cuda9.0 support

2017-11-07 Thread GitBox
yajiedesign commented on issue #8572: use first class cuda with cmake 3.9 and cuda9.0 support URL: https://github.com/apache/incubator-mxnet/pull/8572#issuecomment-342659887 @cjolivier01 if you want use cuda9.0,the new cmake is must.

[GitHub] eric-haibin-lin commented on a change in pull request #8555: Update PR & Issue Template

2017-11-07 Thread GitBox
eric-haibin-lin commented on a change in pull request #8555: Update PR & Issue Template URL: https://github.com/apache/incubator-mxnet/pull/8555#discussion_r149535629 ## File path: .github/PULL_REQUEST_TEMPLATE.md ## @@ -6,7 +6,7 @@ - [ ] Passed code style checking

[GitHub] zhreshold commented on issue #8519: More details to the windows build process

2017-11-07 Thread GitBox
zhreshold commented on issue #8519: More details to the windows build process URL: https://github.com/apache/incubator-mxnet/pull/8519#issuecomment-342650722 Please rebase to fix CI This is an automated message from the

[GitHub] eric-haibin-lin opened a new issue #8583: Improve custom cross-entropy operator example

2017-11-07 Thread GitBox
eric-haibin-lin opened a new issue #8583: Improve custom cross-entropy operator example URL: https://github.com/apache/incubator-mxnet/issues/8583 In the recommender example, the custom cross-entropy operator can be implemented using NDArray instead of numpy, which gives much better

[GitHub] reminisce commented on a change in pull request #8558: slice operator supporting arbitrary values of step

2017-11-07 Thread GitBox
reminisce commented on a change in pull request #8558: slice operator supporting arbitrary values of step URL: https://github.com/apache/incubator-mxnet/pull/8558#discussion_r149528683 ## File path: src/operator/tensor/matrix_op-inl.h ## @@ -414,16 +420,6 @@ inline TShape

[GitHub] zhreshold opened a new pull request #8582: Yolo2 pr

2017-11-07 Thread GitBox
zhreshold opened a new pull request #8582: Yolo2 pr URL: https://github.com/apache/incubator-mxnet/pull/8582 ## Description ## Operator used by yolo2 symbol implementation. ## Checklist ## ### Essentials ### - [x] Passed code style checking (`make lint`) - [x] Changes are

[GitHub] ptrendx commented on issue #8294: NCCL integration

2017-11-07 Thread GitBox
ptrendx commented on issue #8294: NCCL integration URL: https://github.com/apache/incubator-mxnet/pull/8294#issuecomment-342648788 Not really, no. This is an automated message from the Apache Git Service. To respond to the

[GitHub] eric-haibin-lin commented on issue #8294: NCCL integration

2017-11-07 Thread GitBox
eric-haibin-lin commented on issue #8294: NCCL integration URL: https://github.com/apache/incubator-mxnet/pull/8294#issuecomment-342647982 Thanks for addressing all these review comments. Is anyone helping you to setup the CI test with NCCL?

[GitHub] ykim362 commented on issue #7931: MKL-DNN integration: request for reviews

2017-11-07 Thread GitBox
ykim362 commented on issue #7931: MKL-DNN integration: request for reviews URL: https://github.com/apache/incubator-mxnet/pull/7931#issuecomment-342644868 @sbodenstein From my understanding, this PR is not going to be directly merged. It's going to be merged with another revision with

[GitHub] cjolivier01 commented on a change in pull request #8579: Automatic OMP operator tuning based upon kernel operation workload

2017-11-07 Thread GitBox
cjolivier01 commented on a change in pull request #8579: Automatic OMP operator tuning based upon kernel operation workload URL: https://github.com/apache/incubator-mxnet/pull/8579#discussion_r149518876 ## File path: src/operator/mxnet_op.h ## @@ -288,55 +291,84 @@ struct

[GitHub] eric-haibin-lin commented on a change in pull request #8259: check_format of sparse ndrray

2017-11-07 Thread GitBox
eric-haibin-lin commented on a change in pull request #8259: check_format of sparse ndrray URL: https://github.com/apache/incubator-mxnet/pull/8259#discussion_r149509544 ## File path: src/ndarray/ndarray.cc ## @@ -1214,6 +1214,41 @@ void NDArray::SyncCopyToCPU(void *data,

[GitHub] eric-haibin-lin commented on a change in pull request #8259: check_format of sparse ndrray

2017-11-07 Thread GitBox
eric-haibin-lin commented on a change in pull request #8259: check_format of sparse ndrray URL: https://github.com/apache/incubator-mxnet/pull/8259#discussion_r149509374 ## File path: src/ndarray/ndarray.cc ## @@ -1214,6 +1214,41 @@ void NDArray::SyncCopyToCPU(void *data,

[GitHub] eric-haibin-lin commented on a change in pull request #8259: check_format of sparse ndrray

2017-11-07 Thread GitBox
eric-haibin-lin commented on a change in pull request #8259: check_format of sparse ndrray URL: https://github.com/apache/incubator-mxnet/pull/8259#discussion_r149508628 ## File path: tests/python/unittest/test_sparse_ndarray.py ## @@ -735,6 +735,58 @@ def

[GitHub] eric-haibin-lin commented on a change in pull request #8259: check_format of sparse ndrray

2017-11-07 Thread GitBox
eric-haibin-lin commented on a change in pull request #8259: check_format of sparse ndrray URL: https://github.com/apache/incubator-mxnet/pull/8259#discussion_r149510743 ## File path: tests/python/unittest/test_sparse_ndarray.py ## @@ -735,6 +735,58 @@ def

[GitHub] eric-haibin-lin commented on a change in pull request #8259: check_format of sparse ndrray

2017-11-07 Thread GitBox
eric-haibin-lin commented on a change in pull request #8259: check_format of sparse ndrray URL: https://github.com/apache/incubator-mxnet/pull/8259#discussion_r149510310 ## File path: src/common/utils.h ## @@ -43,9 +43,170 @@ #include #include +#include

[GitHub] eric-haibin-lin commented on a change in pull request #8259: check_format of sparse ndrray

2017-11-07 Thread GitBox
eric-haibin-lin commented on a change in pull request #8259: check_format of sparse ndrray URL: https://github.com/apache/incubator-mxnet/pull/8259#discussion_r149509978 ## File path: src/common/utils.h ## @@ -43,9 +43,170 @@ #include #include +#include

[GitHub] eric-haibin-lin commented on a change in pull request #8259: check_format of sparse ndrray

2017-11-07 Thread GitBox
eric-haibin-lin commented on a change in pull request #8259: check_format of sparse ndrray URL: https://github.com/apache/incubator-mxnet/pull/8259#discussion_r149516527 ## File path: src/common/utils.h ## @@ -43,9 +43,170 @@ #include #include +#include

[GitHub] eric-haibin-lin commented on a change in pull request #8259: check_format of sparse ndrray

2017-11-07 Thread GitBox
eric-haibin-lin commented on a change in pull request #8259: check_format of sparse ndrray URL: https://github.com/apache/incubator-mxnet/pull/8259#discussion_r149507608 ## File path: include/mxnet/c_api.h ## @@ -405,6 +405,12 @@ MXNET_DLL int

[GitHub] eric-haibin-lin commented on a change in pull request #8259: check_format of sparse ndrray

2017-11-07 Thread GitBox
eric-haibin-lin commented on a change in pull request #8259: check_format of sparse ndrray URL: https://github.com/apache/incubator-mxnet/pull/8259#discussion_r149507450 ## File path: include/mxnet/ndarray.h ## @@ -447,6 +456,12 @@ class NDArray { */ void

[GitHub] eric-haibin-lin commented on issue #7335: Exception in threads kills entire process

2017-11-07 Thread GitBox
eric-haibin-lin commented on issue #7335: Exception in threads kills entire process URL: https://github.com/apache/incubator-mxnet/issues/7335#issuecomment-342635531 @anirudh2290 This is an automated message from the

[GitHub] mccollum-amzn opened a new issue #7335: Exception in threads kills entire process

2017-11-07 Thread GitBox
mccollum-amzn opened a new issue #7335: Exception in threads kills entire process URL: https://github.com/apache/incubator-mxnet/issues/7335 For bugs or installation issues, please provide the following information. The more information you provide, the more likely people will be able

[GitHub] piiswrong closed pull request #8460: cpu sparse embedding op

2017-11-07 Thread GitBox
piiswrong closed pull request #8460: cpu sparse embedding op URL: https://github.com/apache/incubator-mxnet/pull/8460 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull

[incubator-mxnet] branch master updated: cpu sparse embedding op (#8460)

2017-11-07 Thread jxie
This is an automated email from the ASF dual-hosted git repository. jxie 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 4862c41 cpu sparse embedding op (#8460)

[GitHub] SrivastavaKshitij opened a new issue #8581: More examples ?

2017-11-07 Thread GitBox
SrivastavaKshitij opened a new issue #8581: More examples ? URL: https://github.com/apache/incubator-mxnet/issues/8581 Hi There I am trying to use mxnet and gluon for text classification using cnn. I can't find extensive examples for mxnet. All the gluon tutorials , show models

[GitHub] rahul003 closed pull request #8580: fix some memory leaks, and initializes a variable

2017-11-07 Thread GitBox
rahul003 closed pull request #8580: fix some memory leaks, and initializes a variable URL: https://github.com/apache/incubator-mxnet/pull/8580 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As

[GitHub] rahul003 opened a new pull request #8580: fix some memory leaks, and uninitialized variable errors

2017-11-07 Thread GitBox
rahul003 opened a new pull request #8580: fix some memory leaks, and uninitialized variable errors URL: https://github.com/apache/incubator-mxnet/pull/8580 ## Description ## 1. Fixes memory leaks 2. Sets value to uninitialized variable 3. Adds a default case to a switch statement

[GitHub] eric-haibin-lin commented on a change in pull request #8558: slice operator supporting arbitrary values of step

2017-11-07 Thread GitBox
eric-haibin-lin commented on a change in pull request #8558: slice operator supporting arbitrary values of step URL: https://github.com/apache/incubator-mxnet/pull/8558#discussion_r149505905 ## File path: src/operator/tensor/matrix_op-inl.h ## @@ -414,16 +420,6 @@ inline

[GitHub] eric-haibin-lin commented on a change in pull request #8331: [sparse] slice for csr on two dimensions, cpu implementation

2017-11-07 Thread GitBox
eric-haibin-lin commented on a change in pull request #8331: [sparse] slice for csr on two dimensions, cpu implementation URL: https://github.com/apache/incubator-mxnet/pull/8331#discussion_r149505376 ## File path: src/operator/tensor/matrix_op-inl.h ## @@ -601,18

[GitHub] eric-haibin-lin commented on a change in pull request #8331: [sparse] slice for csr on two dimensions, cpu implementation

2017-11-07 Thread GitBox
eric-haibin-lin commented on a change in pull request #8331: [sparse] slice for csr on two dimensions, cpu implementation URL: https://github.com/apache/incubator-mxnet/pull/8331#discussion_r149505293 ## File path: src/operator/tensor/matrix_op-inl.h ## @@ -551,48 +550,43

[GitHub] marcoabreu commented on issue #8554: Disable long running tests

2017-11-07 Thread GitBox
marcoabreu commented on issue #8554: Disable long running tests URL: https://github.com/apache/incubator-mxnet/pull/8554#issuecomment-342623637 Only the crashing ones are disabled. The long running tests are classified as nightly and are of such a complexity that they should not be

[GitHub] piiswrong commented on issue #8554: Disable long running tests

2017-11-07 Thread GitBox
piiswrong commented on issue #8554: Disable long running tests URL: https://github.com/apache/incubator-mxnet/pull/8554#issuecomment-342611101 I think its better to try to make them run faster than disabling them This is an

[GitHub] cjolivier01 commented on issue #8572: use first class cuda with cmake 3.9 and cuda9.0 support

2017-11-07 Thread GitBox
cjolivier01 commented on issue #8572: use first class cuda with cmake 3.9 and cuda9.0 support URL: https://github.com/apache/incubator-mxnet/pull/8572#issuecomment-342608301 Can you please describe what the net benefits are when using the newer cmake?

[GitHub] cjolivier01 commented on a change in pull request #8572: use first class cuda with cmake 3.9 and cuda9.0 support

2017-11-07 Thread GitBox
cjolivier01 commented on a change in pull request #8572: use first class cuda with cmake 3.9 and cuda9.0 support URL: https://github.com/apache/incubator-mxnet/pull/8572#discussion_r149488762 ## File path: CMakeLists.txt ## @@ -1,17 +1,41 @@

[GitHub] cjolivier01 commented on a change in pull request #8572: use first class cuda with cmake 3.9 and cuda9.0 support

2017-11-07 Thread GitBox
cjolivier01 commented on a change in pull request #8572: use first class cuda with cmake 3.9 and cuda9.0 support URL: https://github.com/apache/incubator-mxnet/pull/8572#discussion_r149490485 ## File path: cmake/Blas.cmake ## @@ -0,0 +1,43 @@ +set(mshadow_LINKER_LIBS "")

[GitHub] cjolivier01 commented on a change in pull request #8572: use first class cuda with cmake 3.9 and cuda9.0 support

2017-11-07 Thread GitBox
cjolivier01 commented on a change in pull request #8572: use first class cuda with cmake 3.9 and cuda9.0 support URL: https://github.com/apache/incubator-mxnet/pull/8572#discussion_r149489696 ## File path: CMakeLists.txt ## @@ -1,17 +1,41 @@

[GitHub] cjolivier01 commented on a change in pull request #8572: use first class cuda with cmake 3.9 and cuda9.0 support

2017-11-07 Thread GitBox
cjolivier01 commented on a change in pull request #8572: use first class cuda with cmake 3.9 and cuda9.0 support URL: https://github.com/apache/incubator-mxnet/pull/8572#discussion_r149490617 ## File path: cmake/FirstCuda.cmake ## @@ -0,0 +1,233 @@ +# Licensed to the

[GitHub] cjolivier01 commented on a change in pull request #8572: use first class cuda with cmake 3.9 and cuda9.0 support

2017-11-07 Thread GitBox
cjolivier01 commented on a change in pull request #8572: use first class cuda with cmake 3.9 and cuda9.0 support URL: https://github.com/apache/incubator-mxnet/pull/8572#discussion_r149490084 ## File path: CMakeLists.txt ## @@ -128,14 +152,20 @@ endif()

[GitHub] cjolivier01 commented on a change in pull request #8572: use first class cuda with cmake 3.9 and cuda9.0 support

2017-11-07 Thread GitBox
cjolivier01 commented on a change in pull request #8572: use first class cuda with cmake 3.9 and cuda9.0 support URL: https://github.com/apache/incubator-mxnet/pull/8572#discussion_r149489043 ## File path: CMakeLists.txt ## @@ -1,17 +1,41 @@

[GitHub] piiswrong closed pull request #8540: div_scalar for sparse

2017-11-07 Thread GitBox
piiswrong closed pull request #8540: div_scalar for sparse URL: https://github.com/apache/incubator-mxnet/pull/8540 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull

[incubator-mxnet] branch master updated: div_scalar for sparse (#8540)

2017-11-07 Thread jxie
This is an automated email from the ASF dual-hosted git repository. jxie 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 c64c307 div_scalar for sparse (#8540)

[GitHub] piiswrong closed pull request #8550: csr ndarray iter with shuffle

2017-11-07 Thread GitBox
piiswrong closed pull request #8550: csr ndarray iter with shuffle URL: https://github.com/apache/incubator-mxnet/pull/8550 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign

[incubator-mxnet] branch master updated: csr ndarray iter with shuffle (#8550)

2017-11-07 Thread jxie
This is an automated email from the ASF dual-hosted git repository. jxie 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 938c25f csr ndarray iter with shuffle

[GitHub] piiswrong commented on issue #8578: Add development instructions and update installation instructions

2017-11-07 Thread GitBox
piiswrong commented on issue #8578: Add development instructions and update installation instructions URL: https://github.com/apache/incubator-mxnet/pull/8578#issuecomment-342594203 why add a new page instead of putting it into build from source section?

[GitHub] pracheer closed issue #8455: [CoreML Converter] Installer for python 3

2017-11-07 Thread GitBox
pracheer closed issue #8455: [CoreML Converter] Installer for python 3 URL: https://github.com/apache/incubator-mxnet/issues/8455 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] pracheer commented on issue #8455: [CoreML Converter] Installer for python 3

2017-11-07 Thread GitBox
pracheer commented on issue #8455: [CoreML Converter] Installer for python 3 URL: https://github.com/apache/incubator-mxnet/issues/8455#issuecomment-342587928 Missed that fact. Resolving this since this it is not needed for the time being.

[GitHub] azai91 commented on issue #8527: send _send_command_to_servers must be string

2017-11-07 Thread GitBox
azai91 commented on issue #8527: send _send_command_to_servers must be string URL: https://github.com/apache/incubator-mxnet/pull/8527#issuecomment-342585022 @piiswrong any reason this can't be built? This is an automated

[GitHub] szha commented on a change in pull request #8578: Add development instructions and update installation instructions

2017-11-07 Thread GitBox
szha commented on a change in pull request #8578: Add development instructions and update installation instructions URL: https://github.com/apache/incubator-mxnet/pull/8578#discussion_r149457822 ## File path: docs/install/index.md ## @@ -65,7 +65,7 @@ Indicate your

  1   2   >