[GitHub] aaronmarkham commented on issue #10010: Documentation for build_version_doc scripts folder

2018-03-07 Thread GitBox
aaronmarkham commented on issue #10010: Documentation for build_version_doc scripts folder URL: https://github.com/apache/incubator-mxnet/pull/10010#issuecomment-371226453 https://issues.apache.org/jira/projects/MXNET/issues/MXNET-47

[GitHub] zhanghang1989 commented on issue #10027: Fix bug/typo for Dropout using axes

2018-03-07 Thread GitBox
zhanghang1989 commented on issue #10027: Fix bug/typo for Dropout using axes URL: https://github.com/apache/incubator-mxnet/pull/10027#issuecomment-371269252 @szha this should fix the problem of dropout using axes. This is an

[GitHub] Jerryzcn commented on a change in pull request #9977: Cpu lstm inference

2018-03-07 Thread GitBox
Jerryzcn commented on a change in pull request #9977: Cpu lstm inference URL: https://github.com/apache/incubator-mxnet/pull/9977#discussion_r172979255 ## File path: src/operator/rnn-inl.h ## @@ -287,8 +507,9 @@ class RNNProp : public OperatorProperty { const

[GitHub] thinksanky opened a new pull request #57: fixed redirection of the install page

2018-03-07 Thread GitBox
thinksanky opened a new pull request #57: fixed redirection of the install page URL: https://github.com/apache/incubator-mxnet-site/pull/57 ## Description ## - Added redirection to install page to mitigate the issue. This

[GitHub] aaronmarkham commented on issue #10013: update on setting up Scala with MXNet and the IntelliJ IDE

2018-03-07 Thread GitBox
aaronmarkham commented on issue #10013: update on setting up Scala with MXNet and the IntelliJ IDE URL: https://github.com/apache/incubator-mxnet/pull/10013#issuecomment-371228893 https://issues.apache.org/jira/projects/MXNET/issues/MXNET-48

[GitHub] sergeykolychev commented on a change in pull request #9988: [Perl] Sparse feature.

2018-03-07 Thread GitBox
sergeykolychev commented on a change in pull request #9988: [Perl] Sparse feature. URL: https://github.com/apache/incubator-mxnet/pull/9988#discussion_r172942895 ## File path: perl-package/AI-MXNet/lib/AI/MXNet/NDArray/Sparse.pm ## @@ -0,0 +1,1342 @@ +# Licensed to the

[GitHub] sxjscience commented on issue #10016: MxNet hangs up during bind.

2018-03-07 Thread GitBox
sxjscience commented on issue #10016: MxNet hangs up during bind. URL: https://github.com/apache/incubator-mxnet/issues/10016#issuecomment-371244313 Hi @kandoiNikhil , what kind of GPU are you using? Sometimes it's due to the JIT compilation. You can cache the result of the JIT

[GitHub] zhanghang1989 commented on issue #9989: Cannot train example gluon style transfer

2018-03-07 Thread GitBox
zhanghang1989 commented on issue #9989: Cannot train example gluon style transfer URL: https://github.com/apache/incubator-mxnet/issues/9989#issuecomment-371252011 @piiswrong Need some help about API changes. In the style transfer example, the `set_data()` function was okay for

[GitHub] zhanghang1989 commented on issue #9989: Cannot train example gluon style transfer

2018-03-07 Thread GitBox
zhanghang1989 commented on issue #9989: Cannot train example gluon style transfer URL: https://github.com/apache/incubator-mxnet/issues/9989#issuecomment-371252011 @piiswrong Need some help about API changes. In the style transfer example, the `set_data()` function was okay for

[GitHub] zhanghang1989 opened a new pull request #10027: Fix bug/typo for Dropout using axes

2018-03-07 Thread GitBox
zhanghang1989 opened a new pull request #10027: Fix bug/typo for Dropout using axes URL: https://github.com/apache/incubator-mxnet/pull/10027 ## Description ## Fix bug for a recent PR https://github.com/apache/incubator-mxnet/pull/9931 ## Checklist ## ### Essentials ### - [

[GitHub] Jerryzcn commented on a change in pull request #9977: Cpu lstm inference

2018-03-07 Thread GitBox
Jerryzcn commented on a change in pull request #9977: Cpu lstm inference URL: https://github.com/apache/incubator-mxnet/pull/9977#discussion_r172978477 ## File path: src/operator/rnn-inl.h ## @@ -78,10 +82,12 @@ inline int rnn_param_size(int layerNum, int size =

[incubator-mxnet-site] branch asf-site updated (7083943 -> 465896d)

2018-03-07 Thread liuyizhi
This is an automated email from the ASF dual-hosted git repository. liuyizhi pushed a change to branch asf-site in repository https://gitbox.apache.org/repos/asf/incubator-mxnet-site.git. from 7083943 Merge pull request #56 from thinksanky/touched_file_to_refresh_site add 3d841e3

