[GitHub] rahul003 commented on issue #10504: Can MXNet operator profiling work well using gluon model?

2018-04-11 Thread GitBox
rahul003 commented on issue #10504: Can MXNet operator profiling work well using gluon model? URL: https://github.com/apache/incubator-mxnet/issues/10504#issuecomment-380686965 Have you tried this ?

[GitHub] jinhuang415 commented on issue #10504: Can MXNet operator profiling work well using gluon model?

2018-04-11 Thread GitBox
jinhuang415 commented on issue #10504: Can MXNet operator profiling work well using gluon model? URL: https://github.com/apache/incubator-mxnet/issues/10504#issuecomment-380686018 @cjolivier01 We noticed there is profiling enhancement recently, may I know how to get below statistics

[GitHub] szha commented on issue #6857: Links on the page with the RNN tutorials are broken

2018-04-11 Thread GitBox
szha commented on issue #6857: Links on the page with the RNN tutorials are broken URL: https://github.com/apache/incubator-mxnet/issues/6857#issuecomment-380684644 that specific tutorial is no longer offered. This is an

[GitHub] szha closed issue #6857: Links on the page with the RNN tutorials are broken

2018-04-11 Thread GitBox
szha closed issue #6857: Links on the page with the RNN tutorials are broken URL: https://github.com/apache/incubator-mxnet/issues/6857 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] sugi1229 opened a new issue #10516: How to run model trained with python on Scala

2018-04-11 Thread GitBox
sugi1229 opened a new issue #10516: How to run model trained with python on Scala URL: https://github.com/apache/incubator-mxnet/issues/10516 I trained model with python. I got x.params and x.json files. However, there files coludn’t load scala. In this case

[GitHub] msurguy commented on issue #10230: libomp.so is missing in Docker builds for armv6

2018-04-11 Thread GitBox
msurguy commented on issue #10230: libomp.so is missing in Docker builds for armv6 URL: https://github.com/apache/incubator-mxnet/issues/10230#issuecomment-380678772 I can confirm that everything works on the Pi Zero now! Thanks @lebeg !!!

[GitHub] ThomasDelteil commented on issue #10508: MXNet much slower than TensorFlow

2018-04-11 Thread GitBox
ThomasDelteil commented on issue #10508: MXNet much slower than TensorFlow URL: https://github.com/apache/incubator-mxnet/issues/10508#issuecomment-380675020 You have an error in your tensorflow code @altosaar You are setting `t0 = time.time()` right before computing: `(time.time() -

[GitHub] ThomasDelteil commented on issue #10508: MXNet much slower than TensorFlow

2018-04-11 Thread GitBox
ThomasDelteil commented on issue #10508: MXNet much slower than TensorFlow URL: https://github.com/apache/incubator-mxnet/issues/10508#issuecomment-380675020 You have an error in your tensorflow code @altosaar You are setting `t0 = time.time()` right before computing: `(time.time() -

[GitHub] ThomasDelteil commented on issue #10508: MXNet much slower than TensorFlow

2018-04-11 Thread GitBox
ThomasDelteil commented on issue #10508: MXNet much slower than TensorFlow URL: https://github.com/apache/incubator-mxnet/issues/10508#issuecomment-380675020 You have an error in your tensorflow code @altosaar You are setting `t0 = time.time()` right before computing: `(time.time() -

[GitHub] mrkn opened a new pull request #10515: Prevent partially update of ParameterDict

2018-04-11 Thread GitBox
mrkn opened a new pull request #10515: Prevent partially update of ParameterDict URL: https://github.com/apache/incubator-mxnet/pull/10515 As you can see in the following execution log, `mxnet.gluon.parameter.ParameterDict#update` method partially updates the receiver when the assertion

[GitHub] pengzhao-intel commented on issue #10508: MXNet much slower than TensorFlow

2018-04-11 Thread GitBox
pengzhao-intel commented on issue #10508: MXNet much slower than TensorFlow URL: https://github.com/apache/incubator-mxnet/issues/10508#issuecomment-380672169 @altosaar you can try CPU with MKL-DNN backend by lastest master branch. I think it will be much faster.

[GitHub] chinakook commented on issue #10508: MXNet much slower than TensorFlow

2018-04-11 Thread GitBox
chinakook commented on issue #10508: MXNet much slower than TensorFlow URL: https://github.com/apache/incubator-mxnet/issues/10508#issuecomment-380673011 MNIST is too small to bench. IO is the main bottleneck. This is an

[GitHub] pengzhao-intel commented on issue #10508: MXNet much slower than TensorFlow

2018-04-11 Thread GitBox
pengzhao-intel commented on issue #10508: MXNet much slower than TensorFlow URL: https://github.com/apache/incubator-mxnet/issues/10508#issuecomment-380672169 @altosaar you can try CPU with MKL-DNN backend by lastest master branch. I think it will be much faster.

[GitHub] idealboy commented on issue #10513: pure virtual function call

