[GitHub] CodingCat commented on issue #9552: [REQUEST FOR REVIEW | DO NOT MERGE] Model Quantization with Calibration

2018-03-06 Thread GitBox
CodingCat commented on issue #9552: [REQUEST FOR REVIEW | DO NOT MERGE] Model Quantization with Calibration URL: https://github.com/apache/incubator-mxnet/pull/9552#issuecomment-371021235 Hi, the community has passed to vote about associating the code changes with JIRA (https://lists.apac

[GitHub] CodingCat commented on issue #9496: Draw labels name

2018-03-06 Thread GitBox
CodingCat commented on issue #9496: Draw labels name URL: https://github.com/apache/incubator-mxnet/pull/9496#issuecomment-371021278 Hi, the community has passed to vote about associating the code changes with JIRA (https://lists.apache.org/thread.html/ab22cf0e35f1bce2c3bf3bec2bc5b85a9583a

[GitHub] CodingCat commented on issue #9603: Btaskaya/stringformatting

2018-03-06 Thread GitBox
CodingCat commented on issue #9603: Btaskaya/stringformatting URL: https://github.com/apache/incubator-mxnet/pull/9603#issuecomment-371021123 Hi, the community has passed to vote about associating the code changes with JIRA (https://lists.apache.org/thread.html/ab22cf0e35f1bce2c3bf3bec2bc5

[GitHub] CodingCat commented on issue #9318: simplify import of citation metadata

2018-03-06 Thread GitBox
CodingCat commented on issue #9318: simplify import of citation metadata URL: https://github.com/apache/incubator-mxnet/pull/9318#issuecomment-371021300 Hi, the community has passed to vote about associating the code changes with JIRA (https://lists.apache.org/thread.html/ab22cf0e35f1bce2c

[GitHub] CodingCat commented on issue #9466: WIP (testing windows builds) Enable CUDA NVTX extensions for profiler

2018-03-06 Thread GitBox
CodingCat commented on issue #9466: WIP (testing windows builds) Enable CUDA NVTX extensions for profiler URL: https://github.com/apache/incubator-mxnet/pull/9466#issuecomment-371021291 Hi, the community has passed to vote about associating the code changes with JIRA (https://lists.apache

[GitHub] CodingCat commented on issue #9195: [WIP]NCE loss gluon

2018-03-06 Thread GitBox
CodingCat commented on issue #9195: [WIP]NCE loss gluon URL: https://github.com/apache/incubator-mxnet/pull/9195#issuecomment-371021350 Hi, the community has passed to vote about associating the code changes with JIRA (https://lists.apache.org/thread.html/ab22cf0e35f1bce2c3bf3bec2bc5b85a95

[GitHub] CodingCat commented on issue #9273: Enhancements to the build mechanism

2018-03-06 Thread GitBox
CodingCat commented on issue #9273: Enhancements to the build mechanism URL: https://github.com/apache/incubator-mxnet/pull/9273#issuecomment-371021319 Hi, the community has passed to vote about associating the code changes with JIRA (https://lists.apache.org/thread.html/ab22cf0e35f1bce2c3

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

2018-03-06 Thread GitBox
CodingCat commented on issue #9152: tutorial for distributed training URL: https://github.com/apache/incubator-mxnet/pull/9152#issuecomment-371021385 Hi, the community has passed to vote about associating the code changes with JIRA (https://lists.apache.org/thread.html/ab22cf0e35f1bce2c3bf

[GitHub] CodingCat commented on issue #9263: Fixes #9210: Cosine Loss Formula

2018-03-06 Thread GitBox
CodingCat commented on issue #9263: Fixes #9210: Cosine Loss Formula URL: https://github.com/apache/incubator-mxnet/pull/9263#issuecomment-371021334 Hi, the community has passed to vote about associating the code changes with JIRA (https://lists.apache.org/thread.html/ab22cf0e35f1bce2c3bf3

[GitHub] CodingCat commented on issue #9029: Update ndarray binary ops to use kernel launch instead of mshadow operations

2018-03-06 Thread GitBox
CodingCat commented on issue #9029: Update ndarray binary ops to use kernel launch instead of mshadow operations URL: https://github.com/apache/incubator-mxnet/pull/9029#issuecomment-371021408 Hi, the community has passed to vote about associating the code changes with JIRA (https://lists

[GitHub] CodingCat commented on issue #9142: add lstm sentimental analysis tutorial

