[GitHub] ThomasDelteil commented on issue #10385: allow user to define unknown token symbol

2018-04-07 Thread GitBox
ThomasDelteil commented on issue #10385: allow user to define unknown token symbol URL: https://github.com/apache/incubator-mxnet/pull/10385#issuecomment-379520960 it would be nice to have a test This is an automated message

[GitHub] ThomasDelteil commented on a change in pull request #10388: [MXNET-265] Update optimizer doc to clarify wd behaviors

2018-04-07 Thread GitBox
ThomasDelteil commented on a change in pull request #10388: [MXNET-265] Update optimizer doc to clarify wd behaviors URL: https://github.com/apache/incubator-mxnet/pull/10388#discussion_r179935219 ## File path: python/mxnet/optimizer.py ## @@ -538,7 +538,7 @@ def

[GitHub] ThomasDelteil commented on a change in pull request #10388: [MXNET-265] Update optimizer doc to clarify wd behaviors

2018-04-07 Thread GitBox
ThomasDelteil commented on a change in pull request #10388: [MXNET-265] Update optimizer doc to clarify wd behaviors URL: https://github.com/apache/incubator-mxnet/pull/10388#discussion_r179935219 ## File path: python/mxnet/optimizer.py ## @@ -538,7 +538,7 @@ def

[GitHub] piiswrong commented on a change in pull request #10424: Improved error message

2018-04-07 Thread GitBox
piiswrong commented on a change in pull request #10424: Improved error message URL: https://github.com/apache/incubator-mxnet/pull/10424#discussion_r179932850 ## File path: python/mxnet/gluon/block.py ## @@ -458,7 +458,11 @@ def _build_cache(self, *args):

[GitHub] piiswrong closed pull request #10434: [MXNET-155] Add support for np.int32 and np.int64 as basic index types

2018-04-07 Thread GitBox
piiswrong closed pull request #10434: [MXNET-155] Add support for np.int32 and np.int64 as basic index types URL: https://github.com/apache/incubator-mxnet/pull/10434 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the

