[GitHub] zhaoxy2018 commented on issue #10634: How to get internal output with C++ API?

2018-05-14 Thread GitBox
zhaoxy2018 commented on issue #10634: How to get internal output with C++ API? URL: https://github.com/apache/incubator-mxnet/issues/10634#issuecomment-388706491 @chinakook Can you kindly point me a complete code example?

[GitHub] feevos commented on issue #10930: gluon documentation bug

2018-05-14 Thread GitBox
feevos commented on issue #10930: gluon documentation bug URL: https://github.com/apache/incubator-mxnet/issues/10930#issuecomment-388723853 Hi @szha, not at all but I don't know how to do so. This is an automated message

[GitHub] haojin2 opened a new pull request #10931: [MXNET-349] [WIP] Histogram Operator

2018-05-14 Thread GitBox
haojin2 opened a new pull request #10931: [MXNET-349] [WIP] Histogram Operator URL: https://github.com/apache/incubator-mxnet/pull/10931 ## Description ## This PR implements histogram operator. ## Checklist ## ### Essentials ### - [x] Changes are complete (i.e. I finished

[GitHub] marcoabreu commented on a change in pull request #10917: [MXNET-416] Add docker cache

2018-05-14 Thread GitBox
marcoabreu commented on a change in pull request #10917: [MXNET-416] Add docker cache URL: https://github.com/apache/incubator-mxnet/pull/10917#discussion_r187869036 ## File path: ci/build.py ## @@ -33,6 +33,7 @@ import shutil import subprocess import sys +import

[GitHub] KellenSunderland commented on a change in pull request #10917: [MXNET-416] Add docker cache

2018-05-14 Thread GitBox
KellenSunderland commented on a change in pull request #10917: [MXNET-416] Add docker cache URL: https://github.com/apache/incubator-mxnet/pull/10917#discussion_r187876960 ## File path: Jenkinsfile ## @@ -92,24 +92,34 @@ echo ${libs} | sed -e 's/,/ /g' | xargs md5sum """

[GitHub] diyang commented on issue #10878: Typo Erro in Optimizer.R

2018-05-14 Thread GitBox
diyang commented on issue #10878: Typo Erro in Optimizer.R URL: https://github.com/apache/incubator-mxnet/issues/10878#issuecomment-388766320 @harryprince Done This is an automated message from the Apache Git Service. To

[GitHub] ivshli commented on issue #10003: C++ API Documentation and Tutorial

2018-05-14 Thread GitBox
ivshli commented on issue #10003: C++ API Documentation and Tutorial URL: https://github.com/apache/incubator-mxnet/issues/10003#issuecomment-388733644 Hi, Does this request is in process? I still can't find intuitive example or doc about c++ Thanks

[GitHub] diyang commented on issue #10878: Typo Erro in Optimizer.R

2018-05-14 Thread GitBox
diyang commented on issue #10878: Typo Erro in Optimizer.R URL: https://github.com/apache/incubator-mxnet/issues/10878#issuecomment-388747677 @jeremiedb Yes, it crashed in all my training scenarios, so I have prompt my own function to bypass it.

[GitHub] larroy commented on a change in pull request #10917: [MXNET-416] Add docker cache

2018-05-14 Thread GitBox
larroy commented on a change in pull request #10917: [MXNET-416] Add docker cache URL: https://github.com/apache/incubator-mxnet/pull/10917#discussion_r187897853 ## File path: ci/Jenkinsfile_docker_cache ## @@ -0,0 +1,80 @@ +// -*- mode: groovy -*- + +// Licensed to the

[GitHub] larroy commented on a change in pull request #10917: [MXNET-416] Add docker cache

2018-05-14 Thread GitBox
larroy commented on a change in pull request #10917: [MXNET-416] Add docker cache URL: https://github.com/apache/incubator-mxnet/pull/10917#discussion_r187897734 ## File path: ci/Jenkinsfile_docker_cache ## @@ -0,0 +1,80 @@ +// -*- mode: groovy -*- + +// Licensed to the

[GitHub] eric-haibin-lin commented on a change in pull request #10882: move exec.reshape to backend

2018-05-14 Thread GitBox
eric-haibin-lin commented on a change in pull request #10882: move exec.reshape to backend URL: https://github.com/apache/incubator-mxnet/pull/10882#discussion_r187844712 ## File path: src/executor/graph_executor.cc ## @@ -1043,6 +1043,108 @@ void

[GitHub] eric-haibin-lin commented on a change in pull request #10882: move exec.reshape to backend

2018-05-14 Thread GitBox
eric-haibin-lin commented on a change in pull request #10882: move exec.reshape to backend URL: https://github.com/apache/incubator-mxnet/pull/10882#discussion_r187845247 ## File path: src/executor/graph_executor.cc ## @@ -1043,6 +1043,108 @@ void

[GitHub] marcoabreu commented on a change in pull request #10917: [MXNET-416] Add docker cache

2018-05-14 Thread GitBox
marcoabreu commented on a change in pull request #10917: [MXNET-416] Add docker cache URL: https://github.com/apache/incubator-mxnet/pull/10917#discussion_r187868534 ## File path: ci/Jenkinsfile_docker_cache ## @@ -0,0 +1,84 @@ +// -*- mode: groovy -*- + +// Licensed to