2018-03-06 Thread GitBox
CodingCat commented on issue #9142: add lstm sentimental analysis tutorial URL: https://github.com/apache/incubator-mxnet/pull/9142#issuecomment-371021398 Hi, the community has passed to vote about associating the code changes with JIRA (https://lists.apache.org/thread.html/ab22cf0e35f1bce

[GitHub] CodingCat commented on issue #8107: "add warmup lr_scheduler" create a new pr

2018-03-06 Thread GitBox
CodingCat commented on issue #8107: "add warmup lr_scheduler" create a new pr URL: https://github.com/apache/incubator-mxnet/pull/8107#issuecomment-371021481 Hi, the community has passed to vote about associating the code changes with JIRA (https://lists.apache.org/thread.html/ab22cf0e35f

[GitHub] CodingCat commented on issue #8582: Yolo2 operator

2018-03-06 Thread GitBox
CodingCat commented on issue #8582: Yolo2 operator URL: https://github.com/apache/incubator-mxnet/pull/8582#issuecomment-371021434 Hi, the community has passed to vote about associating the code changes with JIRA (https://lists.apache.org/thread.html/ab22cf0e35f1bce2c3bf3bec2bc5b85a9583a3f

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

2018-03-06 Thread GitBox
CodingCat commented on issue #8915: NVLink communication pattern updated URL: https://github.com/apache/incubator-mxnet/pull/8915#issuecomment-371021419 Hi, the community has passed to vote about associating the code changes with JIRA (https://lists.apache.org/thread.html/ab22cf0e35f1bce2

[GitHub] CodingCat commented on issue #8797: fix errors for reinterpret_cast usage

2018-03-06 Thread GitBox
CodingCat commented on issue #8797: fix errors for reinterpret_cast usage URL: https://github.com/apache/incubator-mxnet/pull/8797#issuecomment-371021427 Hi, the community has passed to vote about associating the code changes with JIRA (https://lists.apache.org/thread.html/ab22cf0e35f1bce2

[GitHub] CodingCat commented on issue #8027: Optional reshape of predictions in Perplexity metric

2018-03-06 Thread GitBox
CodingCat commented on issue #8027: Optional reshape of predictions in Perplexity metric URL: https://github.com/apache/incubator-mxnet/pull/8027#issuecomment-371021494 Hi, the community has passed to vote about associating the code changes with JIRA (https://lists.apache.org/thread.html/

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

2018-03-06 Thread GitBox
CodingCat commented on issue #8423: Re-implement segnet in MXnet URL: https://github.com/apache/incubator-mxnet/pull/8423#issuecomment-371021462 Hi, the community has passed to vote about associating the code changes with JIRA (https://lists.apache.org/thread.html/ab22cf0e35f1bce2c3bf3bec2

[GitHub] CodingCat commented on issue #8547: Revised validating process of a bounding box in ImageDetIter

2018-03-06 Thread GitBox
CodingCat commented on issue #8547: Revised validating process of a bounding box in ImageDetIter URL: https://github.com/apache/incubator-mxnet/pull/8547#issuecomment-371021444 Hi, the community has passed to vote about associating the code changes with JIRA (https://lists.apache.org/thre

[GitHub] CodingCat commented on issue #8107: "add warmup lr_scheduler" create a new pr

2018-03-06 Thread GitBox
CodingCat commented on issue #8107: "add warmup lr_scheduler" create a new pr URL: https://github.com/apache/incubator-mxnet/pull/8107#issuecomment-371021474 Hi, the community has passed to vote about associating the code changes with JIRA (https://lists.apache.org/thread.html/ab22cf0e35f

[GitHub] Mabinogiysk opened a new issue #10020: How can I build C++ package?

2018-03-06 Thread GitBox
Mabinogiysk opened a new issue #10020: How can I build C++ package? URL: https://github.com/apache/incubator-mxnet/issues/10020 I can not find anything about installing mxnet for C++. The tutorials about [C++](https://mxnet.incubator.apache.org/tutorials/c++/basics.html) mentions ["bui

[GitHub] starimpact commented on issue #8695: Hangs training on P100

2018-03-06 Thread GitBox
starimpact commented on issue #8695: Hangs training on P100 URL: https://github.com/apache/incubator-mxnet/issues/8695#issuecomment-371023082 i encountered the deadlock also mxnet version is v0.8.0 ``` #0 pthread_cond_wait@@GLIBC_2.3.2 () #1 0x7f39f58198dc in std::con

[GitHub] nihui commented on issue #9960: [MXNET-30] Add label augmentation support in image recordio2