[GitHub] zhanghang1989 commented on issue #9989: Cannot train example gluon style transfer

2018-03-07 Thread GitBox
zhanghang1989 commented on issue #9989: Cannot train example gluon style transfer URL: https://github.com/apache/incubator-mxnet/issues/9989#issuecomment-371242715 Hi @samhodge , you shouldn't move L82 out of autograd.record scope, because the gradient of siamese network won't

[GitHub] marcoabreu opened a new issue #10026: MXNET_MKLDNN_DEBUG=1 produces errors

2018-03-07 Thread GitBox
marcoabreu opened a new issue #10026: MXNET_MKLDNN_DEBUG=1 produces errors URL: https://github.com/apache/incubator-mxnet/issues/10026 http://jenkins.mxnet-ci.amazon-ml.com/blue/organizations/jenkins/incubator-mxnet/detail/PR-9995/32/pipeline/483 Setting ``MXNET_MKLDNN_DEBUG=1`` as

[GitHub] zheng-da commented on a change in pull request #9977: Cpu lstm inference

2018-03-07 Thread GitBox
zheng-da commented on a change in pull request #9977: Cpu lstm inference URL: https://github.com/apache/incubator-mxnet/pull/9977#discussion_r172944904 ## File path: src/operator/rnn-inl.h ## @@ -287,8 +507,9 @@ class RNNProp : public OperatorProperty { const

[GitHub] zheng-da commented on a change in pull request #9977: Cpu lstm inference

2018-03-07 Thread GitBox
zheng-da commented on a change in pull request #9977: Cpu lstm inference URL: https://github.com/apache/incubator-mxnet/pull/9977#discussion_r172712476 ## File path: src/operator/rnn-inl.h ## @@ -144,15 +149,224 @@ class RNNOp : public Operator {

[GitHub] zheng-da commented on a change in pull request #9977: Cpu lstm inference

2018-03-07 Thread GitBox
zheng-da commented on a change in pull request #9977: Cpu lstm inference URL: https://github.com/apache/incubator-mxnet/pull/9977#discussion_r172948107 ## File path: src/operator/rnn-inl.h ## @@ -144,15 +149,224 @@ class RNNOp : public Operator {

[GitHub] zheng-da commented on a change in pull request #9977: Cpu lstm inference

2018-03-07 Thread GitBox
zheng-da commented on a change in pull request #9977: Cpu lstm inference URL: https://github.com/apache/incubator-mxnet/pull/9977#discussion_r172710204 ## File path: src/operator/rnn-inl.h ## @@ -78,10 +82,12 @@ inline int rnn_param_size(int layerNum, int size =

[GitHub] Jerryzcn commented on a change in pull request #9977: Cpu lstm inference

2018-03-07 Thread GitBox
Jerryzcn commented on a change in pull request #9977: Cpu lstm inference URL: https://github.com/apache/incubator-mxnet/pull/9977#discussion_r172979255 ## File path: src/operator/rnn-inl.h ## @@ -287,8 +507,9 @@ class RNNProp : public OperatorProperty { const

[GitHub] yzhliu closed pull request #57: fixed redirection of the install page

2018-03-07 Thread GitBox
yzhliu closed pull request #57: fixed redirection of the install page URL: https://github.com/apache/incubator-mxnet-site/pull/57 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-site] 01/01: Merge pull request #57 from thinksanky/fix_getting_started_install_redirection

2018-03-07 Thread liuyizhi
This is an automated email from the ASF dual-hosted git repository. liuyizhi pushed a commit to branch asf-site in repository https://gitbox.apache.org/repos/asf/incubator-mxnet-site.git commit 465896d944c1020e80c67e33d5565893b2c18e76 Merge: 7083943 3d841e3 Author: Yizhi Liu

[GitHub] yzhliu opened a new pull request #10031: Sync master with v1.1.0 branch

2018-03-07 Thread GitBox
yzhliu opened a new pull request #10031: Sync master with v1.1.0 branch URL: https://github.com/apache/incubator-mxnet/pull/10031 I have went through the commits and checked all the changes on v1.1.0 branch has been merged into master, except the LICENSE and NEWS below. But I did

[GitHub] zhanghang1989 opened a new pull request #10028: fix bug for dropout with axes

2018-03-07 Thread GitBox
zhanghang1989 opened a new pull request #10028: fix bug for dropout with axes URL: https://github.com/apache/incubator-mxnet/pull/10028 ## Description ## fix bug for a previous PR https://github.com/apache/incubator-mxnet/pull/9931 ## Checklist ## ### Essentials ### - [ ]

[GitHub] OferRosenberg commented on issue #8917: Build using android.arm64 android.armv7 dockers fails, while arm64 and arm7 dockers work