[incubator-mxnet] branch master updated: [MXNET-357] New Scala API Design (Symbol) (#10660)

2018-05-14 Thread nswamy
This is an automated email from the ASF dual-hosted git repository. nswamy 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 b011ecc [MXNET-357] New Scala API

[GitHub] xinyu-intel commented on issue #10933: remove unnecessary checks on convolution parameters

2018-05-14 Thread GitBox
xinyu-intel commented on issue #10933: remove unnecessary checks on convolution parameters URL: https://github.com/apache/incubator-mxnet/pull/10933#issuecomment-388745011 According to https://github.com/apache/incubator-mxnet/blob/master/src/operator/nn/convolution.cc#L359, the dilate

[GitHub] szha commented on issue #10930: gluon documentation bug

2018-05-14 Thread GitBox
szha commented on issue #10930: gluon documentation bug URL: https://github.com/apache/incubator-mxnet/issues/10930#issuecomment-388721739 Indeed. Sorry that I missed it. Would you mind adding it to the list at the top?

[GitHub] marcoabreu commented on a change in pull request #10917: [MXNET-416] Add docker cache

2018-05-14 Thread GitBox
marcoabreu commented on a change in pull request #10917: [MXNET-416] Add docker cache URL: https://github.com/apache/incubator-mxnet/pull/10917#discussion_r187871185 ## File path: ci/docker_cache.py ## @@ -0,0 +1,257 @@ +#!/usr/bin/env python3 +# -*- coding: utf-8 -*- +

[GitHub] ivshli commented on issue #10003: C++ API Documentation and Tutorial

2018-05-14 Thread GitBox
ivshli commented on issue #10003: C++ API Documentation and Tutorial URL: https://github.com/apache/incubator-mxnet/issues/10003#issuecomment-388733644 Hi, Does this request is in process? I still can't find intuitive example or doc about c++ Thanks

[GitHub] feevos opened a new issue #10930: gluon documentation bug

2018-05-14 Thread GitBox
feevos opened a new issue #10930: gluon documentation bug URL: https://github.com/apache/incubator-mxnet/issues/10930 Dear all, in the gluon documentation of the [neural network layers](https://mxnet.incubator.apache.org/api/python/gluon/nn.html) ```(Versions(master))```, the

[GitHub] xinyu-intel opened a new pull request #10933: remove unnecessary checks on convolution parameters

2018-05-14 Thread GitBox
xinyu-intel opened a new pull request #10933: remove unnecessary checks on convolution parameters URL: https://github.com/apache/incubator-mxnet/pull/10933 ## Description ## Since dilate of 2-D convolution must be {1, 1}, remove some unnecessary code. @zheng-da ## Checklist ##

[GitHub] KellenSunderland commented on a change in pull request #10917: [MXNET-416] Add docker cache

2018-05-14 Thread GitBox
KellenSunderland commented on a change in pull request #10917: [MXNET-416] Add docker cache URL: https://github.com/apache/incubator-mxnet/pull/10917#discussion_r187877275 ## File path: Jenkinsfile ## @@ -92,24 +92,34 @@ echo ${libs} | sed -e 's/,/ /g' | xargs md5sum """

[GitHub] jonbakerfish opened a new issue #10935: When are the images converted to range(0, 1) in pre-trained resnet50_v2 on imagenet?

2018-05-14 Thread GitBox
jonbakerfish opened a new issue #10935: When are the images converted to range(0,1) in pre-trained resnet50_v2 on imagenet? URL: https://github.com/apache/incubator-mxnet/issues/10935 From the Gluon Model Zoo: > All pre-trained models expect input images normalized in the same way,

[GitHub] chinakook commented on issue #10634: How to get internal output with C++ API?

2018-05-14 Thread GitBox
chinakook commented on issue #10634: How to get internal output with C++ API? URL: https://github.com/apache/incubator-mxnet/issues/10634#issuecomment-388708914 https://github.com/apache/incubator-mxnet/blob/master/example/image-classification/predict-cpp/image-classification-predict.cc

[GitHub] chinakook commented on issue #10634: How to get internal output with C++ API?

2018-05-14 Thread GitBox
chinakook commented on issue #10634: How to get internal output with C++ API? URL: https://github.com/apache/incubator-mxnet/issues/10634#issuecomment-383368141 ``` int device_type = 1; int device_id = 0; const char *input_keys[1]; input_keys[0] = "data";

[GitHub] zhaoxy2018 commented on issue #10634: How to get internal output with C++ API?

2018-05-14 Thread GitBox
zhaoxy2018 commented on issue #10634: How to get internal output with C++ API? URL: https://github.com/apache/incubator-mxnet/issues/10634#issuecomment-388744801 @chinakook thanks~~ This is an automated message from the

[GitHub] zhaoxy2018 closed issue #10634: How to get internal output with C++ API?

2018-05-14 Thread GitBox
zhaoxy2018 closed issue #10634: How to get internal output with C++ API? URL: https://github.com/apache/incubator-mxnet/issues/10634 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] GoodJoey opened a new issue #10932: about the data translating tool im2rec, is there any option to add new images to the existing rec file?

2018-05-14 Thread GitBox
GoodJoey opened a new issue #10932: about the data translating tool im2rec, is there any option to add new images to the existing rec file? URL: https://github.com/apache/incubator-mxnet/issues/10932 Hi, I have about 50G images for training, and it take me about 2 hours to translate

[GitHub] marcoabreu commented on a change in pull request #10917: [MXNET-416] Add docker cache

2018-05-14 Thread GitBox
marcoabreu commented on a change in pull request #10917: [MXNET-416] Add docker cache URL: https://github.com/apache/incubator-mxnet/pull/10917#discussion_r187904549 ## File path: ci/Jenkinsfile_docker_cache ## @@ -0,0 +1,80 @@ +// -*- mode: groovy -*- + +// Licensed to

[GitHub] marcoabreu commented on a change in pull request #10917: [MXNET-416] Add docker cache

2018-05-14 Thread GitBox
marcoabreu commented on a change in pull request #10917: [MXNET-416] Add docker cache URL: https://github.com/apache/incubator-mxnet/pull/10917#discussion_r187904578 ## File path: ci/Jenkinsfile_docker_cache ## @@ -0,0 +1,80 @@ +// -*- mode: groovy -*- + +// Licensed to

[GitHub] marcoabreu commented on a change in pull request #10917: [MXNET-416] Add docker cache

2018-05-14 Thread GitBox
marcoabreu commented on a change in pull request #10917: [MXNET-416] Add docker cache URL: https://github.com/apache/incubator-mxnet/pull/10917#discussion_r187913717 ## File path: ci/docker_cache.py ## @@ -0,0 +1,267 @@ +#!/usr/bin/env python3 +# -*- coding: utf-8 -*- +

[GitHub] marcoabreu commented on a change in pull request #10917: [MXNET-416] Add docker cache

2018-05-14 Thread GitBox
marcoabreu commented on a change in pull request #10917: [MXNET-416] Add docker cache URL: https://github.com/apache/incubator-mxnet/pull/10917#discussion_r187913728 ## File path: ci/docker_cache.py ## @@ -0,0 +1,267 @@ +#!/usr/bin/env python3 +# -*- coding: utf-8 -*- +

[GitHub] marcoabreu commented on a change in pull request #10917: [MXNET-416] Add docker cache

2018-05-14 Thread GitBox
marcoabreu commented on a change in pull request #10917: [MXNET-416] Add docker cache URL: https://github.com/apache/incubator-mxnet/pull/10917#discussion_r187914456 ## File path: ci/docker_cache.py ## @@ -0,0 +1,267 @@ +#!/usr/bin/env python3 +# -*- coding: utf-8 -*- +

[GitHub] larroy commented on a change in pull request #10917: [MXNET-416] Add docker cache

2018-05-14 Thread GitBox
larroy commented on a change in pull request #10917: [MXNET-416] Add docker cache URL: https://github.com/apache/incubator-mxnet/pull/10917#discussion_r187914480 ## File path: ci/docker_cache.py ## @@ -0,0 +1,267 @@ +#!/usr/bin/env python3 +# -*- coding: utf-8 -*- + +#

[GitHub] larroy commented on a change in pull request #10917: [MXNET-416] Add docker cache

2018-05-14 Thread GitBox
larroy commented on a change in pull request #10917: [MXNET-416] Add docker cache URL: https://github.com/apache/incubator-mxnet/pull/10917#discussion_r187914480 ## File path: ci/docker_cache.py ## @@ -0,0 +1,267 @@ +#!/usr/bin/env python3 +# -*- coding: utf-8 -*- + +#

[GitHub] larroy commented on a change in pull request #10917: [MXNET-416] Add docker cache

2018-05-14 Thread GitBox
larroy commented on a change in pull request #10917: [MXNET-416] Add docker cache URL: https://github.com/apache/incubator-mxnet/pull/10917#discussion_r187914480 ## File path: ci/docker_cache.py ## @@ -0,0 +1,267 @@ +#!/usr/bin/env python3 +# -*- coding: utf-8 -*- + +#

[GitHub] marcoabreu commented on a change in pull request #10917: [MXNET-416] Add docker cache

2018-05-14 Thread GitBox
marcoabreu commented on a change in pull request #10917: [MXNET-416] Add docker cache URL: https://github.com/apache/incubator-mxnet/pull/10917#discussion_r187920956 ## File path: ci/docker_cache.py ## @@ -0,0 +1,267 @@ +#!/usr/bin/env python3 +# -*- coding: utf-8 -*- +

[GitHub] marcoabreu commented on a change in pull request #10917: [MXNET-416] Add docker cache

2018-05-14 Thread GitBox
marcoabreu commented on a change in pull request #10917: [MXNET-416] Add docker cache URL: https://github.com/apache/incubator-mxnet/pull/10917#discussion_r187904345 ## File path: Jenkinsfile ## @@ -92,24 +92,34 @@ echo ${libs} | sed -e 's/,/ /g' | xargs md5sum """ }

[GitHub] marcoabreu commented on a change in pull request #10917: [MXNET-416] Add docker cache

2018-05-14 Thread GitBox
marcoabreu commented on a change in pull request #10917: [MXNET-416] Add docker cache URL: https://github.com/apache/incubator-mxnet/pull/10917#discussion_r187920618 ## File path: ci/docker_cache.py ## @@ -0,0 +1,267 @@ +#!/usr/bin/env python3 +# -*- coding: utf-8 -*- +

[GitHub] larroy commented on a change in pull request #10917: [MXNET-416] Add docker cache

2018-05-14 Thread GitBox
larroy commented on a change in pull request #10917: [MXNET-416] Add docker cache URL: https://github.com/apache/incubator-mxnet/pull/10917#discussion_r187912072 ## File path: ci/docker_cache.py ## @@ -0,0 +1,267 @@ +#!/usr/bin/env python3 +# -*- coding: utf-8 -*- + +#

[GitHub] larroy commented on a change in pull request #10917: [MXNET-416] Add docker cache

2018-05-14 Thread GitBox
larroy commented on a change in pull request #10917: [MXNET-416] Add docker cache URL: https://github.com/apache/incubator-mxnet/pull/10917#discussion_r187911906 ## File path: ci/docker_cache.py ## @@ -0,0 +1,267 @@ +#!/usr/bin/env python3 +# -*- coding: utf-8 -*- + +#

[GitHub] larroy commented on a change in pull request #10917: [MXNET-416] Add docker cache

2018-05-14 Thread GitBox
larroy commented on a change in pull request #10917: [MXNET-416] Add docker cache URL: https://github.com/apache/incubator-mxnet/pull/10917#discussion_r187911938 ## File path: ci/docker_cache.py ## @@ -0,0 +1,267 @@ +#!/usr/bin/env python3 +# -*- coding: utf-8 -*- + +#

[GitHub] larroy commented on a change in pull request #10917: [MXNET-416] Add docker cache

2018-05-14 Thread GitBox
larroy commented on a change in pull request #10917: [MXNET-416] Add docker cache URL: https://github.com/apache/incubator-mxnet/pull/10917#discussion_r187911986 ## File path: ci/docker_cache.py ## @@ -0,0 +1,267 @@ +#!/usr/bin/env python3 +# -*- coding: utf-8 -*- + +#

[GitHub] larroy commented on a change in pull request #10917: [MXNET-416] Add docker cache

2018-05-14 Thread GitBox
larroy commented on a change in pull request #10917: [MXNET-416] Add docker cache URL: https://github.com/apache/incubator-mxnet/pull/10917#discussion_r187915989 ## File path: ci/docker_cache.py ## @@ -0,0 +1,267 @@ +#!/usr/bin/env python3 +# -*- coding: utf-8 -*- + +#

[GitHub] ashkanaev opened a new issue #10937: c++ API, fixed batch size for predictor.

2018-05-14 Thread GitBox
ashkanaev opened a new issue #10937: c++ API, fixed batch size for predictor. URL: https://github.com/apache/incubator-mxnet/issues/10937 Hello! Is't possible to make predictor with c++ API for arbitrary batch size? From

[GitHub] marcoabreu commented on a change in pull request #10917: [MXNET-416] Add docker cache

2018-05-14 Thread GitBox
marcoabreu commented on a change in pull request #10917: [MXNET-416] Add docker cache URL: https://github.com/apache/incubator-mxnet/pull/10917#discussion_r187913925 ## File path: ci/docker_cache.py ## @@ -0,0 +1,267 @@ +#!/usr/bin/env python3 +# -*- coding: utf-8 -*- +

[GitHub] liuzx32 opened a new issue #10936: aws-ec2 make -j4 and throw the error

2018-05-14 Thread GitBox
liuzx32 opened a new issue #10936: aws-ec2 make -j4 and throw the error URL: https://github.com/apache/incubator-mxnet/issues/10936 n_softmax.o In file included from /home/mobdev/liuzx/incubator-mxnet/3rdparty/mshadow/mshadow/tensor.h:16:0, from

[GitHub] chinakook commented on issue #10935: When are the images converted to range(0, 1) in pre-trained resnet50_v2 on imagenet?

2018-05-14 Thread GitBox
chinakook commented on issue #10935: When are the images converted to range(0,1) in pre-trained resnet50_v2 on imagenet? URL: https://github.com/apache/incubator-mxnet/issues/10935#issuecomment-388773802

[GitHub] marcoabreu commented on a change in pull request #10917: [MXNET-416] Add docker cache

2018-05-14 Thread GitBox
marcoabreu commented on a change in pull request #10917: [MXNET-416] Add docker cache URL: https://github.com/apache/incubator-mxnet/pull/10917#discussion_r187913170 ## File path: ci/build.py ## @@ -61,17 +62,44 @@ def get_docker_binary(use_nvidia_docker: bool) -> str:

[GitHub] larroy commented on a change in pull request #10917: [MXNET-416] Add docker cache

2018-05-14 Thread GitBox
larroy commented on a change in pull request #10917: [MXNET-416] Add docker cache URL: https://github.com/apache/incubator-mxnet/pull/10917#discussion_r187914644 ## File path: ci/docker_cache.py ## @@ -0,0 +1,267 @@ +#!/usr/bin/env python3 +# -*- coding: utf-8 -*- + +#

[GitHub] larroy commented on a change in pull request #10917: [MXNET-416] Add docker cache

2018-05-14 Thread GitBox
larroy commented on a change in pull request #10917: [MXNET-416] Add docker cache URL: https://github.com/apache/incubator-mxnet/pull/10917#discussion_r187912216 ## File path: ci/docker_cache.py ## @@ -0,0 +1,267 @@ +#!/usr/bin/env python3 +# -*- coding: utf-8 -*- + +#

[GitHub] larroy commented on a change in pull request #10917: [MXNET-416] Add docker cache

2018-05-14 Thread GitBox
larroy commented on a change in pull request #10917: [MXNET-416] Add docker cache URL: https://github.com/apache/incubator-mxnet/pull/10917#discussion_r187912169 ## File path: ci/docker_cache.py ## @@ -0,0 +1,267 @@ +#!/usr/bin/env python3 +# -*- coding: utf-8 -*- + +#

[GitHub] larroy commented on a change in pull request #10917: [MXNET-416] Add docker cache

2018-05-14 Thread GitBox
larroy commented on a change in pull request #10917: [MXNET-416] Add docker cache URL: https://github.com/apache/incubator-mxnet/pull/10917#discussion_r187994860 ## File path: Jenkinsfile ## @@ -92,24 +92,34 @@ echo ${libs} | sed -e 's/,/ /g' | xargs md5sum """ } +def

[GitHub] vdantu commented on issue #10903: how to set output value but not output possibility

2018-05-14 Thread GitBox
vdantu commented on issue #10903: how to set output value but not output possibility URL: https://github.com/apache/incubator-mxnet/issues/10903#issuecomment-30005 @Rangocold : Could you post this on discuss.mxnet.io. You will get a broader audience.

[GitHub] jonbakerfish commented on issue #10935: When are the images converted to range(0, 1) in pre-trained resnet50_v2 on imagenet?

2018-05-14 Thread GitBox
jonbakerfish commented on issue #10935: When are the images converted to range(0,1) in pre-trained resnet50_v2 on imagenet? URL: https://github.com/apache/incubator-mxnet/issues/10935#issuecomment-388846795 Hi @chinakook, If you run this code: im =

[GitHub] pengzhao-intel commented on issue #10104: [WIP][MXNET-107] Fused LSTM implementation for CPU

2018-05-14 Thread GitBox
pengzhao-intel commented on issue #10104: [WIP][MXNET-107] Fused LSTM implementation for CPU URL: https://github.com/apache/incubator-mxnet/pull/10104#issuecomment-388832244 @eric-haibin-lin @piiswrong @szha @Jerryzcn the comments are solved. Please help take a review again. After this

[GitHub] vdantu commented on issue #10936: aws-ec2 make -j4 and throw the error

2018-05-14 Thread GitBox
vdantu commented on issue #10936: aws-ec2 make -j4 and throw the error URL: https://github.com/apache/incubator-mxnet/issues/10936#issuecomment-388875612 @liuzx32 : Could you provide minimum reproducible steps for this failure.

[GitHub] marcoabreu commented on a change in pull request #10917: [MXNET-416] Add docker cache

2018-05-14 Thread GitBox
marcoabreu commented on a change in pull request #10917: [MXNET-416] Add docker cache URL: https://github.com/apache/incubator-mxnet/pull/10917#discussion_r187903974 ## File path: Jenkinsfile ## @@ -92,24 +92,34 @@ echo ${libs} | sed -e 's/,/ /g' | xargs md5sum """ }

[GitHub] piiswrong commented on a change in pull request #10104: [WIP][MXNET-107] Fused LSTM implementation for CPU

2018-05-14 Thread GitBox
piiswrong commented on a change in pull request #10104: [WIP][MXNET-107] Fused LSTM implementation for CPU URL: https://github.com/apache/incubator-mxnet/pull/10104#discussion_r188068402 ## File path: src/operator/rnn_impl.h ## @@ -0,0 +1,457 @@ +/* + * Licensed to the

[GitHub] piiswrong commented on a change in pull request #10104: [WIP][MXNET-107] Fused LSTM implementation for CPU

2018-05-14 Thread GitBox
piiswrong commented on a change in pull request #10104: [WIP][MXNET-107] Fused LSTM implementation for CPU URL: https://github.com/apache/incubator-mxnet/pull/10104#discussion_r188068060 ## File path: src/operator/rnn_impl.h ## @@ -0,0 +1,457 @@ +/* + * Licensed to the

[GitHub] zheng-da commented on a change in pull request #10913: Fix _backward_norm op registration

2018-05-14 Thread GitBox
zheng-da commented on a change in pull request #10913: Fix _backward_norm op registration URL: https://github.com/apache/incubator-mxnet/pull/10913#discussion_r188084204 ## File path: src/operator/optimizer_op.cc ## @@ -305,7 +305,7 @@ inline bool SGDStorageType(const

[GitHub] zheng-da commented on a change in pull request #10913: Fix _backward_norm op registration

2018-05-14 Thread GitBox
zheng-da commented on a change in pull request #10913: Fix _backward_norm op registration URL: https://github.com/apache/incubator-mxnet/pull/10913#discussion_r188084767 ## File path: src/operator/tensor/broadcast_reduce_op.h ## @@ -707,18 +707,16 @@ void ReduceCsr(const

[GitHub] zheng-da commented on a change in pull request #10913: Fix _backward_norm op registration

2018-05-14 Thread GitBox
zheng-da commented on a change in pull request #10913: Fix _backward_norm op registration URL: https://github.com/apache/incubator-mxnet/pull/10913#discussion_r188086270 ## File path: src/operator/tensor/broadcast_reduce_op.h ## @@ -880,17 +878,30 @@ inline bool

[GitHub] eric-haibin-lin commented on a change in pull request #10913: Fix _backward_norm op registration

2018-05-14 Thread GitBox
eric-haibin-lin commented on a change in pull request #10913: Fix _backward_norm op registration URL: https://github.com/apache/incubator-mxnet/pull/10913#discussion_r188091468 ## File path: src/operator/tensor/broadcast_reduce_op.h ## @@ -880,17 +878,30 @@ inline bool

[GitHub] anirudh2290 commented on issue #10846: flaky tests in tutorial CI

2018-05-14 Thread GitBox
anirudh2290 commented on issue #10846: flaky tests in tutorial CI URL: https://github.com/apache/incubator-mxnet/issues/10846#issuecomment-388962817 Failed again here: http://jenkins.mxnet-ci.amazon-ml.com/blue/organizations/jenkins/incubator-mxnet/detail/PR-10938/1/pipeline/

[GitHub] conradwt commented on issue #10185: [Documentation][Installation][macOS] Building From Source Has Incomplete Instructions For macOS 10.13.3

2018-05-14 Thread GitBox
conradwt commented on issue #10185: [Documentation][Installation][macOS] Building From Source Has Incomplete Instructions For macOS 10.13.3 URL: https://github.com/apache/incubator-mxnet/issues/10185#issuecomment-388966282 @bhavinthaker The library which needs the libmxnet.dylib is the

[GitHub] marcoabreu commented on a change in pull request #10917: [MXNET-416] Add docker cache

2018-05-14 Thread GitBox
marcoabreu commented on a change in pull request #10917: [MXNET-416] Add docker cache URL: https://github.com/apache/incubator-mxnet/pull/10917#discussion_r188070539 ## File path: Jenkinsfile ## @@ -92,24 +92,34 @@ echo ${libs} | sed -e 's/,/ /g' | xargs md5sum """ }

[incubator-mxnet] branch master updated: [WIP][MXNET-107] Fused LSTM implementation for CPU (#10104)

2018-05-14 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 275378a [WIP][MXNET-107] Fused LSTM

[GitHub] piiswrong closed pull request #10104: [WIP][MXNET-107] Fused LSTM implementation for CPU

2018-05-14 Thread GitBox
piiswrong closed pull request #10104: [WIP][MXNET-107] Fused LSTM implementation for CPU URL: https://github.com/apache/incubator-mxnet/pull/10104 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance:

[GitHub] ThomasDelteil commented on issue #10900: [MXNET-414] Tutorial on visualizing CNN decisions using Grad-CAM

2018-05-14 Thread GitBox
ThomasDelteil commented on issue #10900: [MXNET-414] Tutorial on visualizing CNN decisions using Grad-CAM URL: https://github.com/apache/incubator-mxnet/pull/10900#issuecomment-388944374 @indhub I think you forgot to edit the docs/tutorials/index.md file to add your tutorial to the list

[GitHub] eric-haibin-lin commented on a change in pull request #10913: Fix _backward_norm op registration

2018-05-14 Thread GitBox
eric-haibin-lin commented on a change in pull request #10913: Fix _backward_norm op registration URL: https://github.com/apache/incubator-mxnet/pull/10913#discussion_r188090416 ## File path: src/operator/optimizer_op.cc ## @@ -305,7 +305,7 @@ inline bool

[GitHub] bhavinthaker commented on issue #8155: Error when compiling MXNet from source with VS2013 / Windows 10

2018-05-14 Thread GitBox
bhavinthaker commented on issue #8155: Error when compiling MXNet from source with VS2013 / Windows 10 URL: https://github.com/apache/incubator-mxnet/issues/8155#issuecomment-388961823 @sandeep-krishnamurthy @eric-haibin-lin Could one of you please close this issue?

[GitHub] bhavinthaker commented on issue #9949: Mac build from source script validation fails

2018-05-14 Thread GitBox
bhavinthaker commented on issue #9949: Mac build from source script validation fails URL: https://github.com/apache/incubator-mxnet/issues/9949#issuecomment-388961783 @sandeep-krishnamurthy @eric-haibin-lin Could one of you please close this issue?

[GitHub] bhavinthaker commented on issue #8016: mxnet installation GPU validation issue/error

2018-05-14 Thread GitBox
bhavinthaker commented on issue #8016: mxnet installation GPU validation issue/error URL: https://github.com/apache/incubator-mxnet/issues/8016#issuecomment-388962407 @sandeep-krishnamurthy @eric-haibin-lin Could one of you please close this issue?

[GitHub] reminisce commented on a change in pull request #10882: move exec.reshape to backend

2018-05-14 Thread GitBox
reminisce commented on a change in pull request #10882: move exec.reshape to backend URL: https://github.com/apache/incubator-mxnet/pull/10882#discussion_r188030832 ## File path: src/executor/graph_executor.cc ## @@ -1043,6 +1043,108 @@ void

[GitHub] reminisce commented on a change in pull request #10882: move exec.reshape to backend

2018-05-14 Thread GitBox
reminisce commented on a change in pull request #10882: move exec.reshape to backend URL: https://github.com/apache/incubator-mxnet/pull/10882#discussion_r188031230 ## File path: src/executor/graph_executor.cc ## @@ -1043,6 +1043,108 @@ void

[GitHub] reminisce commented on a change in pull request #10882: move exec.reshape to backend

2018-05-14 Thread GitBox
reminisce commented on a change in pull request #10882: move exec.reshape to backend URL: https://github.com/apache/incubator-mxnet/pull/10882#discussion_r188031132 ## File path: src/executor/graph_executor.cc ## @@ -1043,6 +1043,108 @@ void

[GitHub] bhavinthaker commented on issue #10185: [Documentation][Installation][macOS] Building From Source Has Incomplete Instructions For macOS 10.13.3

2018-05-14 Thread GitBox
bhavinthaker commented on issue #10185: [Documentation][Installation][macOS] Building From Source Has Incomplete Instructions For macOS 10.13.3 URL: https://github.com/apache/incubator-mxnet/issues/10185#issuecomment-388931339 @conradwt Hello, could you please clarify which library needs

[GitHub] piiswrong commented on a change in pull request #10104: [WIP][MXNET-107] Fused LSTM implementation for CPU

2018-05-14 Thread GitBox
piiswrong commented on a change in pull request #10104: [WIP][MXNET-107] Fused LSTM implementation for CPU URL: https://github.com/apache/incubator-mxnet/pull/10104#discussion_r188067608 ## File path: src/operator/rnn_impl.h ## @@ -0,0 +1,457 @@ +/* + * Licensed to the

[GitHub] eric-haibin-lin commented on a change in pull request #10913: Fix _backward_norm op registration

2018-05-14 Thread GitBox
eric-haibin-lin commented on a change in pull request #10913: Fix _backward_norm op registration URL: https://github.com/apache/incubator-mxnet/pull/10913#discussion_r188084643 ## File path: tests/python/unittest/test_sparse_ndarray.py ## @@ -918,18 +918,22 @@ def

[GitHub] eric-haibin-lin commented on a change in pull request #10913: Fix _backward_norm op registration

2018-05-14 Thread GitBox
eric-haibin-lin commented on a change in pull request #10913: Fix _backward_norm op registration URL: https://github.com/apache/incubator-mxnet/pull/10913#discussion_r188091468 ## File path: src/operator/tensor/broadcast_reduce_op.h ## @@ -880,17 +878,30 @@ inline bool

[GitHub] reminisce commented on a change in pull request #10882: move exec.reshape to backend

2018-05-14 Thread GitBox
reminisce commented on a change in pull request #10882: move exec.reshape to backend URL: https://github.com/apache/incubator-mxnet/pull/10882#discussion_r188032349 ## File path: src/c_api/c_api_executor.cc ## @@ -510,6 +510,93 @@ int MXExecutorSimpleBind(SymbolHandle

[GitHub] reminisce commented on a change in pull request #10882: move exec.reshape to backend

2018-05-14 Thread GitBox
reminisce commented on a change in pull request #10882: move exec.reshape to backend URL: https://github.com/apache/incubator-mxnet/pull/10882#discussion_r188032054 ## File path: src/c_api/c_api_executor.cc ## @@ -510,6 +510,93 @@ int MXExecutorSimpleBind(SymbolHandle

[GitHub] eftiquar commented on issue #8075: scala on Windows

2018-05-14 Thread GitBox
eftiquar commented on issue #8075: scala on Windows URL: https://github.com/apache/incubator-mxnet/issues/8075#issuecomment-388931247 1. Steps for building on windows have been revised and merged. We need to use VS 2015, not VS 2013. ( PR -

[GitHub] piiswrong commented on issue #10104: [WIP][MXNET-107] Fused LSTM implementation for CPU

2018-05-14 Thread GitBox
piiswrong commented on issue #10104: [WIP][MXNET-107] Fused LSTM implementation for CPU URL: https://github.com/apache/incubator-mxnet/pull/10104#issuecomment-388934411 I can merge this first. But I think the memset and memcpy statements are superfluous. We should get ride of them later

[GitHub] eric-haibin-lin commented on a change in pull request #10913: Fix _backward_norm op registration

2018-05-14 Thread GitBox
eric-haibin-lin commented on a change in pull request #10913: Fix _backward_norm op registration URL: https://github.com/apache/incubator-mxnet/pull/10913#discussion_r188090653 ## File path: src/operator/tensor/broadcast_reduce_op.h ## @@ -707,18 +707,16 @@ void

[GitHub] piiswrong commented on a change in pull request #10931: [MXNET-349] [WIP] Histogram Operator

2018-05-14 Thread GitBox
piiswrong commented on a change in pull request #10931: [MXNET-349] [WIP] Histogram Operator URL: https://github.com/apache/incubator-mxnet/pull/10931#discussion_r188034226 ## File path: python/mxnet/ndarray/ndarray.py ## @@ -3740,3 +3740,29 @@ def empty(shape, ctx=None,

[GitHub] anirudh2290 opened a new pull request #10938: Add Apachev2 License for contrib

2018-05-14 Thread GitBox
anirudh2290 opened a new pull request #10938: Add Apachev2 License for contrib URL: https://github.com/apache/incubator-mxnet/pull/10938 ## Description ## We were asked this question on general@ on why so many files were excluded in rat-excludes. This is an attempt to remove the

[GitHub] kpmurali commented on issue #10895: [MXNET-413] Fixing the broken links - Week of 5/7

2018-05-14 Thread GitBox
kpmurali commented on issue #10895: [MXNET-413] Fixing the broken links - Week of 5/7 URL: https://github.com/apache/incubator-mxnet/pull/10895#issuecomment-388896739 Re-run this build? This is an automated message from the

[GitHub] piiswrong closed pull request #10934: Fix typo "weights"

2018-05-14 Thread GitBox
piiswrong closed pull request #10934: Fix typo "weights" URL: https://github.com/apache/incubator-mxnet/pull/10934 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] zhreshold commented on issue #10915: [MXNET-9704] An assertion check for invalid layout

2018-05-14 Thread GitBox
zhreshold commented on issue #10915: [MXNET-9704] An assertion check for invalid layout URL: https://github.com/apache/incubator-mxnet/pull/10915#issuecomment-388893309 @szha I am not quite sure about current supported formats. Do you have any source?

[GitHub] piiswrong commented on a change in pull request #10931: [MXNET-349] [WIP] Histogram Operator

2018-05-14 Thread GitBox
piiswrong commented on a change in pull request #10931: [MXNET-349] [WIP] Histogram Operator URL: https://github.com/apache/incubator-mxnet/pull/10931#discussion_r188034390 ## File path: python/mxnet/ndarray/ndarray.py ## @@ -3740,3 +3740,29 @@ def empty(shape, ctx=None,

[GitHub] piiswrong commented on a change in pull request #10931: [MXNET-349] [WIP] Histogram Operator

2018-05-14 Thread GitBox
piiswrong commented on a change in pull request #10931: [MXNET-349] [WIP] Histogram Operator URL: https://github.com/apache/incubator-mxnet/pull/10931#discussion_r188034390 ## File path: python/mxnet/ndarray/ndarray.py ## @@ -3740,3 +3740,29 @@ def empty(shape, ctx=None,

[GitHub] haojin2 commented on a change in pull request #10931: [MXNET-349] [WIP] Histogram Operator

2018-05-14 Thread GitBox
haojin2 commented on a change in pull request #10931: [MXNET-349] [WIP] Histogram Operator URL: https://github.com/apache/incubator-mxnet/pull/10931#discussion_r188034559 ## File path: python/mxnet/ndarray/ndarray.py ## @@ -3740,3 +3740,29 @@ def empty(shape, ctx=None,

[GitHub] szha commented on issue #10915: [MXNET-9704] An assertion check for invalid layout

2018-05-14 Thread GitBox
szha commented on issue #10915: [MXNET-9704] An assertion check for invalid layout URL: https://github.com/apache/incubator-mxnet/pull/10915#issuecomment-388906473 Needs to check for the used operators. For convolution:

[GitHub] szha commented on issue #10930: gluon documentation bug

2018-05-14 Thread GitBox
szha commented on issue #10930: gluon documentation bug URL: https://github.com/apache/incubator-mxnet/issues/10930#issuecomment-388907677 @feevos thanks! You can add the missing entries in this file https://github.com/apache/incubator-mxnet/blob/master/docs/api/python/gluon/nn.md

[GitHub] piiswrong closed pull request #10861: split trainer.step into allreduce and update

2018-05-14 Thread GitBox
piiswrong closed pull request #10861: split trainer.step into allreduce and update URL: https://github.com/apache/incubator-mxnet/pull/10861 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] gautamkmr commented on issue #9949: Mac build from source script validation fails

2018-05-14 Thread GitBox
gautamkmr commented on issue #9949: Mac build from source script validation fails URL: https://github.com/apache/incubator-mxnet/issues/9949#issuecomment-388927308 Dev decided to have user upgrade the python rather than using the old version. Please see the discussion here

[GitHub] piiswrong closed pull request #10929: fix example in smooth l1 document

2018-05-14 Thread GitBox
piiswrong closed pull request #10929: fix example in smooth l1 document URL: https://github.com/apache/incubator-mxnet/pull/10929 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] piiswrong commented on a change in pull request #10931: [MXNET-349] [WIP] Histogram Operator

2018-05-14 Thread GitBox
piiswrong commented on a change in pull request #10931: [MXNET-349] [WIP] Histogram Operator URL: https://github.com/apache/incubator-mxnet/pull/10931#discussion_r188037090 ## File path: src/operator/tensor/histogram.cc ## @@ -0,0 +1,163 @@ +/* + * Licensed to the Apache

[incubator-mxnet] branch master updated: fix example in smooth l1 document (#10929)

2018-05-14 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 89ffab9 fix example in smooth l1

  1   2   >