2018-04-11 Thread GitBox
idealboy commented on issue #10513: pure virtual function call URL: https://github.com/apache/incubator-mxnet/issues/10513#issuecomment-380672233 (gdb) backtrace #0 0x7f29b8b844da in malloc_consolidate () from /lib64/libc.so.6 #1 0x7f29b8b85087 in _int_free () from

[GitHub] pengzhao-intel commented on issue #10508: MXNet much slower than TensorFlow

2018-04-11 Thread GitBox
pengzhao-intel commented on issue #10508: MXNet much slower than TensorFlow URL: https://github.com/apache/incubator-mxnet/issues/10508#issuecomment-380672169 @altosaar you can try CPU with MKL-DNN backend by lastest master branch. I think it will be much faster.

[GitHub] opringle opened a new pull request #10514: new NER example

2018-04-11 Thread GitBox
opringle opened a new pull request #10514: new NER example URL: https://github.com/apache/incubator-mxnet/pull/10514 ## Description ## Added example of named entity recognition using bucketing, custom metrics etc.

[GitHub] jinhuang415 commented on issue #10504: Can MXNet operator profiling work well using gluon model?

2018-04-11 Thread GitBox
jinhuang415 commented on issue #10504: Can MXNet operator profiling work well using gluon model? URL: https://github.com/apache/incubator-mxnet/issues/10504#issuecomment-380671980 @pengzhao-intel This is an automated

[GitHub] ysfalo commented on issue #9823: RCNN example fails for using latest mxnet

2018-04-11 Thread GitBox
ysfalo commented on issue #9823: RCNN example fails for using latest mxnet URL: https://github.com/apache/incubator-mxnet/issues/9823#issuecomment-380663649 It happened on single GPU by chance in my machine with mxnet 1.2.0.

[GitHub] jinhuang415 commented on issue #10504: Can MXNet operator profiling work well using gluon model?

2018-04-11 Thread GitBox
jinhuang415 commented on issue #10504: Can MXNet operator profiling work well using gluon model? URL: https://github.com/apache/incubator-mxnet/issues/10504#issuecomment-380660550 Btw, is there any way to selectively do the profiling? Like only perform operator profiling, I asked this

[GitHub] jinhuang415 commented on issue #10504: Can MXNet operator profiling work well using gluon model?

2018-04-11 Thread GitBox
jinhuang415 commented on issue #10504: Can MXNet operator profiling work well using gluon model? URL: https://github.com/apache/incubator-mxnet/issues/10504#issuecomment-380661980 For selective profiling, I figured out myself that we need to use

[GitHub] jinhuang415 commented on issue #10504: Can MXNet operator profiling work well using gluon model?

2018-04-11 Thread GitBox
jinhuang415 commented on issue #10504: Can MXNet operator profiling work well using gluon model? URL: https://github.com/apache/incubator-mxnet/issues/10504#issuecomment-380661980 For selective profiling, I figured out myself that we need to use

[GitHub] jinhuang415 commented on issue #10504: Can MXNet operator profiling work well using gluon model?

2018-04-11 Thread GitBox
jinhuang415 commented on issue #10504: Can MXNet operator profiling work well using gluon model? URL: https://github.com/apache/incubator-mxnet/issues/10504#issuecomment-380661980 For selective profiling, I figured out myself that we need to use "mx.profiler.set_config(profile_all=False,

[GitHub] jinhuang415 commented on issue #10504: Can MXNet operator profiling work well using gluon model?

2018-04-11 Thread GitBox
jinhuang415 commented on issue #10504: Can MXNet operator profiling work well using gluon model? URL: https://github.com/apache/incubator-mxnet/issues/10504#issuecomment-380660550 Btw, is there any way to selectively do the profiling? Like only perform operator profiling, I asked this

[GitHub] jinhuang415 commented on issue #10504: Can MXNet operator profiling work well using gluon model?

2018-04-11 Thread GitBox
jinhuang415 commented on issue #10504: Can MXNet operator profiling work well using gluon model? URL: https://github.com/apache/incubator-mxnet/issues/10504#issuecomment-380660550 Btw, is there any way to selectively do the profiling? Like only perform operator profiling, I asked this

[GitHub] aaronmarkham commented on issue #10485: [MXNET-304][RFC] Jenkins docs build

2018-04-11 Thread GitBox
aaronmarkham commented on issue #10485: [MXNET-304][RFC] Jenkins docs build URL: https://github.com/apache/incubator-mxnet/pull/10485#issuecomment-380657955 Are we good on the changes for now? This is an automated message

[GitHub] ThomasDelteil commented on issue #10391: [MXNET-139] Tutorial for mixed precision training with float16

2018-04-11 Thread GitBox
ThomasDelteil commented on issue #10391: [MXNET-139] Tutorial for mixed precision training with float16 URL: https://github.com/apache/incubator-mxnet/pull/10391#issuecomment-380653939 Great tutorial @rahul003 and well documented 