2018-03-06 Thread GitBox
nihui commented on issue #9960: [MXNET-30] Add label augmentation support in image recordio2 URL: https://github.com/apache/incubator-mxnet/pull/9960#issuecomment-371023666 jenkins please retest changes This is an automated m

[GitHub] piiswrong commented on issue #10018: Bug Fix and performance optimized for rtc

2018-03-06 Thread GitBox
piiswrong commented on issue #10018: Bug Fix and performance optimized for rtc URL: https://github.com/apache/incubator-mxnet/pull/10018#issuecomment-371026295 @CodingCat When was this? JIRA is ancient and arcane. This only adds unnecessary overhead. ---

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

2018-03-06 Thread GitBox
pengzhao-intel 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-371026517 @wkcn thanks for the detail profiling data. Regarding sorting, you can use M

[GitHub] piiswrong commented on a change in pull request #10014: Fix crash with mx.nd.ones

2018-03-06 Thread GitBox
piiswrong commented on a change in pull request #10014: Fix crash with mx.nd.ones URL: https://github.com/apache/incubator-mxnet/pull/10014#discussion_r172745323 ## File path: src/imperative/imperative_utils.h ## @@ -70,12 +70,14 @@ inline Context GetContext(const nnvm::No

[incubator-mxnet] branch master updated: add SameType as default type inference function in imperative mode (#9942)

2018-03-06 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 0fe04e9 add SameType as default type inf

[GitHub] piiswrong closed pull request #9942: add SameType as default type inference function in imperative mode

2018-03-06 Thread GitBox
piiswrong closed pull request #9942: add SameType as default type inference function in imperative mode URL: https://github.com/apache/incubator-mxnet/pull/9942 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of

[GitHub] ashokei opened a new pull request #10021: [MXNET-33] Mkldnn pooling convention crash

2018-03-06 Thread GitBox
ashokei opened a new pull request #10021: [MXNET-33] Mkldnn pooling convention crash URL: https://github.com/apache/incubator-mxnet/pull/10021 ## Description ## As crash is seen if pooling convention is "full" with Max pool. This PR resolves the crash by defaulting to CPU implementation

[GitHub] eric-haibin-lin closed issue #9957: R Package Fails to Build since #9882

2018-03-06 Thread GitBox
eric-haibin-lin closed issue #9957: R Package Fails to Build since #9882 URL: https://github.com/apache/incubator-mxnet/issues/9957 This is an automated message from the Apache Git Service. To respond to the message, please l

[GitHub] ashokei commented on issue #10021: [MXNET-33] Mkldnn pooling convention crash

2018-03-06 Thread GitBox
ashokei commented on issue #10021: [MXNET-33] Mkldnn pooling convention crash URL: https://github.com/apache/incubator-mxnet/pull/10021#issuecomment-371031468 @zheng-da @TaoLv please review. I notice we do not have proper "full" pooling convention support, this PR disables mkl-dnn pooling f

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

2018-03-06 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 Yes. It takes any function. So you could have sth like: ``` class A: def __init__(self):

[GitHub] Mabinogiysk commented on issue #9814: C++ Convolution layer usage produces "Segmentation fault"

2018-03-06 Thread GitBox
Mabinogiysk commented on issue #9814: C++ Convolution layer usage produces "Segmentation fault" URL: https://github.com/apache/incubator-mxnet/issues/9814#issuecomment-371033697 Hi, could you tell me where to find the tutorials about building C++ package? -

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

2018-03-06 Thread GitBox
piiswrong commented on issue #9934: [MXNET-31] Support variable sequence length in gluon.RecurrentCell URL: https://github.com/apache/incubator-mxnet/pull/9934#issuecomment-371034064 @szha Is this ready? This is an automated

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

2018-03-06 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-371034568 I need to test the VariationalDropoutCell. Will add it after I finish the implementation of LayerNorm. Ho

[GitHub] TaoLv commented on issue #10021: [MXNET-33] Mkldnn pooling convention crash

2018-03-06 Thread GitBox
TaoLv commented on issue #10021: [MXNET-33] Mkldnn pooling convention crash URL: https://github.com/apache/incubator-mxnet/pull/10021#issuecomment-371034685 There is a test case for max pooling with `full` convention: https://github.com/apache/incubator-mxnet/blob/master/tests/python/gpu/te

[GitHub] piiswrong commented on a change in pull request #9960: [MXNET-30] Add label augmentation support in image recordio2

2018-03-06 Thread GitBox
piiswrong commented on a change in pull request #9960: [MXNET-30] Add label augmentation support in image recordio2 URL: https://github.com/apache/incubator-mxnet/pull/9960#discussion_r172749317 ## File path: src/io/image_iter_common.h ## @@ -85,6 +85,14 @@ class ImageLabe

[GitHub] piiswrong commented on a change in pull request #9029: Update ndarray binary ops to use kernel launch instead of mshadow operations

2018-03-06 Thread GitBox
piiswrong commented on a change in pull request #9029: Update ndarray binary ops to use kernel launch instead of mshadow operations URL: https://github.com/apache/incubator-mxnet/pull/9029#discussion_r172749967 ## File path: src/ndarray/ndarray.cc ## @@ -805,15 +808,71 @@

[GitHub] piiswrong commented on a change in pull request #9029: Update ndarray binary ops to use kernel launch instead of mshadow operations

2018-03-06 Thread GitBox
piiswrong commented on a change in pull request #9029: Update ndarray binary ops to use kernel launch instead of mshadow operations URL: https://github.com/apache/incubator-mxnet/pull/9029#discussion_r172750286 ## File path: src/ndarray/ndarray_function-inl.h ## @@ -446,6

[GitHub] piiswrong commented on a change in pull request #10021: [MXNET-33] Mkldnn pooling convention crash

2018-03-06 Thread GitBox
piiswrong commented on a change in pull request #10021: [MXNET-33] Mkldnn pooling convention crash URL: https://github.com/apache/incubator-mxnet/pull/10021#discussion_r172750399 ## File path: src/operator/nn/mkldnn/mkldnn_pooling-inl.h ## @@ -92,6 +92,8 @@ inline bool Sup

[GitHub] piiswrong commented on a change in pull request #10018: Bug Fix and performance optimized for rtc

2018-03-06 Thread GitBox
piiswrong commented on a change in pull request #10018: Bug Fix and performance optimized for rtc URL: https://github.com/apache/incubator-mxnet/pull/10018#discussion_r172750653 ## File path: example/numpy-ops/custom_softmax_rtc.py ## @@ -23,78 +23,91 @@ class Softmax(m

[GitHub] anirudh2290 commented on a change in pull request #10014: Fix crash with mx.nd.ones

2018-03-06 Thread GitBox
anirudh2290 commented on a change in pull request #10014: Fix crash with mx.nd.ones URL: https://github.com/apache/incubator-mxnet/pull/10014#discussion_r172751325 ## File path: src/imperative/imperative_utils.h ## @@ -70,12 +70,14 @@ inline Context GetContext(const nnvm::

[GitHub] nihui commented on a change in pull request #9960: [MXNET-30] Add label augmentation support in image recordio2

2018-03-06 Thread GitBox
nihui commented on a change in pull request #9960: [MXNET-30] Add label augmentation support in image recordio2 URL: https://github.com/apache/incubator-mxnet/pull/9960#discussion_r172752595 ## File path: src/io/image_iter_common.h ## @@ -85,6 +85,14 @@ class ImageLabelMap

[GitHub] rahul003 commented on a change in pull request #9029: [MXNET-36] Update ndarray binary ops to use kernel launch instead of mshadow operations

2018-03-06 Thread GitBox
rahul003 commented on a change in pull request #9029: [MXNET-36] Update ndarray binary ops to use kernel launch instead of mshadow operations URL: https://github.com/apache/incubator-mxnet/pull/9029#discussion_r172760891 ## File path: src/ndarray/ndarray_function-inl.h ##

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

2018-03-06 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-370703827 @piiswrong Are you worried about polluting the global namespace with specific mathematical functions?

[GitHub] lightingghost opened a new issue #10002: _backward_Convolution is non-differentiable

2018-03-06 Thread GitBox
lightingghost opened a new issue #10002: _backward_Convolution is non-differentiable URL: https://github.com/apache/incubator-mxnet/issues/10002 As I saw mxnet has `autograd` package to support high order gradients, I tried to implement wgan-gp with mxnet. But I got an error of ```

[GitHub] ahkarami opened a new issue #10003: [feature request] C++ API Documentation and Tutorial

2018-03-06 Thread GitBox
ahkarami opened a new issue #10003: [feature request] C++ API Documentation and Tutorial URL: https://github.com/apache/incubator-mxnet/issues/10003 Thanks the MXNet team for releasing the framework. However, I think its current documentation for C++ API is not complete (_e.g._, How to ins

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

2018-03-06 Thread GitBox
marcoabreu 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-370716483 Hi, you can just create an empty commit to trigger the CI yourself :) JackieWu

