[GitHub] formath commented on issue #7913: what are the dependencies between each modules

2017-09-20 Thread git
formath commented on issue #7913: what are the dependencies between each modules URL: https://github.com/apache/incubator-mxnet/issues/7913#issuecomment-330755045 Functionality is clear between modules but api is not. Some modules may use inner variables or implicit functions in the other

[GitHub] formath commented on issue #7959: executor not expose its inner variables to symbol

2017-09-20 Thread git
formath commented on issue #7959: executor not expose its inner variables to symbol URL: https://github.com/apache/incubator-mxnet/pull/7959#issuecomment-330753643 @piiswrong This is an automated message from the Apache

[GitHub] x10000year commented on issue #7934: Bug of group2ctx? Wrong device placement?

2017-09-20 Thread git
x1year commented on issue #7934: Bug of group2ctx? Wrong device placement? URL: https://github.com/apache/incubator-mxnet/issues/7934#issuecomment-330757441 @formath So if a symbol a has input b, and b is assigned to cpu device, then a will also be automatically assigned to cpu device

[GitHub] KeyKy commented on issue #7899: Need help: numpy array to mxnet ndarray is too slow.

2017-09-20 Thread git
KeyKy commented on issue #7899: Need help: numpy array to mxnet ndarray is too slow. URL: https://github.com/apache/incubator-mxnet/issues/7899#issuecomment-330502892 @madjam image_data is (128, 3, 224, 224) label_data is (128,) it needs to load batch of data before doing net training.

[GitHub] formath commented on issue #7934: Bug of group2ctx? Wrong device placement?

2017-09-20 Thread git
formath commented on issue #7934: Bug of group2ctx? Wrong device placement? URL: https://github.com/apache/incubator-mxnet/issues/7934#issuecomment-330759033 `exe = x.bind(mx.gpu(), {}, group2ctx={"a": mx.cpu(), "b": mx.gpu()})` The default context is just indicate this executor run on

[GitHub] x10000year commented on issue #7934: Bug of group2ctx? Wrong device placement?

2017-09-20 Thread git
x1year commented on issue #7934: Bug of group2ctx? Wrong device placement? URL: https://github.com/apache/incubator-mxnet/issues/7934#issuecomment-330759842 @formath Is executor just a scheduler? What does it mean by "executor run on this context"? What executor runs? Parameter

[GitHub] ZiyueHuang commented on issue #7958: Engine shutdown

2017-09-20 Thread git
ZiyueHuang commented on issue #7958: Engine shutdown URL: https://github.com/apache/incubator-mxnet/issues/7958#issuecomment-330762854 It seems not relevant to the engine. `src/io/image_io.cc:165: Check failed: !dst.empty()`

