[GitHub] reminisce commented on issue #9997: Flaky test: test_sparse_embedding

2018-03-08 Thread GitBox
reminisce commented on issue #9997: Flaky test: test_sparse_embedding URL: https://github.com/apache/incubator-mxnet/issues/9997#issuecomment-371739006 The `atol` cannot be zero here. When comparing two values, the following inequality is checked: `|a-b| < atol + b * rtol`. In this

[GitHub] reminisce commented on issue #9916: precision bug in batchnorm.

2018-03-08 Thread GitBox
reminisce commented on issue #9916: precision bug in batchnorm. URL: https://github.com/apache/incubator-mxnet/issues/9916#issuecomment-371736378 After some investigation, I think it's due to the low precision that FP16 has. When doing batch norm backward pass, gradients of `gamma`

[GitHub] Jerryzcn commented on a change in pull request #9977: Cpu lstm inference

2018-03-08 Thread GitBox
Jerryzcn commented on a change in pull request #9977: Cpu lstm inference URL: https://github.com/apache/incubator-mxnet/pull/9977#discussion_r173381025 ## File path: tests/python/unittest/test_gluon_rnn.py ## @@ -67,6 +67,22 @@ def test_lstm_forget_bias():

[GitHub] Jerryzcn commented on a change in pull request #9977: Cpu lstm inference

2018-03-08 Thread GitBox
Jerryzcn commented on a change in pull request #9977: Cpu lstm inference URL: https://github.com/apache/incubator-mxnet/pull/9977#discussion_r173381025 ## File path: tests/python/unittest/test_gluon_rnn.py ## @@ -67,6 +67,22 @@ def test_lstm_forget_bias():

[GitHub] EternalSaga opened a new issue #10049: Building MxNet on Win10 with CPP package, CUDA9 and MKL 2018

2018-03-08 Thread GitBox
EternalSaga opened a new issue #10049: Building MxNet on Win10 with CPP package, CUDA9 and MKL 2018 URL: https://github.com/apache/incubator-mxnet/issues/10049 Note: Providing complete information in the most concise form is the best way to get help. This issue template serves as the

[GitHub] szha commented on a change in pull request #9977: Cpu lstm inference

2018-03-08 Thread GitBox
szha commented on a change in pull request #9977: Cpu lstm inference URL: https://github.com/apache/incubator-mxnet/pull/9977#discussion_r173372031 ## File path: src/operator/rnn-inl.h ## @@ -144,15 +149,223 @@ class RNNOp : public Operator { const

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

2018-03-08 Thread GitBox
eric-haibin-lin commented on issue #9921: [DISCUSSION] module.contrib.SparseModule API URL: https://github.com/apache/incubator-mxnet/issues/9921#issuecomment-371722591 @zheng-da Thanks for the comments! For the first point, however, the graph_executor/operators are not aware of the

[GitHub] szha commented on a change in pull request #9977: Cpu lstm inference

2018-03-08 Thread GitBox
szha commented on a change in pull request #9977: Cpu lstm inference URL: https://github.com/apache/incubator-mxnet/pull/9977#discussion_r173371568 ## File path: tests/python/unittest/test_gluon_rnn.py ## @@ -67,6 +67,22 @@ def test_lstm_forget_bias():

[GitHub] Sherry-Zhang commented on a change in pull request #9977: Cpu lstm inference

2018-03-08 Thread GitBox
Sherry-Zhang commented on a change in pull request #9977: Cpu lstm inference URL: https://github.com/apache/incubator-mxnet/pull/9977#discussion_r173371417 ## File path: src/operator/rnn-inl.h ## @@ -114,7 +120,8 @@ struct RNNParam : public dmlc::Parameter {

[GitHub] szha commented on a change in pull request #9995: [WIP] CI docker revamp; Add Jetson, Raspberry and CentOS 7 build [MXNET-42][MXNET-43][MXNET-44][MXNET-57]

2018-03-08 Thread GitBox
szha commented on a change in pull request #9995: [WIP] CI docker revamp; Add Jetson, Raspberry and CentOS 7 build [MXNET-42][MXNET-43][MXNET-44][MXNET-57] URL: https://github.com/apache/incubator-mxnet/pull/9995#discussion_r173371249 ## File path:

[GitHub] szha commented on a change in pull request #9995: [WIP] CI docker revamp; Add Jetson, Raspberry and CentOS 7 build [MXNET-42][MXNET-43][MXNET-44][MXNET-57]

2018-03-08 Thread GitBox
szha commented on a change in pull request #9995: [WIP] CI docker revamp; Add Jetson, Raspberry and CentOS 7 build [MXNET-42][MXNET-43][MXNET-44][MXNET-57] URL: https://github.com/apache/incubator-mxnet/pull/9995#discussion_r173370939 ## File path:

[GitHub] TaoLv commented on a change in pull request #9977: Cpu lstm inference

2018-03-08 Thread GitBox
TaoLv commented on a change in pull request #9977: Cpu lstm inference URL: https://github.com/apache/incubator-mxnet/pull/9977#discussion_r173370569 ## File path: src/operator/rnn-inl.h ## @@ -144,15 +149,223 @@ class RNNOp : public Operator {