[GitHub] marcoabreu closed pull request #9999: R fix

2018-03-06 Thread GitBox
marcoabreu closed pull request #: R fix URL: https://github.com/apache/incubator-mxnet/pull/ 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 fork),

[incubator-mxnet] branch v1.1.0 updated: R fix (#9999)

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

[GitHub] marcoabreu commented on a change in pull request #9995: [DO NOT MERGE] Start CI docker revamp

2018-03-06 Thread GitBox
marcoabreu commented on a change in pull request #9995: [DO NOT MERGE] Start CI docker revamp URL: https://github.com/apache/incubator-mxnet/pull/9995#discussion_r172461200 ## File path: ci/docker/install/ubuntu_perl.sh ## @@ -0,0 +1,26 @@ +#!/bin/bash + +# Licensed to the

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

2018-03-06 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-370730412 @marcoabreu Thank you:) I will commit it soon. -

[GitHub] larroy commented on issue #9993: cmake cannot build mxnet

2018-03-06 Thread GitBox
larroy commented on issue #9993: cmake cannot build mxnet URL: https://github.com/apache/incubator-mxnet/issues/9993#issuecomment-370737102 Well, it's clear from the logic in the CMake file what the problem is. I can fix it, but honestly I would like to test the build system better or be m

[GitHub] larroy commented on issue #9993: cmake cannot build mxnet