[incubator-mxnet] branch master updated: simplify CTC forward after namespace reorganize (#7936)

2017-09-20 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 e68298c simplify CTC forward after

[incubator-mxnet] branch szha-patch-1 deleted (was 7e3c820)

2017-09-20 Thread zhasheng
This is an automated email from the ASF dual-hosted git repository. zhasheng pushed a change to branch szha-patch-1 in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. was 7e3c820 simplify CTC forward after namespace reorganize The revisions that were on this branch

[GitHub] szha closed pull request #7936: simplify CTC forward after namespace reorganize

2017-09-20 Thread git
szha closed pull request #7936: simplify CTC forward after namespace reorganize URL: https://github.com/apache/incubator-mxnet/pull/7936 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] Godricly opened a new issue #7961: fixed_param in module

2017-09-20 Thread git
Godricly opened a new issue #7961: fixed_param in module URL: https://github.com/apache/incubator-mxnet/issues/7961 Hi, The current fixed_param is only compatible with params fixed at early stage. For example. fixing A here. ``` input -> A -> B ``` But if I have another

[GitHub] Harold-Zhang commented on issue #7958: Engine shutdown

2017-09-20 Thread git
Harold-Zhang commented on issue #7958: Engine shutdown URL: https://github.com/apache/incubator-mxnet/issues/7958#issuecomment-330757502 @piiswrong @szha This is an automated message from the Apache Git Service. To respond

[GitHub] formath commented on issue #7934: Bug of group2ctx? Wrong device placement?

2017-09-20 Thread git
formath commented on issue #7934: Bug of group2ctx? Wrong device placement? URL: https://github.com/apache/incubator-mxnet/issues/7934#issuecomment-330759033 `exe = x.bind(mx.gpu(), {}, group2ctx={"a": mx.cpu(), "b": mx.gpu()})` The default context just indicate this executor run on this

[GitHub] zhreshold commented on issue #7958: Engine shutdown

2017-09-20 Thread git
zhreshold commented on issue #7958: Engine shutdown URL: https://github.com/apache/incubator-mxnet/issues/7958#issuecomment-330766154 Possibly failed in imdecode to the dst Image not exists or image broken? This is an

[GitHub] formath commented on issue #7942: Adam optimizer consistent with paper

2017-09-20 Thread git
formath commented on issue #7942: Adam optimizer consistent with paper URL: https://github.com/apache/incubator-mxnet/pull/7942#issuecomment-330748814 @sergeykolychev Have been merged. @piiswrong Jenkins error: libdc1394 error: Failed to initialize libdc1394

[GitHub] formath closed pull request #7959: executor not expose its inner variables to symbol

2017-09-20 Thread git
formath closed pull request #7959: executor not expose its inner variables to symbol URL: https://github.com/apache/incubator-mxnet/pull/7959 This is an automated message from the Apache Git Service. To respond to the

[GitHub] formath commented on issue #7959: executor not expose its inner variables to symbol

2017-09-20 Thread git
formath commented on issue #7959: executor not expose its inner variables to symbol URL: https://github.com/apache/incubator-mxnet/pull/7959#issuecomment-330753643 @piiswrong This is an automated message from the Apache

[GitHub] Prasad9 opened a new pull request #7962: SSD Example: Fixed training bug

2017-09-20 Thread git
Prasad9 opened a new pull request #7962: SSD Example: Fixed training bug URL: https://github.com/apache/incubator-mxnet/pull/7962 * Fixed import error bugs * Fixed character encoding issue * Converted Python 2 to Python 3

[GitHub] szha commented on issue #7960: Dropout layer crashes jupyter kernel

2017-09-20 Thread git
szha commented on issue #7960: Dropout layer crashes jupyter kernel URL: https://github.com/apache/incubator-mxnet/issues/7960#issuecomment-330924924 I noticed that the version you're using is 0.10. Do you mind getting onto 0.11 and give it a try?

[GitHub] cjolivier01 commented on a change in pull request #7903: Refactor AdaGrad optimizer to support sparse tensors

2017-09-20 Thread git
cjolivier01 commented on a change in pull request #7903: Refactor AdaGrad optimizer to support sparse tensors URL: https://github.com/apache/incubator-mxnet/pull/7903#discussion_r140057752 ## File path: tests/python/unittest/test_sparse_operator.py ## @@ -697,6 +697,19 @@

[GitHub] cjolivier01 commented on a change in pull request #7903: Refactor AdaGrad optimizer to support sparse tensors

2017-09-20 Thread git
cjolivier01 commented on a change in pull request #7903: Refactor AdaGrad optimizer to support sparse tensors URL: https://github.com/apache/incubator-mxnet/pull/7903#discussion_r140057718 ## File path: tests/python/unittest/test_sparse_operator.py ## @@ -697,6 +697,19 @@

[GitHub] piiswrong commented on a change in pull request #7949: account for batch padding when updating metrics

2017-09-20 Thread git
piiswrong commented on a change in pull request #7949: account for batch padding when updating metrics URL: https://github.com/apache/incubator-mxnet/pull/7949#discussion_r140037644 ## File path: python/mxnet/module/executor_group.py ## @@ -96,6 +96,19 @@ def

[GitHub] szha commented on issue #7962: SSD Example: Fixed training bug

2017-09-20 Thread git
szha commented on issue #7962: SSD Example: Fixed training bug URL: https://github.com/apache/incubator-mxnet/pull/7962#issuecomment-330925146 @zhreshold This is an automated message from the Apache Git Service. To respond

[GitHub] szha commented on issue #7962: SSD Example: Fixed training bug

2017-09-20 Thread git
szha commented on issue #7962: SSD Example: Fixed training bug URL: https://github.com/apache/incubator-mxnet/pull/7962#issuecomment-330925146 @zhresho This is an automated message from the Apache Git Service. To respond to

[GitHub] piiswrong closed pull request #7941: DO NOT MERGE: Just a test of adding properties to JenkinsFile

2017-09-20 Thread git
piiswrong closed pull request #7941: DO NOT MERGE: Just a test of adding properties to JenkinsFile URL: https://github.com/apache/incubator-mxnet/pull/7941 This is an automated message from the Apache Git Service. To

[GitHub] piiswrong commented on issue #7967: Fix computation of num_inputs for Python API create_operator_entry

2017-09-20 Thread git
piiswrong commented on issue #7967: Fix computation of num_inputs for Python API create_operator_entry URL: https://github.com/apache/incubator-mxnet/pull/7967#issuecomment-330920726 I think this is correct. Could you PR the same thing to 0.11.0?

[GitHub] piiswrong commented on issue #7942: Adam optimizer consistent with paper

2017-09-20 Thread git
piiswrong commented on issue #7942: Adam optimizer consistent with paper URL: https://github.com/apache/incubator-mxnet/pull/7942#issuecomment-330931609 @sxjscience Could you verify this against the paper? @formath The default for rho should be 1.0 to keep the original default

[GitHub] cjolivier01 commented on a change in pull request #7903: Refactor AdaGrad optimizer to support sparse tensors

2017-09-20 Thread git
cjolivier01 commented on a change in pull request #7903: Refactor AdaGrad optimizer to support sparse tensors URL: https://github.com/apache/incubator-mxnet/pull/7903#discussion_r140046020 ## File path: python/mxnet/optimizer.py ## @@ -664,27 +667,45 @@ class

[GitHub] piiswrong commented on issue #7942: Adam optimizer consistent with paper

2017-09-20 Thread git
piiswrong commented on issue #7942: Adam optimizer consistent with paper URL: https://github.com/apache/incubator-mxnet/pull/7942#issuecomment-330931609 @sxjscience Could you verify this against the paper? @formath The default for rho should be 1.0 to keep the original default

[GitHub] jeremiedb commented on issue #7524: Is there a tutorial of using mxnet R LSTM for time series forecasting?

2017-09-20 Thread git
jeremiedb commented on issue #7524: Is there a tutorial of using mxnet R LSTM for time series forecasting? URL: https://github.com/apache/incubator-mxnet/issues/7524#issuecomment-330938609 The refactor RNN API has now been merged and the [demo on its

[GitHub] lichen11 opened a new issue #7968: Transfer Learning using VGG-16

2017-09-20 Thread git
lichen11 opened a new issue #7968: Transfer Learning using VGG-16 URL: https://github.com/apache/incubator-mxnet/issues/7968 I downloaded the pre-trained vgg-16 model from data.dmlc.ml/mxnet/models/imagenet/vgg/. I would like to retrain the last three layers. I followed the tutorial of

[GitHub] kevinthesun commented on a change in pull request #7950: Changed How To String to FAQ and added Discuss Tab to the Navigation bar

2017-09-20 Thread git
kevinthesun commented on a change in pull request #7950: Changed How To String to FAQ and added Discuss Tab to the Navigation bar URL: https://github.com/apache/incubator-mxnet/pull/7950#discussion_r140050062 ## File path: docs/_static/js/navbar.js ## @@ -1,6 +1,6 @@ var

[GitHub] cswiercz commented on issue #7781: Implement Khatri-Rao operator

2017-09-20 Thread git
cswiercz commented on issue #7781: Implement Khatri-Rao operator URL: https://github.com/apache/incubator-mxnet/pull/7781#issuecomment-330922986 @mli @piiswrong - Two MXNet contributors who know this function have approved this PR. Is there anything I'm missing?

[GitHub] zjost commented on issue #7960: Dropout layer crashes jupyter kernel

2017-09-20 Thread git
zjost commented on issue #7960: Dropout layer crashes jupyter kernel URL: https://github.com/apache/incubator-mxnet/issues/7960#issuecomment-330944880 This is a shared box on my team and we're relying on the internal setup of the Deep Learning AMI, so I lack permissions for global

[GitHub] thinksanky commented on a change in pull request #7950: Changed How To String to FAQ and added Discuss Tab to the Navigation bar

2017-09-20 Thread git
thinksanky commented on a change in pull request #7950: Changed How To String to FAQ and added Discuss Tab to the Navigation bar URL: https://github.com/apache/incubator-mxnet/pull/7950#discussion_r140063057 ## File path: docs/_static/js/navbar.js ## @@ -1,6 +1,6 @@ var

[incubator-mxnet] branch master updated: R-package RNN refactor (#7476)

2017-09-20 Thread qkou
This is an automated email from the ASF dual-hosted git repository. qkou 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 9cdac22 R-package RNN refactor (#7476)

[GitHub] thirdwing closed pull request #7476: R-package RNN refactor

2017-09-20 Thread git
thirdwing closed pull request #7476: R-package RNN refactor URL: https://github.com/apache/incubator-mxnet/pull/7476 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub

[GitHub] szha commented on issue #7957: add densenet

2017-09-20 Thread git
szha commented on issue #7957: add densenet URL: https://github.com/apache/incubator-mxnet/pull/7957#issuecomment-330924232 For hybrid block when symbol is used they should be exactly the same. In that case the gluon interface is just a modular build tool.

[GitHub] fhieber commented on issue #7967: Fix computation of num_inputs for Python API create_operator_entry

2017-09-20 Thread git
fhieber commented on issue #7967: Fix computation of num_inputs for Python API create_operator_entry URL: https://github.com/apache/incubator-mxnet/pull/7967#issuecomment-330878891 @tdomhan This is an automated message from

[GitHub] fhieber opened a new pull request #7967: Fix computation of num_inputs for Python API create_operator_entry

2017-09-20 Thread git
fhieber opened a new pull request #7967: Fix computation of num_inputs for Python API create_operator_entry URL: https://github.com/apache/incubator-mxnet/pull/7967 This solves the following problem in MXNet v.0.11.x (not present in v.0.10.0): If a user creates a CustomOperator in

[GitHub] zjost commented on issue #7960: Dropout layer crashes jupyter kernel

2017-09-20 Thread git
zjost commented on issue #7960: Dropout layer crashes jupyter kernel URL: https://github.com/apache/incubator-mxnet/issues/7960#issuecomment-330904210 I don't get a stack trace, the Jupyter notebook kernel crashes. It says "The kernel appears to have died. It will restart automatically."

[GitHub] traveller59 closed issue #7927: can't perform operations between mx.sym.var and mx.nd.array

2017-09-20 Thread git
traveller59 closed issue #7927: can't perform operations between mx.sym.var and mx.nd.array URL: https://github.com/apache/incubator-mxnet/issues/7927 This is an automated message from the Apache Git Service. To respond to

[GitHub] jeremiedb commented on issue #7476: R-package RNN refactor

2017-09-20 Thread git
jeremiedb commented on issue #7476: R-package RNN refactor URL: https://github.com/apache/incubator-mxnet/pull/7476#issuecomment-330895488 @thirdwing CI has issue with some Python test, not related to this pull. This is an

[GitHub] traveller59 commented on issue #7927: can't perform operations between mx.sym.var and mx.nd.array

2017-09-20 Thread git
traveller59 commented on issue #7927: can't perform operations between mx.sym.var and mx.nd.array URL: https://github.com/apache/incubator-mxnet/issues/7927#issuecomment-330887561 Thanks, I will consider gluon firstly.

[GitHub] FrancisTse8 commented on issue #7852: Trouble installing MXNet on Raspberry Pi 3

2017-09-20 Thread git
FrancisTse8 commented on issue #7852: Trouble installing MXNet on Raspberry Pi 3 URL: https://github.com/apache/incubator-mxnet/issues/7852#issuecomment-330896157 I was able to follow the instructions on

[GitHub] piiswrong commented on a change in pull request #7892: allow users to specify repo url for downloadable contents

2017-09-20 Thread git
piiswrong commented on a change in pull request #7892: allow users to specify repo url for downloadable contents URL: https://github.com/apache/incubator-mxnet/pull/7892#discussion_r140072134 ## File path: python/mxnet/gluon/utils.py ## @@ -207,4 +207,14 @@ def

[GitHub] piiswrong commented on a change in pull request #7938: instance norm and reflection padding

2017-09-20 Thread git
piiswrong commented on a change in pull request #7938: instance norm and reflection padding URL: https://github.com/apache/incubator-mxnet/pull/7938#discussion_r140076823 ## File path: python/mxnet/gluon/nn/conv_layers.py ## @@ -1009,3 +1009,32 @@ def __init__(self,

[GitHub] piiswrong commented on a change in pull request #7938: instance norm and reflection padding

2017-09-20 Thread git
piiswrong commented on a change in pull request #7938: instance norm and reflection padding URL: https://github.com/apache/incubator-mxnet/pull/7938#discussion_r140077132 ## File path: python/mxnet/gluon/nn/conv_layers.py ## @@ -1009,3 +1009,32 @@ def __init__(self,

[GitHub] piiswrong commented on a change in pull request #7938: instance norm and reflection padding

2017-09-20 Thread git
piiswrong commented on a change in pull request #7938: instance norm and reflection padding URL: https://github.com/apache/incubator-mxnet/pull/7938#discussion_r140076823 ## File path: python/mxnet/gluon/nn/conv_layers.py ## @@ -1009,3 +1009,32 @@ def __init__(self,

[GitHub] piiswrong commented on a change in pull request #7864: Enhancement for distributed sparse linear regression example

2017-09-20 Thread git
piiswrong commented on a change in pull request #7864: Enhancement for distributed sparse linear regression example URL: https://github.com/apache/incubator-mxnet/pull/7864#discussion_r140078814 ## File path: python/mxnet/metric.py ## @@ -917,6 +917,73 @@ def

[GitHub] piiswrong commented on a change in pull request #7864: Enhancement for distributed sparse linear regression example

2017-09-20 Thread git
piiswrong commented on a change in pull request #7864: Enhancement for distributed sparse linear regression example URL: https://github.com/apache/incubator-mxnet/pull/7864#discussion_r140078898 ## File path: python/mxnet/metric.py ## @@ -917,6 +917,73 @@ def

[GitHub] piiswrong commented on issue #7958: Engine shutdown

2017-09-20 Thread git
piiswrong commented on issue #7958: Engine shutdown URL: https://github.com/apache/incubator-mxnet/issues/7958#issuecomment-330973360 @zhreshold Could you change imdecode to imread and improve the error message to output which image is broken?

[GitHub] adstraw commented on issue #7810: Enable MKL support for full pooling

2017-09-20 Thread git
adstraw commented on issue #7810: Enable MKL support for full pooling URL: https://github.com/apache/incubator-mxnet/pull/7810#issuecomment-330971832 Tested with the following unit test per feedback from @glingyan ``` import numpy as np from mxnet.test_utils import

[GitHub] szha commented on a change in pull request #7892: allow users to specify repo url for downloadable contents

2017-09-20 Thread git
szha commented on a change in pull request #7892: allow users to specify repo url for downloadable contents URL: https://github.com/apache/incubator-mxnet/pull/7892#discussion_r140095478 ## File path: python/mxnet/gluon/utils.py ## @@ -207,4 +207,14 @@ def download(url,

[GitHub] kevinthesun commented on issue #7946: Defining Residual Convolution Structures in Block from gluon Module

2017-09-20 Thread git
kevinthesun commented on issue #7946: Defining Residual Convolution Structures in Block from gluon Module URL: https://github.com/apache/incubator-mxnet/issues/7946#issuecomment-330987090 1. self.ramp(self.conv(x)) vs mx.gluon.nn.Conv1D(activation='relu')(x) Yes. The latter applies a

[GitHub] kevinthesun commented on issue #7956: simple_bind error when using rotation matrix & batch_dot

2017-09-20 Thread git
kevinthesun commented on issue #7956: simple_bind error when using rotation matrix & batch_dot URL: https://github.com/apache/incubator-mxnet/issues/7956#issuecomment-330988418 @reminisce Any idea? This is an automated

[GitHub] jmerkow commented on issue #7949: account for batch padding when updating metrics

2017-09-20 Thread git
jmerkow commented on issue #7949: account for batch padding when updating metrics URL: https://github.com/apache/incubator-mxnet/pull/7949#issuecomment-330955000 @piiswrong or @mikerossgithub, do you know if padding handled correctly when update (for parameters) is called?

[GitHub] piiswrong commented on a change in pull request #7854: Basic CPU Kernel OMP selection based upon whether GPU has been used

2017-09-20 Thread git
piiswrong commented on a change in pull request #7854: Basic CPU Kernel OMP selection based upon whether GPU has been used URL: https://github.com/apache/incubator-mxnet/pull/7854#discussion_r140073255 ## File path: src/engine/threaded_engine.h ## @@ -360,6 +380,21 @@

[GitHub] piiswrong closed pull request #7874: Default shape and context attributes for operators

2017-09-20 Thread git
piiswrong closed pull request #7874: Default shape and context attributes for operators URL: https://github.com/apache/incubator-mxnet/pull/7874 This is an automated message from the Apache Git Service. To respond to the

[GitHub] piiswrong closed pull request #7937: Increase the tolerance

2017-09-20 Thread git
piiswrong closed pull request #7937: Increase the tolerance URL: https://github.com/apache/incubator-mxnet/pull/7937 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub

[GitHub] cjolivier01 commented on a change in pull request #7854: Basic CPU Kernel OMP selection based upon whether GPU has been used

2017-09-20 Thread git
cjolivier01 commented on a change in pull request #7854: Basic CPU Kernel OMP selection based upon whether GPU has been used URL: https://github.com/apache/incubator-mxnet/pull/7854#discussion_r140077864 ## File path: src/engine/threaded_engine.h ## @@ -360,6 +380,21 @@

[GitHub] szha commented on a change in pull request #7854: Basic CPU Kernel OMP selection based upon whether GPU has been used

2017-09-20 Thread git
szha commented on a change in pull request #7854: Basic CPU Kernel OMP selection based upon whether GPU has been used URL: https://github.com/apache/incubator-mxnet/pull/7854#discussion_r140080582 ## File path: src/engine/threaded_engine.h ## @@ -293,6 +299,20 @@ class

[GitHub] cjolivier01 commented on a change in pull request #7854: Basic CPU Kernel OMP selection based upon whether GPU has been used

2017-09-20 Thread git
cjolivier01 commented on a change in pull request #7854: Basic CPU Kernel OMP selection based upon whether GPU has been used URL: https://github.com/apache/incubator-mxnet/pull/7854#discussion_r140080388 ## File path: src/engine/threaded_engine.h ## @@ -293,6 +299,20 @@

[GitHub] cjolivier01 commented on a change in pull request #7854: Basic CPU Kernel OMP selection based upon whether GPU has been used

2017-09-20 Thread git
cjolivier01 commented on a change in pull request #7854: Basic CPU Kernel OMP selection based upon whether GPU has been used URL: https://github.com/apache/incubator-mxnet/pull/7854#discussion_r140082242 ## File path: src/engine/threaded_engine.h ## @@ -293,6 +299,20 @@

[GitHub] szha commented on issue #7877: sequential multiple add

2017-09-20 Thread git
szha commented on issue #7877: sequential multiple add URL: https://github.com/apache/incubator-mxnet/pull/7877#issuecomment-330980673 Any feed back other than dropping the feature? This is an automated message from the

[GitHub] piiswrong commented on a change in pull request #7892: allow users to specify repo url for downloadable contents

2017-09-20 Thread git
piiswrong commented on a change in pull request #7892: allow users to specify repo url for downloadable contents URL: https://github.com/apache/incubator-mxnet/pull/7892#discussion_r140095325 ## File path: python/mxnet/gluon/utils.py ## @@ -207,4 +207,14 @@ def

[GitHub] piiswrong commented on a change in pull request #7264: gluon conv rnns

2017-09-20 Thread git
piiswrong commented on a change in pull request #7264: gluon conv rnns URL: https://github.com/apache/incubator-mxnet/pull/7264#discussion_r140096260 ## File path: python/mxnet/gluon/contrib/rnn/conv_rnn_cell.py ## @@ -0,0 +1,971 @@ +# Licensed to the Apache Software

[GitHub] kevinthesun commented on issue #7933: CoreML conversion with finetuned model

2017-09-20 Thread git
kevinthesun commented on issue #7933: CoreML conversion with finetuned model URL: https://github.com/apache/incubator-mxnet/issues/7933#issuecomment-330986289 @pracheer This is an automated message from the Apache Git

[GitHub] kevinthesun commented on issue #7950: Changed How To String to FAQ and added Discuss Tab to the Navigation bar

2017-09-20 Thread git
kevinthesun commented on issue #7950: Changed How To String to FAQ and added Discuss Tab to the Navigation bar URL: https://github.com/apache/incubator-mxnet/pull/7950#issuecomment-330956519 LGTM This is an automated

[incubator-mxnet] branch master updated: Add Kaggle w. gluon pipeline and k fold cross validation (#7915)

2017-09-20 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 67c0ad7 Add Kaggle w. gluon pipeline

[GitHub] piiswrong closed pull request #7915: Add Kaggle w. gluon pipeline and k fold cross validation

2017-09-20 Thread git
piiswrong closed pull request #7915: Add Kaggle w. gluon pipeline and k fold cross validation URL: https://github.com/apache/incubator-mxnet/pull/7915 This is an automated message from the Apache Git Service. To respond to

[GitHub] eric-haibin-lin commented on a change in pull request #7864: Enhancement for distributed sparse linear regression example

2017-09-20 Thread git
eric-haibin-lin commented on a change in pull request #7864: Enhancement for distributed sparse linear regression example URL: https://github.com/apache/incubator-mxnet/pull/7864#discussion_r140079384 ## File path: python/mxnet/metric.py ## @@ -917,6 +917,73 @@ def

[GitHub] piiswrong commented on issue #7810: Enable MKL support for full pooling

2017-09-20 Thread git
piiswrong commented on issue #7810: Enable MKL support for full pooling URL: https://github.com/apache/incubator-mxnet/pull/7810#issuecomment-330966776 @ykim362 Could you confirm? This is an automated message from the Apache

[GitHub] eric-haibin-lin commented on a change in pull request #7864: Enhancement for distributed sparse linear regression example

2017-09-20 Thread git
eric-haibin-lin commented on a change in pull request #7864: Enhancement for distributed sparse linear regression example URL: https://github.com/apache/incubator-mxnet/pull/7864#discussion_r140079503 ## File path: python/mxnet/metric.py ## @@ -917,6 +917,73 @@ def

[GitHub] szha commented on a change in pull request #7264: gluon conv rnns

2017-09-20 Thread git
szha commented on a change in pull request #7264: gluon conv rnns URL: https://github.com/apache/incubator-mxnet/pull/7264#discussion_r140100643 ## File path: python/mxnet/gluon/contrib/rnn/conv_rnn_cell.py ## @@ -0,0 +1,971 @@ +# Licensed to the Apache Software Foundation

[GitHub] szha commented on a change in pull request #7854: Basic CPU Kernel OMP selection based upon whether GPU has been used

2017-09-20 Thread git
szha commented on a change in pull request #7854: Basic CPU Kernel OMP selection based upon whether GPU has been used URL: https://github.com/apache/incubator-mxnet/pull/7854#discussion_r140078454 ## File path: src/engine/threaded_engine.h ## @@ -293,6 +299,20 @@ class

[GitHub] cjolivier01 commented on a change in pull request #7854: Basic CPU Kernel OMP selection based upon whether GPU has been used

2017-09-20 Thread git
cjolivier01 commented on a change in pull request #7854: Basic CPU Kernel OMP selection based upon whether GPU has been used URL: https://github.com/apache/incubator-mxnet/pull/7854#discussion_r140080388 ## File path: src/engine/threaded_engine.h ## @@ -293,6 +299,20 @@

[GitHub] piiswrong closed pull request #7969: update doc icon

2017-09-20 Thread git
piiswrong closed pull request #7969: update doc icon URL: https://github.com/apache/incubator-mxnet/pull/7969 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use

[GitHub] cjolivier01 commented on a change in pull request #7854: Basic CPU Kernel OMP selection based upon whether GPU has been used

2017-09-20 Thread git
cjolivier01 commented on a change in pull request #7854: Basic CPU Kernel OMP selection based upon whether GPU has been used URL: https://github.com/apache/incubator-mxnet/pull/7854#discussion_r140080388 ## File path: src/engine/threaded_engine.h ## @@ -293,6 +299,20 @@

[GitHub] piiswrong commented on a change in pull request #7868: Issue 7264: MKL supports 2D/3D/4D input tensor only for concatenation

2017-09-20 Thread git
piiswrong commented on a change in pull request #7868: Issue 7264: MKL supports 2D/3D/4D input tensor only for concatenation URL: https://github.com/apache/incubator-mxnet/pull/7868#discussion_r140095993 ## File path: src/operator/concat.cc ## @@ -33,10 +33,21 @@

[GitHub] szha commented on a change in pull request #7892: allow users to specify repo url for downloadable contents

2017-09-20 Thread git
szha commented on a change in pull request #7892: allow users to specify repo url for downloadable contents URL: https://github.com/apache/incubator-mxnet/pull/7892#discussion_r140096017 ## File path: python/mxnet/gluon/utils.py ## @@ -207,4 +207,14 @@ def download(url,

[GitHub] piiswrong commented on a change in pull request #7264: gluon conv rnns

2017-09-20 Thread git
piiswrong commented on a change in pull request #7264: gluon conv rnns URL: https://github.com/apache/incubator-mxnet/pull/7264#discussion_r140097191 ## File path: python/mxnet/gluon/contrib/rnn/conv_rnn_cell.py ## @@ -0,0 +1,971 @@ +# Licensed to the Apache Software

[GitHub] piiswrong commented on a change in pull request #7892: allow users to specify repo url for downloadable contents

2017-09-20 Thread git
piiswrong commented on a change in pull request #7892: allow users to specify repo url for downloadable contents URL: https://github.com/apache/incubator-mxnet/pull/7892#discussion_r140071775 ## File path: python/mxnet/gluon/model_zoo/vision/densenet.py ## @@ -131,12

[GitHub] szha commented on a change in pull request #7892: allow users to specify repo url for downloadable contents

2017-09-20 Thread git
szha commented on a change in pull request #7892: allow users to specify repo url for downloadable contents URL: https://github.com/apache/incubator-mxnet/pull/7892#discussion_r140073989 ## File path: python/mxnet/gluon/model_zoo/vision/densenet.py ## @@ -131,12 +132,16

[GitHub] szha commented on a change in pull request #7892: allow users to specify repo url for downloadable contents

2017-09-20 Thread git
szha commented on a change in pull request #7892: allow users to specify repo url for downloadable contents URL: https://github.com/apache/incubator-mxnet/pull/7892#discussion_r140074208 ## File path: python/mxnet/gluon/utils.py ## @@ -207,4 +207,14 @@ def download(url,

[incubator-mxnet] branch master updated: Fix computation of num_inputs for Python API create_operator_entry for custom operators with 0 arguments. (#7967)

2017-09-20 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 ec9f82e Fix computation of num_inputs

[GitHub] piiswrong closed pull request #7967: Fix computation of num_inputs for Python API create_operator_entry

2017-09-20 Thread git
piiswrong closed pull request #7967: Fix computation of num_inputs for Python API create_operator_entry URL: https://github.com/apache/incubator-mxnet/pull/7967 This is an automated message from the Apache Git Service. To

[GitHub] jmerkow commented on a change in pull request #7949: account for batch padding when updating metrics

2017-09-20 Thread git
jmerkow commented on a change in pull request #7949: account for batch padding when updating metrics URL: https://github.com/apache/incubator-mxnet/pull/7949#discussion_r140069817 ## File path: python/mxnet/module/executor_group.py ## @@ -400,6 +414,12 @@ def

[GitHub] piiswrong commented on a change in pull request #7938: instance norm and reflection padding

2017-09-20 Thread git
piiswrong commented on a change in pull request #7938: instance norm and reflection padding URL: https://github.com/apache/incubator-mxnet/pull/7938#discussion_r140076384 ## File path: python/mxnet/gluon/nn/conv_layers.py ## @@ -1009,3 +1009,32 @@ def __init__(self,

[GitHub] piiswrong commented on a change in pull request #7938: instance norm and reflection padding

2017-09-20 Thread git
piiswrong commented on a change in pull request #7938: instance norm and reflection padding URL: https://github.com/apache/incubator-mxnet/pull/7938#discussion_r140076426 ## File path: python/mxnet/gluon/nn/conv_layers.py ## @@ -1009,3 +1009,32 @@ def __init__(self,

[GitHub] piiswrong commented on a change in pull request #7938: instance norm and reflection padding

2017-09-20 Thread git
piiswrong commented on a change in pull request #7938: instance norm and reflection padding URL: https://github.com/apache/incubator-mxnet/pull/7938#discussion_r140076341 ## File path: python/mxnet/gluon/nn/basic_layers.py ## @@ -442,3 +442,94 @@ def hybrid_forward(self,

[GitHub] piiswrong commented on a change in pull request #7938: instance norm and reflection padding

2017-09-20 Thread git
piiswrong commented on a change in pull request #7938: instance norm and reflection padding URL: https://github.com/apache/incubator-mxnet/pull/7938#discussion_r140076562 ## File path: python/mxnet/gluon/nn/conv_layers.py ## @@ -1009,3 +1009,32 @@ def __init__(self,

[GitHub] piiswrong commented on a change in pull request #7938: instance norm and reflection padding

2017-09-20 Thread git
piiswrong commented on a change in pull request #7938: instance norm and reflection padding URL: https://github.com/apache/incubator-mxnet/pull/7938#discussion_r140076648 ## File path: python/mxnet/gluon/nn/conv_layers.py ## @@ -1009,3 +1009,32 @@ def __init__(self,

[GitHub] piiswrong commented on issue #7938: instance norm and reflection padding

2017-09-20 Thread git
piiswrong commented on issue #7938: instance norm and reflection padding URL: https://github.com/apache/incubator-mxnet/pull/7938#issuecomment-330964117 Please add tests. This is an automated message from the Apache Git

[GitHub] piiswrong commented on issue #7898: Stop the build if there are Python syntax errors

2017-09-20 Thread git
piiswrong commented on issue #7898: Stop the build if there are Python syntax errors URL: https://github.com/apache/incubator-mxnet/pull/7898#issuecomment-330966368 @gautamkmr This is an automated message from the Apache

[GitHub] adstraw commented on issue #7810: Enable MKL support for full pooling

2017-09-20 Thread git
adstraw commented on issue #7810: Enable MKL support for full pooling URL: https://github.com/apache/incubator-mxnet/pull/7810#issuecomment-330971832 Tested with the following unit test per feedback from @glingyan `import numpy as np from mxnet.test_utils import check_consistency

[incubator-mxnet] branch master updated: update doc icon (#7969)

2017-09-20 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 6f764e9 update doc icon (#7969) 6f764e9

[GitHub] piiswrong commented on a change in pull request #7264: gluon conv rnns

2017-09-20 Thread git
piiswrong commented on a change in pull request #7264: gluon conv rnns URL: https://github.com/apache/incubator-mxnet/pull/7264#discussion_r140097722 ## File path: python/mxnet/gluon/contrib/rnn/conv_rnn_cell.py ## @@ -0,0 +1,971 @@ +# Licensed to the Apache Software

[GitHub] szha commented on a change in pull request #7264: gluon conv rnns

2017-09-20 Thread git
szha commented on a change in pull request #7264: gluon conv rnns URL: https://github.com/apache/incubator-mxnet/pull/7264#discussion_r140098969 ## File path: python/mxnet/gluon/contrib/rnn/conv_rnn_cell.py ## @@ -0,0 +1,971 @@ +# Licensed to the Apache Software Foundation

[GitHub] sergeykolychev commented on issue #7965: Perl AI::MXNet

2017-09-20 Thread git
sergeykolychev commented on issue #7965: Perl AI::MXNet URL: https://github.com/apache/incubator-mxnet/issues/7965#issuecomment-331013939 @xurc old perl has not av_top_index defined, for now please change all av_top_index to av_len I'll patch the master code later when I'll be

  1   2   >