[GitHub] samhodge commented on issue #9813: Unable to save gluon model to symbolic network : neural style

2018-02-24 Thread GitBox
samhodge commented on issue #9813: Unable to save gluon model to symbolic network : neural style URL: https://github.com/apache/incubator-mxnet/issues/9813#issuecomment-368286926 This clumsy attempt wont work because you cannot convert a symbol to an int.

[GitHub] samhodge commented on issue #9813: Unable to save gluon model to symbolic network : neural style

2018-02-24 Thread GitBox
samhodge commented on issue #9813: Unable to save gluon model to symbolic network : neural style URL: https://github.com/apache/incubator-mxnet/issues/9813#issuecomment-368286692 I have seen this: https://github.com/apache/incubator-mxnet/issues/6087 and I have also seen this:

[GitHub] samhodge commented on issue #9813: Unable to save gluon model to symbolic network : neural style

2018-02-24 Thread GitBox
samhodge commented on issue #9813: Unable to save gluon model to symbolic network : neural style URL: https://github.com/apache/incubator-mxnet/issues/9813#issuecomment-368286692 I have seen this: https://github.com/apache/incubator-mxnet/issues/6087 and I have also seen this:

[GitHub] ashokei commented on a change in pull request #9810: remove MKL_EXPERIMENTAL and update make files for MKL-DNN

2018-02-24 Thread GitBox
ashokei commented on a change in pull request #9810: remove MKL_EXPERIMENTAL and update make files for MKL-DNN URL: https://github.com/apache/incubator-mxnet/pull/9810#discussion_r170442004 ## File path: docker_multiarch/arm.crosscompile.mk ## @@ -115,13 +100,10 @@

[GitHub] tqchen commented on issue #9880: TVM bridge support to JIT NDArray Function by TVM

2018-02-24 Thread GitBox
tqchen commented on issue #9880: TVM bridge support to JIT NDArray Function by TVM URL: https://github.com/apache/incubator-mxnet/pull/9880#issuecomment-368284709 TVM's side of PR https://github.com/dmlc/tvm/pull/930 This is

[GitHub] tqchen commented on issue #9880: TVM bridge support

2018-02-24 Thread GitBox
tqchen commented on issue #9880: TVM bridge support URL: https://github.com/apache/incubator-mxnet/pull/9880#issuecomment-368284643 cc @piiswrong This is an automated message from the Apache Git Service. To respond to the

[GitHub] tqchen commented on issue #9880: TVM bridge support

2018-02-24 Thread GitBox
tqchen commented on issue #9880: TVM bridge support URL: https://github.com/apache/incubator-mxnet/pull/9880#issuecomment-368284637 based on an early version of work by @ZihengJiang This is an automated message from the

[GitHub] tqchen commented on issue #9880: TVM bridge support

2018-02-24 Thread GitBox
tqchen commented on issue #9880: TVM bridge support URL: https://github.com/apache/incubator-mxnet/pull/9880#issuecomment-368284643 cc @piiswrong @sxjscience This is an automated message from the Apache Git Service. To

[GitHub] tqchen opened a new pull request #9880: TVM bridge support

2018-02-24 Thread GitBox
tqchen opened a new pull request #9880: TVM bridge support URL: https://github.com/apache/incubator-mxnet/pull/9880 Support wrap TVM compiled function as an NDArray function. This enables use TVM as RTC module for MX's async function ## Example ```python def

[GitHub] yian2271368 commented on issue #2454: input image size smaller than input shape when I train inception-bn-28-small using my own data set

2018-02-24 Thread GitBox
yian2271368 commented on issue #2454: input image size smaller than input shape when I train inception-bn-28-small using my own data set URL: https://github.com/apache/incubator-mxnet/issues/2454#issuecomment-368281632 @zhudelong hey, i faced the same problem, how did you fix it ?

[GitHub] iblis17 commented on issue #9677: Refactor operators and add MKLDNN

2018-02-24 Thread GitBox
iblis17 commented on issue #9677: Refactor operators and add MKLDNN URL: https://github.com/apache/incubator-mxnet/pull/9677#issuecomment-368280425 @marcoabreu About the reason of hosting Julia code in another repository, Julia's package manager is built on top of git, and it

[GitHub] iblis17 commented on issue #9677: Refactor operators and add MKLDNN

2018-02-24 Thread GitBox
iblis17 commented on issue #9677: Refactor operators and add MKLDNN URL: https://github.com/apache/incubator-mxnet/pull/9677#issuecomment-368280425 @marcoabreu About the reason of hosting Julia code in another repository, Julia's package manager is built on top of git, and it