2018-03-07 Thread GitBox
OferRosenberg commented on issue #8917: Build using android.arm64 android.armv7 dockers fails, while arm64 and arm7 dockers work URL: https://github.com/apache/incubator-mxnet/issues/8917#issuecomment-371295031 Thanks ! checked it out and it works perfectly.

[GitHub] OferRosenberg closed issue #8917: Build using android.arm64 android.armv7 dockers fails, while arm64 and arm7 dockers work

2018-03-07 Thread GitBox
OferRosenberg closed issue #8917: Build using android.arm64 android.armv7 dockers fails, while arm64 and arm7 dockers work URL: https://github.com/apache/incubator-mxnet/issues/8917 This is an automated message from the

[GitHub] cjolivier01 closed pull request #10007: add name

2018-03-07 Thread GitBox
cjolivier01 closed pull request #10007: add name URL: https://github.com/apache/incubator-mxnet/pull/10007 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 request (from a

[GitHub] zheng-da commented on a change in pull request #9977: Cpu lstm inference

2018-03-07 Thread GitBox
zheng-da commented on a change in pull request #9977: Cpu lstm inference URL: https://github.com/apache/incubator-mxnet/pull/9977#discussion_r173038522 ## File path: src/operator/rnn-inl.h ## @@ -287,8 +507,9 @@ class RNNProp : public OperatorProperty { const

[GitHub] zheng-da commented on issue #9977: Cpu lstm inference

2018-03-07 Thread GitBox
zheng-da commented on issue #9977: Cpu lstm inference URL: https://github.com/apache/incubator-mxnet/pull/9977#issuecomment-371345496 @Jerryzcn can you provide the workload for speed evaluation? so later on, other people can reproduce the result if they want.

[GitHub] zhanghang1989 closed pull request #10027: Fix bug/typo for Dropout using axes

2018-03-07 Thread GitBox
zhanghang1989 closed pull request #10027: Fix bug/typo for Dropout using axes URL: https://github.com/apache/incubator-mxnet/pull/10027 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] alihashmiii commented on issue #8936: Issue on MXNet R installation with GPU support on Windows

2018-03-07 Thread GitBox
alihashmiii commented on issue #8936: Issue on MXNet R installation with GPU support on Windows URL: https://github.com/apache/incubator-mxnet/issues/8936#issuecomment-371310815 Same problem here. why is not the GPU support easier for R mxnet. Python GPU support was so much easier

[GitHub] sxjscience opened a new pull request #10029: Layer Norm

2018-03-07 Thread GitBox
sxjscience opened a new pull request #10029: Layer Norm URL: https://github.com/apache/incubator-mxnet/pull/10029 ## Description ## 1. Directly implement layer normalization in C++. The speed and memory cost are both better than the way of stacking the broadcast/reduce OPs. Solves

[GitHub] aaronmarkham opened a new pull request #58: Redirect via mod_rewrite

2018-03-07 Thread GitBox
aaronmarkham opened a new pull request #58: Redirect via mod_rewrite URL: https://github.com/apache/incubator-mxnet-site/pull/58 ## Description ## Even though #57 should have kicked in I'm still seeing a 404 for: http://mxnet.incubator.apache.org/get_started/install.html This

[GitHub] pengzhao-intel commented on issue #9918: Update mkldnn to the newest & Add clang build test with mkldnn.

2018-03-07 Thread GitBox
pengzhao-intel commented on issue #9918: Update mkldnn to the newest & Add clang build test with mkldnn. URL: https://github.com/apache/incubator-mxnet/pull/9918#issuecomment-371342064 @marcoabreu @KellenSunderland @cjolivier01 please help confirm the final changes. And this issue will

[GitHub] marcoabreu commented on a change in pull request #9963: Onnx Module to import onnx models into mxnet

2018-03-07 Thread GitBox
marcoabreu commented on a change in pull request #9963: Onnx Module to import onnx models into mxnet URL: https://github.com/apache/incubator-mxnet/pull/9963#discussion_r172999228 ## File path: tests/python/unittest/test_layers.py ## @@ -50,5 +50,29 @@ def

[GitHub] zhanghang1989 opened a new pull request #10030: Fix bug for Dropout with axes, also adding unit test

2018-03-07 Thread GitBox
zhanghang1989 opened a new pull request #10030: Fix bug for Dropout with axes, also adding unit test URL: https://github.com/apache/incubator-mxnet/pull/10030 ## Description ## (Brief description on what this PR is about) ## Checklist ## ### Essentials ### - [ ] Passed code