2018-03-06 Thread GitBox
larroy commented on issue #9993: cmake cannot build mxnet URL: https://github.com/apache/incubator-mxnet/issues/9993#issuecomment-370738175 Another problem right now is what we discussed previously with @cjolivier01 : When I build in CLion: ``` CMake Error at CMakeLists.txt

[GitHub] sbodenstein commented on issue #9543: Variable Length Support for cuDNN RNN

2018-03-06 Thread GitBox
sbodenstein commented on issue #9543: Variable Length Support for cuDNN RNN URL: https://github.com/apache/incubator-mxnet/issues/9543#issuecomment-370758230 So it seems as though we should wait a little bit before jumping in to add support. It seems as though Intel will have their RNN ope

[GitHub] larroy opened a new issue #10004: [ARM][Jetson] assertion failed in unit test

2018-03-06 Thread GitBox
larroy opened a new issue #10004: [ARM][Jetson] assertion failed in unit test URL: https://github.com/apache/incubator-mxnet/issues/10004 ## Description Unit test faillure / assertion triggered in testing forward for mobilenetv2_0.25 ## Environment info (Required) ```

[GitHub] Mabinogiysk opened a new issue #10005: Where should I put my dll after building from source?

2018-03-06 Thread GitBox
Mabinogiysk opened a new issue #10005: Where should I put my dll after building from source? URL: https://github.com/apache/incubator-mxnet/issues/10005 ## Description I have built mxnet from source and I got error when I import mxnet in python: ``` D:\Anaconda3\lib\ctypes\__init__

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

2018-03-06 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-370795671 If things have to be moved to a namespace, I'd need help on how to do that: I don't know how this works.

[GitHub] lixiangchun opened a new issue #10006: How to visualise filters of MXNet model?

2018-03-06 Thread GitBox
lixiangchun opened a new issue #10006: How to visualise filters of MXNet model? URL: https://github.com/apache/incubator-mxnet/issues/10006 How to visualise filters of MXNet model like what has been done in keras - [https://blog.keras.io/how-convolutional-neural-networks-see-the-world.html]

[GitHub] cjolivier01 opened a new pull request #10007: add name

2018-03-06 Thread GitBox
cjolivier01 opened a new pull request #10007: add name URL: https://github.com/apache/incubator-mxnet/pull/10007 ## Description ## (Brief description on what this PR is about) ## Checklist ## ### Essentials ### - [ ] Passed code style checking (`make lint`) - [ ] Changes a

[GitHub] precedenceguo commented on a change in pull request #9939: add multi proposal operator (cpu version) and fix the bug in proposal op (gpu version)

2018-03-06 Thread GitBox
precedenceguo commented on a change in pull request #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#discussion_r172564668 ## File path: src/operator/contrib/proposal.cu ##

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

2018-03-06 Thread GitBox
tlby commented on a change in pull request #9988: [Perl] Sparse feature. URL: https://github.com/apache/incubator-mxnet/pull/9988#discussion_r172571957 ## File path: perl-package/AI-MXNetCAPI/mxnet_typemaps.i ## @@ -112,6 +112,32 @@ } } +%typemap(in,numinputs=0) (co

[GitHub] sxjscience commented on issue #10002: _backward_Convolution is non-differentiable