[GitHub] iblis17 commented on issue #8175: WIP: Julia CI build

2018-02-24 Thread GitBox
iblis17 commented on issue #8175: WIP: Julia CI build URL: https://github.com/apache/incubator-mxnet/pull/8175#issuecomment-368280486 bump again. This is an automated message from the Apache Git Service. To respond to the

[GitHub] iblis17 commented on issue #9677: Refactor operators and add MKLDNN

2018-02-24 Thread GitBox
iblis17 commented on issue #9677: Refactor operators and add MKLDNN URL: https://github.com/apache/incubator-mxnet/pull/9677#issuecomment-368280425 @marcoabreu About the reason of hosting Julia code in another repository, Julia's package manager is built on top of git, and it

[GitHub] eric-haibin-lin commented on a change in pull request #9625: sparse regression operators

2018-02-24 Thread GitBox
eric-haibin-lin commented on a change in pull request #9625: sparse regression operators URL: https://github.com/apache/incubator-mxnet/pull/9625#discussion_r170439654 ## File path: src/operator/regression_output-inl.h ## @@ -95,12 +125,35 @@ void RegressionForward(const

[GitHub] eric-haibin-lin commented on a change in pull request #9625: sparse regression operators

2018-02-24 Thread GitBox
eric-haibin-lin commented on a change in pull request #9625: sparse regression operators URL: https://github.com/apache/incubator-mxnet/pull/9625#discussion_r170439239 ## File path: tests/python/unittest/test_operator.py ## @@ -219,13 +219,13 @@ def

[GitHub] eric-haibin-lin commented on a change in pull request #9625: sparse regression operators

2018-02-24 Thread GitBox
eric-haibin-lin commented on a change in pull request #9625: sparse regression operators URL: https://github.com/apache/incubator-mxnet/pull/9625#discussion_r170439526 ## File path: src/operator/regression_output-inl.h ## @@ -121,6 +228,67 @@ void RegressionBackward(const

[GitHub] eric-haibin-lin commented on a change in pull request #9625: sparse regression operators

2018-02-24 Thread GitBox
eric-haibin-lin commented on a change in pull request #9625: sparse regression operators URL: https://github.com/apache/incubator-mxnet/pull/9625#discussion_r170439324 ## File path: src/operator/regression_output.cc ## @@ -74,12 +79,17 @@ then the squared loss estimated

[GitHub] marcoabreu closed pull request #9873: add doc for gluon contrib

2018-02-24 Thread GitBox
marcoabreu closed pull request #9873: add doc for gluon contrib URL: https://github.com/apache/incubator-mxnet/pull/9873 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