[incubator-mxnet] branch master updated: add name (#10007)

2018-03-07 Thread cjolivier01
This is an automated email from the ASF dual-hosted git repository. cjolivier01 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 098c9d9 add name (#10007)

[GitHub] szha commented on issue #10028: fix bug for dropout with axes

2018-03-07 Thread GitBox
szha commented on issue #10028: fix bug for dropout with axes URL: https://github.com/apache/incubator-mxnet/pull/10028#issuecomment-371324917 Please post the fix and tests together. This is an automated message from the

[GitHub] szha closed pull request #10028: fix bug for dropout with axes

2018-03-07 Thread GitBox
szha closed pull request #10028: fix bug for dropout with axes URL: https://github.com/apache/incubator-mxnet/pull/10028 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] sxjscience commented on issue #10029: Layer Norm

2018-03-07 Thread GitBox
sxjscience commented on issue #10029: Layer Norm URL: https://github.com/apache/incubator-mxnet/pull/10029#issuecomment-371330053 @fhieber @tdomhan You could try this after it gets merged. This is an automated message from

[GitHub] zhanghang1989 commented on issue #10027: Fix bug/typo for Dropout using axes

2018-03-07 Thread GitBox
zhanghang1989 commented on issue #10027: Fix bug/typo for Dropout using axes URL: https://github.com/apache/incubator-mxnet/pull/10027#issuecomment-371293067 will create another one. This is an automated message from the

[GitHub] alihashmiii commented on issue #8936: Issue on MXNet R installation with GPU support on Windows

2018-03-07 Thread GitBox
alihashmiii commented on issue #8936: Issue on MXNet R installation with GPU support on Windows URL: https://github.com/apache/incubator-mxnet/issues/8936#issuecomment-371310815 I am facing the same problem here. why is not the GPU support easier for R mxnet. Python GPU support was so

[GitHub] marcoabreu commented on issue #9995: CI docker revamp; Add Jetson, Raspberry and CentOS 7 build [MXNET-42][MXNET-43][MXNET-44][MXNET-57]

2018-03-07 Thread GitBox
marcoabreu commented on issue #9995: CI docker revamp; Add Jetson, Raspberry and CentOS 7 build [MXNET-42][MXNET-43][MXNET-44][MXNET-57] URL: https://github.com/apache/incubator-mxnet/pull/9995#issuecomment-371318315 @szha @sergeykolychev @tqchen @cjolivier01 @eric-haibin-lin can you

[GitHub] szha closed pull request #10030: Fix bug for Dropout with axes, also adding unit test

2018-03-07 Thread GitBox
szha closed pull request #10030: Fix bug for Dropout with axes, also adding unit test URL: https://github.com/apache/incubator-mxnet/pull/10030 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] szha commented on issue #10030: Fix bug for Dropout with axes, also adding unit test

2018-03-07 Thread GitBox
szha commented on issue #10030: Fix bug for Dropout with axes, also adding unit test URL: https://github.com/apache/incubator-mxnet/pull/10030#issuecomment-371349152 Thanks. Let's add a jira retroactively when you figure out how.

[GitHub] cjolivier01 commented on issue #9688: Adding BilinearResize2D and AdaptiveAvgPool2d operators

2018-03-07 Thread GitBox
cjolivier01 commented on issue #9688: Adding BilinearResize2D and AdaptiveAvgPool2d operators URL: https://github.com/apache/incubator-mxnet/pull/9688#issuecomment-371360311 just curious, does opencv not have an optimized bilinear interpolation resize call? Intel makes a really fast

[GitHub] CoinCheung commented on a change in pull request #10000: fix average pooling kernel size assignment error

2018-03-07 Thread GitBox
CoinCheung commented on a change in pull request #1: fix average pooling kernel size assignment error URL: https://github.com/apache/incubator-mxnet/pull/1#discussion_r173058901 ## File path: tests/python/gpu/test_operator_gpu.py ## @@ -904,86 +904,87 @@ def

[GitHub] eric-haibin-lin commented on a change in pull request #9988: [Perl] Sparse feature.

2018-03-07 Thread GitBox
eric-haibin-lin commented on a change in pull request #9988: [Perl] Sparse feature. URL: https://github.com/apache/incubator-mxnet/pull/9988#discussion_r173079707 ## File path: perl-package/AI-MXNet/lib/AI/MXNet/NDArray/Sparse.pm ## @@ -0,0 +1,1342 @@ +# Licensed to the

[GitHub] eric-haibin-lin commented on a change in pull request #9988: [Perl] Sparse feature.

2018-03-07 Thread GitBox
eric-haibin-lin commented on a change in pull request #9988: [Perl] Sparse feature. URL: https://github.com/apache/incubator-mxnet/pull/9988#discussion_r173078142 ## File path: perl-package/AI-MXNetCAPI/mxnet.i ## @@ -338,6 +351,37 @@ int MXNDArrayCreateEx(const mx_uint

[GitHub] eric-haibin-lin commented on a change in pull request #9988: [Perl] Sparse feature.

2018-03-07 Thread GitBox
eric-haibin-lin commented on a change in pull request #9988: [Perl] Sparse feature. URL: https://github.com/apache/incubator-mxnet/pull/9988#discussion_r173079068 ## File path: perl-package/AI-MXNet/t/test_kvstore.t ## @@ -73,6 +75,93 @@ sub test_list_kv_pair } }

[GitHub] eric-haibin-lin commented on issue #10031: Sync master with v1.1.0 branch

2018-03-07 Thread GitBox
eric-haibin-lin commented on issue #10031: Sync master with v1.1.0 branch URL: https://github.com/apache/incubator-mxnet/pull/10031#issuecomment-371350293 Looks good if README is also updated. This is an automated message

[GitHub] sxjscience commented on a change in pull request #10000: fix average pooling kernel size assignment error

2018-03-07 Thread GitBox
sxjscience commented on a change in pull request #1: fix average pooling kernel size assignment error URL: https://github.com/apache/incubator-mxnet/pull/1#discussion_r173059994 ## File path: tests/python/gpu/test_operator_gpu.py ## @@ -904,86 +904,87 @@ def

[GitHub] zheng-da commented on a change in pull request #10025: Language model with Google's billion words dataset

2018-03-07 Thread GitBox
zheng-da commented on a change in pull request #10025: Language model with Google's billion words dataset URL: https://github.com/apache/incubator-mxnet/pull/10025#discussion_r173047522 ## File path: src/operator/nn/fully_connected.cc ## @@ -87,8 +90,16 @@ void

[GitHub] szha opened a new pull request #10033: correct resnet link in model zoo page

2018-03-07 Thread GitBox
szha opened a new pull request #10033: correct resnet link in model zoo page URL: https://github.com/apache/incubator-mxnet/pull/10033 ## Description ## correct resnet paper link in model zoo page ## Checklist ## ### Essentials ### - [x] Code is well-documented: ###

[GitHub] fhieber commented on issue #10029: [MXNET-58]Layer Normalization in C++

2018-03-07 Thread GitBox
fhieber commented on issue #10029: [MXNET-58]Layer Normalization in C++ URL: https://github.com/apache/incubator-mxnet/pull/10029#issuecomment-371400507 @sxjscience fantastic, thank you! We will definitely try this as soon as its available!

svn commit: r25579 - /release/incubator/mxnet/1.0.0/

2018-03-07 Thread liuyizhi
Author: liuyizhi Date: Thu Mar 8 02:04:22 2018 New Revision: 25579 Log: remove the previous release folder (1.0.0) Removed: release/incubator/mxnet/1.0.0/

svn commit: r25578 - in /dev/incubator/mxnet: 1.1.0.rc0/ 1.1.0.rc1/

2018-03-07 Thread liuyizhi
Author: liuyizhi Date: Thu Mar 8 02:02:50 2018 New Revision: 25578 Log: remove the previous release candidate folders (1.1.0.rc*) Removed: dev/incubator/mxnet/1.1.0.rc0/ dev/incubator/mxnet/1.1.0.rc1/

[GitHub] zhanghang1989 commented on issue #9688: Adding BilinearResize2D and AdaptiveAvgPool2d operators

2018-03-07 Thread GitBox
zhanghang1989 commented on issue #9688: Adding BilinearResize2D and AdaptiveAvgPool2d operators URL: https://github.com/apache/incubator-mxnet/pull/9688#issuecomment-371360712 @cjolivier01 I guess opencv won't support batch dimension. We have to support 4D input (Batch x Channel x Height

[GitHub] sxjscience commented on a change in pull request #10000: fix average pooling kernel size assignment error

2018-03-07 Thread GitBox
sxjscience commented on a change in pull request #1: fix average pooling kernel size assignment error URL: https://github.com/apache/incubator-mxnet/pull/1#discussion_r173062611 ## File path: tests/python/gpu/test_operator_gpu.py ## @@ -904,86 +904,87 @@ def

[GitHub] szha closed pull request #10022: Fix ndarray assignment issue with basic indexing

2018-03-07 Thread GitBox
szha closed pull request #10022: Fix ndarray assignment issue with basic indexing URL: https://github.com/apache/incubator-mxnet/pull/10022 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: Fix ndarray assignment issue with basic indexing (#10022)

2018-03-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 39c0fd8 Fix ndarray assignment

[GitHub] sxjscience commented on issue #9934: [MXNET-31] Support variable sequence length in gluon.RecurrentCell

2018-03-07 Thread GitBox
sxjscience commented on issue #9934: [MXNET-31] Support variable sequence length in gluon.RecurrentCell URL: https://github.com/apache/incubator-mxnet/pull/9934#issuecomment-371383734 @szha @piiswrong I've added the test of VariationalDropoutCell.

[GitHub] eric-haibin-lin commented on a change in pull request #10025: Language model with Google's billion words dataset

2018-03-07 Thread GitBox
eric-haibin-lin commented on a change in pull request #10025: Language model with Google's billion words dataset URL: https://github.com/apache/incubator-mxnet/pull/10025#discussion_r173073715 ## File path: src/operator/nn/fully_connected.cc ## @@ -87,8 +90,16 @@ void

[GitHub] eric-haibin-lin commented on a change in pull request #9986: gluon language modeling dataset and text token reader

2018-03-07 Thread GitBox
eric-haibin-lin commented on a change in pull request #9986: gluon language modeling dataset and text token reader URL: https://github.com/apache/incubator-mxnet/pull/9986#discussion_r173077682 ## File path: python/mxnet/gluon/data/text/base.py ## @@ -0,0 +1,99 @@ +#

[GitHub] eric-haibin-lin commented on a change in pull request #9986: gluon language modeling dataset and text token reader

2018-03-07 Thread GitBox
eric-haibin-lin commented on a change in pull request #9986: gluon language modeling dataset and text token reader URL: https://github.com/apache/incubator-mxnet/pull/9986#discussion_r173077529 ## File path: python/mxnet/gluon/data/text/base.py ## @@ -0,0 +1,99 @@ +#

[GitHub] eric-haibin-lin commented on a change in pull request #9986: gluon language modeling dataset and text token reader

2018-03-07 Thread GitBox
eric-haibin-lin commented on a change in pull request #9986: gluon language modeling dataset and text token reader URL: https://github.com/apache/incubator-mxnet/pull/9986#discussion_r173077113 ## File path: python/mxnet/gluon/data/text/base.py ## @@ -0,0 +1,99 @@ +#

[incubator-mxnet] branch master updated: Fix bug for Dropout with axes, also adding unit test (#10030)

2018-03-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 117c509 Fix bug for Dropout with

[GitHub] TaoLv commented on a change in pull request #9918: Update mkldnn to the newest & Add clang build test with mkldnn.

2018-03-07 Thread GitBox
TaoLv commented on a change in pull request #9918: Update mkldnn to the newest & Add clang build test with mkldnn. URL: https://github.com/apache/incubator-mxnet/pull/9918#discussion_r173041509 ## File path: src/operator/nn/mkldnn/mkldnn_base.cc ## @@ -237,13 +237,14 @@

[GitHub] TaoLv commented on issue #9152: [MXNET-37] tutorial for distributed training

2018-03-07 Thread GitBox
TaoLv commented on issue #9152: [MXNET-37] tutorial for distributed training URL: https://github.com/apache/incubator-mxnet/pull/9152#issuecomment-371353840 @rahul003 Thanks for your reply. I will try that. Seems some links in this tutorial are broker: [Training with multiple GPUs using

[GitHub] sxjscience commented on issue #10029: [MXNET-58]Layer Normalization in C++

2018-03-07 Thread GitBox
sxjscience commented on issue #10029: [MXNET-58]Layer Normalization in C++ URL: https://github.com/apache/incubator-mxnet/pull/10029#issuecomment-371374211 Here's the new doc of InstanceNorm

[GitHub] sxjscience commented on issue #10029: [MXNET-58]Layer Normalization in C++

2018-03-07 Thread GitBox
sxjscience commented on issue #10029: [MXNET-58]Layer Normalization in C++ URL: https://github.com/apache/incubator-mxnet/pull/10029#issuecomment-371374211 Here's the new doc of InstanceNorm

[GitHub] zihaolucky commented on issue #9990: Update tensorboard.py

2018-03-07 Thread GitBox
zihaolucky commented on issue #9990: Update tensorboard.py URL: https://github.com/apache/incubator-mxnet/pull/9990#issuecomment-371354509 Thanks for your contribution! LGTM. This is an automated message from the Apache Git

[GitHub] szha opened a new pull request #10032: add axes support for dropouts in gluon

2018-03-07 Thread GitBox
szha opened a new pull request #10032: add axes support for dropouts in gluon URL: https://github.com/apache/incubator-mxnet/pull/10032 ## Description ## add axes support for dropouts in gluon ## Checklist ## ### Essentials ### - [x] Passed code style checking (`make lint`)

[GitHub] CoinCheung commented on a change in pull request #10000: fix average pooling kernel size assignment error

2018-03-07 Thread GitBox
CoinCheung commented on a change in pull request #1: fix average pooling kernel size assignment error URL: https://github.com/apache/incubator-mxnet/pull/1#discussion_r173060956 ## File path: tests/python/gpu/test_operator_gpu.py ## @@ -904,86 +904,87 @@ def

[GitHub] eric-haibin-lin commented on a change in pull request #10025: Language model with Google's billion words dataset

2018-03-07 Thread GitBox
eric-haibin-lin commented on a change in pull request #10025: Language model with Google's billion words dataset URL: https://github.com/apache/incubator-mxnet/pull/10025#discussion_r173073054 ## File path: src/operator/nn/fully_connected.cc ## @@ -87,8 +90,16 @@ void

[GitHub] szha commented on a change in pull request #9986: gluon language modeling dataset and text token reader

2018-03-07 Thread GitBox
szha commented on a change in pull request #9986: gluon language modeling dataset and text token reader URL: https://github.com/apache/incubator-mxnet/pull/9986#discussion_r173085151 ## File path: python/mxnet/gluon/data/text/base.py ## @@ -0,0 +1,99 @@ +# Licensed to the

[GitHub] eric-haibin-lin opened a new pull request #10034: Update an incorrect description in the API doc.

2018-03-07 Thread GitBox
eric-haibin-lin opened a new pull request #10034: Update an incorrect description in the API doc. URL: https://github.com/apache/incubator-mxnet/pull/10034 Update an incorrect description in the API doc. This is an

[GitHub] eric-haibin-lin commented on issue #10025: Language model with Google's billion words dataset

2018-03-07 Thread GitBox
eric-haibin-lin commented on issue #10025: Language model with Google's billion words dataset URL: https://github.com/apache/incubator-mxnet/pull/10025#issuecomment-371403506 @zihaolucky This is an automated message from

[GitHub] szha commented on a change in pull request #9986: gluon language modeling dataset and text token reader

2018-03-07 Thread GitBox
szha commented on a change in pull request #9986: gluon language modeling dataset and text token reader URL: https://github.com/apache/incubator-mxnet/pull/9986#discussion_r173084963 ## File path: python/mxnet/gluon/data/text/base.py ## @@ -0,0 +1,99 @@ +# Licensed to the

[GitHub] eric-haibin-lin commented on a change in pull request #9988: [Perl] Sparse feature.

2018-03-07 Thread GitBox
eric-haibin-lin commented on a change in pull request #9988: [Perl] Sparse feature. URL: https://github.com/apache/incubator-mxnet/pull/9988#discussion_r173080890 ## File path: perl-package/AI-MXNet/lib/AI/MXNet/NDArray/Sparse.pm ## @@ -0,0 +1,1342 @@ +# Licensed to the

[GitHub] rahul003 commented on a change in pull request #9933: [MXNET-23] Adding support to profile kvstore server during distributed training

2018-03-07 Thread GitBox
rahul003 commented on a change in pull request #9933: [MXNET-23] Adding support to profile kvstore server during distributed training URL: https://github.com/apache/incubator-mxnet/pull/9933#discussion_r172764504 ## File path: src/kvstore/kvstore_dist_server.h ## @@

[GitHub] jeremiedb opened a new pull request #10023: R graph.viz fix

2018-03-07 Thread GitBox
jeremiedb opened a new pull request #10023: R graph.viz fix URL: https://github.com/apache/incubator-mxnet/pull/10023 As requested in #9975. Patch for v1.1.0 seems already done (#) This is an automated message from

[GitHub] rahul003 commented on issue #9152: tutorial for distributed training

2018-03-07 Thread GitBox
rahul003 commented on issue #9152: tutorial for distributed training URL: https://github.com/apache/incubator-mxnet/pull/9152#issuecomment-371059065 @TaoLv Sorry I missed your comment. You can profile the worker process similar to a single machine case. ```

[GitHub] eric-haibin-lin commented on issue #9921: [DISCUSSION] module.contrib.SparseModule API

2018-03-07 Thread GitBox
eric-haibin-lin commented on issue #9921: [DISCUSSION] module.contrib.SparseModule API URL: https://github.com/apache/incubator-mxnet/issues/9921#issuecomment-371033223 @anirudh2290 How much would a function call cost? I don't think static dictionary would be a common case.. Usually you

[GitHub] xinyu-intel commented on issue #9918: Update mkldnn to the newest & Add clang build test with mkldnn.

2018-03-07 Thread GitBox
xinyu-intel commented on issue #9918: Update mkldnn to the newest & Add clang build test with mkldnn. URL: https://github.com/apache/incubator-mxnet/pull/9918#issuecomment-371060912 @cjolivier01 This is an automated message

[GitHub] mseeger commented on issue #9982: Unary ops logcdf_normal, derivlogcdf_normal for log CDF of standard n?

2018-03-07 Thread GitBox
mseeger commented on issue #9982: Unary ops logcdf_normal, derivlogcdf_normal for log CDF of standard n? URL: https://github.com/apache/incubator-mxnet/pull/9982#issuecomment-371070814 @piiswrong Tensorflow has log_cdf in tf.distributions.Normal, see

[GitHub] jeremiedb commented on issue #9803: R Metrics

2018-03-07 Thread GitBox
jeremiedb commented on issue #9803: R Metrics URL: https://github.com/apache/incubator-mxnet/pull/9803#issuecomment-371074856 Sure, solving the root cause is definitely preferable. Help would however be welcomed to track the memory leak; I'm lacking resources to do so. Given there

[GitHub] zlin3000 opened a new pull request #10024: fixed calling error while preparing data in ssd

2018-03-07 Thread GitBox
zlin3000 opened a new pull request #10024: fixed calling error while preparing data in ssd URL: https://github.com/apache/incubator-mxnet/pull/10024 fixed calling error while calling im2rec.py in prepare_dataset.py add threads support in prepare_dataset.py ## Description ##

[GitHub] wkcn commented on issue #9939: add multi proposal operator (cpu version) and fix the bug in proposal op (gpu version)

2018-03-07 Thread GitBox
wkcn commented on issue #9939: add multi proposal operator (cpu version) and fix the bug in proposal op (gpu version) URL: https://github.com/apache/incubator-mxnet/pull/9939#issuecomment-371060094 @CodingCat I will read it. Thank you!

[GitHub] jeremiedb commented on issue #9967: Error in output.shape[[output.names]]

2018-03-07 Thread GitBox
jeremiedb commented on issue #9967: Error in output.shape[[output.names]] URL: https://github.com/apache/incubator-mxnet/issues/9967#issuecomment-371073140 `mx.model.FeedForward.create` is a function intended for training of the model, it is therefore expected to have a label passed as

[incubator-mxnet] branch v1.0.0 updated: R graph.viz fix (#10023)

2018-03-07 Thread marcoabreu
This is an automated email from the ASF dual-hosted git repository. marcoabreu pushed a commit to branch v1.0.0 in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git The following commit(s) were added to refs/heads/v1.0.0 by this push: new 3df9bf8 R graph.viz fix (#10023)

[GitHub] marcoabreu closed pull request #10023: R graph.viz fix

2018-03-07 Thread GitBox
marcoabreu closed pull request #10023: R graph.viz fix URL: https://github.com/apache/incubator-mxnet/pull/10023 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 request

[GitHub] eric-haibin-lin opened a new pull request #10025: Language model with Google's billion words dataset

2018-03-07 Thread GitBox
eric-haibin-lin opened a new pull request #10025: Language model with Google's billion words dataset URL: https://github.com/apache/incubator-mxnet/pull/10025 ## Description ## This example reproduces the result (~42 perplexity) on [Exploring the Limits of Language

[GitHub] Ldpe2G commented on issue #9984: [MXNET-38]add reshape predicator function to c_predict_api

2018-03-07 Thread GitBox
Ldpe2G commented on issue #9984: [MXNET-38]add reshape predicator function to c_predict_api URL: https://github.com/apache/incubator-mxnet/pull/9984#issuecomment-371159253 @cjolivier01 I have updated the pr This is an

[GitHub] marcoabreu commented on issue #10020: How can I build C++ package?

2018-03-07 Thread GitBox
marcoabreu commented on issue #10020: How can I build C++ package? URL: https://github.com/apache/incubator-mxnet/issues/10020#issuecomment-371121048 @aaronmarkham see comment about website This is an automated message from

[GitHub] asitstands closed pull request #9991: Random shuffle implementation

2018-03-07 Thread GitBox
asitstands closed pull request #9991: Random shuffle implementation URL: https://github.com/apache/incubator-mxnet/pull/9991 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] asitstands commented on issue #9991: Random shuffle implementation

2018-03-07 Thread GitBox
asitstands commented on issue #9991: Random shuffle implementation URL: https://github.com/apache/incubator-mxnet/pull/9991#issuecomment-371105396 Ok, I'll make a new PR with an implementation complying to numpy. This is an

[GitHub] edmBernard commented on issue #10020: How can I build C++ package?

2018-03-07 Thread GitBox
edmBernard commented on issue #10020: How can I build C++ package? URL: https://github.com/apache/incubator-mxnet/issues/10020#issuecomment-37564 You can use this parameter when you run your make command : `USE_CPP_PACKAGE=1`

[GitHub] marcoabreu commented on issue #9906: Add CPU optimized docker which will be compiled with MKL-DNN

2018-03-07 Thread GitBox
marcoabreu commented on issue #9906: Add CPU optimized docker which will be compiled with MKL-DNN URL: https://github.com/apache/incubator-mxnet/pull/9906#issuecomment-371115687 I can really only speak from a CI perspective, maybe @eric-haibin-lin or @szha could shed a bit of light into

[GitHub] marcoabreu commented on issue #9803: R Metrics

2018-03-07 Thread GitBox
marcoabreu commented on issue #9803: R Metrics URL: https://github.com/apache/incubator-mxnet/pull/9803#issuecomment-371117644 I understand. In that case I'm fine with merging as long as the problem is tracked somewhere.

[GitHub] TaoLv commented on a change in pull request #10025: Language model with Google's billion words dataset

2018-03-07 Thread GitBox
TaoLv commented on a change in pull request #10025: Language model with Google's billion words dataset URL: https://github.com/apache/incubator-mxnet/pull/10025#discussion_r172866473 ## File path: src/operator/nn/fully_connected.cc ## @@ -214,9 +238,7 @@ If ``no_bias`` is

  1   2   >