[GitHub] TaoLv commented on a change in pull request #9977: Cpu lstm inference

2018-03-08 Thread GitBox
TaoLv commented on a change in pull request #9977: Cpu lstm inference URL: https://github.com/apache/incubator-mxnet/pull/9977#discussion_r173370326 ## File path: src/operator/rnn-inl.h ## @@ -114,7 +120,8 @@ struct RNNParam : public dmlc::Parameter {

[GitHub] eric-haibin-lin commented on a change in pull request #9947: [WIP] Performance optimization for dot(csr, rsp) on cpu

2018-03-08 Thread GitBox
eric-haibin-lin commented on a change in pull request #9947: [WIP] Performance optimization for dot(csr, rsp) on cpu URL: https://github.com/apache/incubator-mxnet/pull/9947#discussion_r173369215 ## File path: src/operator/tensor/dot-inl.h ## @@ -793,16 +764,21 @@ inline

[GitHub] szha commented on a change in pull request #9995: [WIP] CI docker revamp; Add Jetson, Raspberry and CentOS 7 build [MXNET-42][MXNET-43][MXNET-44][MXNET-57]

2018-03-08 Thread GitBox
szha commented on a change in pull request #9995: [WIP] CI docker revamp; Add Jetson, Raspberry and CentOS 7 build [MXNET-42][MXNET-43][MXNET-44][MXNET-57] URL: https://github.com/apache/incubator-mxnet/pull/9995#discussion_r173367898 ## File path:

[GitHub] szha commented on a change in pull request #9995: [WIP] CI docker revamp; Add Jetson, Raspberry and CentOS 7 build [MXNET-42][MXNET-43][MXNET-44][MXNET-57]

2018-03-08 Thread GitBox
szha commented on a change in pull request #9995: [WIP] CI docker revamp; Add Jetson, Raspberry and CentOS 7 build [MXNET-42][MXNET-43][MXNET-44][MXNET-57] URL: https://github.com/apache/incubator-mxnet/pull/9995#discussion_r173367713 ## File path: ci/build.py ## @@ -0,0

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

2018-03-08 Thread GitBox
wkcn commented on issue #9939: [MXNET-40]add multi proposal operator (cpu version) and fix the bug in proposal op (gpu version) URL: https://github.com/apache/incubator-mxnet/pull/9939#issuecomment-370285916 @pengzhao-intel @xinyu-intel Thank you! I will have a try. The performance

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

2018-03-08 Thread GitBox
wkcn commented on issue #9939: [MXNET-40]add multi proposal operator (cpu version) and fix the bug in proposal op (gpu version) URL: https://github.com/apache/incubator-mxnet/pull/9939#issuecomment-370285916 @pengzhao-intel @xinyu-intel Thank you! I will have a try. The performance

[GitHub] asitstands opened a new pull request #10048: Random shuffle implementation

2018-03-08 Thread GitBox
asitstands opened a new pull request #10048: Random shuffle implementation URL: https://github.com/apache/incubator-mxnet/pull/10048 ## Description ## This operator randomly shuffles an NDArray along the first axis. The order of the elements in each subarray does not change. For

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

2018-03-08 Thread GitBox
wkcn commented on issue #9939: [MXNET-40]add multi proposal operator (cpu version) and fix the bug in proposal op (gpu version) URL: https://github.com/apache/incubator-mxnet/pull/9939#issuecomment-370133350 @pengzhao-intel Here is the testing code.

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

2018-03-08 Thread GitBox
wkcn commented on issue #9939: [MXNET-40]add multi proposal operator (cpu version) and fix the bug in proposal op (gpu version) URL: https://github.com/apache/incubator-mxnet/pull/9939#issuecomment-370133350 @pengzhao-intel Here is the testing code.

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

2018-03-08 Thread GitBox
zihaolucky commented on issue #9195: [WIP]NCE loss gluon URL: https://github.com/apache/incubator-mxnet/pull/9195#issuecomment-371705097 Close this PR and create another one. This is an automated message from the Apache Git

[GitHub] zihaolucky closed pull request #9195: [WIP]NCE loss gluon

2018-03-08 Thread GitBox
zihaolucky closed pull request #9195: [WIP]NCE loss gluon URL: https://github.com/apache/incubator-mxnet/pull/9195 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request

[GitHub] chinakook opened a new issue #10047: Can "NHWC" layout support come back?

2018-03-08 Thread GitBox
chinakook opened a new issue #10047: Can "NHWC" layout support come back? URL: https://github.com/apache/incubator-mxnet/issues/10047 MXNetError: [11:16:26] src/operator/nn/./convolution-inl.h:163: Check failed: param_.layout.value() == mshadow::kNCW || param_.layout.value() ==

[GitHub] cjolivier01 commented on a change in pull request #9958: Parallelization for ROIpooling OP

2018-03-08 Thread GitBox
cjolivier01 commented on a change in pull request #9958: Parallelization for ROIpooling OP URL: https://github.com/apache/incubator-mxnet/pull/9958#discussion_r173356859 ## File path: src/operator/roi_pooling.cc ## @@ -74,7 +79,13 @@ inline void ROIPoolForward(const

[GitHub] reminisce commented on a change in pull request #9947: [WIP] Performance optimization for dot(csr, rsp) on cpu