2018-03-06 Thread GitBox
sxjscience commented on issue #10002: _backward_Convolution is non-differentiable URL: https://github.com/apache/incubator-mxnet/issues/10002#issuecomment-370864594 Supporting higher order gradients is on the agenda. Refer to the discussion in https://github.com/apache/incubator-mxnet/iss

[GitHub] sxjscience commented on issue #10002: _backward_Convolution is non-differentiable

2018-03-06 Thread GitBox
sxjscience commented on issue #10002: _backward_Convolution is non-differentiable URL: https://github.com/apache/incubator-mxnet/issues/10002#issuecomment-370865067 I find the issue is similar as https://github.com/apache/incubator-mxnet/issues/9979 . Let's move the discussion there. --

[GitHub] sxjscience closed issue #10002: _backward_Convolution is non-differentiable

2018-03-06 Thread GitBox
sxjscience closed issue #10002: _backward_Convolution is non-differentiable URL: https://github.com/apache/incubator-mxnet/issues/10002 This is an automated message from the Apache Git Service. To respond to the message, plea

[GitHub] cjolivier01 closed pull request #9955: [DO NOT MERGE] Cython CI check

2018-03-06 Thread GitBox
cjolivier01 closed pull request #9955: [DO NOT MERGE] Cython CI check URL: https://github.com/apache/incubator-mxnet/pull/9955 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 p

[GitHub] szha opened a new pull request #10008: gluon dataset download refactor

2018-03-06 Thread GitBox
szha opened a new pull request #10008: gluon dataset download refactor URL: https://github.com/apache/incubator-mxnet/pull/10008 ## Description ## refactor of download logic in #9986 ## Checklist ## ### Essentials ### - [x] Passed code style checking (`make lint`) - [x] C

[GitHub] larroy opened a new issue #10009: [CMAKE][IoT] Remove pthread from android_arm64 build

2018-03-06 Thread GitBox
larroy opened a new issue #10009: [CMAKE][IoT] Remove pthread from android_arm64 build URL: https://github.com/apache/incubator-mxnet/issues/10009 Logic to detect the target platform should be added to CMake by using try_run and #error as done in openmp ---

[GitHub] aaronmarkham opened a new pull request #10010: Documentation for build_version_doc scripts folder

2018-03-06 Thread GitBox
aaronmarkham opened a new pull request #10010: Documentation for build_version_doc scripts folder URL: https://github.com/apache/incubator-mxnet/pull/10010 ## Description ## Adding documentation for the `build_version_doc` scripts folder.

[GitHub] zhanghang1989 commented on issue #9931: Add axes support to Dropout for variational dropout in NLP

2018-03-06 Thread GitBox
zhanghang1989 commented on issue #9931: Add axes support to Dropout for variational dropout in NLP URL: https://github.com/apache/incubator-mxnet/pull/9931#issuecomment-370880704 @szha The performance of dropout should be the same as before, when no axes are given. --

[GitHub] cjolivier01 commented on issue #9931: Add axes support to Dropout for variational dropout in NLP

2018-03-06 Thread GitBox
cjolivier01 commented on issue #9931: Add axes support to Dropout for variational dropout in NLP URL: https://github.com/apache/incubator-mxnet/pull/9931#issuecomment-370880967 If you have a request for changes from a committer, you can't merge per Apache guidelines. -

[GitHub] cjolivier01 commented on issue #9931: Add axes support to Dropout for variational dropout in NLP

2018-03-06 Thread GitBox
cjolivier01 commented on issue #9931: Add axes support to Dropout for variational dropout in NLP URL: https://github.com/apache/incubator-mxnet/pull/9931#issuecomment-370881129 What is the performance impact of these changes for default axes behavior compared ot the older code? --

[GitHub] szha commented on issue #9931: Add axes support to Dropout for variational dropout in NLP

2018-03-06 Thread GitBox
szha commented on issue #9931: Add axes support to Dropout for variational dropout in NLP URL: https://github.com/apache/incubator-mxnet/pull/9931#issuecomment-370881524 Hang suggested that if `axes` is empty, the behavior is exactly the same as before the change https://github.com/apache

[GitHub] szha commented on issue #9931: Add axes support to Dropout for variational dropout in NLP

2018-03-06 Thread GitBox
szha commented on issue #9931: Add axes support to Dropout for variational dropout in NLP URL: https://github.com/apache/incubator-mxnet/pull/9931#issuecomment-370881902 Did the guideline explain how committers deal with stale reviews? --

[GitHub] cjolivier01 commented on issue #9931: Add axes support to Dropout for variational dropout in NLP