[GitHub] ThomasDelteil commented on a change in pull request #10391: [MXNET-139] Tutorial for mixed precision training with float16

2018-04-11 Thread GitBox
ThomasDelteil commented on a change in pull request #10391: [MXNET-139] Tutorial for mixed precision training with float16 URL: https://github.com/apache/incubator-mxnet/pull/10391#discussion_r18094 ## File path: docs/tutorials/python/float16.md ## @@ -0,0 +1,280 @@

[GitHub] ThomasDelteil commented on a change in pull request #10391: [MXNET-139] Tutorial for mixed precision training with float16

2018-04-11 Thread GitBox
ThomasDelteil commented on a change in pull request #10391: [MXNET-139] Tutorial for mixed precision training with float16 URL: https://github.com/apache/incubator-mxnet/pull/10391#discussion_r180948389 ## File path: docs/tutorials/python/float16.md ## @@ -0,0 +1,280 @@

[GitHub] idealboy opened a new issue #10513: pure virtual function call

2018-04-11 Thread GitBox
idealboy opened a new issue #10513: pure virtual function call URL: https://github.com/apache/incubator-mxnet/issues/10513 Note: Providing complete information in the most concise form is the best way to get help. This issue template serves as the checklist for essential information to

[GitHub] ThomasDelteil commented on a change in pull request #10391: [MXNET-139] Tutorial for mixed precision training with float16

2018-04-11 Thread GitBox
ThomasDelteil commented on a change in pull request #10391: [MXNET-139] Tutorial for mixed precision training with float16 URL: https://github.com/apache/incubator-mxnet/pull/10391#discussion_r180948389 ## File path: docs/tutorials/python/float16.md ## @@ -0,0 +1,280 @@

[GitHub] ThomasDelteil commented on a change in pull request #10391: [MXNET-139] Tutorial for mixed precision training with float16

2018-04-11 Thread GitBox
ThomasDelteil commented on a change in pull request #10391: [MXNET-139] Tutorial for mixed precision training with float16 URL: https://github.com/apache/incubator-mxnet/pull/10391#discussion_r180947924 ## File path: docs/tutorials/python/float16.md ## @@ -0,0 +1,280 @@

[GitHub] ThomasDelteil commented on a change in pull request #10391: [MXNET-139] Tutorial for mixed precision training with float16

2018-04-11 Thread GitBox
ThomasDelteil commented on a change in pull request #10391: [MXNET-139] Tutorial for mixed precision training with float16 URL: https://github.com/apache/incubator-mxnet/pull/10391#discussion_r180947924 ## File path: docs/tutorials/python/float16.md ## @@ -0,0 +1,280 @@

[GitHub] ThomasDelteil commented on a change in pull request #10391: [MXNET-139] Tutorial for mixed precision training with float16

2018-04-11 Thread GitBox
ThomasDelteil commented on a change in pull request #10391: [MXNET-139] Tutorial for mixed precision training with float16 URL: https://github.com/apache/incubator-mxnet/pull/10391#discussion_r180947641 ## File path: docs/tutorials/python/float16.md ## @@ -0,0 +1,280 @@

[GitHub] jinhuang415 commented on issue #10504: Can MXNet operator profiling work well using gluon model?

2018-04-11 Thread GitBox
jinhuang415 commented on issue #10504: Can MXNet operator profiling work well using gluon model? URL: https://github.com/apache/incubator-mxnet/issues/10504#issuecomment-380650540 @rahul003 Thanks, the way you mentioned works. For the environment variable way, do we also need to support

[GitHub] ThomasDelteil commented on issue #10511: add naming tutorial

2018-04-11 Thread GitBox
ThomasDelteil commented on issue #10511: add naming tutorial URL: https://github.com/apache/incubator-mxnet/pull/10511#issuecomment-380650174 Great tutorial, it was much needed and clarifies what is going on under the hood with the naming scopes 

[GitHub] ThomasDelteil commented on a change in pull request #10511: add naming tutorial

2018-04-11 Thread GitBox
ThomasDelteil commented on a change in pull request #10511: add naming tutorial URL: https://github.com/apache/incubator-mxnet/pull/10511#discussion_r180946437 ## File path: docs/tutorials/gluon/naming.md ## @@ -0,0 +1,236 @@ + +# Naming of Gluon Parameter and Blocks + +In

[GitHub] ThomasDelteil commented on a change in pull request #10511: add naming tutorial

2018-04-11 Thread GitBox
ThomasDelteil commented on a change in pull request #10511: add naming tutorial URL: https://github.com/apache/incubator-mxnet/pull/10511#discussion_r180945890 ## File path: docs/tutorials/gluon/naming.md ## @@ -0,0 +1,236 @@ + +# Naming of Gluon Parameter and Blocks + +In

[GitHub] ThomasDelteil commented on a change in pull request #10511: add naming tutorial