2018-03-08 Thread GitBox
reminisce commented on a change in pull request #9947: [WIP] Performance optimization for dot(csr, rsp) on cpu URL: https://github.com/apache/incubator-mxnet/pull/9947#discussion_r173354681 ## File path: src/operator/tensor/dot-inl.h ## @@ -793,16 +764,21 @@ inline void

[GitHub] zhaodongsun opened a new issue #10046: mx.nd.UpSampling issue

2018-03-08 Thread GitBox
zhaodongsun opened a new issue #10046: mx.nd.UpSampling issue URL: https://github.com/apache/incubator-mxnet/issues/10046 An error occurs when I run this code. `a=mx.nd.random.uniform(shape=(4,3,2,2)) ` `b=mx.nd.UpSampling(a,scale=2,sample_type='bilinear')` MXNetError:

[incubator-mxnet] branch nlp_toolkit updated (329acde -> a61fd99)

2018-03-08 Thread zhasheng
This is an automated email from the ASF dual-hosted git repository. zhasheng pushed a change to branch nlp_toolkit in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. discard 329acde gluon language modeling dataset and text token reader (#9986) new a61fd99 gluon

[incubator-mxnet] 01/01: gluon language modeling dataset and text token reader (#9986)

2018-03-08 Thread zhasheng
This is an automated email from the ASF dual-hosted git repository. zhasheng pushed a commit to branch nlp_toolkit in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git commit a61fd99b44318aa1ed5a43ca91d666e4c0134d15 Author: Sheng Zha AuthorDate:

[GitHub] wangx404 commented on issue #10019: ModuleNotFoundError

2018-03-08 Thread GitBox
wangx404 commented on issue #10019: ModuleNotFoundError URL: https://github.com/apache/incubator-mxnet/issues/10019#issuecomment-371692811 3Q. I think this may be the reason. I have forgotten mxnet using C code to spped up.

[GitHub] yuewu001 commented on issue #9557: update_on_kvstore error setting with multiple machines

2018-03-08 Thread GitBox
yuewu001 commented on issue #9557: update_on_kvstore error setting with multiple machines URL: https://github.com/apache/incubator-mxnet/issues/9557#issuecomment-371690862 Thank you for your reply. Do you have any ideas on how to save_optimizer_states property in distributed training,

[GitHub] pengzhao-intel commented on issue #9958: Parallelization for ROIpooling OP

2018-03-08 Thread GitBox
pengzhao-intel commented on issue #9958: Parallelization for ROIpooling OP URL: https://github.com/apache/incubator-mxnet/pull/9958#issuecomment-371685877 @cjolivier01 Could you help review again? Our customer requested this features and want to get the usable code from official repo

[incubator-mxnet] branch master updated: Parallelization for ROIpooling OP (#9958)

2018-03-08 Thread cjolivier01
This is an automated email from the ASF dual-hosted git repository. cjolivier01 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 884408a Parallelization for

[GitHub] cjolivier01 closed pull request #9958: Parallelization for ROIpooling OP

2018-03-08 Thread GitBox
cjolivier01 closed pull request #9958: Parallelization for ROIpooling OP URL: https://github.com/apache/incubator-mxnet/pull/9958 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] cjolivier01 commented on issue #9958: Parallelization for ROIpooling OP

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

[GitHub] cjolivier01 commented on a change in pull request #9958: Parallelization for ROIpooling OP

2018-03-08 Thread GitBox
cjolivier01 commented on a change in pull request #9958: Parallelization for ROIpooling OP URL: https://github.com/apache/incubator-mxnet/pull/9958#discussion_r173347236 ## File path: src/operator/roi_pooling.cc ## @@ -74,7 +79,13 @@ inline void ROIPoolForward(const

[GitHub] pengzhao-intel commented on issue #9958: Parallelization for ROIpooling OP

2018-03-08 Thread GitBox
pengzhao-intel commented on issue #9958: Parallelization for ROIpooling OP URL: https://github.com/apache/incubator-mxnet/pull/9958#issuecomment-371685877 @cjolivier01 Could you help review again? Our customer requested this features and want to get the usable code from official repo

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

2018-03-08 Thread GitBox
wkcn commented on issue #9939: [MXNET-40]add multi proposal operator (cpu version) and fix the bug in proposal op (gpu version) URL: https://github.com/apache/incubator-mxnet/pull/9939#issuecomment-370285916 @pengzhao-intel @xinyu-intel Thank you! I will have a try. The performance

[GitHub] zheng-da commented on a change in pull request #9947: [WIP] Performance optimization for dot(csr, rsp) on cpu

2018-03-08 Thread GitBox
zheng-da commented on a change in pull request #9947: [WIP] Performance optimization for dot(csr, rsp) on cpu URL: https://github.com/apache/incubator-mxnet/pull/9947#discussion_r173340887 ## File path: src/operator/tensor/dot-inl.h ## @@ -793,16 +764,21 @@ inline void

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

2018-03-08 Thread GitBox
wkcn commented on issue #9939: [MXNET-40]add multi proposal operator (cpu version) and fix the bug in proposal op (gpu version) URL: https://github.com/apache/incubator-mxnet/pull/9939#issuecomment-370285916 @pengzhao-intel @xinyu-intel Thank you! I will have a try. The performance

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

2018-03-08 Thread GitBox
wkcn commented on issue #9939: [MXNET-40]add multi proposal operator (cpu version) and fix the bug in proposal op (gpu version) URL: https://github.com/apache/incubator-mxnet/pull/9939#issuecomment-370285916 @pengzhao-intel @xinyu-intel Thank you! I will have a try. The performance

[GitHub] pengzhao-intel commented on a change in pull request #10025: Language model with Google's billion words dataset

2018-03-08 Thread GitBox
pengzhao-intel commented on a change in pull request #10025: Language model with Google's billion words dataset URL: https://github.com/apache/incubator-mxnet/pull/10025#discussion_r173338682 ## File path: src/operator/nn/fully_connected.cc ## @@ -73,22 +76,47 @@ static

[GitHub] sxjscience commented on issue #10025: Language model with Google's billion words dataset

2018-03-08 Thread GitBox
sxjscience commented on issue #10025: Language model with Google's billion words dataset URL: https://github.com/apache/incubator-mxnet/pull/10025#issuecomment-371674634 @eric-haibin-lin The RNN.unroll is just updated to support variable length. Need to resolve the conflict.

[GitHub] anirudhacharya commented on a change in pull request #9963: [MXNET-34] Onnx Module to import onnx models into mxnet

2018-03-08 Thread GitBox
anirudhacharya commented on a change in pull request #9963: [MXNET-34] Onnx Module to import onnx models into mxnet URL: https://github.com/apache/incubator-mxnet/pull/9963#discussion_r173336298 ## File path: python/mxnet/contrib/__init__.py ## @@ -28,5 +28,5 @@ from .

[GitHub] anirudhacharya commented on a change in pull request #9963: [MXNET-34] Onnx Module to import onnx models into mxnet

2018-03-08 Thread GitBox
anirudhacharya commented on a change in pull request #9963: [MXNET-34] Onnx Module to import onnx models into mxnet URL: https://github.com/apache/incubator-mxnet/pull/9963#discussion_r173335708 ## File path: example/onnx/test_super_resolution.py ## @@ -0,0 +1,62 @@ +#

[GitHub] yifeim opened a new issue #10045: Despite so many declarations, Module is still confused by extra names in DataIter

2018-03-08 Thread GitBox
yifeim opened a new issue #10045: Despite so many declarations, Module is still confused by extra names in DataIter URL: https://github.com/apache/incubator-mxnet/issues/10045 ## Description I provide data_names to module declaration, module binding, and DataIter. Still, my module

[GitHub] cjolivier01 commented on a change in pull request #9947: [WIP] Performance optimization for dot(csr, rsp) on cpu

2018-03-08 Thread GitBox
cjolivier01 commented on a change in pull request #9947: [WIP] Performance optimization for dot(csr, rsp) on cpu URL: https://github.com/apache/incubator-mxnet/pull/9947#discussion_r17323 ## File path: src/operator/tensor/dot-inl.h ## @@ -793,16 +764,21 @@ inline void

[GitHub] cjolivier01 commented on a change in pull request #9995: [WIP] CI docker revamp; Add Jetson, Raspberry and CentOS 7 build [MXNET-42][MXNET-43][MXNET-44][MXNET-57]

2018-03-08 Thread GitBox
cjolivier01 commented on a change in pull request #9995: [WIP] CI docker revamp; Add Jetson, Raspberry and CentOS 7 build [MXNET-42][MXNET-43][MXNET-44][MXNET-57] URL: https://github.com/apache/incubator-mxnet/pull/9995#discussion_r173332947 ## File path: python/mxnet/libinfo.py

[GitHub] cjolivier01 commented on a change in pull request #9995: [WIP] CI docker revamp; Add Jetson, Raspberry and CentOS 7 build [MXNET-42][MXNET-43][MXNET-44][MXNET-57]

2018-03-08 Thread GitBox
cjolivier01 commented on a change in pull request #9995: [WIP] CI docker revamp; Add Jetson, Raspberry and CentOS 7 build [MXNET-42][MXNET-43][MXNET-44][MXNET-57] URL: https://github.com/apache/incubator-mxnet/pull/9995#discussion_r173332972 ## File path: python/mxnet/libinfo.py

[GitHub] mongoose54 opened a new issue #10044: How can I use im2rec.py to generate .bin files?

2018-03-08 Thread GitBox
mongoose54 opened a new issue #10044: How can I use im2rec.py to generate .bin files? URL: https://github.com/apache/incubator-mxnet/issues/10044 I have modified `im2rec.py` to be more flexible with various image manipulations? However I see that `im2rec.py` is not generating .bin files

[GitHub] marcoabreu commented on a change in pull request #9995: [WIP] CI docker revamp; Add Jetson, Raspberry and CentOS 7 build [MXNET-42][MXNET-43][MXNET-44][MXNET-57]

2018-03-08 Thread GitBox
marcoabreu commented on a change in pull request #9995: [WIP] CI docker revamp; Add Jetson, Raspberry and CentOS 7 build [MXNET-42][MXNET-43][MXNET-44][MXNET-57] URL: https://github.com/apache/incubator-mxnet/pull/9995#discussion_r173332212 ## File path: python/mxnet/libinfo.py

[GitHub] cjolivier01 commented on a change in pull request #9995: [WIP] CI docker revamp; Add Jetson, Raspberry and CentOS 7 build [MXNET-42][MXNET-43][MXNET-44][MXNET-57]

2018-03-08 Thread GitBox
cjolivier01 commented on a change in pull request #9995: [WIP] CI docker revamp; Add Jetson, Raspberry and CentOS 7 build [MXNET-42][MXNET-43][MXNET-44][MXNET-57] URL: https://github.com/apache/incubator-mxnet/pull/9995#discussion_r173331575 ## File path: python/mxnet/libinfo.py

[GitHub] marcoabreu commented on a change in pull request #9995: [WIP] CI docker revamp; Add Jetson, Raspberry and CentOS 7 build [MXNET-42][MXNET-43][MXNET-44][MXNET-57]

2018-03-08 Thread GitBox
marcoabreu commented on a change in pull request #9995: [WIP] CI docker revamp; Add Jetson, Raspberry and CentOS 7 build [MXNET-42][MXNET-43][MXNET-44][MXNET-57] URL: https://github.com/apache/incubator-mxnet/pull/9995#discussion_r173330602 ## File path: python/mxnet/libinfo.py

[GitHub] cjolivier01 commented on a change in pull request #9995: [WIP] CI docker revamp; Add Jetson, Raspberry and CentOS 7 build [MXNET-42][MXNET-43][MXNET-44][MXNET-57]

2018-03-08 Thread GitBox
cjolivier01 commented on a change in pull request #9995: [WIP] CI docker revamp; Add Jetson, Raspberry and CentOS 7 build [MXNET-42][MXNET-43][MXNET-44][MXNET-57] URL: https://github.com/apache/incubator-mxnet/pull/9995#discussion_r173331575 ## File path: python/mxnet/libinfo.py

[GitHub] zheng-da commented on a change in pull request #9947: [WIP] Performance optimization for dot(csr, rsp) on cpu

2018-03-08 Thread GitBox
zheng-da commented on a change in pull request #9947: [WIP] Performance optimization for dot(csr, rsp) on cpu URL: https://github.com/apache/incubator-mxnet/pull/9947#discussion_r173331483 ## File path: src/operator/tensor/dot-inl.h ## @@ -793,16 +764,21 @@ inline void

[GitHub] marcoabreu commented on a change in pull request #9995: [WIP] CI docker revamp; Add Jetson, Raspberry and CentOS 7 build [MXNET-42][MXNET-43][MXNET-44][MXNET-57]

2018-03-08 Thread GitBox
marcoabreu commented on a change in pull request #9995: [WIP] CI docker revamp; Add Jetson, Raspberry and CentOS 7 build [MXNET-42][MXNET-43][MXNET-44][MXNET-57] URL: https://github.com/apache/incubator-mxnet/pull/9995#discussion_r173330808 ## File path: python/mxnet/libinfo.py

[GitHub] marcoabreu commented on a change in pull request #9963: [MXNET-34] Onnx Module to import onnx models into mxnet

2018-03-08 Thread GitBox
marcoabreu commented on a change in pull request #9963: [MXNET-34] Onnx Module to import onnx models into mxnet URL: https://github.com/apache/incubator-mxnet/pull/9963#discussion_r173329728 ## File path: Jenkinsfile ## @@ -649,7 +649,20 @@ try { }

[GitHub] marcoabreu commented on a change in pull request #9963: [MXNET-34] Onnx Module to import onnx models into mxnet

2018-03-08 Thread GitBox
marcoabreu commented on a change in pull request #9963: [MXNET-34] Onnx Module to import onnx models into mxnet URL: https://github.com/apache/incubator-mxnet/pull/9963#discussion_r173329488 ## File path: python/mxnet/contrib/__init__.py ## @@ -28,5 +28,5 @@ from .

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

2018-03-08 Thread GitBox
marcoabreu commented on issue #10021: [MXNET-33] Mkldnn pooling convention crash URL: https://github.com/apache/incubator-mxnet/pull/10021#issuecomment-371665164 Feel free to develop and add an appropriate unit test yourself. Please don't hesitate to reach out to us in case you need

[GitHub] marcoabreu closed pull request #10031: [MXNET-67] Sync master with v1.1.0 branch

2018-03-08 Thread GitBox
marcoabreu closed pull request #10031: [MXNET-67] Sync master with v1.1.0 branch URL: https://github.com/apache/incubator-mxnet/pull/10031 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-67] Sync master with v1.1.0 branch (#10031)

2018-03-08 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 ccf0469 [MXNET-67] Sync master

[GitHub] cjolivier01 commented on issue #9995: [WIP] CI docker revamp; Add Jetson, Raspberry and CentOS 7 build [MXNET-42][MXNET-43][MXNET-44][MXNET-57]

2018-03-08 Thread GitBox
cjolivier01 commented on issue #9995: [WIP] CI docker revamp; Add Jetson, Raspberry and CentOS 7 build [MXNET-42][MXNET-43][MXNET-44][MXNET-57] URL: https://github.com/apache/incubator-mxnet/pull/9995#issuecomment-371663585 LGTM

[GitHub] cjolivier01 commented on a change in pull request #9995: [WIP] CI docker revamp; Add Jetson, Raspberry and CentOS 7 build [MXNET-42][MXNET-43][MXNET-44][MXNET-57]

2018-03-08 Thread GitBox
cjolivier01 commented on a change in pull request #9995: [WIP] CI docker revamp; Add Jetson, Raspberry and CentOS 7 build [MXNET-42][MXNET-43][MXNET-44][MXNET-57] URL: https://github.com/apache/incubator-mxnet/pull/9995#discussion_r173327315 ## File path: python/mxnet/libinfo.py

[GitHub] marcoabreu commented on a change in pull request #9995: [WIP] CI docker revamp; Add Jetson, Raspberry and CentOS 7 build [MXNET-42][MXNET-43][MXNET-44][MXNET-57]

2018-03-08 Thread GitBox
marcoabreu commented on a change in pull request #9995: [WIP] CI docker revamp; Add Jetson, Raspberry and CentOS 7 build [MXNET-42][MXNET-43][MXNET-44][MXNET-57] URL: https://github.com/apache/incubator-mxnet/pull/9995#discussion_r173325670 ## File path: python/mxnet/libinfo.py

[GitHub] mbaijal closed pull request #10043: [Post 1.1.0] Merge LICENSE file changes from 1.1.0 release

2018-03-08 Thread GitBox
mbaijal closed pull request #10043: [Post 1.1.0] Merge LICENSE file changes from 1.1.0 release URL: https://github.com/apache/incubator-mxnet/pull/10043 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of

[GitHub] mbaijal commented on issue #10043: [Post 1.1.0] Merge LICENSE file changes from 1.1.0 release

2018-03-08 Thread GitBox
mbaijal commented on issue #10043: [Post 1.1.0] Merge LICENSE file changes from 1.1.0 release URL: https://github.com/apache/incubator-mxnet/pull/10043#issuecomment-371660013 Closing this since it is a duplicate of PR https://github.com/apache/incubator-mxnet/pull/10031

[GitHub] mbaijal opened a new pull request #10043: [Post 1.1.0]

2018-03-08 Thread GitBox
mbaijal opened a new pull request #10043: [Post 1.1.0] URL: https://github.com/apache/incubator-mxnet/pull/10043 ## Description ## Fixes Issue #9702 Apply changes to top Level LICENSE file based on comments from 1.1.0 release ## Checklist ## ### Essentials ### - [ ]

[GitHub] marcoabreu commented on issue #58: Redirect via mod_rewrite

2018-03-08 Thread GitBox
marcoabreu commented on issue #58: Redirect via mod_rewrite URL: https://github.com/apache/incubator-mxnet-site/pull/58#issuecomment-371656409 I agree that it's a good solution for the problem in its current form. We just have to make sure to note down all requirement and revisit all

[GitHub] rahul003 commented on issue #9774: mx.io.ImageRecordIter does not respect dtype argument / FP16 performance on Volta

2018-03-08 Thread GitBox
rahul003 commented on issue #9774: mx.io.ImageRecordIter does not respect dtype argument / FP16 performance on Volta URL: https://github.com/apache/incubator-mxnet/issues/9774#issuecomment-371654013 Both suggestions didn't help improve the speed unforunately. Using

[GitHub] rahul003 commented on issue #9774: mx.io.ImageRecordIter does not respect dtype argument / FP16 performance on Volta

2018-03-08 Thread GitBox
rahul003 commented on issue #9774: mx.io.ImageRecordIter does not respect dtype argument / FP16 performance on Volta URL: https://github.com/apache/incubator-mxnet/issues/9774#issuecomment-371654013 Both suggestions didn't help improve the speed unforunately. Using

[GitHub] rahul003 commented on issue #9774: mx.io.ImageRecordIter does not respect dtype argument / FP16 performance on Volta

2018-03-08 Thread GitBox
rahul003 commented on issue #9774: mx.io.ImageRecordIter does not respect dtype argument / FP16 performance on Volta URL: https://github.com/apache/incubator-mxnet/issues/9774#issuecomment-371654013 Both suggestions didn't help improve the speed unforunately. Using

[GitHub] rahul003 commented on issue #9774: mx.io.ImageRecordIter does not respect dtype argument / FP16 performance on Volta

2018-03-08 Thread GitBox
rahul003 commented on issue #9774: mx.io.ImageRecordIter does not respect dtype argument / FP16 performance on Volta URL: https://github.com/apache/incubator-mxnet/issues/9774#issuecomment-371654013 Both suggestions didn't help improve the speed unforunately. Using

[GitHub] rahul003 commented on issue #9774: mx.io.ImageRecordIter does not respect dtype argument / FP16 performance on Volta

2018-03-08 Thread GitBox
rahul003 commented on issue #9774: mx.io.ImageRecordIter does not respect dtype argument / FP16 performance on Volta URL: https://github.com/apache/incubator-mxnet/issues/9774#issuecomment-371654013 Both suggestions didn't help improve the speed unforunately. Using

[GitHub] aaronmarkham commented on issue #58: Redirect via mod_rewrite

2018-03-08 Thread GitBox
aaronmarkham commented on issue #58: Redirect via mod_rewrite URL: https://github.com/apache/incubator-mxnet-site/pull/58#issuecomment-371652299 Well, I was planning on incorporating it in the site generation process. I was thinking the s3 solution was just for staging the site and

[GitHub] marcoabreu commented on issue #58: Redirect via mod_rewrite

2018-03-08 Thread GitBox
marcoabreu commented on issue #58: Redirect via mod_rewrite URL: https://github.com/apache/incubator-mxnet-site/pull/58#issuecomment-371650217 This entire solution is temporary anyways, considering all these changes will be reverted once we turn on website generation Sheng Zha

[GitHub] szha commented on issue #58: Redirect via mod_rewrite

2018-03-08 Thread GitBox
szha commented on issue #58: Redirect via mod_rewrite URL: https://github.com/apache/incubator-mxnet-site/pull/58#issuecomment-371648475 Usage of .htaccess assumes apache http server. If we migrate to static hosting using s3, a different solution will be needed.

[GitHub] haojin2 commented on issue #9715: IndexError in labels when size of training dataset is not multiple of batch size

2018-03-08 Thread GitBox
haojin2 commented on issue #9715: IndexError in labels when size of training dataset is not multiple of batch size URL: https://github.com/apache/incubator-mxnet/issues/9715#issuecomment-371643865 @nazikus Hello, I tried reproducing your issue with: 1. Generating a similar dataset with

[GitHub] haojin2 commented on issue #9715: IndexError in labels when size of training dataset is not multiple of batch size

2018-03-08 Thread GitBox
haojin2 commented on issue #9715: IndexError in labels when size of training dataset is not multiple of batch size URL: https://github.com/apache/incubator-mxnet/issues/9715#issuecomment-371643865 @nazikus Hello, I tried reproducing your issue with: 0) Generating a similar dataset with

[GitHub] haojin2 commented on issue #9715: IndexError in labels when size of training dataset is not multiple of batch size

2018-03-08 Thread GitBox
haojin2 commented on issue #9715: IndexError in labels when size of training dataset is not multiple of batch size URL: https://github.com/apache/incubator-mxnet/issues/9715#issuecomment-371643865 @nazikus Hello, I tried reproducing your issue with: 1 Generating a similar dataset with

[GitHub] haojin2 commented on issue #9715: IndexError in labels when size of training dataset is not multiple of batch size

2018-03-08 Thread GitBox
haojin2 commented on issue #9715: IndexError in labels when size of training dataset is not multiple of batch size URL: https://github.com/apache/incubator-mxnet/issues/9715#issuecomment-371643865 @nazikus Hello, I tried reproducing your issue with: 0) Generating a similar dataset with

[GitHub] zheng-da commented on a change in pull request #10025: Language model with Google's billion words dataset

2018-03-08 Thread GitBox
zheng-da commented on a change in pull request #10025: Language model with Google's billion words dataset URL: https://github.com/apache/incubator-mxnet/pull/10025#discussion_r173305447 ## File path: src/operator/nn/fully_connected.cc ## @@ -73,22 +76,47 @@ static bool

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

2018-03-08 Thread GitBox
ashokei commented on issue #10021: [WIP] [MXNET-33] Mkldnn pooling convention crash URL: https://github.com/apache/incubator-mxnet/pull/10021#issuecomment-371638503 @marcoabreu @TaoLv @zheng-da Unit-tests currently do not seem to test backward/training mode for pooling very well. Below is

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

2018-03-08 Thread GitBox
ashokei commented on issue #10021: [WIP] [MXNET-33] Mkldnn pooling convention crash URL: https://github.com/apache/incubator-mxnet/pull/10021#issuecomment-371638503 @marcoabreu @TaoLv @zheng-da Unit-tests currently do not seem to test backward/training mode for pooling very well. Below is

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

2018-03-08 Thread GitBox
szha commented on issue #9986: gluon language modeling dataset and text token reader URL: https://github.com/apache/incubator-mxnet/pull/9986#issuecomment-371637476 Also, for large dataset it might be worth changing to store indices-only. Need to explore more.

[GitHub] sxjscience commented on issue #10042: Gluon dataloader crash on speech recognition training

2018-03-08 Thread GitBox
sxjscience commented on issue #10042: Gluon dataloader crash on speech recognition training URL: https://github.com/apache/incubator-mxnet/issues/10042#issuecomment-371636821 Thanks for reporting! Is there any code to reproduce the error message?

[incubator-mxnet] branch nlp_toolkit updated: gluon language modeling dataset and text token reader (#9986)

2018-03-08 Thread sxjscience
This is an automated email from the ASF dual-hosted git repository. sxjscience pushed a commit to branch nlp_toolkit in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git The following commit(s) were added to refs/heads/nlp_toolkit by this push: new 329acde gluon language

[GitHub] sxjscience closed pull request #9986: gluon language modeling dataset and text token reader

2018-03-08 Thread GitBox
sxjscience closed pull request #9986: gluon language modeling dataset and text token reader URL: https://github.com/apache/incubator-mxnet/pull/9986 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance:

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

2018-03-08 Thread GitBox
szha commented on issue #9986: gluon language modeling dataset and text token reader URL: https://github.com/apache/incubator-mxnet/pull/9986#issuecomment-371630223 Discussed offline on the design, and it would be worth having a data reader that returns raw sentences to facilitate use

[GitHub] jeremiedb commented on issue #10038: MxNet with R on Windows : DiagrammeR dependency isn't resolved properly

2018-03-08 Thread GitBox
jeremiedb commented on issue #10038: MxNet with R on Windows : DiagrammeR dependency isn't resolved properly URL: https://github.com/apache/incubator-mxnet/issues/10038#issuecomment-371629153 DiagrammeR 1.0.0 brought on incompatibility with the graph.viz. This has been fixed in the

[GitHub] cjolivier01 commented on a change in pull request #9995: [WIP] CI docker revamp; Add Jetson, Raspberry and CentOS 7 build [MXNET-42][MXNET-43][MXNET-44][MXNET-57]

2018-03-08 Thread GitBox
cjolivier01 commented on a change in pull request #9995: [WIP] CI docker revamp; Add Jetson, Raspberry and CentOS 7 build [MXNET-42][MXNET-43][MXNET-44][MXNET-57] URL: https://github.com/apache/incubator-mxnet/pull/9995#discussion_r173294247 ## File path: CMakeLists.txt

[GitHub] cjolivier01 commented on a change in pull request #9995: [WIP] CI docker revamp; Add Jetson, Raspberry and CentOS 7 build [MXNET-42][MXNET-43][MXNET-44][MXNET-57]

2018-03-08 Thread GitBox
cjolivier01 commented on a change in pull request #9995: [WIP] CI docker revamp; Add Jetson, Raspberry and CentOS 7 build [MXNET-42][MXNET-43][MXNET-44][MXNET-57] URL: https://github.com/apache/incubator-mxnet/pull/9995#discussion_r173294029 ## File path: CMakeLists.txt

[GitHub] Jerryzcn opened a new issue #10042: Gluon dataloader crash on speech recognition training

2018-03-08 Thread GitBox
Jerryzcn opened a new issue #10042: Gluon dataloader crash on speech recognition training URL: https://github.com/apache/incubator-mxnet/issues/10042 Note: Providing complete information in the most concise form is the best way to get help. This issue template serves as the checklist for

[GitHub] sxjscience commented on issue #10019: ModuleNotFoundError

2018-03-08 Thread GitBox
sxjscience commented on issue #10019: ModuleNotFoundError URL: https://github.com/apache/incubator-mxnet/issues/10019#issuecomment-371623118 I think you may need to install the cython pacakge in the RCNN folder https://github.com/apache/incubator-mxnet/tree/master/example/rcnn/rcnn/cython

[GitHub] sxjscience commented on issue #10035: Who can help me solve this error??batch_loss.backward() error?

2018-03-08 Thread GitBox
sxjscience commented on issue #10035: Who can help me solve this error??batch_loss.backward() error? URL: https://github.com/apache/incubator-mxnet/issues/10035#issuecomment-371622551 The for-loop here should be the problem ```python for step in range(...): ... ``` We

[GitHub] Jerryzcn commented on issue #9977: Cpu lstm inference

2018-03-08 Thread GitBox
Jerryzcn commented on issue #9977: Cpu lstm inference URL: https://github.com/apache/incubator-mxnet/pull/9977#issuecomment-371608311 ``` x = nd.ones(shape=(800,1,40)) model = mx.gluon.nn.Sequential() with model.name_scope(): model.add(mx.gluon.rnn.LSTM(320, num_layers=4,

[GitHub] sxjscience commented on issue #10038: MxNet with R on Windows : DiagrammeR dependency isn't resolved properly

2018-03-08 Thread GitBox
sxjscience commented on issue #10038: MxNet with R on Windows : DiagrammeR dependency isn't resolved properly URL: https://github.com/apache/incubator-mxnet/issues/10038#issuecomment-371620482 @nswamy @jeremiedb This is an

[incubator-mxnet] branch nlp_toolkit updated (29f7f43 -> 649b086)

2018-03-08 Thread zhasheng
This is an automated email from the ASF dual-hosted git repository. zhasheng pushed a change to branch nlp_toolkit in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. from 29f7f43 gluon dataset download refactor (#10008) add 2a9c7d9 [Perl] Sparse feature. (#9988)

[GitHub] sxjscience closed pull request #10032: add axes support for dropouts in gluon

2018-03-08 Thread GitBox
sxjscience closed pull request #10032: add axes support for dropouts in gluon URL: https://github.com/apache/incubator-mxnet/pull/10032 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

[incubator-mxnet] branch master updated: add axes support for dropouts in gluon (#10032)

2018-03-08 Thread sxjscience
This is an automated email from the ASF dual-hosted git repository. sxjscience 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 649b086 add axes support for

[GitHub] sxjscience commented on a change in pull request #10032: add axes support for dropouts in gluon

2018-03-08 Thread GitBox
sxjscience commented on a change in pull request #10032: add axes support for dropouts in gluon URL: https://github.com/apache/incubator-mxnet/pull/10032#discussion_r173283167 ## File path: tests/python/unittest/test_gluon.py ## @@ -827,6 +827,46 @@ def selu(x): x =

  1   2   >