2018-03-06 Thread GitBox
cjolivier01 commented on issue #9931: Add axes support to Dropout for variational dropout in NLP URL: https://github.com/apache/incubator-mxnet/pull/9931#issuecomment-370883978 Apache say: "A code-modification proposal may be stopped dead in its tracks by a -1 vote by a qualified v

[GitHub] szha commented on issue #9931: Add axes support to Dropout for variational dropout in NLP

2018-03-06 Thread GitBox
szha commented on issue #9931: Add axes support to Dropout for variational dropout in NLP URL: https://github.com/apache/incubator-mxnet/pull/9931#issuecomment-370884321 Haha OK. Jokes aside, did Hang sufficiently address your concern? --

[GitHub] cjolivier01 commented on issue #9931: Add axes support to Dropout for variational dropout in NLP

2018-03-06 Thread GitBox
cjolivier01 commented on issue #9931: Add axes support to Dropout for variational dropout in NLP URL: https://github.com/apache/incubator-mxnet/pull/9931#issuecomment-370885831 Yeah, I'm good. This is an automated message fro

[GitHub] reminisce commented on issue #10006: How to visualise filters of MXNet model?

2018-03-06 Thread GitBox
reminisce commented on issue #10006: How to visualise filters of MXNet model? URL: https://github.com/apache/incubator-mxnet/issues/10006#issuecomment-370885981 Thanks for the reference. We have been working on enabling visualization of MXNet data in the TensorBoard. You can see the design

[GitHub] szha commented on issue #9986: gluon language modeling dataset and text token reader

2018-03-06 Thread GitBox
szha commented on issue #9986: gluon language modeling dataset and text token reader URL: https://github.com/apache/incubator-mxnet/pull/9986#issuecomment-370887670 I created a separate PR (linked above) for making the download refactoring changes on master, and will rebase this PR once th

[GitHub] anirudhacharya commented on issue #9963: Onnx Module to import onnx models into mxnet

2018-03-06 Thread GitBox
anirudhacharya commented on issue #9963: Onnx Module to import onnx models into mxnet URL: https://github.com/apache/incubator-mxnet/pull/9963#issuecomment-370888340 @piiswrong can you please take a look at the code. We will keep adding more operator support, but can you review the general

[GitHub] larroy opened a new issue #10011: [CMake][IoT][Jetson] Building with CMake produces libomp.so

2018-03-06 Thread GitBox
larroy opened a new issue #10011: [CMake][IoT][Jetson] Building with CMake produces libomp.so URL: https://github.com/apache/incubator-mxnet/issues/10011 we should use -fopenmp and not 3rd party openmp depending on the target platform -

[GitHub] piiswrong commented on issue #9991: Random shuffle implementation

2018-03-06 Thread GitBox
piiswrong commented on issue #9991: Random shuffle implementation URL: https://github.com/apache/incubator-mxnet/pull/9991#issuecomment-370892094 Our principle is to try to conform to numpy API as much as possible. Especially in cases like this where users could silently get confusing resul

[GitHub] piiswrong closed issue #9976: The assignment problem about NDArray

2018-03-06 Thread GitBox
piiswrong closed issue #9976: The assignment problem about NDArray URL: https://github.com/apache/incubator-mxnet/issues/9976 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] piiswrong closed pull request #9981: [fix issue#9976] The assignment problem in NDArray

2018-03-06 Thread GitBox
piiswrong closed pull request #9981: [fix issue#9976] The assignment problem in NDArray URL: https://github.com/apache/incubator-mxnet/pull/9981 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