2018-04-11 Thread GitBox
ThomasDelteil commented on a change in pull request #10511: add naming tutorial URL: https://github.com/apache/incubator-mxnet/pull/10511#discussion_r180944301 ## File path: docs/tutorials/gluon/naming.md ## @@ -0,0 +1,236 @@ + +# Naming of Gluon Parameter and Blocks + +In

[GitHub] rahul003 commented on issue #10504: Can MXNet operator profiling work well using gluon model?

2018-04-11 Thread GitBox
rahul003 commented on issue #10504: Can MXNet operator profiling work well using gluon model? URL: https://github.com/apache/incubator-mxnet/issues/10504#issuecomment-380647097 Could you use mx.profiler.set_config and set the option profile_all=True, and try again? Note that this is a

[GitHub] ankkhedia commented on a change in pull request #10424: [MXNET-185] Improved error message

2018-04-11 Thread GitBox
ankkhedia commented on a change in pull request #10424: [MXNET-185] Improved error message URL: https://github.com/apache/incubator-mxnet/pull/10424#discussion_r180941675 ## File path: python/mxnet/gluon/block.py ## @@ -520,11 +520,11 @@ def _infer_attrs(self, infer_fn,

[GitHub] piiswrong commented on a change in pull request #10511: add naming tutorial

2018-04-11 Thread GitBox
piiswrong commented on a change in pull request #10511: add naming tutorial URL: https://github.com/apache/incubator-mxnet/pull/10511#discussion_r180939805 ## File path: docs/tutorials/gluon/naming.md ## @@ -0,0 +1,236 @@ + +# Naming of Gluon Parameter and Blocks + +In

