[GitHub] zheng-da commented on a change in pull request #8302: Refactor operators & MKLDNN

2017-12-08 Thread GitBox
zheng-da commented on a change in pull request #8302: Refactor operators & MKLDNN URL: https://github.com/apache/incubator-mxnet/pull/8302#discussion_r155914245 ## File path: src/executor/infer_graph_attr_pass.cc ## @@ -416,11 +416,6 @@ nnvm::Graph

[incubator-mxnet] branch master updated: Fix Example example/dec/dec.py (#8990)

2017-12-08 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 3ee1ca2 Fix Example example/dec/dec.py

[GitHub] piiswrong closed pull request #8990: Fix Example example/dec/dec.py

2017-12-08 Thread GitBox
piiswrong closed pull request #8990: Fix Example example/dec/dec.py URL: https://github.com/apache/incubator-mxnet/pull/8990 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign

[GitHub] piiswrong closed pull request #9005: update python-howto examples

2017-12-08 Thread GitBox
piiswrong closed pull request #9005: update python-howto examples URL: https://github.com/apache/incubator-mxnet/pull/9005 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: update readme (#9005)

2017-12-08 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 ac4369c update readme (#9005) ac4369c

[GitHub] fanlu commented on issue #8861: The first GPU cost more memory than the others in version 0.12.1

2017-12-08 Thread GitBox
fanlu commented on issue #8861: The first GPU cost more memory than the others in version 0.12.1 URL: https://github.com/apache/incubator-mxnet/issues/8861#issuecomment-350421473 I find the same problem in 1.0.0 either

[GitHub] marfago commented on issue #8849: support monodimensional sparse array

2017-12-08 Thread GitBox
marfago commented on issue #8849: support monodimensional sparse array URL: https://github.com/apache/incubator-mxnet/pull/8849#issuecomment-350420577 @piiswrong Maybe I don't get your point, but this code works ``` >>> import numpy as np >>> from scipy.sparse import csr_matrix

[GitHub] marfago commented on issue #8849: support monodimensional sparse array

2017-12-08 Thread GitBox
marfago commented on issue #8849: support monodimensional sparse array URL: https://github.com/apache/incubator-mxnet/pull/8849#issuecomment-350420577 @piiswrong Maybe I don't get your point, but this code works ``` >>> import numpy as np >>> from scipy.sparse import csr_matrix

[GitHub] marfago commented on issue #8849: support monodimensional sparse array

2017-12-08 Thread GitBox
marfago commented on issue #8849: support monodimensional sparse array URL: https://github.com/apache/incubator-mxnet/pull/8849#issuecomment-350420577 @piiswrong Maybe I don't get your point, but this code works ``` >>> import numpy as np >>> from scipy.sparse import csr_matrix

[GitHub] marfago commented on issue #8849: support monodimensional sparse array

2017-12-08 Thread GitBox
marfago commented on issue #8849: support monodimensional sparse array URL: https://github.com/apache/incubator-mxnet/pull/8849#issuecomment-350420577 @piiswrong Maybe I don't get your point, but this code works ``` >>> import numpy as np >>> from scipy.sparse import csr_matrix

[GitHub] weixingzhang commented on issue #8849: support monodimensional sparse array

2017-12-08 Thread GitBox
weixingzhang commented on issue #8849: support monodimensional sparse array URL: https://github.com/apache/incubator-mxnet/pull/8849#issuecomment-350419800 @eric-haibin-lin yes, I agree. I am waiting for the final decision and then make a new change.

[GitHub] eric-haibin-lin commented on issue #7631: windows MXNET_CUDNN_AUTOTUNE_DEFAULT = 0 no work.

2017-12-08 Thread GitBox
eric-haibin-lin commented on issue #7631: windows MXNET_CUDNN_AUTOTUNE_DEFAULT = 0 no work. URL: https://github.com/apache/incubator-mxnet/issues/7631#issuecomment-350415591 @nchafni what's your version? Are you building from source?

[GitHub] ptrendx commented on issue #9004: test_nccl.py script causes a core dump on P2.16xlarge instance when ran against NCCL enabled MXNet build.

2017-12-08 Thread GitBox
ptrendx commented on issue #9004: test_nccl.py script causes a core dump on P2.16xlarge instance when ran against NCCL enabled MXNet build. URL: https://github.com/apache/incubator-mxnet/issues/9004#issuecomment-350415135 @leleamol Could you run with env variable NCCL_DEBUG=INFO and post

[GitHub] taliesinb commented on issue #8949: New layer: split_like.

2017-12-08 Thread GitBox
taliesinb commented on issue #8949: New layer: split_like. URL: https://github.com/apache/incubator-mxnet/pull/8949#issuecomment-350414149 @piiswrong what name makes more sense to you than `split_layer`? perhaps `unflatten_layer`, because it is doing the opposite of flattening the first

[GitHub] astonzhang commented on a change in pull request #8763: [WIP] Add text apis

2017-12-08 Thread GitBox
astonzhang commented on a change in pull request #8763: [WIP] Add text apis URL: https://github.com/apache/incubator-mxnet/pull/8763#discussion_r155907745 ## File path: python/mxnet/text/__init__.py ## @@ -0,0 +1,25 @@ +# Licensed to the Apache Software Foundation (ASF)

[GitHub] astonzhang commented on a change in pull request #8763: [WIP] Add text apis

2017-12-08 Thread GitBox
astonzhang commented on a change in pull request #8763: [WIP] Add text apis URL: https://github.com/apache/incubator-mxnet/pull/8763#discussion_r155908281 ## File path: python/mxnet/text/glossary.py ## @@ -0,0 +1,654 @@ +# Licensed to the Apache Software Foundation (ASF)

[GitHub] anirudh2290 opened a new pull request #9008: Usability improvement mxnet vae

2017-12-08 Thread GitBox
anirudh2290 opened a new pull request #9008: Usability improvement mxnet vae URL: https://github.com/apache/incubator-mxnet/pull/9008 ## Description ## Usability improvement for mxnet vae ## Checklist ## ### Essentials ### - [x] Passed code style checking (`make lint`) -

[incubator-mxnet] branch master updated: Locate liblapack.a in /usr/lib64 (#8073)

2017-12-08 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 84fde4d Locate liblapack.a in

[GitHub] piiswrong closed pull request #8073: Locate liblapack.a in /usr/lib64

2017-12-08 Thread GitBox
piiswrong closed pull request #8073: Locate liblapack.a in /usr/lib64 URL: https://github.com/apache/incubator-mxnet/pull/8073 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign

[GitHub] piiswrong commented on a change in pull request #8851: Use backtrace from dmlc-core (dmlc::stack_trace())

2017-12-08 Thread GitBox
piiswrong commented on a change in pull request #8851: Use backtrace from dmlc-core (dmlc::stack_trace()) URL: https://github.com/apache/incubator-mxnet/pull/8851#discussion_r155907400 ## File path: make/config.mk ## @@ -33,8 +33,8 @@ DEBUG = 0 # whether compile with

[GitHub] zheng-da commented on issue #8302: Refactor operators & MKLDNN

2017-12-08 Thread GitBox
zheng-da commented on issue #8302: Refactor operators & MKLDNN URL: https://github.com/apache/incubator-mxnet/pull/8302#issuecomment-350410800 @cjolivier01 the reason code refactoring and MKLDNN are put in the same PR is that we don't want to refactor original MKL code. We don't want a PR

[GitHub] piiswrong commented on a change in pull request #8851: Use backtrace from dmlc-core (dmlc::stack_trace())

2017-12-08 Thread GitBox
piiswrong commented on a change in pull request #8851: Use backtrace from dmlc-core (dmlc::stack_trace()) URL: https://github.com/apache/incubator-mxnet/pull/8851#discussion_r155907381 ## File path: make/config.mk ## @@ -33,8 +33,8 @@ DEBUG = 0 # whether compile with

[GitHub] bhavinthaker commented on issue #8968: 1.0.0 crash with Google perf tools

2017-12-08 Thread GitBox
bhavinthaker commented on issue #8968: 1.0.0 crash with Google perf tools URL: https://github.com/apache/incubator-mxnet/issues/8968#issuecomment-350410651 Let's try a coverity run too: @rahul003 This is an automated

[GitHub] zheng-da commented on a change in pull request #8302: Refactor operators & MKLDNN

2017-12-08 Thread GitBox
zheng-da commented on a change in pull request #8302: Refactor operators & MKLDNN URL: https://github.com/apache/incubator-mxnet/pull/8302#discussion_r155907169 ## File path: src/operator/nn/convolution.cc ## @@ -38,63 +36,361 @@ namespace mxnet { namespace op {

[GitHub] piiswrong commented on issue #8849: support monodimensional sparse array

2017-12-08 Thread GitBox
piiswrong commented on issue #8849: support monodimensional sparse array URL: https://github.com/apache/incubator-mxnet/pull/8849#issuecomment-350410335 Are you sure csr sparse vector is a thing? Is it supported in scipy?

[incubator-mxnet] branch master updated: [clang][warning] fix warnings about braced initializer (fixes warning in concurrenc… (#8870)

2017-12-08 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 1c22bc7 [clang][warning] fix warnings

[GitHub] piiswrong closed pull request #8870: [clang][warning] fix warnings about braced initializer (fixes warning in concurrenc?

2017-12-08 Thread GitBox
piiswrong closed pull request #8870: [clang][warning] fix warnings about braced initializer (fixes warning in concurrenc? URL: https://github.com/apache/incubator-mxnet/pull/8870 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed

[GitHub] piiswrong commented on issue #8872: Minor refactor: prevent string copying, list -> vector, shared_ptr by?

2017-12-08 Thread GitBox
piiswrong commented on issue #8872: Minor refactor: prevent string copying, list -> vector, shared_ptr by? URL: https://github.com/apache/incubator-mxnet/pull/8872#issuecomment-350409951 LGTM. Please resolve conflicts

[GitHub] zheng-da commented on a change in pull request #8302: Refactor operators & MKLDNN

2017-12-08 Thread GitBox
zheng-da commented on a change in pull request #8302: Refactor operators & MKLDNN URL: https://github.com/apache/incubator-mxnet/pull/8302#discussion_r155907018 ## File path: src/kvstore/kvstore_dist.h ## @@ -32,11 +32,6 @@ #include "mxnet/engine.h" #include "ps/ps.h"

[GitHub] piiswrong commented on a change in pull request #8919: [Vision] add test cases for flip, normalize, to_tensor

2017-12-08 Thread GitBox
piiswrong commented on a change in pull request #8919: [Vision] add test cases for flip, normalize, to_tensor URL: https://github.com/apache/incubator-mxnet/pull/8919#discussion_r155906779 ## File path: python/mxnet/gluon/data/vision/transforms.py ## @@ -184,26 +184,44 @@

[GitHub] piiswrong commented on a change in pull request #8919: [Vision] add test cases for flip, normalize, to_tensor

2017-12-08 Thread GitBox
piiswrong commented on a change in pull request #8919: [Vision] add test cases for flip, normalize, to_tensor URL: https://github.com/apache/incubator-mxnet/pull/8919#discussion_r155906779 ## File path: python/mxnet/gluon/data/vision/transforms.py ## @@ -184,26 +184,44 @@

[GitHub] sandeep-krishnamurthy commented on issue #8941: MXNET backend in Keras 2

2017-12-08 Thread GitBox
sandeep-krishnamurthy commented on issue #8941: MXNET backend in Keras 2 URL: https://github.com/apache/incubator-mxnet/issues/8941#issuecomment-350409542 Here is the repo we are working on - https://github.com/deep-learning-tools/keras/tree/keras2_mxnet_backend We are looking at

[GitHub] piiswrong commented on a change in pull request #8919: [Vision] add test cases for flip, normalize, to_tensor

2017-12-08 Thread GitBox
piiswrong commented on a change in pull request #8919: [Vision] add test cases for flip, normalize, to_tensor URL: https://github.com/apache/incubator-mxnet/pull/8919#discussion_r155906736 ## File path: python/mxnet/gluon/data/vision/transforms.py ## @@ -184,26 +184,44 @@

[GitHub] piiswrong commented on a change in pull request #8919: [Vision] add test cases for flip, normalize, to_tensor

2017-12-08 Thread GitBox
piiswrong commented on a change in pull request #8919: [Vision] add test cases for flip, normalize, to_tensor URL: https://github.com/apache/incubator-mxnet/pull/8919#discussion_r155906736 ## File path: python/mxnet/gluon/data/vision/transforms.py ## @@ -184,26 +184,44 @@

[GitHub] piiswrong closed pull request #8935: change workspace allocation in random samplers

2017-12-08 Thread GitBox
piiswrong closed pull request #8935: change workspace allocation in random samplers URL: https://github.com/apache/incubator-mxnet/pull/8935 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] piiswrong closed pull request #8970: [Do not merge] Trigger build

2017-12-08 Thread GitBox
piiswrong closed pull request #8970: [Do not merge] Trigger build URL: https://github.com/apache/incubator-mxnet/pull/8970 This is an automated message from the Apache Git Service. To respond to the message, please log on

[incubator-mxnet] branch master updated: change workspace allocation in random samplers (#8935)

2017-12-08 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 01b7ae3 change workspace allocation in

[GitHub] piiswrong commented on issue #8949: New layer: split_like.

2017-12-08 Thread GitBox
piiswrong commented on issue #8949: New layer: split_like. URL: https://github.com/apache/incubator-mxnet/pull/8949#issuecomment-350409300 This operator feels weird. First split_like is not a good name because it behaves very differently from split operator. Second splitting x[0] to y[0],

[GitHub] szha commented on a change in pull request #8763: [WIP] Add text apis

2017-12-08 Thread GitBox
szha commented on a change in pull request #8763: [WIP] Add text apis URL: https://github.com/apache/incubator-mxnet/pull/8763#discussion_r155906426 ## File path: python/mxnet/text/glossary.py ## @@ -0,0 +1,654 @@ +# Licensed to the Apache Software Foundation (ASF) under

[GitHub] szha commented on a change in pull request #8763: [WIP] Add text apis

2017-12-08 Thread GitBox
szha commented on a change in pull request #8763: [WIP] Add text apis URL: https://github.com/apache/incubator-mxnet/pull/8763#discussion_r155906426 ## File path: python/mxnet/text/glossary.py ## @@ -0,0 +1,654 @@ +# Licensed to the Apache Software Foundation (ASF) under

[GitHub] eric-haibin-lin commented on a change in pull request #8949: New layer: split_like.

2017-12-08 Thread GitBox
eric-haibin-lin commented on a change in pull request #8949: New layer: split_like. URL: https://github.com/apache/incubator-mxnet/pull/8949#discussion_r155906385 ## File path: src/operator/tensor/elemwise_unary_op_basic.cc ## @@ -295,6 +295,51 @@

[GitHub] piiswrong closed pull request #8975: Correct the alexnet url.

2017-12-08 Thread GitBox
piiswrong closed pull request #8975: Correct the alexnet url. URL: https://github.com/apache/incubator-mxnet/pull/8975 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: [CI] Fix indentation of Jenkinsfile, no code changes (#8988)

2017-12-08 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 3d34f2e [CI] Fix indentation of

[GitHub] piiswrong closed pull request #8984: Updates for R readme and example

2017-12-08 Thread GitBox
piiswrong closed pull request #8984: Updates for R readme and example URL: https://github.com/apache/incubator-mxnet/pull/8984 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign

[GitHub] piiswrong closed pull request #8988: [CI] Fix indentation of Jenkinsfile, no code changes

2017-12-08 Thread GitBox
piiswrong closed pull request #8988: [CI] Fix indentation of Jenkinsfile, no code changes URL: https://github.com/apache/incubator-mxnet/pull/8988 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance:

[GitHub] piiswrong commented on a change in pull request #9002: Misc performance changes

2017-12-08 Thread GitBox
piiswrong commented on a change in pull request #9002: Misc performance changes URL: https://github.com/apache/incubator-mxnet/pull/9002#discussion_r155906043 ## File path: src/imperative/imperative_utils.h ## @@ -453,7 +453,7 @@ inline void PushOperator(const OpStatePtr&

[GitHub] piiswrong closed pull request #9003: Update PULL_REQUEST_TEMPLATE.md

2017-12-08 Thread GitBox
piiswrong closed pull request #9003: Update PULL_REQUEST_TEMPLATE.md URL: https://github.com/apache/incubator-mxnet/pull/9003 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 (d77d937 -> 918fac3)

2017-12-08 Thread jxie
This is an automated email from the ASF dual-hosted git repository. jxie pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. from d77d937 Fix __repr__ for gluon.Parameter (#8956) add 918fac3 Update PULL_REQUEST_TEMPLATE.md (#9003)

[GitHub] szha commented on a change in pull request #8763: [WIP] Add text apis

2017-12-08 Thread GitBox
szha commented on a change in pull request #8763: [WIP] Add text apis URL: https://github.com/apache/incubator-mxnet/pull/8763#discussion_r155905841 ## File path: python/mxnet/text/glossary.py ## @@ -0,0 +1,654 @@ +# Licensed to the Apache Software Foundation (ASF) under

[GitHub] szha commented on a change in pull request #8763: [WIP] Add text apis

2017-12-08 Thread GitBox
szha commented on a change in pull request #8763: [WIP] Add text apis URL: https://github.com/apache/incubator-mxnet/pull/8763#discussion_r155905718 ## File path: python/mxnet/text/glossary.py ## @@ -0,0 +1,654 @@ +# Licensed to the Apache Software Foundation (ASF) under

[GitHub] szha commented on a change in pull request #8763: [WIP] Add text apis

2017-12-08 Thread GitBox
szha commented on a change in pull request #8763: [WIP] Add text apis URL: https://github.com/apache/incubator-mxnet/pull/8763#discussion_r155905718 ## File path: python/mxnet/text/glossary.py ## @@ -0,0 +1,654 @@ +# Licensed to the Apache Software Foundation (ASF) under

[incubator-mxnet] branch master updated: Fix __repr__ for gluon.Parameter (#8956)

2017-12-08 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 d77d937 Fix __repr__ for

[GitHub] piiswrong closed pull request #8956: Fix __repr__ for gluon.Parameter

2017-12-08 Thread GitBox
piiswrong closed pull request #8956: Fix __repr__ for gluon.Parameter URL: https://github.com/apache/incubator-mxnet/pull/8956 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign

[GitHub] szha commented on a change in pull request #8763: [WIP] Add text apis

2017-12-08 Thread GitBox
szha commented on a change in pull request #8763: [WIP] Add text apis URL: https://github.com/apache/incubator-mxnet/pull/8763#discussion_r155904984 ## File path: python/mxnet/text/glossary.py ## @@ -0,0 +1,654 @@ +# Licensed to the Apache Software Foundation (ASF) under

[GitHub] szha commented on a change in pull request #8763: [WIP] Add text apis

2017-12-08 Thread GitBox
szha commented on a change in pull request #8763: [WIP] Add text apis URL: https://github.com/apache/incubator-mxnet/pull/8763#discussion_r155904593 ## File path: python/mxnet/text/glossary.py ## @@ -0,0 +1,654 @@ +# Licensed to the Apache Software Foundation (ASF) under

[GitHub] indhub closed pull request #8939: Minor changes to Caffe Translator

2017-12-08 Thread GitBox
indhub closed pull request #8939: Minor changes to Caffe Translator URL: https://github.com/apache/incubator-mxnet/pull/8939 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: Minor changes to Caffe Translator (#8939)

2017-12-08 Thread indhub
This is an automated email from the ASF dual-hosted git repository. indhub 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 9d4bb9c Minor changes to Caffe

[GitHub] szha commented on issue #7805: Enhancement : Operators for Deformable ConvNets not implemented to work on CPU mode

2017-12-08 Thread GitBox
szha commented on issue #7805: Enhancement : Operators for Deformable ConvNets not implemented to work on CPU mode URL: https://github.com/apache/incubator-mxnet/issues/7805#issuecomment-350406104 This issue is closed due to lack of activity in the last 90 days. Feel free to ping me to

[GitHub] szha closed issue #7805: Enhancement : Operators for Deformable ConvNets not implemented to work on CPU mode

2017-12-08 Thread GitBox
szha closed issue #7805: Enhancement : Operators for Deformable ConvNets not implemented to work on CPU mode URL: https://github.com/apache/incubator-mxnet/issues/7805 This is an automated message from the Apache Git

[GitHub] JulianSlzr commented on issue #8956: Fix __repr__ for gluon.Parameter

2017-12-08 Thread GitBox
JulianSlzr commented on issue #8956: Fix __repr__ for gluon.Parameter URL: https://github.com/apache/incubator-mxnet/pull/8956#issuecomment-350405837 Wrote test and rebased, should be good now This is an automated message

[GitHub] szha commented on a change in pull request #8763: [WIP] Add text apis

2017-12-08 Thread GitBox
szha commented on a change in pull request #8763: [WIP] Add text apis URL: https://github.com/apache/incubator-mxnet/pull/8763#discussion_r155904102 ## File path: python/mxnet/text/glossary.py ## @@ -0,0 +1,654 @@ +# Licensed to the Apache Software Foundation (ASF) under

[GitHub] taliesinb opened a new issue #9007: float16 argmax breaks on negative inputs

2017-12-08 Thread GitBox
taliesinb opened a new issue #9007: float16 argmax breaks on negative inputs URL: https://github.com/apache/incubator-mxnet/issues/9007 ## Description float16 implementation of argmax seems to treat negative numbers as if they were zero. ## Environment info (Required)

[GitHub] cjolivier01 commented on a change in pull request #9002: Misc performance changes

2017-12-08 Thread GitBox
cjolivier01 commented on a change in pull request #9002: Misc performance changes URL: https://github.com/apache/incubator-mxnet/pull/9002#discussion_r155902211 ## File path: src/imperative/imperative_utils.h ## @@ -453,7 +453,7 @@ inline void PushOperator(const

[GitHub] taliesinb commented on a change in pull request #8949: New layer: split_like.

2017-12-08 Thread GitBox
taliesinb commented on a change in pull request #8949: New layer: split_like. URL: https://github.com/apache/incubator-mxnet/pull/8949#discussion_r155900554 ## File path: src/operator/tensor/elemwise_unary_op_basic.cc ## @@ -295,6 +295,51 @@ NNVM_REGISTER_OP(reshape_like)

[GitHub] eric-haibin-lin commented on a change in pull request #8302: Refactor operators & MKLDNN

2017-12-08 Thread GitBox
eric-haibin-lin commented on a change in pull request #8302: Refactor operators & MKLDNN URL: https://github.com/apache/incubator-mxnet/pull/8302#discussion_r155899439 ## File path: src/operator/nn/convolution.cc ## @@ -38,63 +36,361 @@ namespace mxnet { namespace op {

[GitHub] eric-haibin-lin commented on a change in pull request #8302: Refactor operators & MKLDNN

2017-12-08 Thread GitBox
eric-haibin-lin commented on a change in pull request #8302: Refactor operators & MKLDNN URL: https://github.com/apache/incubator-mxnet/pull/8302#discussion_r155898841 ## File path: src/operator/nn/convolution.cc ## @@ -38,63 +36,361 @@ namespace mxnet { namespace op {

[incubator-mxnet] branch master updated: Factorization machine example & sparse example folder re-org (#8767)

2017-12-08 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 24ea5b0 Factorization machine

[GitHub] szha closed pull request #8767: Factorization machine example & sparse example folder re-org

2017-12-08 Thread GitBox
szha closed pull request #8767: Factorization machine example & sparse example folder re-org URL: https://github.com/apache/incubator-mxnet/pull/8767 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

[incubator-mxnet] branch master updated: Reduce the verbose warnings in CI (#8981)

2017-12-08 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 c9e440d Reduce the verbose warnings

[GitHub] szha closed pull request #8981: Reduce the verbose warnings in CI

2017-12-08 Thread GitBox
szha closed pull request #8981: Reduce the verbose warnings in CI URL: https://github.com/apache/incubator-mxnet/pull/8981 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

[GitHub] szha commented on issue #8671: Discussion and troubleshooting on PyPI (pip) installation

2017-12-08 Thread GitBox
szha commented on issue #8671: Discussion and troubleshooting on PyPI (pip) installation URL: https://github.com/apache/incubator-mxnet/issues/8671#issuecomment-350398194 I will add a section for the flags This is an

[GitHub] szha commented on a change in pull request #8763: [WIP] Add text apis

2017-12-08 Thread GitBox
szha commented on a change in pull request #8763: [WIP] Add text apis URL: https://github.com/apache/incubator-mxnet/pull/8763#discussion_r155898721 ## File path: python/mxnet/text/__init__.py ## @@ -0,0 +1,25 @@ +# Licensed to the Apache Software Foundation (ASF) under

[GitHub] eric-haibin-lin commented on issue #8861: The first GPU cost more memory than the others in version 0.12.1

2017-12-08 Thread GitBox
eric-haibin-lin commented on issue #8861: The first GPU cost more memory than the others in version 0.12.1 URL: https://github.com/apache/incubator-mxnet/issues/8861#issuecomment-350397214 I didn't see relevant changes regarding kvstore buffer distributed in kvstore comm between 0.10 and

[GitHub] eric-haibin-lin commented on issue #8849: support monodimensional sparse array

2017-12-08 Thread GitBox
eric-haibin-lin commented on issue #8849: support monodimensional sparse array URL: https://github.com/apache/incubator-mxnet/pull/8849#issuecomment-350393232 `zeros` will not be the only function which require changes if we enable such feature.

[GitHub] eric-haibin-lin commented on issue #8817: Creating a monodimensional sparse array returns an error

2017-12-08 Thread GitBox
eric-haibin-lin commented on issue #8817: Creating a monodimensional sparse array returns an error URL: https://github.com/apache/incubator-mxnet/issues/8817#issuecomment-350393045 From an implementation point of view, it's hard to enable such mono-dimensional array. The 1-D

[GitHub] astonzhang commented on issue #8763: [WIP] Add text apis

2017-12-08 Thread GitBox
astonzhang commented on issue #8763: [WIP] Add text apis URL: https://github.com/apache/incubator-mxnet/pull/8763#issuecomment-350390587 @marcoabreu resolved. This is an automated message from the Apache Git Service. To

[GitHub] eric-haibin-lin commented on issue #9004: test_nccl.py script causes a core dump on P2.16xlarge instance when ran against NCCL enabled MXNet build.

2017-12-08 Thread GitBox
eric-haibin-lin commented on issue #9004: test_nccl.py script causes a core dump on P2.16xlarge instance when ran against NCCL enabled MXNet build. URL: https://github.com/apache/incubator-mxnet/issues/9004#issuecomment-350387205 @ptrendx

[GitHub] eric-haibin-lin opened a new issue #9006: Move MNISTIterator from tests/python/common to mxnet.test_utils

2017-12-08 Thread GitBox
eric-haibin-lin opened a new issue #9006: Move MNISTIterator from tests/python/common to mxnet.test_utils URL: https://github.com/apache/incubator-mxnet/issues/9006 The `MNISTIterator` in https://github.com/apache/incubator-mxnet/blob/master/tests/python/common/get_data.py#L60 introduced

[GitHub] cjolivier01 commented on issue #8997: gluon hang-up unexpectedly

2017-12-08 Thread GitBox
cjolivier01 commented on issue #8997: gluon hang-up unexpectedly URL: https://github.com/apache/incubator-mxnet/issues/8997#issuecomment-350386748 I will take a look This is an automated message from the Apache Git Service.

[GitHub] eric-haibin-lin opened a new pull request #9005: update python-howto examples

2017-12-08 Thread GitBox
eric-haibin-lin opened a new pull request #9005: update python-howto examples URL: https://github.com/apache/incubator-mxnet/pull/9005 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] piiswrong commented on issue #8997: gluon hang-up unexpectedly

2017-12-08 Thread GitBox
piiswrong commented on issue #8997: gluon hang-up unexpectedly URL: https://github.com/apache/incubator-mxnet/issues/8997#issuecomment-350384329 @cjolivier01 looks like another shutdown problem? This is an automated message

[GitHub] eric-haibin-lin commented on issue #8302: Refactor operators & MKLDNN

2017-12-08 Thread GitBox
eric-haibin-lin commented on issue #8302: Refactor operators & MKLDNN URL: https://github.com/apache/incubator-mxnet/pull/8302#issuecomment-350381039 The path to the conv op mentioned here have to be updated, too

[GitHub] eric-haibin-lin commented on a change in pull request #8949: New layer: split_like.

2017-12-08 Thread GitBox
eric-haibin-lin commented on a change in pull request #8949: New layer: split_like. URL: https://github.com/apache/incubator-mxnet/pull/8949#discussion_r155881293 ## File path: src/operator/tensor/elemwise_unary_op_basic.cc ## @@ -295,6 +295,51 @@

[GitHub] eric-haibin-lin commented on a change in pull request #8938: Add operator for dot(dns, csr) = csr

2017-12-08 Thread GitBox
eric-haibin-lin commented on a change in pull request #8938: Add operator for dot(dns, csr) = csr URL: https://github.com/apache/incubator-mxnet/pull/8938#discussion_r155879961 ## File path: src/operator/tensor/dot-inl.h ## @@ -811,6 +891,94 @@ inline void

[GitHub] eric-haibin-lin commented on a change in pull request #8938: Add operator for dot(dns, csr) = csr

2017-12-08 Thread GitBox
eric-haibin-lin commented on a change in pull request #8938: Add operator for dot(dns, csr) = csr URL: https://github.com/apache/incubator-mxnet/pull/8938#discussion_r155880396 ## File path: src/operator/tensor/dot-inl.h ## @@ -811,6 +891,94 @@ inline void

[GitHub] eric-haibin-lin commented on a change in pull request #8938: Add operator for dot(dns, csr) = csr

2017-12-08 Thread GitBox
eric-haibin-lin commented on a change in pull request #8938: Add operator for dot(dns, csr) = csr URL: https://github.com/apache/incubator-mxnet/pull/8938#discussion_r155879219 ## File path: src/operator/tensor/dot-inl.h ## @@ -231,6 +231,12 @@ inline bool

[GitHub] eric-haibin-lin commented on a change in pull request #8938: Add operator for dot(dns, csr) = csr

2017-12-08 Thread GitBox
eric-haibin-lin commented on a change in pull request #8938: Add operator for dot(dns, csr) = csr URL: https://github.com/apache/incubator-mxnet/pull/8938#discussion_r155879484 ## File path: src/operator/tensor/dot-inl.h ## @@ -811,6 +891,94 @@ inline void

[GitHub] eric-haibin-lin commented on a change in pull request #8938: Add operator for dot(dns, csr) = csr

2017-12-08 Thread GitBox
eric-haibin-lin commented on a change in pull request #8938: Add operator for dot(dns, csr) = csr URL: https://github.com/apache/incubator-mxnet/pull/8938#discussion_r155879592 ## File path: src/operator/tensor/dot-inl.h ## @@ -811,6 +891,94 @@ inline void

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

2017-12-08 Thread GitBox
rahul003 commented on issue #8527: send _send_command_to_servers must be string URL: https://github.com/apache/incubator-mxnet/pull/8527#issuecomment-350375482 @azai91 This doesn't fix the issue. It would still throw the error at Line 392. Either way, I believe your issue would be

[GitHub] eric-haibin-lin commented on a change in pull request #8738: Fix custom op - infer_storage_type_backward

2017-12-08 Thread GitBox
eric-haibin-lin commented on a change in pull request #8738: Fix custom op - infer_storage_type_backward URL: https://github.com/apache/incubator-mxnet/pull/8738#discussion_r155878265 ## File path: tests/python/unittest/test_operator.py ## @@ -3624,15 +3640,93 @@ def

[GitHub] eric-haibin-lin commented on a change in pull request #8738: Fix custom op - infer_storage_type_backward

2017-12-08 Thread GitBox
eric-haibin-lin commented on a change in pull request #8738: Fix custom op - infer_storage_type_backward URL: https://github.com/apache/incubator-mxnet/pull/8738#discussion_r155877202 ## File path: example/numpy-ops/custom_sparse_sqr.py ## @@ -0,0 +1,93 @@ +# Licensed to

[GitHub] eric-haibin-lin commented on a change in pull request #8738: Fix custom op - infer_storage_type_backward

2017-12-08 Thread GitBox
eric-haibin-lin commented on a change in pull request #8738: Fix custom op - infer_storage_type_backward URL: https://github.com/apache/incubator-mxnet/pull/8738#discussion_r155877035 ## File path: example/numpy-ops/custom_sparse_sqr.py ## @@ -0,0 +1,93 @@ +# Licensed to

[GitHub] b0noI commented on issue #9004: test_nccl.py script causes a core dump on P2.16xlarge instance when ran against NCCL enabled MXNet build.

2017-12-08 Thread GitBox
b0noI commented on issue #9004: test_nccl.py script causes a core dump on P2.16xlarge instance when ran against NCCL enabled MXNet build. URL: https://github.com/apache/incubator-mxnet/issues/9004#issuecomment-350374826 NCCL is 2.1 with CUDA9

[GitHub] leleamol opened a new issue #9004: test_nccl.py script causes a core dump on P2.16xlarge instance when ran against NCCL enabled MXNet build.

2017-12-08 Thread GitBox
leleamol opened a new issue #9004: test_nccl.py script causes a core dump on P2.16xlarge instance when ran against NCCL enabled MXNet build. URL: https://github.com/apache/incubator-mxnet/issues/9004 Note: Providing complete information in the most concise form is the best way to get

[GitHub] rahul003 commented on issue #8915: NVLink communication pattern updated

2017-12-08 Thread GitBox
rahul003 commented on issue #8915: NVLink communication pattern updated URL: https://github.com/apache/incubator-mxnet/pull/8915#issuecomment-350372457 @Laurawly Can we also make use of this feature for ReduceCompressed function?

[GitHub] ashokei commented on issue #8974: MXNET compatibility with MKL libraries bundled in Microsoft R Open

2017-12-08 Thread GitBox
ashokei commented on issue #8974: MXNET compatibility with MKL libraries bundled in Microsoft R Open URL: https://github.com/apache/incubator-mxnet/issues/8974#issuecomment-350371732 @mjmg if you do not want to use full mkl (core), please try the following. export

[GitHub] mbaijal commented on a change in pull request #8990: Fix Example example/dec/dec.py

2017-12-08 Thread GitBox
mbaijal commented on a change in pull request #8990: Fix Example example/dec/dec.py URL: https://github.com/apache/incubator-mxnet/pull/8990#discussion_r155874494 ## File path: example/deep-embedded-clustering/dec.py ## @@ -82,8 +82,8 @@ def list_arguments(self):

[GitHub] marcoabreu commented on issue #8763: [WIP] Add text apis

2017-12-08 Thread GitBox
marcoabreu commented on issue #8763: [WIP] Add text apis URL: https://github.com/apache/incubator-mxnet/pull/8763#issuecomment-350366355 @astonzhang Again, please rebase your PR as it creates invalid CI requests This is an

[GitHub] astonzhang commented on a change in pull request #8763: [WIP] Add text apis

2017-12-08 Thread GitBox
astonzhang commented on a change in pull request #8763: [WIP] Add text apis URL: https://github.com/apache/incubator-mxnet/pull/8763#discussion_r155869635 ## File path: python/mxnet/text/__init__.py ## @@ -0,0 +1,25 @@ +# Licensed to the Apache Software Foundation (ASF)

[GitHub] cjolivier01 commented on a change in pull request #9002: Misc performance changes

2017-12-08 Thread GitBox
cjolivier01 commented on a change in pull request #9002: Misc performance changes URL: https://github.com/apache/incubator-mxnet/pull/9002#discussion_r155867731 ## File path: src/imperative/imperative_utils.h ## @@ -453,7 +453,7 @@ inline void PushOperator(const

  1   2   >