[incubator-mxnet] branch master updated: [fix issue#9976] The assignment problem in NDArray (#9981)

2018-03-06 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 e2b1a56 [fix issue#9976] The assignment

[GitHub] yzhliu closed pull request #9931: Add axes support to Dropout for variational dropout in NLP

2018-03-06 Thread GitBox
yzhliu closed pull request #9931: Add axes support to Dropout for variational dropout in NLP URL: https://github.com/apache/incubator-mxnet/pull/9931 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance:

[incubator-mxnet] branch master updated: Add axes support to Dropout for variational dropout in NLP (#9931)

2018-03-06 Thread liuyizhi
This is an automated email from the ASF dual-hosted git repository. liuyizhi 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 40de6ab Add axes support to Dropout

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

2018-03-06 Thread GitBox
piiswrong 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-370896548 Is this exactly the same with scipy.stats.norm.logcdf? If so maybe we call it logcdf too? derivlogcd

[GitHub] piiswrong commented on issue #9958: Parallelization for ROIpooling OP

2018-03-06 Thread GitBox
piiswrong commented on issue #9958: Parallelization for ROIpooling OP URL: https://github.com/apache/incubator-mxnet/pull/9958#issuecomment-370899560 @cjolivier01 This is an automated message from the Apache Git Service. To r

[GitHub] piiswrong closed pull request #10008: gluon dataset download refactor

2018-03-06 Thread GitBox
piiswrong closed pull request #10008: gluon dataset download refactor URL: https://github.com/apache/incubator-mxnet/pull/10008 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: gluon dataset download refactor (#10008)

2018-03-06 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 29f7f43 gluon dataset download refactor

[GitHub] lightingghost commented on issue #10002: _backward_Convolution is non-differentiable

2018-03-06 Thread GitBox
lightingghost commented on issue #10002: _backward_Convolution is non-differentiable URL: https://github.com/apache/incubator-mxnet/issues/10002#issuecomment-370905340 @sxjscience I think #9979 is a question rather than a feature request. In his case, he should use `autograd.grad` and set

[incubator-mxnet] branch nlp_toolkit updated (02dd89a -> 29f7f43)

2018-03-06 Thread zhasheng
This is an automated email from the ASF dual-hosted git repository. zhasheng pushed a change to branch nlp_toolkit in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. from 02dd89a Non-blocking row_sparse_pull. Fix incorrect indices generated by device kvstore.row_sparse_

[GitHub] szha commented on issue #9986: gluon language modeling dataset and text token reader

2018-03-06 Thread GitBox
szha commented on issue #9986: gluon language modeling dataset and text token reader URL: https://github.com/apache/incubator-mxnet/pull/9986#issuecomment-370906543 Rebased. This is an automated message from the Apache Git Se

[GitHub] aidan-plenert-macdonald commented on issue #9979: Computing the second derivative

2018-03-06 Thread GitBox
aidan-plenert-macdonald commented on issue #9979: Computing the second derivative URL: https://github.com/apache/incubator-mxnet/issues/9979#issuecomment-370912952 @sxjscience If you tell me how, I can help add support for higher order gradients in more operators

[GitHub] aidan-plenert-macdonald commented on issue #9979: Computing the second derivative

2018-03-06 Thread GitBox
aidan-plenert-macdonald commented on issue #9979: Computing the second derivative URL: https://github.com/apache/incubator-mxnet/issues/9979#issuecomment-370912952 @sxjscience If you tell me how, I can help add support for higher order gradients in more operators. A working example and po

[GitHub] sxjscience commented on issue #9979: Computing the second derivative

2018-03-06 Thread GitBox
sxjscience commented on issue #9979: Computing the second derivative URL: https://github.com/apache/incubator-mxnet/issues/9979#issuecomment-370914218 @lightingghost Thanks a lot! Could you help give the solution to the question in this PR? I'll double confirm with Eric on the current plan

[GitHub] lightingghost opened a new issue #10002: _backward_Convolution is non-differentiable

2018-03-06 Thread GitBox
lightingghost opened a new issue #10002: _backward_Convolution is non-differentiable URL: https://github.com/apache/incubator-mxnet/issues/10002 As I saw mxnet has `autograd` package to support high order gradients, I tried to implement wgan-gp with mxnet. But I got an error of ```

[GitHub] lightingghost commented on issue #9979: Computing the second derivative

2018-03-06 Thread GitBox
lightingghost commented on issue #9979: Computing the second derivative URL: https://github.com/apache/incubator-mxnet/issues/9979#issuecomment-370919814 I will suggest using `autograd.grad` for computing second order derivatives. like [here](https://mxnet.apache.org/api/python/autograd/au

[GitHub] lightingghost commented on issue #9979: Computing the second derivative

2018-03-06 Thread GitBox
lightingghost commented on issue #9979: Computing the second derivative URL: https://github.com/apache/incubator-mxnet/issues/9979#issuecomment-370919814 I will suggest using `autograd.grad` for computing second order derivatives. like [here](https://mxnet.apache.org/api/python/autograd/au

[GitHub] aidan-plenert-macdonald commented on issue #9979: Computing the second derivative

2018-03-06 Thread GitBox
aidan-plenert-macdonald commented on issue #9979: Computing the second derivative URL: https://github.com/apache/incubator-mxnet/issues/9979#issuecomment-370922133 Gave me ``` MXNetError: [12:44:29] src/pass/gradient.cc:187: Operator _backward_power_scalar is non-differentiable

  1   2   3   >