[incubator-mxnet] branch master updated: add doc for gluon contrib (#9873)

2018-02-24 Thread marcoabreu
This is an automated email from the ASF dual-hosted git repository. marcoabreu pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git The following commit(s) were added to refs/heads/master by this push: new c6a7aba add doc for gluon contrib

[GitHub] marcoabreu commented on issue #9879: Versions patch

2018-02-24 Thread GitBox
marcoabreu commented on issue #9879: Versions patch URL: https://github.com/apache/incubator-mxnet/pull/9879#issuecomment-368273802 Could you separate your changes between this PR and #9718? This is an automated message from

[GitHub] marcoabreu commented on a change in pull request #9879: Versions patch

2018-02-24 Thread GitBox
marcoabreu commented on a change in pull request #9879: Versions patch URL: https://github.com/apache/incubator-mxnet/pull/9879#discussion_r170437946 ## File path: docs/build_version_doc/AddVersion.py ## @@ -57,6 +57,9 @@ for name in files: if not

[GitHub] marcoabreu commented on a change in pull request #9878: Docs build all versions refactor

2018-02-24 Thread GitBox
marcoabreu commented on a change in pull request #9878: Docs build all versions refactor URL: https://github.com/apache/incubator-mxnet/pull/9878#discussion_r170437422 ## File path: docs/build_version_doc/build_all_version.sh ## @@ -59,27 +65,8 @@ for tag in $tag_list; do

[GitHub] marcoabreu commented on a change in pull request #9878: Docs build all versions refactor

2018-02-24 Thread GitBox
marcoabreu commented on a change in pull request #9878: Docs build all versions refactor URL: https://github.com/apache/incubator-mxnet/pull/9878#discussion_r170437313 ## File path: docs/build_version_doc/build_all_version.sh ## @@ -29,10 +29,16 @@ tag_list="1.1.0 1.0.0

[GitHub] marcoabreu commented on a change in pull request #9878: Docs build all versions refactor

2018-02-24 Thread GitBox
marcoabreu commented on a change in pull request #9878: Docs build all versions refactor URL: https://github.com/apache/incubator-mxnet/pull/9878#discussion_r170437313 ## File path: docs/build_version_doc/build_all_version.sh ## @@ -29,10 +29,16 @@ tag_list="1.1.0 1.0.0

[GitHub] marcoabreu commented on a change in pull request #9878: Docs build all versions refactor

2018-02-24 Thread GitBox
marcoabreu commented on a change in pull request #9878: Docs build all versions refactor URL: https://github.com/apache/incubator-mxnet/pull/9878#discussion_r170437286 ## File path: docs/build_version_doc/update_all_version.sh ## @@ -0,0 +1,62 @@ +#!/bin/bash + +#

[GitHub] marcoabreu commented on a change in pull request #9878: Docs build all versions refactor

2018-02-24 Thread GitBox
marcoabreu commented on a change in pull request #9878: Docs build all versions refactor URL: https://github.com/apache/incubator-mxnet/pull/9878#discussion_r170437247 ## File path: docs/build_version_doc/setup_docs_ubuntu.sh ## @@ -0,0 +1,42 @@ +# If you need to build <=

[GitHub] marcoabreu commented on a change in pull request #9878: Docs build all versions refactor

2018-02-24 Thread GitBox
marcoabreu commented on a change in pull request #9878: Docs build all versions refactor URL: https://github.com/apache/incubator-mxnet/pull/9878#discussion_r170437214 ## File path: docs/build_version_doc/setup_docker.sh ## @@ -0,0 +1,17 @@ +# Setup Docker Review

[GitHub] gigasquid commented on issue #8971: Clojure Library for mxnet

2018-02-24 Thread GitBox
gigasquid commented on issue #8971: Clojure Library for mxnet URL: https://github.com/apache/incubator-mxnet/issues/8971#issuecomment-366543008 An update - I've ported over the MnistModule example to clojure - yay! ?

[GitHub] marcoabreu commented on a change in pull request #9810: remove MKL_EXPERIMENTAL and update make files for MKL-DNN

2018-02-24 Thread GitBox
marcoabreu commented on a change in pull request #9810: remove MKL_EXPERIMENTAL and update make files for MKL-DNN URL: https://github.com/apache/incubator-mxnet/pull/9810#discussion_r170436506 ## File path: docker_multiarch/arm.crosscompile.mk ## @@ -115,13 +100,10 @@

[GitHub] marcoabreu commented on a change in pull request #9810: remove MKL_EXPERIMENTAL and update make files for MKL-DNN

2018-02-24 Thread GitBox
marcoabreu commented on a change in pull request #9810: remove MKL_EXPERIMENTAL and update make files for MKL-DNN URL: https://github.com/apache/incubator-mxnet/pull/9810#discussion_r170436502 ## File path: docker_multiarch/arm.crosscompile.mk ## @@ -115,13 +100,10 @@

[GitHub] marcoabreu commented on issue #9677: Refactor operators and add MKLDNN

2018-02-24 Thread GitBox
marcoabreu commented on issue #9677: Refactor operators and add MKLDNN URL: https://github.com/apache/incubator-mxnet/pull/9677#issuecomment-368269119 Could somebody please elaborate why that package is hosted in another repository and thus not part of our PR validation chain?

[GitHub] zheng-da commented on issue #9677: Refactor operators and add MKLDNN

2018-02-24 Thread GitBox
zheng-da commented on issue #9677: Refactor operators and add MKLDNN URL: https://github.com/apache/incubator-mxnet/pull/9677#issuecomment-368254930 I guess the reason is that NNVM handles operator arguments differently? @piiswrong @szha I see mx.sym.Convolution has an argument called

[GitHub] smorrel1 commented on issue #1090: How to get layer shapes

2018-02-24 Thread GitBox
smorrel1 commented on issue #1090: How to get layer shapes URL: https://github.com/apache/incubator-mxnet/issues/1090#issuecomment-368244698 The link above is broken. Use [this

[GitHub] aaronmarkham opened a new pull request #9879: Versions patch

2018-02-24 Thread GitBox
aaronmarkham opened a new pull request #9879: Versions patch URL: https://github.com/apache/incubator-mxnet/pull/9879 ## Description ## This overlays on my previous one and fixes layout issues on the install page. Must merge my other layout PR too. #9718 ## Checklist ## ###

[GitHub] iblis17 commented on issue #9677: Refactor operators and add MKLDNN

2018-02-24 Thread GitBox
iblis17 commented on issue #9677: Refactor operators and add MKLDNN URL: https://github.com/apache/incubator-mxnet/pull/9677#issuecomment-368242469 I built the python one. ```python In [6]: mx.sym.Convolution(attr={'a': 42}) ``` and got similar error message. This PR

[GitHub] iblis17 commented on issue #9677: Refactor operators and add MKLDNN

2018-02-24 Thread GitBox
iblis17 commented on issue #9677: Refactor operators and add MKLDNN URL: https://github.com/apache/incubator-mxnet/pull/9677#issuecomment-368241751 I did bisect. This patch caused MXNet.jl testing failure. (Ref https://travis-ci.org/dmlc/MXNet.jl/jobs/345129890#L1486) The

[GitHub] aaronmarkham opened a new pull request #9878: Docs build all versions refactor

2018-02-24 Thread GitBox
aaronmarkham opened a new pull request #9878: Docs build all versions refactor URL: https://github.com/apache/incubator-mxnet/pull/9878 ## Description ## I updated the build_all_versions.sh logic so that it works locally and can possibly repair the docs site when the version builds

[GitHub] ashokei commented on issue #9810: remove MKL_EXPERIMENTAL and update make files for MKL-DNN

2018-02-24 Thread GitBox
ashokei commented on issue #9810: remove MKL_EXPERIMENTAL and update make files for MKL-DNN URL: https://github.com/apache/incubator-mxnet/pull/9810#issuecomment-368241273 @marcoabreu i made suggested changes...can you please review/merge if ok.

[GitHub] ashokei commented on issue #9810: remove MKL_EXPERIMENTAL and update make files for MKL-DNN

2018-02-24 Thread GitBox
ashokei commented on issue #9810: remove MKL_EXPERIMENTAL and update make files for MKL-DNN URL: https://github.com/apache/incubator-mxnet/pull/9810#issuecomment-368241273 @marcoabreu i made your requested changes...can you please approve/merge if ok.

[GitHub] eric-haibin-lin commented on issue #9848: add infer_type for regression ops, fix issues#9847

2018-02-24 Thread GitBox
eric-haibin-lin commented on issue #9848: add infer_type for regression ops, fix issues#9847 URL: https://github.com/apache/incubator-mxnet/pull/9848#issuecomment-368226704 @piiswrong any special reason for missing default inference function in imperative mode?

[GitHub] ZiyueHuang commented on issue #9848: add infer_type for regression ops, fix issues#9847

2018-02-24 Thread GitBox
ZiyueHuang commented on issue #9848: add infer_type for regression ops, fix issues#9847 URL: https://github.com/apache/incubator-mxnet/pull/9848#issuecomment-368226581 Is this good to merge? @piiswrong This is an automated

[GitHub] eric-haibin-lin commented on a change in pull request #9625: [WIP] sparse regression operators

2018-02-24 Thread GitBox
eric-haibin-lin commented on a change in pull request #9625: [WIP] sparse regression operators URL: https://github.com/apache/incubator-mxnet/pull/9625#discussion_r170420190 ## File path: src/operator/regression_output-inl.h ## @@ -77,22 +78,73 @@ inline bool

[GitHub] zheng-da commented on issue #9868: MKL and CMake

2018-02-24 Thread GitBox
zheng-da commented on issue #9868: MKL and CMake URL: https://github.com/apache/incubator-mxnet/issues/9868#issuecomment-368220757 Ideally, we should use only USE_MKLDNN. But I needed to set up a PR to change mshadow first. Without merging mkldnn, changing mshadow would break

[GitHub] marcoabreu commented on issue #9868: MKL and CMake

2018-02-24 Thread GitBox
marcoabreu commented on issue #9868: MKL and CMake URL: https://github.com/apache/incubator-mxnet/issues/9868#issuecomment-368219084 @zheng-da This is an automated message from the Apache Git Service. To respond to the

[GitHub] dabraude commented on issue #9573: Fix bug with symbol loading

2018-02-24 Thread GitBox
dabraude commented on issue #9573: Fix bug with symbol loading URL: https://github.com/apache/incubator-mxnet/pull/9573#issuecomment-368215088 this has fallen way behind the master I'm going to close this pull request and do the two parts separately