[incubator-mxnet] branch master updated: [MXNET-155] Add support for np.int32 and np.int64 as basic index types (#10434)

2018-04-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 4fb5241 [MXNET-155] Add support for

[GitHub] piiswrong commented on issue #10437: [MXNET-171] Fix a bug that was causing training accuracy to be printed as nan sometimes

2018-04-07 Thread GitBox
piiswrong commented on issue #10437: [MXNET-171] Fix a bug that was causing training accuracy to be printed as nan sometimes URL: https://github.com/apache/incubator-mxnet/pull/10437#issuecomment-379513190 Deep copy is slow and can cause a lot of problems. We can't do this kind of hack in

[GitHub] piiswrong closed pull request #10457: fix string representation of rnn layer

2018-04-07 Thread GitBox
piiswrong closed pull request #10457: fix string representation of rnn layer URL: https://github.com/apache/incubator-mxnet/pull/10457 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

[incubator-mxnet] branch master updated: fix string representation of rnn layer (#10457)

2018-04-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 de7269b fix string representation of

[GitHub] zheng-da commented on issue #10442: use correct prop_kind for mkl-dnn FC layer

2018-04-07 Thread GitBox
zheng-da commented on issue #10442: use correct prop_kind for mkl-dnn FC layer URL: https://github.com/apache/incubator-mxnet/pull/10442#issuecomment-379511109 I think this should count as bug fix. This is an automated

[GitHub] marcoabreu commented on a change in pull request #10435: [MXNET-289] Allow specifying number of batches to run in an epoch to fix hang in dist sync for last epoch

2018-04-07 Thread GitBox
marcoabreu commented on a change in pull request #10435: [MXNET-289] Allow specifying number of batches to run in an epoch to fix hang in dist sync for last epoch URL: https://github.com/apache/incubator-mxnet/pull/10435#discussion_r179932034 ## File path:

[GitHub] marcoabreu commented on issue #10442: use correct prop_kind for mkl-dnn FC layer

2018-04-07 Thread GitBox
marcoabreu commented on issue #10442: use correct prop_kind for mkl-dnn FC layer URL: https://github.com/apache/incubator-mxnet/pull/10442#issuecomment-379510572 Just to clarify, is this a bugfix, performance improvement or something else?

[incubator-mxnet] branch master updated: Updated OpenBLAS release tag for armv6 build (#10441)

2018-04-07 Thread marcoabreu
This is an automated email from the ASF dual-hosted git repository. marcoabreu 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 1b2805c Updated OpenBLAS release

[GitHub] marcoabreu closed pull request #10441: Fixed OpenBLAS release tag for armv6 build

2018-04-07 Thread GitBox
marcoabreu closed pull request #10441: Fixed OpenBLAS release tag for armv6 build URL: https://github.com/apache/incubator-mxnet/pull/10441 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

[incubator-mxnet] branch master updated: [MXNET-292] Added social media handles (#10448)

2018-04-07 Thread marcoabreu
This is an automated email from the ASF dual-hosted git repository. marcoabreu 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 432ec20 [MXNET-292] Added social

[GitHub] marcoabreu closed pull request #10448: [MXNET-292] Added social media handles

2018-04-07 Thread GitBox
marcoabreu closed pull request #10448: [MXNET-292] Added social media handles URL: https://github.com/apache/incubator-mxnet/pull/10448 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

[GitHub] marcoabreu commented on issue #10443: bug fix

2018-04-07 Thread GitBox
marcoabreu commented on issue #10443: bug fix URL: https://github.com/apache/incubator-mxnet/pull/10443#issuecomment-379510364 Hello @alues, could you please provide a description and a proper title for your PR? This is an

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

2018-04-07 Thread GitBox
marcoabreu commented on issue #8915: NVLink communication pattern updated URL: https://github.com/apache/incubator-mxnet/pull/8915#issuecomment-379510129 @Jerryzcn we're waiting for @Laurawly to address the review comments

[GitHub] msurguy commented on issue #10230: libomp.so is missing in Docker builds for armv6

2018-04-07 Thread GitBox
msurguy commented on issue #10230: libomp.so is missing in Docker builds for armv6 URL: https://github.com/apache/incubator-mxnet/issues/10230#issuecomment-379506971 @lebeg yeah. I tried installing it via the command you provided ( I assume it should be automatically added to the path)

[incubator-mxnet] branch master updated: Update row_sparse.md (#10454)

2018-04-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 4be7be7 Update row_sparse.md

[GitHub] szha closed pull request #10454: Improve row_sparse tutorial

2018-04-07 Thread GitBox
szha closed pull request #10454: Improve row_sparse tutorial URL: https://github.com/apache/incubator-mxnet/pull/10454 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] eric-haibin-lin commented on issue #10454: Improve row_sparse tutorial

2018-04-07 Thread GitBox
eric-haibin-lin commented on issue #10454: Improve row_sparse tutorial URL: https://github.com/apache/incubator-mxnet/pull/10454#issuecomment-379498121 I don’t think Tiny PR like this needs one but yeah I can create a JIRA item..

[GitHub] anirudh2290 commented on issue #10454: Improve row_sparse tutorial

2018-04-07 Thread GitBox
anirudh2290 commented on issue #10454: Improve row_sparse tutorial URL: https://github.com/apache/incubator-mxnet/pull/10454#issuecomment-379496284 can you associate this with a jira ? This is an automated message from the

[GitHub] zheng-da commented on issue #10451: [WIP] Add Foreach

2018-04-07 Thread GitBox
zheng-da commented on issue #10451: [WIP] Add Foreach URL: https://github.com/apache/incubator-mxnet/pull/10451#issuecomment-379493798 we potentially can parallelize among iterations. most likely, there is dependency between iterations. so parallelization among iterations may not be very

[GitHub] szha opened a new pull request #10457: fix string representation of rnn layer

2018-04-07 Thread GitBox
szha opened a new pull request #10457: fix string representation of rnn layer URL: https://github.com/apache/incubator-mxnet/pull/10457 ## Description ## fix string representation of rnn layer ## Checklist ## ### Essentials ### Please feel free to remove inapplicable items

[GitHub] singleye commented on issue #797: a short introduction to mxnet design and implementation (chinese)

2018-04-07 Thread GitBox
singleye commented on issue #797: a short introduction to mxnet design and implementation (chinese) URL: https://github.com/apache/incubator-mxnet/issues/797#issuecomment-379477803 Cool! This is an automated message from

[GitHub] lzhou-arch commented on issue #4887: multi-node training hangs occasionally

2018-04-07 Thread GitBox
lzhou-arch commented on issue #4887: multi-node training hangs occasionally URL: https://github.com/apache/incubator-mxnet/issues/4887#issuecomment-379475277 @feiyulv Have you figured it out? I have exactly the same issue. After I launch the job on a two-machine cluster, it hangs

[GitHub] kpot commented on issue #8337: mx.autograd.grad works or fails depending on use of slices

2018-04-07 Thread GitBox
kpot commented on issue #8337: mx.autograd.grad works or fails depending on use of slices URL: https://github.com/apache/incubator-mxnet/issues/8337#issuecomment-379471987 @roywei To be honest, over the last months I've already lost the context and don't remember what I was using the

[GitHub] marcoabreu commented on a change in pull request #10449: [MXNET-293] Navigation updates

2018-04-07 Thread GitBox
marcoabreu commented on a change in pull request #10449: [MXNET-293] Navigation updates URL: https://github.com/apache/incubator-mxnet/pull/10449#discussion_r179917229 ## File path: docs/_static/mxnet-theme/navbar.html ## @@ -7,47 +7,39 @@ Install -

[GitHub] marcoabreu commented on issue #10451: [WIP] Add Foreach

2018-04-07 Thread GitBox
marcoabreu commented on issue #10451: [WIP] Add Foreach URL: https://github.com/apache/incubator-mxnet/pull/10451#issuecomment-379470746 Very interesting! Our of curiosity, is this operator going to be entirely parallelized since it basically splits the graph into multiple subgraphs or how

[GitHub] MoritzMaxeiner commented on issue #8836: Backward shape inconsistent with custom HybridBlock and gluon.loss

2018-04-07 Thread GitBox
MoritzMaxeiner commented on issue #8836: Backward shape inconsistent with custom HybridBlock and gluon.loss URL: https://github.com/apache/incubator-mxnet/issues/8836#issuecomment-379465681 @reminisce Sure, I just didn't close for the reasons mentioned

[GitHub] marcoabreu commented on issue #10445: Make the numpy version compatible with Official Mac OS system python-2.7.10's numpy

2018-04-07 Thread GitBox
marcoabreu commented on issue #10445: Make the numpy version compatible with Official Mac OS system python-2.7.10's numpy URL: https://github.com/apache/incubator-mxnet/pull/10445#issuecomment-379465501 > Since pinning the version for numpy is causing an issue

[GitHub] marcoabreu commented on issue #10445: Make the numpy version compatible with Official Mac OS system python-2.7.10's numpy

2018-04-07 Thread GitBox
marcoabreu commented on issue #10445: Make the numpy version compatible with Official Mac OS system python-2.7.10's numpy URL: https://github.com/apache/incubator-mxnet/pull/10445#issuecomment-379465501 > Since pinning the version for numpy is causing an issue

[GitHub] marcoabreu commented on issue #10445: Make the numpy version compatible with Official Mac OS system python-2.7.10's numpy

2018-04-07 Thread GitBox
marcoabreu commented on issue #10445: Make the numpy version compatible with Official Mac OS system python-2.7.10's numpy URL: https://github.com/apache/incubator-mxnet/pull/10445#issuecomment-379465501 > Since pinning the version for numpy is causing an issue

[GitHub] marcoabreu commented on issue #10445: Make the numpy version compatible with Official Mac OS system python-2.7.10's numpy

2018-04-07 Thread GitBox
marcoabreu commented on issue #10445: Make the numpy version compatible with Official Mac OS system python-2.7.10's numpy URL: https://github.com/apache/incubator-mxnet/pull/10445#issuecomment-379465359 1.7.1 according to https://pkgs.org/download/numpy

[GitHub] liangyong928 commented on issue #9517: Check failed: err == CUDNN_STATUS_SUCCESS (6 vs. 0) CUDNN_STATUS_ARCH_MISMATCH

2018-04-07 Thread GitBox
liangyong928 commented on issue #9517: Check failed: err == CUDNN_STATUS_SUCCESS (6 vs. 0) CUDNN_STATUS_ARCH_MISMATCH URL: https://github.com/apache/incubator-mxnet/issues/9517#issuecomment-379456056 @Lorna-Liu Maybe Video card is bad. I bought a used graphics card. 可能显卡有问题,二手显卡。

[GitHub] liangyong928 commented on issue #9517: Check failed: err == CUDNN_STATUS_SUCCESS (6 vs. 0) CUDNN_STATUS_ARCH_MISMATCH

2018-04-07 Thread GitBox
liangyong928 commented on issue #9517: Check failed: err == CUDNN_STATUS_SUCCESS (6 vs. 0) CUDNN_STATUS_ARCH_MISMATCH URL: https://github.com/apache/incubator-mxnet/issues/9517#issuecomment-379455944 @ankkhedia No. Maybe Video card is bad. I bought a used graphics card.

[GitHub] oscarbg opened a new issue #10456: WinML or DirectML support planned?

2018-04-07 Thread GitBox
oscarbg opened a new issue #10456: WinML or DirectML support planned? URL: https://github.com/apache/incubator-mxnet/issues/10456 Windows only, but this should support AMD and Intel GPUs using D3D12 backend or even DirectML which with custom metacommands seems even able to use tensor

[GitHub] VikingMew commented on issue #8238: different behaviour of customop in latest MXNet

2018-04-07 Thread GitBox
VikingMew commented on issue #8238: different behaviour of customop in latest MXNet URL: https://github.com/apache/incubator-mxnet/issues/8238#issuecomment-379448411 The customop ``` class Maskeddense: def list_arguments(self): return ['data',

[GitHub] hetong007 commented on issue #10450: [MXNET-212] [R] Fix Random Samplers from Uniform and Gaussian distribution in R bindings

2018-04-07 Thread GitBox
hetong007 commented on issue #10450: [MXNET-212] [R] Fix Random Samplers from Uniform and Gaussian distribution in R bindings URL: https://github.com/apache/incubator-mxnet/pull/10450#issuecomment-379447803 Please add test for it.

[GitHub] hetong007 commented on issue #10450: [MXNET-212] [R] Fix Random Samplers from Uniform and Gaussian distribution in R bindings

2018-04-07 Thread GitBox
hetong007 commented on issue #10450: [MXNET-212] [R] Fix Random Samplers from Uniform and Gaussian distribution in R bindings URL: https://github.com/apache/incubator-mxnet/pull/10450#issuecomment-379447803 Please add test for it.

[GitHub] majia-yu opened a new issue #10455: Bug of group2ctxs for model parallelism

2018-04-07 Thread GitBox
majia-yu opened a new issue #10455: Bug of group2ctxs for model parallelism URL: https://github.com/apache/incubator-mxnet/issues/10455 ## Description I use mx.mod.Module with group2ctxs to split a very larger softmax to multiple gpus. When I use less than 8 gpus, everything is fine,

[GitHub] anirudh2290 commented on a change in pull request #10374: Sparse support for Custom Op

2018-04-07 Thread GitBox
anirudh2290 commented on a change in pull request #10374: Sparse support for Custom Op URL: https://github.com/apache/incubator-mxnet/pull/10374#discussion_r179910358 ## File path: python/mxnet/operator.py ## @@ -522,6 +525,71 @@ def infer_type(self, in_type):

[GitHub] anirudh2290 commented on a change in pull request #10374: Sparse support for Custom Op

2018-04-07 Thread GitBox
anirudh2290 commented on a change in pull request #10374: Sparse support for Custom Op URL: https://github.com/apache/incubator-mxnet/pull/10374#discussion_r179910359 ## File path: src/operator/custom/custom.cc ## @@ -266,97 +292,243 @@ OpStatePtr CreateState(const

[GitHub] anirudh2290 commented on a change in pull request #10374: Sparse support for Custom Op

2018-04-07 Thread GitBox
anirudh2290 commented on a change in pull request #10374: Sparse support for Custom Op URL: https://github.com/apache/incubator-mxnet/pull/10374#discussion_r179910356 ## File path: tests/python/unittest/test_operator.py ## @@ -4059,14 +4080,100 @@ def

[GitHub] anirudh2290 commented on a change in pull request #10374: Sparse support for Custom Op

2018-04-07 Thread GitBox
anirudh2290 commented on a change in pull request #10374: Sparse support for Custom Op URL: https://github.com/apache/incubator-mxnet/pull/10374#discussion_r179910335 ## File path: tests/python/unittest/test_operator.py ## @@ -4059,14 +4080,100 @@ def

[GitHub] Godricly opened a new issue #9704: Misleading Document for conv1d and max1d in gluon

2018-04-07 Thread GitBox
Godricly opened a new issue #9704: Misleading Document for conv1d and max1d in gluon URL: https://github.com/apache/incubator-mxnet/issues/9704 The document claims to support 'NWC' layout, while it actually not. my current mxnet version is mxnet-cu80 (1.0.0.post4). @zhreshold

[GitHub] Godricly commented on issue #9704: Misleading Document for conv1d and max1d in gluon

2018-04-07 Thread GitBox
Godricly commented on issue #9704: Misleading Document for conv1d and max1d in gluon URL: https://github.com/apache/incubator-mxnet/issues/9704#issuecomment-379437062 @gianlucacorrado reopened This is an automated message

[GitHub] eric-haibin-lin commented on issue #10453: Bug of CuDNN RNN with variable sequence length

2018-04-07 Thread GitBox
eric-haibin-lin commented on issue #10453: Bug of CuDNN RNN with variable sequence length URL: https://github.com/apache/incubator-mxnet/issues/10453#issuecomment-379435652 @szha how much memory consumption did you observe? One thing to try is to can run the code step by step and

[GitHub] szha commented on issue #10453: Bug of CuDNN RNN with variable sequence length

2018-04-07 Thread GitBox
szha commented on issue #10453: Bug of CuDNN RNN with variable sequence length URL: https://github.com/apache/incubator-mxnet/issues/10453#issuecomment-379435474 I was able to finish running the script by setting `export MXNET_GPU_MEM_POOL_RESERVE=7`

[GitHub] haojin2 commented on issue #10454: Improve row_sparse tutorial

2018-04-07 Thread GitBox
haojin2 commented on issue #10454: Improve row_sparse tutorial URL: https://github.com/apache/incubator-mxnet/pull/10454#issuecomment-379435064 LGTM! This is an automated message from the Apache Git Service. To respond to the