[incubator-mxnet] branch master updated: Update LICENSE (#10370)

2018-04-02 Thread jxie
This is an automated email from the ASF dual-hosted git repository. jxie pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git The following commit(s) were added to refs/heads/master by this push: new e0df199 Update LICENSE (#10370) e0df199

[GitHub] piiswrong closed pull request #10370: [MXNET-16] Update LICENSE for nnvm-fusion path

2018-04-02 Thread GitBox
piiswrong closed pull request #10370: [MXNET-16] Update LICENSE for nnvm-fusion path URL: https://github.com/apache/incubator-mxnet/pull/10370 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As

[GitHub] ThomasDelteil opened a new pull request #10372: Adding CNN-character level example notebook

2018-04-02 Thread GitBox
ThomasDelteil opened a new pull request #10372: Adding CNN-character level example notebook URL: https://github.com/apache/incubator-mxnet/pull/10372 adding an example to the example list This is an automated message from

[GitHub] piiswrong commented on a change in pull request #10315: [MXNET-249] Add inplace support to mkldnn sum

2018-04-02 Thread GitBox
piiswrong commented on a change in pull request #10315: [MXNET-249] Add inplace support to mkldnn sum URL: https://github.com/apache/incubator-mxnet/pull/10315#discussion_r178621496 ## File path: src/operator/nn/mkldnn/mkldnn_sum.cc ## @@ -49,23 +49,42 @@ void Sum(const

[GitHub] piiswrong closed pull request #10336: Fix MKLDNN sigmoid/softrelu issue

2018-04-02 Thread GitBox
piiswrong closed pull request #10336: Fix MKLDNN sigmoid/softrelu issue URL: https://github.com/apache/incubator-mxnet/pull/10336 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: Fix MKLDNN sigmoid/softrelu issue (#10336)

2018-04-02 Thread jxie
This is an automated email from the ASF dual-hosted git repository. jxie pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git The following commit(s) were added to refs/heads/master by this push: new e0df25c Fix MKLDNN sigmoid/softrelu

[GitHub] cgraywang commented on issue #10363: Fix windows setup doc using VS 2017

2018-04-02 Thread GitBox
cgraywang commented on issue #10363: Fix windows setup doc using VS 2017 URL: https://github.com/apache/incubator-mxnet/pull/10363#issuecomment-378009760 @piiswrong Put 2017 as option 1, and add one more detail about setting up the vs 2017 version suggested by @yajiedesign .

[GitHub] indhub commented on issue #9733: BUG in MultiBoxTargetForward when there is single box label

2018-04-02 Thread GitBox
indhub commented on issue #9733: BUG in MultiBoxTargetForward when there is single box label URL: https://github.com/apache/incubator-mxnet/issues/9733#issuecomment-378007151 @zhreshold Thoughts? This is an automated

[GitHub] piiswrong commented on issue #10345: allow block setattr to reset the prefix when setting new block

2018-04-02 Thread GitBox
piiswrong commented on issue #10345: allow block setattr to reset the prefix when setting new block URL: https://github.com/apache/incubator-mxnet/pull/10345#issuecomment-378005952 We should raise a warning when assign blocks with non-matching prefix. In python copy on assignment only

[GitHub] anirudhacharya commented on issue #9358: Why does running 1 round of an MXNET model training produce Train-mse=NaN?

2018-04-02 Thread GitBox
anirudhacharya commented on issue #9358: Why does running 1 round of an MXNET model training produce Train-mse=NaN? URL: https://github.com/apache/incubator-mxnet/issues/9358#issuecomment-378004852 @alexmosc can you verify the bug fix and close the issue accordingly.

[incubator-mxnet] branch master updated: Replace std::swap_ranges with memcpy (#10351)

2018-04-02 Thread jxie
This is an automated email from the ASF dual-hosted git repository. jxie pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git The following commit(s) were added to refs/heads/master by this push: new b37b3f5 Replace std::swap_ranges with

[GitHub] piiswrong closed pull request #10351: [MXNET-259] Performance improvement of random.shuffle

2018-04-02 Thread GitBox
piiswrong closed pull request #10351: [MXNET-259] Performance improvement of random.shuffle URL: https://github.com/apache/incubator-mxnet/pull/10351 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] piiswrong commented on a change in pull request #10360: extend ndarray in-place reshape

2018-04-02 Thread GitBox
piiswrong commented on a change in pull request #10360: extend ndarray in-place reshape URL: https://github.com/apache/incubator-mxnet/pull/10360#discussion_r178609880 ## File path: python/mxnet/ndarray/ndarray.py ## @@ -996,10 +994,10 @@ def reshape(self, *shape,

[GitHub] piiswrong commented on a change in pull request #10360: extend ndarray in-place reshape

2018-04-02 Thread GitBox
piiswrong commented on a change in pull request #10360: extend ndarray in-place reshape URL: https://github.com/apache/incubator-mxnet/pull/10360#discussion_r178609994 ## File path: python/mxnet/ndarray/ndarray.py ## @@ -943,13 +943,8 @@ def reshape(self, *shape,

[GitHub] indhub closed pull request #10039: [MXNET-103] Added tutorial on types of data augmentations.

2018-04-02 Thread GitBox
indhub closed pull request #10039: [MXNET-103] Added tutorial on types of data augmentations. URL: https://github.com/apache/incubator-mxnet/pull/10039 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

[incubator-mxnet] branch master updated: [MXNET-103] Added tutorial on types of data augmentations. (#10039)

2018-04-02 Thread indhub
This is an automated email from the ASF dual-hosted git repository. indhub 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 fe6220b [MXNET-103] Added tutorial on

[GitHub] themummy commented on issue #7792: Multiplatform docker based builds

2018-04-02 Thread GitBox
themummy commented on issue #7792: Multiplatform docker based builds URL: https://github.com/apache/incubator-mxnet/pull/7792#issuecomment-377996886 Has anyone successfully amalgamated mxnet for Android x86-64? I need to run mxnet on an Android Emulator on Jenkins.

[incubator-mxnet] branch master updated: Update docs for multiple functions (#10362)

2018-04-02 Thread jxie
This is an automated email from the ASF dual-hosted git repository. jxie pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git The following commit(s) were added to refs/heads/master by this push: new e208f4c Update docs for multiple

[GitHub] piiswrong closed pull request #10362: Update docs for multiple functions

2018-04-02 Thread GitBox
piiswrong closed pull request #10362: Update docs for multiple functions URL: https://github.com/apache/incubator-mxnet/pull/10362 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] szha commented on issue #10364: [MXNET-260]remove use_fast_math

2018-04-02 Thread GitBox
szha commented on issue #10364: [MXNET-260]remove use_fast_math URL: https://github.com/apache/incubator-mxnet/pull/10364#issuecomment-377996353 Makefile doesn't have that flag for nvcc compilation lines. It doesn't seem like the cmake development includes the consistency of compilation

[GitHub] piiswrong commented on issue #10363: Fix windows setup doc using VS 2017

2018-04-02 Thread GitBox
piiswrong commented on issue #10363: Fix windows setup doc using VS 2017 URL: https://github.com/apache/incubator-mxnet/pull/10363#issuecomment-377996125 I think we should put 2017 as option 1 This is an automated message

[GitHub] cjolivier01 commented on a change in pull request #10365: [MXNET-261]Update MKLDNN & Add CPP Test

2018-04-02 Thread GitBox
cjolivier01 commented on a change in pull request #10365: [MXNET-261]Update MKLDNN & Add CPP Test URL: https://github.com/apache/incubator-mxnet/pull/10365#discussion_r178606123 ## File path: tests/cpp/operator/mkldnn.cc ## @@ -77,4 +77,11 @@ TEST(MKLDNN_UTIL_FUNC,

[GitHub] piiswrong commented on issue #10364: [MXNET-260]remove use_fast_math

2018-04-02 Thread GitBox
piiswrong commented on issue #10364: [MXNET-260]remove use_fast_math URL: https://github.com/apache/incubator-mxnet/pull/10364#issuecomment-377995749 Also need to turn it off for Makefile? This is an automated message from

[GitHub] piiswrong commented on issue #10367: [MXNET-262] Implement mx.random.seed_context to seed random number generators of a specific device context

2018-04-02 Thread GitBox
piiswrong commented on issue #10367: [MXNET-262] Implement mx.random.seed_context to seed random number generators of a specific device context URL: https://github.com/apache/incubator-mxnet/pull/10367#issuecomment-377995564 I would add a `ctx=None` argument to seed instead

[GitHub] anirudhacharya commented on issue #10359: Correct way to detect JSON file contains MXNet model

2018-04-02 Thread GitBox
anirudhacharya commented on issue #10359: Correct way to detect JSON file contains MXNet model URL: https://github.com/apache/incubator-mxnet/issues/10359#issuecomment-377994986 @szha please tag this - "HowTo", "Visualization".

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

2018-04-02 Thread GitBox
zheng-da commented on issue #10365: [MXNET-261]Update MKLDNN & Add CPP Test URL: https://github.com/apache/incubator-mxnet/pull/10365#issuecomment-377994182 are you sure this is a MKL problem? it fails in all configurations.

[GitHub] haojin2 opened a new pull request #10371: [MXNET-263] [WIP] Support for dot(dns, csr) = dns and dot(dns, csr.T) = dns on GPU

2018-04-02 Thread GitBox
haojin2 opened a new pull request #10371: [MXNET-263] [WIP] Support for dot(dns, csr) = dns and dot(dns, csr.T) = dns on GPU URL: https://github.com/apache/incubator-mxnet/pull/10371 ## Description ## Support dot(dns, csr) = dns and dot(dns, csr.T) = dns. ## Checklist ## ###

[GitHub] anirudhacharya commented on issue #10359: Correct way to detect JSON file contains MXNet model

2018-04-02 Thread GitBox
anirudhacharya commented on issue #10359: Correct way to detect JSON file contains MXNet model URL: https://github.com/apache/incubator-mxnet/issues/10359#issuecomment-377993495 Here is a potential solution to this problem, without having any restrictions on the filename of the json

[GitHub] eric-haibin-lin commented on issue #9396: inference speed drop after updating mxnet from 0.10.0 to 1.0.0

2018-04-02 Thread GitBox
eric-haibin-lin commented on issue #9396: inference speed drop after updating mxnet from 0.10.0 to 1.0.0 URL: https://github.com/apache/incubator-mxnet/issues/9396#issuecomment-377986916 @DickJC123 gentle ping - any update? Were you able to reproduce the issue? Just want to check if

[GitHub] haojin2 commented on a change in pull request #10208: [MXNET-117] Sparse operator broadcast_mul/div(csr, dense) = csr

2018-04-02 Thread GitBox
haojin2 commented on a change in pull request #10208: [MXNET-117] Sparse operator broadcast_mul/div(csr, dense) = csr URL: https://github.com/apache/incubator-mxnet/pull/10208#discussion_r178596771 ## File path: python/mxnet/ndarray/sparse.py ## @@ -1151,6 +1177,284 @@

[GitHub] reminisce commented on issue #10368: asscalar is very slow

2018-04-02 Thread GitBox
reminisce commented on issue #10368: asscalar is very slow URL: https://github.com/apache/incubator-mxnet/issues/10368#issuecomment-37798 For OOM problem, seems it's caused by always pushing operations to engine for every mini-batch in an epoch. You need to sync before another

[GitHub] nswamy commented on a change in pull request #10343: [MXNET-116] Optimized functions with batch input

2018-04-02 Thread GitBox
nswamy commented on a change in pull request #10343: [MXNET-116] Optimized functions with batch input URL: https://github.com/apache/incubator-mxnet/pull/10343#discussion_r178595028 ## File path: scala-package/infer/src/main/scala/ml/dmlc/mxnet/infer/ImageClassifier.scala

[GitHub] nswamy commented on a change in pull request #10343: [MXNET-116] Optimized functions with batch input

2018-04-02 Thread GitBox
nswamy commented on a change in pull request #10343: [MXNET-116] Optimized functions with batch input URL: https://github.com/apache/incubator-mxnet/pull/10343#discussion_r178594869 ## File path: scala-package/infer/src/main/scala/ml/dmlc/mxnet/infer/ImageClassifier.scala

[GitHub] reminisce commented on issue #10368: asscalar is very slow

2018-04-02 Thread GitBox
reminisce commented on issue #10368: asscalar is very slow URL: https://github.com/apache/incubator-mxnet/issues/10368#issuecomment-377982754 This timing does not reflect the correct time spent on training and `asscalar` respectively. As said before, all the lines before the second

[GitHub] nju-luke commented on issue #10368: asscalar is very slow

2018-04-02 Thread GitBox
nju-luke commented on issue #10368: asscalar is very slow URL: https://github.com/apache/incubator-mxnet/issues/10368#issuecomment-377979496 Thank you for the answer. And I have another problem, could you please help me fix it. When I use mxnet with gpu context, the first iteration

[GitHub] haojin2 commented on a change in pull request #10208: [MXNET-117] Sparse operator broadcast_mul/div(csr, dense) = csr

2018-04-02 Thread GitBox
haojin2 commented on a change in pull request #10208: [MXNET-117] Sparse operator broadcast_mul/div(csr, dense) = csr URL: https://github.com/apache/incubator-mxnet/pull/10208#discussion_r178591508 ## File path: src/operator/tensor/elemwise_binary_broadcast_op_basic.cc ##

[GitHub] haojin2 commented on a change in pull request #10208: [MXNET-117] Sparse operator broadcast_mul/div(csr, dense) = csr

2018-04-02 Thread GitBox
haojin2 commented on a change in pull request #10208: [MXNET-117] Sparse operator broadcast_mul/div(csr, dense) = csr URL: https://github.com/apache/incubator-mxnet/pull/10208#discussion_r178591486 ## File path: python/mxnet/ndarray/sparse.py ## @@ -1176,6 +1480,7 @@ def

[GitHub] haojin2 commented on a change in pull request #10208: [MXNET-117] Sparse operator broadcast_mul/div(csr, dense) = csr

2018-04-02 Thread GitBox
haojin2 commented on a change in pull request #10208: [MXNET-117] Sparse operator broadcast_mul/div(csr, dense) = csr URL: https://github.com/apache/incubator-mxnet/pull/10208#discussion_r178591121 ## File path: tests/python/unittest/test_sparse_operator.py ## @@ -1675,6

[GitHub] haojin2 commented on a change in pull request #10208: [MXNET-117] Sparse operator broadcast_mul/div(csr, dense) = csr

2018-04-02 Thread GitBox
haojin2 commented on a change in pull request #10208: [MXNET-117] Sparse operator broadcast_mul/div(csr, dense) = csr URL: https://github.com/apache/incubator-mxnet/pull/10208#discussion_r178591078 ## File path: tests/python/unittest/test_sparse_operator.py ## @@ -1675,6

[GitHub] eric-haibin-lin commented on a change in pull request #10231: [MXNET-16] move dmlc-core & nnvm

2018-04-02 Thread GitBox
eric-haibin-lin commented on a change in pull request #10231: [MXNET-16] move dmlc-core & nnvm URL: https://github.com/apache/incubator-mxnet/pull/10231#discussion_r178591087 ## File path: LICENSE ## @@ -220,13 +220,13 @@ 3. scala-package - For details, see,

[GitHub] eric-haibin-lin opened a new pull request #10370: Update LICENSE for nnvm-fusion path

2018-04-02 Thread GitBox
eric-haibin-lin opened a new pull request #10370: Update LICENSE for nnvm-fusion path URL: https://github.com/apache/incubator-mxnet/pull/10370 ## Description ## (Brief description on what this PR is about) @mbaijal ## Checklist ## ### Essentials ### - [ ] Passed code

[GitHub] nju-luke commented on issue #10368: asscalar is very slow

2018-04-02 Thread GitBox
nju-luke commented on issue #10368: asscalar is very slow URL: https://github.com/apache/incubator-mxnet/issues/10368#issuecomment-377979496 This is the version with time, and the third out put time. ``` python print(datetime.datetime.now()) train_loss =

[GitHub] asitstands opened a new issue #10369: Proper seeding of the random number generators for parallel CPU threads and multiple GPU devices

2018-04-02 Thread GitBox
asitstands opened a new issue #10369: Proper seeding of the random number generators for parallel CPU threads and multiple GPU devices URL: https://github.com/apache/incubator-mxnet/issues/10369 It looks like the random number generators for CPU threads are seeded with consecutive

[GitHub] eric-haibin-lin commented on a change in pull request #10293: [MXNET-72] Improve sparse sgd on GPU

2018-04-02 Thread GitBox
eric-haibin-lin commented on a change in pull request #10293: [MXNET-72] Improve sparse sgd on GPU URL: https://github.com/apache/incubator-mxnet/pull/10293#discussion_r178586231 ## File path: src/operator/optimizer_op.cc ## @@ -98,6 +98,38 @@ Where the parameter

[incubator-mxnet] branch master updated: [MXNET-141] Add tutorial Gluon Datasets and DataLoaders (#10251)

2018-04-02 Thread indhub
This is an automated email from the ASF dual-hosted git repository. indhub 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 9a0d002 [MXNET-141] Add tutorial

[GitHub] indhub closed pull request #10251: [MXNET-141] Add tutorial Gluon Datasets and DataLoaders

2018-04-02 Thread GitBox
indhub closed pull request #10251: [MXNET-141] Add tutorial Gluon Datasets and DataLoaders URL: https://github.com/apache/incubator-mxnet/pull/10251 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] sxjscience commented on issue #10363: Fix windows setup doc using VS 2017

2018-04-02 Thread GitBox
sxjscience commented on issue #10363: Fix windows setup doc using VS 2017 URL: https://github.com/apache/incubator-mxnet/pull/10363#issuecomment-377976220 Also, we have another place that shows the windows setup doc.

[GitHub] sxjscience commented on issue #10363: Fix windows setup doc using VS 2017

2018-04-02 Thread GitBox
sxjscience commented on issue #10363: Fix windows setup doc using VS 2017 URL: https://github.com/apache/incubator-mxnet/pull/10363#issuecomment-377975456 Looks great! This is an automated message from the Apache Git Service.

[GitHub] sxjscience commented on issue #10364: [MXNET-260]remove use_fast_math

2018-04-02 Thread GitBox
sxjscience commented on issue #10364: [MXNET-260]remove use_fast_math URL: https://github.com/apache/incubator-mxnet/pull/10364#issuecomment-377974852 This solves https://github.com/apache/incubator-mxnet/issues/9572 . Also, I find that (-1)**2 and (-1)**3 are not tested in the code. We'd

[GitHub] eric-haibin-lin commented on a change in pull request #10293: [MXNET-72] Improve sparse sgd on GPU

2018-04-02 Thread GitBox
eric-haibin-lin commented on a change in pull request #10293: [MXNET-72] Improve sparse sgd on GPU URL: https://github.com/apache/incubator-mxnet/pull/10293#discussion_r178586231 ## File path: src/operator/optimizer_op.cc ## @@ -98,6 +98,38 @@ Where the parameter

[GitHub] haojin2 commented on a change in pull request #10208: [MXNET-117] Sparse operator broadcast_mul/div(csr, dense) = csr

2018-04-02 Thread GitBox
haojin2 commented on a change in pull request #10208: [MXNET-117] Sparse operator broadcast_mul/div(csr, dense) = csr URL: https://github.com/apache/incubator-mxnet/pull/10208#discussion_r178585958 ## File path: src/operator/tensor/elemwise_binary_broadcast_op_basic.cc ##

[GitHub] sxjscience commented on issue #10366: fix bug in sgd

2018-04-02 Thread GitBox
sxjscience commented on issue #10366: fix bug in sgd URL: https://github.com/apache/incubator-mxnet/pull/10366#issuecomment-377972673 I think the temporary workspace is used in the sparse update @eric-haibin-lin This is an

[GitHub] sxjscience commented on issue #10366: fix bug in sgd

2018-04-02 Thread GitBox
sxjscience commented on issue #10366: fix bug in sgd URL: https://github.com/apache/incubator-mxnet/pull/10366#issuecomment-377972673 I think the temporary workspace is used in the code @eric-haibin-lin This is an automated

[GitHub] reminisce commented on issue #10368: asscalar is very slow

2018-04-02 Thread GitBox
reminisce commented on issue #10368: asscalar is very slow URL: https://github.com/apache/incubator-mxnet/issues/10368#issuecomment-377963542 How did you time it? `asscalar` is a blocking function call, while the other lines are async. It would not be accurate to simply time it line by

[GitHub] nju-luke opened a new issue #10368: asscalar is very slow

2018-04-02 Thread GitBox
nju-luke opened a new issue #10368: asscalar is very slow URL: https://github.com/apache/incubator-mxnet/issues/10368 The train part cost 0.01 second, but the asscalar operation cost a few second sometimes more than 10. I install the newest version of mxnet by pip, and this was happened

[GitHub] asitstands opened a new pull request #10367: [MXNET-262] Implement mx.random.seed_context to seed random number generators of a specific device context

2018-04-02 Thread GitBox
asitstands opened a new pull request #10367: [MXNET-262] Implement mx.random.seed_context to seed random number generators of a specific device context URL: https://github.com/apache/incubator-mxnet/pull/10367 ## Description ## This PR introduces a function `mx.random.seed_context`

[GitHub] asitstands commented on a change in pull request #10351: [MXNET-259] Performance improvement of random.shuffle

2018-04-02 Thread GitBox
asitstands commented on a change in pull request #10351: [MXNET-259] Performance improvement of random.shuffle URL: https://github.com/apache/incubator-mxnet/pull/10351#discussion_r178562672 ## File path: src/operator/random/shuffle_op.cc ## @@ -55,18 +56,24 @@ void

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

2018-04-02 Thread GitBox
marcoabreu commented on issue #10365: [MXNET-261]Update MKLDNN & Add CPP Test URL: https://github.com/apache/incubator-mxnet/pull/10365#issuecomment-37790 I'd vote for using a stable version instead of the latest master. @xinyu-intel @pengzhao-intel what's the stability of the

[GitHub] marcoabreu commented on issue #10353: CI does not check with CuDNN=0

2018-04-02 Thread GitBox
marcoabreu commented on issue #10353: CI does not check with CuDNN=0 URL: https://github.com/apache/incubator-mxnet/issues/10353#issuecomment-377909647 I'd propose to add specific excludes (for these non-runnable tests) to the nosetest call in runtime_functions.sh.

[GitHub] marcoabreu commented on issue #10353: CI does not check with CuDNN=0

2018-04-02 Thread GitBox
marcoabreu commented on issue #10353: CI does not check with CuDNN=0 URL: https://github.com/apache/incubator-mxnet/issues/10353#issuecomment-377909647 I'd propose to add specific excludes (for these non-runnable tests) to the nosetest call in runtime_functions.sh. Other options

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

2018-04-02 Thread GitBox
marcoabreu commented on issue #10365: [MXNET-261]Update MKLDNN & Add CPP Test URL: https://github.com/apache/incubator-mxnet/pull/10365#issuecomment-37790 I'd vote for using a stable version instead of the latest master

[GitHub] solin319 opened a new pull request #10366: fix bug in sgd

2018-04-02 Thread GitBox
solin319 opened a new pull request #10366: fix bug in sgd URL: https://github.com/apache/incubator-mxnet/pull/10366 `ResourceRequest` will make `sgd_mom_update` can't overlap the backward computation. #9782 So can we delete `ResourceRequest` when register `sgd_mom_update`.

[GitHub] zheng-da commented on issue #10353: CI does not check with CuDNN=0

2018-04-02 Thread GitBox
zheng-da commented on issue #10353: CI does not check with CuDNN=0 URL: https://github.com/apache/incubator-mxnet/issues/10353#issuecomment-377878897 We should definitely test build. The question is whether we should run the code and how? Some test code can't run without CuDNN as we have

[GitHub] jinhuang415 commented on a change in pull request #10336: Fix MKLDNN sigmoid/softrelu issue

2018-04-02 Thread GitBox
jinhuang415 commented on a change in pull request #10336: Fix MKLDNN sigmoid/softrelu issue URL: https://github.com/apache/incubator-mxnet/pull/10336#discussion_r178503829 ## File path: tests/python/unittest/test_gluon.py ## @@ -717,8 +717,8 @@ def test_lambda():

[GitHub] zheng-da commented on issue #10365: Update MKLDNN & Add CPP Test

2018-04-02 Thread GitBox
zheng-da commented on issue #10365: Update MKLDNN & Add CPP Test URL: https://github.com/apache/incubator-mxnet/pull/10365#issuecomment-377878370 Should we update MKLDNN to the latest commit in its master branch? or should we always attach it to a certain version tag? What rule should we

[GitHub] xinyu-intel opened a new pull request #10365: Update MKLDNN & Add CPP Test

2018-04-02 Thread GitBox
xinyu-intel opened a new pull request #10365: Update MKLDNN & Add CPP Test URL: https://github.com/apache/incubator-mxnet/pull/10365 ## Description ## This pr aims to fix bugs in #8712 by update MKLDNN to the newest. CPP tests are added to monitor data format change of MKL-DNN

[GitHub] zheng-da commented on a change in pull request #10336: Fix MKLDNN sigmoid/softrelu issue

2018-04-02 Thread GitBox
zheng-da commented on a change in pull request #10336: Fix MKLDNN sigmoid/softrelu issue URL: https://github.com/apache/incubator-mxnet/pull/10336#discussion_r178500575 ## File path: tests/python/unittest/test_gluon.py ## @@ -717,8 +717,8 @@ def test_lambda():

[GitHub] zheng-da commented on a change in pull request #10336: Fix MKLDNN sigmoid/softrelu issue

2018-04-02 Thread GitBox
zheng-da commented on a change in pull request #10336: Fix MKLDNN sigmoid/softrelu issue URL: https://github.com/apache/incubator-mxnet/pull/10336#discussion_r178500575 ## File path: tests/python/unittest/test_gluon.py ## @@ -717,8 +717,8 @@ def test_lambda():

[GitHub] cgraywang commented on issue #10363: Fix windows setup doc using VS 2017

2018-04-02 Thread GitBox
cgraywang commented on issue #10363: Fix windows setup doc using VS 2017 URL: https://github.com/apache/incubator-mxnet/pull/10363#issuecomment-377867492 @aaronmarkham This is an automated message from the Apache Git

[GitHub] cgraywang commented on issue #10363: Fix windows setup doc using VS 2017

2018-04-02 Thread GitBox
cgraywang commented on issue #10363: Fix windows setup doc using VS 2017 URL: https://github.com/apache/incubator-mxnet/pull/10363#issuecomment-377866992 @sxjscience please take a look at this as well. This is an automated

[GitHub] yajiedesign opened a new pull request #10364: [MXNET-260]remove use_fast_math

2018-04-02 Thread GitBox
yajiedesign opened a new pull request #10364: [MXNET-260]remove use_fast_math URL: https://github.com/apache/incubator-mxnet/pull/10364 ## Description ## remove use_fast_math ## Checklist ## ### Essentials ### Please feel free to remove inapplicable items for your PR. -

[GitHub] cgraywang opened a new pull request #10363: Fix windows setup doc using VS 2017

2018-04-02 Thread GitBox
cgraywang opened a new pull request #10363: Fix windows setup doc using VS 2017 URL: https://github.com/apache/incubator-mxnet/pull/10363 ## Description ## Update the MXNet windows installation doc by adding the details about how to install MXNet from source on windows using VS

[GitHub] anirudhacharya commented on issue #10283: [MXNET-242][Tutorial] Fine-tuning ONNX model in Gluon

2018-04-02 Thread GitBox
anirudhacharya commented on issue #10283: [MXNET-242][Tutorial] Fine-tuning ONNX model in Gluon URL: https://github.com/apache/incubator-mxnet/pull/10283#issuecomment-377863372 @ThomasDelteil please update this tutorial in the ONNX-MXNet API page -

<    1   2