[incubator-mxnet] branch master updated: [MXNET-311] change test needs a docker with sudo, hence image changed (#10510)

2018-04-11 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 ceb810c [MXNET-311] change test

[GitHub] marcoabreu closed pull request #10510: [MXNET-311] Change the docker image for Installation Guide Test - needs sudo

2018-04-11 Thread GitBox
marcoabreu closed pull request #10510: [MXNET-311] Change the docker image for Installation Guide Test - needs sudo URL: https://github.com/apache/incubator-mxnet/pull/10510 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below

[GitHub] anirudh2290 commented on issue #10510: [MXNET-311] Change the docker image for Installation Guide Test - needs sudo

2018-04-11 Thread GitBox
anirudh2290 commented on issue #10510: [MXNET-311] Change the docker image for Installation Guide Test - needs sudo URL: https://github.com/apache/incubator-mxnet/pull/10510#issuecomment-380636975 @marcoabreu jira is added. Can we merge this ?

[GitHub] eric-haibin-lin commented on issue #10508: MXNet much slower than TensorFlow

2018-04-11 Thread GitBox
eric-haibin-lin commented on issue #10508: MXNet much slower than TensorFlow URL: https://github.com/apache/incubator-mxnet/issues/10508#issuecomment-380635162 @altosaar Did you have a chance to run the code with mxnet profiler and see which operator is the bottleneck?

[GitHub] marcoabreu commented on a change in pull request #10485: [MXNET-304][RFC] Jenkins docs build

2018-04-11 Thread GitBox
marcoabreu commented on a change in pull request #10485: [MXNET-304][RFC] Jenkins docs build URL: https://github.com/apache/incubator-mxnet/pull/10485#discussion_r180933894 ## File path: docs/build_version_doc/build_all_version.sh ## @@ -59,16 +61,21 @@ for tag in

[GitHub] anirudh2290 commented on issue #10424: Improved error message

2018-04-11 Thread GitBox
anirudh2290 commented on issue #10424: Improved error message URL: https://github.com/apache/incubator-mxnet/pull/10424#issuecomment-380633013 @ankkhedia can you please add jira. This is an automated message from the Apache

[GitHub] aaronmarkham commented on a change in pull request #10485: [MXNET-304][RFC] Jenkins docs build

2018-04-11 Thread GitBox
aaronmarkham commented on a change in pull request #10485: [MXNET-304][RFC] Jenkins docs build URL: https://github.com/apache/incubator-mxnet/pull/10485#discussion_r180932594 ## File path: docs/build_version_doc/build_all_version.sh ## @@ -59,16 +61,21 @@ for tag in

[GitHub] aaronmarkham commented on a change in pull request #10485: [MXNET-304][RFC] Jenkins docs build

2018-04-11 Thread GitBox
aaronmarkham commented on a change in pull request #10485: [MXNET-304][RFC] Jenkins docs build URL: https://github.com/apache/incubator-mxnet/pull/10485#discussion_r180932311 ## File path: docs/build_version_doc/build_all_version.sh ## @@ -59,16 +61,21 @@ for tag in

[GitHub] marcoabreu commented on issue #10485: [MXNET-304][RFC] Jenkins docs build

2018-04-11 Thread GitBox
marcoabreu commented on issue #10485: [MXNET-304][RFC] Jenkins docs build URL: https://github.com/apache/incubator-mxnet/pull/10485#issuecomment-380630913 They'll notice pretty quickly if they removed a dependency because our CI won't work in that case and the PR won't get through :)

[GitHub] zheng-da commented on a change in pull request #10410: Fix output names of nn operators.

2018-04-11 Thread GitBox
zheng-da commented on a change in pull request #10410: Fix output names of nn operators. URL: https://github.com/apache/incubator-mxnet/pull/10410#discussion_r180929651 ## File path: tests/python/unittest/test_operator.py ## @@ -5444,6 +5444,21 @@ def

[GitHub] marcoabreu commented on a change in pull request #10495: [MXNET-307] Add tutorials to the CI + Fix them

2018-04-11 Thread GitBox
marcoabreu commented on a change in pull request #10495: [MXNET-307] Add tutorials to the CI + Fix them URL: https://github.com/apache/incubator-mxnet/pull/10495#discussion_r180929532 ## File path: tests/nightly/test_tutorial.py ## @@ -25,87 +25,116 @@ import os import

[GitHub] aaronmarkham commented on issue #10485: [MXNET-304][RFC] Jenkins docs build

2018-04-11 Thread GitBox
aaronmarkham commented on issue #10485: [MXNET-304][RFC] Jenkins docs build URL: https://github.com/apache/incubator-mxnet/pull/10485#issuecomment-380627600 Currently testing the reorg of the install scripts based on the comments. 1. Removed `libatlas-base-dev` from caffe and moved

[GitHub] marcoabreu commented on issue #10485: [MXNET-304][RFC] Jenkins docs build

2018-04-11 Thread GitBox
marcoabreu commented on issue #10485: [MXNET-304][RFC] Jenkins docs build URL: https://github.com/apache/incubator-mxnet/pull/10485#issuecomment-380627272 For the ```ln -s /usr/lib/libopenblas.so /usr/lib/libcblas.so``` error. You can remove that line since cblas is being installed with

[GitHub] marcoabreu commented on issue #10485: [MXNET-304][RFC] Jenkins docs build

2018-04-11 Thread GitBox
marcoabreu commented on issue #10485: [MXNET-304][RFC] Jenkins docs build URL: https://github.com/apache/incubator-mxnet/pull/10485#issuecomment-380627075 Thanks a lot for the quick iteration, Aaron! Would you mind also changing the main Jenkinsfiles doc generation stage to use your

[GitHub] marcoabreu commented on a change in pull request #10485: [MXNET-304][RFC] Jenkins docs build

2018-04-11 Thread GitBox
marcoabreu commented on a change in pull request #10485: [MXNET-304][RFC] Jenkins docs build URL: https://github.com/apache/incubator-mxnet/pull/10485#discussion_r180927140 ## File path: docs/build_version_doc/build_all_version.sh ## @@ -59,16 +61,21 @@ for tag in

[GitHub] marcoabreu commented on a change in pull request #10485: [MXNET-304][RFC] Jenkins docs build

2018-04-11 Thread GitBox
marcoabreu commented on a change in pull request #10485: [MXNET-304][RFC] Jenkins docs build URL: https://github.com/apache/incubator-mxnet/pull/10485#discussion_r180926919 ## File path: ci/docker/install/ubuntu_docs.sh ## @@ -21,8 +21,21 @@ # the whole docker cache for

[GitHub] marcoabreu commented on issue #10504: Can MXNet operator profiling work well using gluon model?

2018-04-11 Thread GitBox
marcoabreu commented on issue #10504: Can MXNet operator profiling work well using gluon model? URL: https://github.com/apache/incubator-mxnet/issues/10504#issuecomment-380625802 @KellenSunderland This is an automated

[GitHub] marcoabreu commented on issue #10508: MXNet much slower than TensorFlow

2018-04-11 Thread GitBox
marcoabreu commented on issue #10508: MXNet much slower than TensorFlow URL: https://github.com/apache/incubator-mxnet/issues/10508#issuecomment-380625348 Hello @altosaar, thanks for your benchmark. Could you please add your compile configuration?

[GitHub] marcoabreu commented on issue #10508: MXNet much slower than TensorFlow

2018-04-11 Thread GitBox
marcoabreu commented on issue #10508: MXNet much slower than TensorFlow URL: https://github.com/apache/incubator-mxnet/issues/10508#issuecomment-380625431 @eric-haibin-lin This is an automated message from the Apache Git

[GitHub] pengzhao-intel commented on issue #10365: [MXNET-261]Update MKLDNN & Add CPP Test

2018-04-11 Thread GitBox
pengzhao-intel commented on issue #10365: [MXNET-261]Update MKLDNN & Add CPP Test URL: https://github.com/apache/incubator-mxnet/pull/10365#issuecomment-380624774 ping @nihui This is an automated message from the Apache Git

[GitHub] marcoabreu commented on issue #10507: Fix infer storage type

2018-04-11 Thread GitBox
marcoabreu commented on issue #10507: Fix infer storage type URL: https://github.com/apache/incubator-mxnet/pull/10507#issuecomment-380624185 Thanks for addressing this so quickly! This is an automated message from the Apache

[GitHub] eric-haibin-lin closed pull request #10488: [MXNET-305] Scala tutorial table fix

2018-04-11 Thread GitBox
eric-haibin-lin closed pull request #10488: [MXNET-305] Scala tutorial table fix URL: https://github.com/apache/incubator-mxnet/pull/10488 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this

[incubator-mxnet] branch master updated: [MXNET-305] Scala tutorial table fix (#10488)

2018-04-11 Thread haibin
This is an automated email from the ASF dual-hosted git repository. haibin 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 ee6dac7 [MXNET-305] Scala tutorial

[GitHub] anirudh2290 commented on issue #10503: CustomOp error with latest master

2018-04-11 Thread GitBox
anirudh2290 commented on issue #10503: CustomOp error with latest master URL: https://github.com/apache/incubator-mxnet/issues/10503#issuecomment-380622555 @fhieber the fix is currently merged. Can you please check if this issue is good to close ?

[GitHub] marcoabreu commented on a change in pull request #10485: [MXNET-304][RFC] Jenkins docs build

2018-04-11 Thread GitBox
marcoabreu commented on a change in pull request #10485: [MXNET-304][RFC] Jenkins docs build URL: https://github.com/apache/incubator-mxnet/pull/10485#discussion_r180921222 ## File path: ci/docker/install/ubuntu_docs.sh ## @@ -21,8 +21,44 @@ # the whole docker cache for

[GitHub] marcoabreu commented on a change in pull request #10485: [MXNET-304][RFC] Jenkins docs build

2018-04-11 Thread GitBox
marcoabreu commented on a change in pull request #10485: [MXNET-304][RFC] Jenkins docs build URL: https://github.com/apache/incubator-mxnet/pull/10485#discussion_r180921222 ## File path: ci/docker/install/ubuntu_docs.sh ## @@ -21,8 +21,44 @@ # the whole docker cache for

[GitHub] marcoabreu commented on a change in pull request #10485: [MXNET-304][RFC] Jenkins docs build

2018-04-11 Thread GitBox
marcoabreu commented on a change in pull request #10485: [MXNET-304][RFC] Jenkins docs build URL: https://github.com/apache/incubator-mxnet/pull/10485#discussion_r180921072 ## File path: ci/docker/install/ubuntu_docs.sh ## @@ -21,8 +21,44 @@ # the whole docker cache for

[GitHub] marcoabreu commented on issue #10495: [MXNET-307] Add tutorials to the CI + Fix them

2018-04-11 Thread GitBox
marcoabreu commented on issue #10495: [MXNET-307] Add tutorials to the CI + Fix them URL: https://github.com/apache/incubator-mxnet/pull/10495#issuecomment-380618897 Ah I see, I'll introduce it to you in our meeting tomorrow :) In them meantime, feel free to have a look at

[GitHub] marcoabreu commented on issue #10510: Change the docker image for Installation Guide Test - needs sudo

2018-04-11 Thread GitBox
marcoabreu commented on issue #10510: Change the docker image for Installation Guide Test - needs sudo URL: https://github.com/apache/incubator-mxnet/pull/10510#issuecomment-380617930 Alrighty, fine with me :) Could you please add a jira ticket before we merge?

[GitHub] msurguy commented on issue #10469: [MXNET-286] Removed OpenMP from armv6 builds

2018-04-11 Thread GitBox
msurguy commented on issue #10469: [MXNET-286] Removed OpenMP from armv6 builds URL: https://github.com/apache/incubator-mxnet/pull/10469#issuecomment-380617845 Thank you @lebeg !!! This is an automated message from the

[GitHub] msurguy commented on issue #10439: [MXNET-287] ARMv6 build with 8-10 times bigger file size

2018-04-11 Thread GitBox
msurguy commented on issue #10439: [MXNET-287] ARMv6 build with 8-10 times bigger file size URL: https://github.com/apache/incubator-mxnet/pull/10439#issuecomment-380617574 Can confirm that the size went down a lot and even more after libomp was removed:

[GitHub] anirudh2290 commented on issue #10488: [MXNET-305] Scala tutorial table fix

2018-04-11 Thread GitBox
anirudh2290 commented on issue #10488: [MXNET-305] Scala tutorial table fix URL: https://github.com/apache/incubator-mxnet/pull/10488#issuecomment-380617326 Sorry, my bad. I see it is fixed in the latest commit. @eric-haibin-lin does this look good to merge ?

[GitHub] ThomasDelteil commented on issue #10512: [MXNET-309] [ONNX-MXNet] Model Metadata API

2018-04-11 Thread GitBox
ThomasDelteil commented on issue #10512: [MXNET-309] [ONNX-MXNet] Model Metadata API URL: https://github.com/apache/incubator-mxnet/pull/10512#issuecomment-380616005 @anirudhacharya not sure, currently it looks like to find the input name you are doing this: ``` data_names =

[GitHub] anirudh2290 commented on issue #10424: Improved error message

2018-04-11 Thread GitBox
anirudh2290 commented on issue #10424: Improved error message URL: https://github.com/apache/incubator-mxnet/pull/10424#issuecomment-380615582 @piiswrong can this be merged ? This is an automated message from the Apache Git

[GitHub] anirudh2290 commented on issue #10488: [MXNET-305] Scala tutorial table fix

2018-04-11 Thread GitBox
anirudh2290 commented on issue #10488: [MXNET-305] Scala tutorial table fix URL: https://github.com/apache/incubator-mxnet/pull/10488#issuecomment-380615287 @aaronmarkham can you also address the image rendering after "Run the Hello World App:" as pointed out by @eric-haibin-lin

[GitHub] yzhliu closed issue #9109: How can I best add support for complex numbers?

2018-04-11 Thread GitBox
yzhliu closed issue #9109: How can I best add support for complex numbers? URL: https://github.com/apache/incubator-mxnet/issues/9109 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] yzhliu commented on issue #9109: How can I best add support for complex numbers?

2018-04-11 Thread GitBox
yzhliu commented on issue #9109: How can I best add support for complex numbers? URL: https://github.com/apache/incubator-mxnet/issues/9109#issuecomment-380614724 Close for now and feel free to reopen if you want to discuss more.

[GitHub] yzhliu commented on issue #9109: How can I best add support for complex numbers?

2018-04-11 Thread GitBox
yzhliu commented on issue #9109: How can I best add support for complex numbers? URL: https://github.com/apache/incubator-mxnet/issues/9109#issuecomment-380614563 I would say it is not easy to add complex number support. basically you need to define how to do + - * / ... for the new type

[GitHub] indhub closed issue #6915: Training Faster R-CNN on own dataset

2018-04-11 Thread GitBox
indhub closed issue #6915: Training Faster R-CNN on own dataset URL: https://github.com/apache/incubator-mxnet/issues/6915 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] indhub commented on issue #6915: Training Faster R-CNN on own dataset

2018-04-11 Thread GitBox
indhub commented on issue #6915: Training Faster R-CNN on own dataset URL: https://github.com/apache/incubator-mxnet/issues/6915#issuecomment-380613956 Closing this since the requester hasn't responded. Please use the [user forum](https://discuss.mxnet.io/) for quicker responses for

[GitHub] mbaijal commented on issue #10510: Change the docker image for Installation Guide Test - needs sudo

2018-04-11 Thread GitBox
mbaijal commented on issue #10510: Change the docker image for Installation Guide Test - needs sudo URL: https://github.com/apache/incubator-mxnet/pull/10510#issuecomment-380613740 Correct, it is far from ideal. But hoping that this CI setup will be deprecated soon and 1.2 is the last

[GitHub] anirudhacharya opened a new pull request #10512: [MXNET-309] [ONNX-MXNet] Model Metadata API

2018-04-11 Thread GitBox
anirudhacharya opened a new pull request #10512: [MXNET-309] [ONNX-MXNet] Model Metadata API URL: https://github.com/apache/incubator-mxnet/pull/10512 ## Description ## A new API in the onnx module to fetch shape and name information of a model. ## Checklist ## ### Essentials

[GitHub] ThomasDelteil commented on issue #10495: [MXNET-307] Add tutorials to the CI + Fix them

2018-04-11 Thread GitBox
ThomasDelteil commented on issue #10495: [MXNET-307] Add tutorials to the CI + Fix them URL: https://github.com/apache/incubator-mxnet/pull/10495#issuecomment-380613367 @marcoabreu I have simply updated the existing test. Not sure why it was done like that in the first place and not

[GitHub] indhub commented on issue #7912: Do I need to change grad_req when sharing weights?

2018-04-11 Thread GitBox
indhub commented on issue #7912: Do I need to change grad_req when sharing weights? URL: https://github.com/apache/incubator-mxnet/issues/7912#issuecomment-380613087 Answered above. This is an automated message from the

[GitHub] ThomasDelteil commented on a change in pull request #10495: [MXNET-307] Add tutorials to the CI + Fix them

2018-04-11 Thread GitBox
ThomasDelteil commented on a change in pull request #10495: [MXNET-307] Add tutorials to the CI + Fix them URL: https://github.com/apache/incubator-mxnet/pull/10495#discussion_r180914315 ## File path: tests/nightly/test_tutorial.py ## @@ -25,87 +25,116 @@ import os

[GitHub] indhub closed issue #7912: Do I need to change grad_req when sharing weights?

2018-04-11 Thread GitBox
indhub closed issue #7912: Do I need to change grad_req when sharing weights? URL: https://github.com/apache/incubator-mxnet/issues/7912 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] ThomasDelteil commented on a change in pull request #10495: [MXNET-307] Add tutorials to the CI + Fix them

2018-04-11 Thread GitBox
ThomasDelteil commented on a change in pull request #10495: [MXNET-307] Add tutorials to the CI + Fix them URL: https://github.com/apache/incubator-mxnet/pull/10495#discussion_r180914065 ## File path: tests/nightly/test_tutorial.py ## @@ -25,87 +25,116 @@ import os

[GitHub] marcoabreu commented on issue #10510: Change the docker image for Installation Guide Test - needs sudo

2018-04-11 Thread GitBox
marcoabreu commented on issue #10510: Change the docker image for Installation Guide Test - needs sudo URL: https://github.com/apache/incubator-mxnet/pull/10510#issuecomment-380612813 I see, thanks for elaborating. So just to clarify: This means if a new slave for the internal CI gets

[GitHub] anirudh2290 commented on a change in pull request #10511: add naming tutorial

2018-04-11 Thread GitBox
anirudh2290 commented on a change in pull request #10511: add naming tutorial URL: https://github.com/apache/incubator-mxnet/pull/10511#discussion_r180910434 ## File path: docs/tutorials/gluon/naming.md ## @@ -0,0 +1,236 @@ + +# Naming of Gluon Parameter and Blocks + +In

[GitHub] anirudh2290 commented on a change in pull request #10511: add naming tutorial

2018-04-11 Thread GitBox
anirudh2290 commented on a change in pull request #10511: add naming tutorial URL: https://github.com/apache/incubator-mxnet/pull/10511#discussion_r180906260 ## File path: docs/tutorials/gluon/naming.md ## @@ -0,0 +1,236 @@ + +# Naming of Gluon Parameter and Blocks + +In

[GitHub] anirudh2290 commented on a change in pull request #10511: add naming tutorial

2018-04-11 Thread GitBox
anirudh2290 commented on a change in pull request #10511: add naming tutorial URL: https://github.com/apache/incubator-mxnet/pull/10511#discussion_r180904852 ## File path: docs/tutorials/gluon/naming.md ## @@ -0,0 +1,236 @@ + +# Naming of Gluon Parameter and Blocks + +In

[GitHub] mbaijal commented on issue #10510: Change the docker image for Installation Guide Test - needs sudo

2018-04-11 Thread GitBox
mbaijal commented on issue #10510: Change the docker image for Installation Guide Test - needs sudo URL: https://github.com/apache/incubator-mxnet/pull/10510#issuecomment-380612454 I will add a dockerfile to the repo for this test but only during migration. Doing so now would require

[incubator-mxnet] branch master updated: Minor simplifications in ci/build.py (#10496)

2018-04-11 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 8097059 Minor simplifications in

[GitHub] marcoabreu commented on issue #10496: Minor simplifications in ci/build.py

2018-04-11 Thread GitBox
marcoabreu commented on issue #10496: Minor simplifications in ci/build.py URL: https://github.com/apache/incubator-mxnet/pull/10496#issuecomment-380612160 Thanks for the modifications :) This is an automated message from the

[GitHub] marcoabreu closed pull request #10496: Minor simplifications in ci/build.py

2018-04-11 Thread GitBox
marcoabreu closed pull request #10496: Minor simplifications in ci/build.py URL: https://github.com/apache/incubator-mxnet/pull/10496 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] marcoabreu commented on a change in pull request #10495: [MXNET-307] Add tutorials to the CI + Fix them

2018-04-11 Thread GitBox
marcoabreu commented on a change in pull request #10495: [MXNET-307] Add tutorials to the CI + Fix them URL: https://github.com/apache/incubator-mxnet/pull/10495#discussion_r180913316 ## File path: tests/nightly/test_tutorial.py ## @@ -25,87 +25,116 @@ import os import

[GitHub] marcoabreu commented on a change in pull request #10495: [MXNET-307] Add tutorials to the CI + Fix them

2018-04-11 Thread GitBox
marcoabreu commented on a change in pull request #10495: [MXNET-307] Add tutorials to the CI + Fix them URL: https://github.com/apache/incubator-mxnet/pull/10495#discussion_r180913205 ## File path: tests/nightly/test_tutorial.py ## @@ -25,87 +25,116 @@ import os import

[GitHub] marcoabreu commented on issue #10495: [MXNET-307] Add tutorials to the CI + Fix them

2018-04-11 Thread GitBox
marcoabreu commented on issue #10495: [MXNET-307] Add tutorials to the CI + Fix them URL: https://github.com/apache/incubator-mxnet/pull/10495#issuecomment-380610987 Thanks a lot for improving the test coverage for tutorials, that's great! One quick question: Why did you write your

[GitHub] mbaijal commented on issue #10510: Change the docker image for Installation Guide Test - needs sudo

2018-04-11 Thread GitBox
mbaijal commented on issue #10510: Change the docker image for Installation Guide Test - needs sudo URL: https://github.com/apache/incubator-mxnet/pull/10510#issuecomment-380610626 Currently, this change is to run the installation guide test locally for the 1.2 release. Yes as you

  1   2   3   >