[GitHub] rahul003 commented on issue #11406: [MXNET-599] Partial shape infer for Slice

2018-07-01 Thread GitBox
rahul003 commented on issue #11406: [MXNET-599] Partial shape infer for Slice URL: https://github.com/apache/incubator-mxnet/pull/11406#issuecomment-401680023 Sorry I realized I've not pushed the latest changes to github. I'll update the PR tomorrow

[GitHub] lupesko commented on issue #9649: TEST(MEMORY_TEST, MemsetAndMemcopyPerformance) sometimes failing with timer failures

2018-07-01 Thread GitBox
lupesko commented on issue #9649: TEST(MEMORY_TEST, MemsetAndMemcopyPerformance) sometimes failing with timer failures URL: https://github.com/apache/incubator-mxnet/issues/9649#issuecomment-401675243 @KellenSunderland the PR removing the test was merged. Kindly close the issue. Thanks!

[GitHub] larroy commented on issue #11483: Compile with LAPACK shared library

2018-07-01 Thread GitBox
larroy commented on issue #11483: Compile with LAPACK shared library URL: https://github.com/apache/incubator-mxnet/pull/11483#issuecomment-401672657 LGTM This is an automated message from the Apache Git Service. To respond

[GitHub] larroy commented on issue #11514: [MXNET-617] - Improve Clojure styles with cljfmt tools

2018-07-01 Thread GitBox
larroy commented on issue #11514: [MXNET-617] - Improve Clojure styles with cljfmt tools URL: https://github.com/apache/incubator-mxnet/pull/11514#issuecomment-401671639 LGTM This is an automated message from the Apache Git

[GitHub] agilecreativity commented on issue #11514: [MXNET-617] - Improve Clojure styles with cljfmt tools

2018-07-01 Thread GitBox
agilecreativity commented on issue #11514: [MXNET-617] - Improve Clojure styles with cljfmt tools URL: https://github.com/apache/incubator-mxnet/pull/11514#issuecomment-401670731 @gigasquid I updated the README with the instruction to how to run it. If you like to edit or adjust

[GitHub] nehaljwani commented on issue #11513: Add path_suffixes for finding cudnn.lib

2018-07-01 Thread GitBox
nehaljwani commented on issue #11513: Add path_suffixes for finding cudnn.lib URL: https://github.com/apache/incubator-mxnet/pull/11513#issuecomment-401669058 Done. This is an automated message from the Apache Git Service. To

[GitHub] larroy commented on issue #11518: Disable test_pooling_versions

2018-07-01 Thread GitBox
larroy commented on issue #11518: Disable test_pooling_versions URL: https://github.com/apache/incubator-mxnet/pull/11518#issuecomment-401669111 Why we disable instead of fixing? This is an automated message from the Apache

[GitHub] larroy commented on a change in pull request #11520: [WIP] Improve Ccache

2018-07-01 Thread GitBox
larroy commented on a change in pull request #11520: [WIP] Improve Ccache URL: https://github.com/apache/incubator-mxnet/pull/11520#discussion_r199381460 ## File path: ci/docker/runtime_functions.sh ## @@ -33,35 +33,23 @@ clean_repo() { git submodule update --init

[GitHub] larroy commented on a change in pull request #11520: [WIP] Improve Ccache

2018-07-01 Thread GitBox
larroy commented on a change in pull request #11520: [WIP] Improve Ccache URL: https://github.com/apache/incubator-mxnet/pull/11520#discussion_r199381404 ## File path: ci/docker/install/ubuntu_adduser.sh ## @@ -34,4 +34,5 @@ then mkdir /work/mxnet mkdir

[GitHub] sandeep-krishnamurthy commented on issue #9116: unit test test_ndarray.test_ndarray_indexing fails

2018-07-01 Thread GitBox
sandeep-krishnamurthy commented on issue #9116: unit test test_ndarray.test_ndarray_indexing fails URL: https://github.com/apache/incubator-mxnet/issues/9116#issuecomment-401664901 Can you please confirm you tried with same build that was failing before? For example, I had a test that

[GitHub] aaronmarkham commented on issue #11503: updating installation info to have latest packages and more clarity

2018-07-01 Thread GitBox
aaronmarkham commented on issue #11503: updating installation info to have latest packages and more clarity URL: https://github.com/apache/incubator-mxnet/pull/11503#issuecomment-401653928 @nswamy - updated Ubuntu instructions as discussed (noting that >16.04 doesn't work).

[GitHub] kiendang commented on a change in pull request #11483: Compile with LAPACK shared library

2018-07-01 Thread GitBox
kiendang commented on a change in pull request #11483: Compile with LAPACK shared library URL: https://github.com/apache/incubator-mxnet/pull/11483#discussion_r199371834 ## File path: Makefile ## @@ -157,9 +157,13 @@ endif ifeq ($(USE_LAPACK), 1) ifeq

[GitHub] marcoabreu opened a new pull request #11520: [WIP] Improve Ccache

2018-07-01 Thread GitBox
marcoabreu opened a new pull request #11520: [WIP] Improve Ccache URL: https://github.com/apache/incubator-mxnet/pull/11520 ## Description ## https://github.com/apache/incubator-mxnet/issues/11516 ## Checklist ## ### Essentials ### Please feel free to remove inapplicable

[GitHub] eric-haibin-lin opened a new pull request #11519: re-enable flaky test #9384

2018-07-01 Thread GitBox
eric-haibin-lin opened a new pull request #11519: re-enable flaky test #9384 URL: https://github.com/apache/incubator-mxnet/pull/11519 ## Description ## Tested locally 20K time, not able to reproduce. ## Checklist ## ### Essentials ### - [ ] Passed code style checking

[incubator-mxnet-site] branch asf-site updated: Bump the publish timestamp.

2018-07-01 Thread zhasheng
This is an automated email from the ASF dual-hosted git repository. zhasheng pushed a commit to branch asf-site in repository https://gitbox.apache.org/repos/asf/incubator-mxnet-site.git The following commit(s) were added to refs/heads/asf-site by this push: new c208393 Bump the publish

[GitHub] chinakook commented on a change in pull request #11003: rnn_cell little bug fixed

2018-07-01 Thread GitBox
chinakook commented on a change in pull request #11003: rnn_cell little bug fixed URL: https://github.com/apache/incubator-mxnet/pull/11003#discussion_r199366887 ## File path: tests/python/unittest/test_gluon_rnn.py ## @@ -196,6 +244,26 @@ def test_zoneout(): assert

[GitHub] chinakook commented on a change in pull request #11003: rnn_cell little bug fixed

2018-07-01 Thread GitBox
chinakook commented on a change in pull request #11003: rnn_cell little bug fixed URL: https://github.com/apache/incubator-mxnet/pull/11003#discussion_r199366887 ## File path: tests/python/unittest/test_gluon_rnn.py ## @@ -196,6 +244,26 @@ def test_zoneout(): assert

[GitHub] anirudhacharya commented on issue #11509: Hanging flaky test test_operator.test_norm @ Python 3: GPU Win

2018-07-01 Thread GitBox
anirudhacharya commented on issue #11509: Hanging flaky test test_operator.test_norm @ Python 3: GPU Win URL: https://github.com/apache/incubator-mxnet/issues/11509#issuecomment-401643400 I will look into it This is an

[GitHub] eric-haibin-lin edited a comment on issue #11173: Flaky test: test_deformable_convolution_options

2018-07-01 Thread GitBox
eric-haibin-lin edited a comment on issue #11173: Flaky test: test_deformable_convolution_options URL: https://github.com/apache/incubator-mxnet/issues/11173#issuecomment-401642974 I'm able to reproduce the err with the following example in test_operator_gpu.py: ``` @with_seed()

[GitHub] eric-haibin-lin commented on issue #11173: Flaky test: test_deformable_convolution_options

2018-07-01 Thread GitBox
eric-haibin-lin commented on issue #11173: Flaky test: test_deformable_convolution_options URL: https://github.com/apache/incubator-mxnet/issues/11173#issuecomment-401642974 I'm able to reproduce the err with the following example in test_operator_gpu.py: ``` @with_seed() def

[GitHub] szha closed issue #8971: Clojure Library for mxnet

2018-07-01 Thread GitBox
szha closed issue #8971: Clojure Library for mxnet URL: https://github.com/apache/incubator-mxnet/issues/8971 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use

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

2018-07-01 Thread GitBox
danielcompton commented on issue #8971: Clojure Library for mxnet URL: https://github.com/apache/incubator-mxnet/issues/8971#issuecomment-401640437 I think this issue can be closed now that #11205 has been merged? This is an

[GitHub] zheng-da commented on issue #11473: batchnorm falls back for sparse arrays.

2018-07-01 Thread GitBox
zheng-da commented on issue #11473: batchnorm falls back for sparse arrays. URL: https://github.com/apache/incubator-mxnet/pull/11473#issuecomment-401639154 @eric-haibin-lin any more comments with this PR? This is an

[GitHub] szha closed pull request #11511: add test for #11003

2018-07-01 Thread GitBox
szha closed pull request #11511: add test for #11003 URL: https://github.com/apache/incubator-mxnet/pull/11511 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] eric-haibin-lin commented on issue #10278: Flaky test_sparse_storage_fallback (?)

2018-07-01 Thread GitBox
eric-haibin-lin commented on issue #10278: Flaky test_sparse_storage_fallback (?) URL: https://github.com/apache/incubator-mxnet/issues/10278#issuecomment-401637668 Cannot reproduce both locally and on CI. closing. This is

[incubator-mxnet] branch master updated: add test for #11003 (#11511)

2018-07-01 Thread zhasheng
This is an automated email from the ASF dual-hosted git repository. zhasheng pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git The following commit(s) were added to refs/heads/master by this push: new cba2b03 add test for #11003

[GitHub] marcoabreu commented on issue #11021: [MXNET-380] count_include_pad argument for Avg Pooling

2018-07-01 Thread GitBox
marcoabreu commented on issue #11021: [MXNET-380] count_include_pad argument for Avg Pooling URL: https://github.com/apache/incubator-mxnet/pull/11021#issuecomment-401637667 Ah sorry, I just thought it might be this PR since it was the last one that modified the test. I've created an

[GitHub] marcoabreu edited a comment on issue #11021: [MXNET-380] count_include_pad argument for Avg Pooling

2018-07-01 Thread GitBox
marcoabreu edited a comment on issue #11021: [MXNET-380] count_include_pad argument for Avg Pooling URL: https://github.com/apache/incubator-mxnet/pull/11021#issuecomment-401637667 Ah sorry, I just thought it might be this PR since it was the last one that modified the overall test. I've

[GitHub] eric-haibin-lin closed issue #10278: Flaky test_sparse_storage_fallback (?)

2018-07-01 Thread GitBox
eric-haibin-lin closed issue #10278: Flaky test_sparse_storage_fallback (?) URL: https://github.com/apache/incubator-mxnet/issues/10278 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] eric-haibin-lin closed pull request #11464: [WIP] fix flaky test #10278

2018-07-01 Thread GitBox
eric-haibin-lin closed pull request #11464: [WIP] fix flaky test #10278 URL: https://github.com/apache/incubator-mxnet/pull/11464 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] haojin2 commented on issue #11021: [MXNET-380] count_include_pad argument for Avg Pooling

2018-07-01 Thread GitBox
haojin2 commented on issue #11021: [MXNET-380] count_include_pad argument for Avg Pooling URL: https://github.com/apache/incubator-mxnet/pull/11021#issuecomment-401637354 `test_2d_pooling('max')` the test is failing at max pooling, which does not even hit my code paths.

[GitHub] agilecreativity commented on a change in pull request #11514: [MXNET-617] - Improve Clojure styles with cljfmt tools

2018-07-01 Thread GitBox
agilecreativity commented on a change in pull request #11514: [MXNET-617] - Improve Clojure styles with cljfmt tools URL: https://github.com/apache/incubator-mxnet/pull/11514#discussion_r199363498 ## File path: contrib/clojure-package/ci-test.sh ## @@ -1,4 +1,5 @@

[GitHub] marcoabreu commented on issue #11021: [MXNET-380] count_include_pad argument for Avg Pooling

2018-07-01 Thread GitBox
marcoabreu commented on issue #11021: [MXNET-380] count_include_pad argument for Avg Pooling URL: https://github.com/apache/incubator-mxnet/pull/11021#issuecomment-401637223 Hi @haojin2 , I think your PR has made test_pooling_versions flaky. See details at

[GitHub] marcoabreu opened a new pull request #11518: Disable test_pooling_versions

2018-07-01 Thread GitBox
marcoabreu opened a new pull request #11518: Disable test_pooling_versions URL: https://github.com/apache/incubator-mxnet/pull/11518 ## Description ## Disable flaky test https://github.com/apache/incubator-mxnet/issues/11517 ## Checklist ## ### Essentials ### Please feel

[GitHub] marcoabreu opened a new issue #11517: Flaky test test_operator_gpu.test_pooling_versions

2018-07-01 Thread GitBox
marcoabreu opened a new issue #11517: Flaky test test_operator_gpu.test_pooling_versions URL: https://github.com/apache/incubator-mxnet/issues/11517 http://jenkins.mxnet-ci.amazon-ml.com/blue/organizations/jenkins/incubator-mxnet/detail/master/1121/pipeline ```

[GitHub] agilecreativity commented on a change in pull request #11514: [MXNET-617] - Improve Clojure styles with cljfmt tools

2018-07-01 Thread GitBox
agilecreativity commented on a change in pull request #11514: [MXNET-617] - Improve Clojure styles with cljfmt tools URL: https://github.com/apache/incubator-mxnet/pull/11514#discussion_r199363016 ## File path: contrib/clojure-package/test/good-test-ndarray.clj ## @@

[GitHub] marcoabreu commented on a change in pull request #11514: [MXNET-617] - Improve Clojure styles with cljfmt tools

2018-07-01 Thread GitBox
marcoabreu commented on a change in pull request #11514: [MXNET-617] - Improve Clojure styles with cljfmt tools URL: https://github.com/apache/incubator-mxnet/pull/11514#discussion_r199362917 ## File path: contrib/clojure-package/ci-test.sh ## @@ -1,4 +1,5 @@ #!/bin/bash

[GitHub] gigasquid commented on a change in pull request #11514: [MXNET-617] - Improve Clojure styles with cljfmt tools

2018-07-01 Thread GitBox
gigasquid commented on a change in pull request #11514: [MXNET-617] - Improve Clojure styles with cljfmt tools URL: https://github.com/apache/incubator-mxnet/pull/11514#discussion_r199361754 ## File path: contrib/clojure-package/test/good-test-symbol.clj ## @@ -1,9 +1,9

[GitHub] gigasquid commented on a change in pull request #11514: [MXNET-617] - Improve Clojure styles with cljfmt tools

2018-07-01 Thread GitBox
gigasquid commented on a change in pull request #11514: [MXNET-617] - Improve Clojure styles with cljfmt tools URL: https://github.com/apache/incubator-mxnet/pull/11514#discussion_r199361705 ## File path: contrib/clojure-package/test/good-test-ndarray.clj ## @@ -22,15

[GitHub] gigasquid commented on a change in pull request #11514: [MXNET-617] - Improve Clojure styles with cljfmt tools

2018-07-01 Thread GitBox
gigasquid commented on a change in pull request #11514: [MXNET-617] - Improve Clojure styles with cljfmt tools URL: https://github.com/apache/incubator-mxnet/pull/11514#discussion_r199361561 ## File path: contrib/clojure-package/ci-test.sh ## @@ -1,4 +1,5 @@ #!/bin/bash

[GitHub] marcoabreu opened a new issue #11516: CCache does not work for NVCC or make-based builds

2018-07-01 Thread GitBox
marcoabreu opened a new issue #11516: CCache does not work for NVCC or make-based builds URL: https://github.com/apache/incubator-mxnet/issues/11516 At the moment, we don't apply ccache to NVCC compilation or to make-based builds in general. This is due to third parties not supporting our

[GitHub] agilecreativity commented on issue #11514: [MXNET-617] - Improve Clojure styles with cljfmt tools

2018-07-01 Thread GitBox
agilecreativity commented on issue #11514: [MXNET-617] - Improve Clojure styles with cljfmt tools URL: https://github.com/apache/incubator-mxnet/pull/11514#issuecomment-401634175 Thanks @marcoabreu for your feedback. I will address the issue shortly and other issue from @gigasquid.

[GitHub] KellenSunderland opened a new pull request #11515: WIP NVTX Windows builds

2018-07-01 Thread GitBox
KellenSunderland opened a new pull request #11515: WIP NVTX Windows builds URL: https://github.com/apache/incubator-mxnet/pull/11515 WIP This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] marcoabreu edited a comment on issue #11514: [MXNET-617] - Improve Clojure styles with cljfmt tools

2018-07-01 Thread GitBox
marcoabreu edited a comment on issue #11514: [MXNET-617] - Improve Clojure styles with cljfmt tools URL: https://github.com/apache/incubator-mxnet/pull/11514#issuecomment-401633292 Hi, thanks a lot for the contribution! Could you add this logic here

[GitHub] marcoabreu edited a comment on issue #11514: [MXNET-617] - Improve Clojure styles with cljfmt tools

2018-07-01 Thread GitBox
marcoabreu edited a comment on issue #11514: [MXNET-617] - Improve Clojure styles with cljfmt tools URL: https://github.com/apache/incubator-mxnet/pull/11514#issuecomment-401633292 Hi, thanks a lot for the contribution! Could you add this to the Makefile and link the logic here

[GitHub] marcoabreu commented on issue #11514: [MXNET-617] - Improve Clojure styles with cljfmt tools

2018-07-01 Thread GitBox
marcoabreu commented on issue #11514: [MXNET-617] - Improve Clojure styles with cljfmt tools URL: https://github.com/apache/incubator-mxnet/pull/11514#issuecomment-401633292 Hi, thanks a lot for the contribution? Could you add this to the Makefile and link the logic here

[GitHub] gigasquid commented on issue #11514: [MXNET-617] - Improve Clojure styles with cljfmt tools

2018-07-01 Thread GitBox
gigasquid commented on issue #11514: [MXNET-617] - Improve Clojure styles with cljfmt tools URL: https://github.com/apache/incubator-mxnet/pull/11514#issuecomment-401632501 Thanks @agilecreativity for this contribution  - I'll take a look at review the changes

[GitHub] KellenSunderland commented on issue #11513: Add path_suffixes for finding cudnn.lib

2018-07-01 Thread GitBox
KellenSunderland commented on issue #11513: Add path_suffixes for finding cudnn.lib URL: https://github.com/apache/incubator-mxnet/pull/11513#issuecomment-401629155 I wonder if you'd also be willing to contribute the change to this project?

[GitHub] nehaljwani commented on issue #11513: Add path_suffixes for finding cudnn.lib

2018-07-01 Thread GitBox
nehaljwani commented on issue #11513: Add path_suffixes for finding cudnn.lib URL: https://github.com/apache/incubator-mxnet/pull/11513#issuecomment-401629131 FWIW, I was using CMake 3.11.1 This is an automated message from

[GitHub] KellenSunderland commented on issue #11513: Add path_suffixes for finding cudnn.lib

2018-07-01 Thread GitBox
KellenSunderland commented on issue #11513: Add path_suffixes for finding cudnn.lib URL: https://github.com/apache/incubator-mxnet/pull/11513#issuecomment-401629067 This for the quick reply. I wanted to verify since this file seems quite out of date to me. I'll try setting CUDNN_ROOT

[GitHub] nswamy closed pull request #11250: [MXNET-531] MNIST Examples for Scala new API

2018-07-01 Thread GitBox
nswamy closed pull request #11250: [MXNET-531] MNIST Examples for Scala new API URL: https://github.com/apache/incubator-mxnet/pull/11250 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

[incubator-mxnet] branch master updated: [MXNET-531] MNIST Examples for Scala new API (#11250)

2018-07-01 Thread nswamy
This is an automated email from the ASF dual-hosted git repository. nswamy 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 5627bc3 [MXNET-531] MNIST Examples

[GitHub] nswamy commented on a change in pull request #11250: [MXNET-531] MNIST Examples for Scala new API

2018-07-01 Thread GitBox
nswamy commented on a change in pull request #11250: [MXNET-531] MNIST Examples for Scala new API URL: https://github.com/apache/incubator-mxnet/pull/11250#discussion_r199359292 ## File path:

[GitHub] lanking520 commented on a change in pull request #11239: [MXNET-319] Javadoc fix

2018-07-01 Thread GitBox
lanking520 commented on a change in pull request #11239: [MXNET-319] Javadoc fix URL: https://github.com/apache/incubator-mxnet/pull/11239#discussion_r199359185 ## File path: scala-package/macros/src/main/scala/org/apache/mxnet/APIDocGenerator.scala ## @@ -55,16 +69,44 @@

[GitHub] nswamy commented on a change in pull request #11239: [MXNET-319] Javadoc fix

2018-07-01 Thread GitBox
nswamy commented on a change in pull request #11239: [MXNET-319] Javadoc fix URL: https://github.com/apache/incubator-mxnet/pull/11239#discussion_r199358997 ## File path: scala-package/macros/src/main/scala/org/apache/mxnet/APIDocGenerator.scala ## @@ -55,16 +69,44 @@

[GitHub] nswamy commented on a change in pull request #11239: [MXNET-319] Javadoc fix

2018-07-01 Thread GitBox
nswamy commented on a change in pull request #11239: [MXNET-319] Javadoc fix URL: https://github.com/apache/incubator-mxnet/pull/11239#discussion_r199358997 ## File path: scala-package/macros/src/main/scala/org/apache/mxnet/APIDocGenerator.scala ## @@ -55,16 +69,44 @@

[GitHub] nehaljwani edited a comment on issue #11513: Add path_suffixes for finding cudnn.lib

2018-07-01 Thread GitBox
nehaljwani edited a comment on issue #11513: Add path_suffixes for finding cudnn.lib URL: https://github.com/apache/incubator-mxnet/pull/11513#issuecomment-401627588 I am setting CUDNN_ROOT as an environment variable, if that makes any difference. Also, I traced it being called from:

[GitHub] nehaljwani commented on issue #11513: Add path_suffixes for finding cudnn.lib

2018-07-01 Thread GitBox
nehaljwani commented on issue #11513: Add path_suffixes for finding cudnn.lib URL: https://github.com/apache/incubator-mxnet/pull/11513#issuecomment-401627645 This is still present in the current HEAD:

[GitHub] nehaljwani commented on issue #11513: Add path_suffixes for finding cudnn.lib

2018-07-01 Thread GitBox
nehaljwani commented on issue #11513: Add path_suffixes for finding cudnn.lib URL: https://github.com/apache/incubator-mxnet/pull/11513#issuecomment-401627588 I am setting CUDNN_ROOT as an environment variable, if it that makes any difference. Also, I traced it being called from:

[GitHub] nehaljwani edited a comment on issue #11513: Add path_suffixes for finding cudnn.lib

2018-07-01 Thread GitBox
nehaljwani edited a comment on issue #11513: Add path_suffixes for finding cudnn.lib URL: https://github.com/apache/incubator-mxnet/pull/11513#issuecomment-401627446 @KellenSunderland Yes, it is actually being called.. That's is how I found out the issue and then set a PR, because

[GitHub] nehaljwani commented on issue #11513: Add path_suffixes for finding cudnn.lib

2018-07-01 Thread GitBox
nehaljwani commented on issue #11513: Add path_suffixes for finding cudnn.lib URL: https://github.com/apache/incubator-mxnet/pull/11513#issuecomment-401627446 @KellenSunderland Yes, I do. That's is how I found out the issue and then set a PR, because CUDNN_INCLUDE was being set, but

[GitHub] agilecreativity opened a new pull request #11514: [MXNET-617] - Improve style with cljfmt tools

2018-07-01 Thread GitBox
agilecreativity opened a new pull request #11514: [MXNET-617] - Improve style with cljfmt tools URL: https://github.com/apache/incubator-mxnet/pull/11514 - Add cljfmt to the plugins vector - Add script to run `lein cljfmt check` to for check for style - Add script to run `lein cljfmt

[GitHub] KellenSunderland commented on issue #11513: Add path_suffixes for finding cudnn.lib

2018-07-01 Thread GitBox
KellenSunderland commented on issue #11513: Add path_suffixes for finding cudnn.lib URL: https://github.com/apache/incubator-mxnet/pull/11513#issuecomment-401627368 Is this code actually called when running CMake on windows? Can you try and modify the message that reads "Found cuDNN

[GitHub] eric-haibin-lin commented on issue #11509: Hanging flaky test test_operator.test_norm @ Python 3: GPU Win

2018-07-01 Thread GitBox
eric-haibin-lin commented on issue #11509: Hanging flaky test test_operator.test_norm @ Python 3: GPU Win URL: https://github.com/apache/incubator-mxnet/issues/11509#issuecomment-401625819 @anirudhacharya This is an

[GitHub] nehaljwani opened a new pull request #11513: Add path_suffixes for finding cudnn.lib

2018-07-01 Thread GitBox
nehaljwani opened a new pull request #11513: Add path_suffixes for finding cudnn.lib URL: https://github.com/apache/incubator-mxnet/pull/11513 On Windows, the directory structure layout for CUDNN is: ``` /t/incubator-mxnet ❯❯❯ unzip -l /tmp/cudnn-9.0-windows10-x64-v7.1.zip

[incubator-mxnet-site] branch asf-site updated: Bump the publish timestamp.

2018-07-01 Thread zhasheng
This is an automated email from the ASF dual-hosted git repository. zhasheng pushed a commit to branch asf-site in repository https://gitbox.apache.org/repos/asf/incubator-mxnet-site.git The following commit(s) were added to refs/heads/asf-site by this push: new 1a7f219 Bump the publish

[GitHub] nswamy commented on a change in pull request #11205: Clojure Contrib Package

2018-07-01 Thread GitBox
nswamy commented on a change in pull request #11205: Clojure Contrib Package URL: https://github.com/apache/incubator-mxnet/pull/11205#discussion_r199356836 ## File path: contrib/clojure-package/src/org/apache/clojure_mxnet/gen/ndarray.clj ## @@ -0,0 +1,2312 @@ +(ns

[GitHub] eric-haibin-lin edited a comment on issue #11429: Improve sparse pull performance for gluon trainer

2018-07-01 Thread GitBox
eric-haibin-lin edited a comment on issue #11429: Improve sparse pull performance for gluon trainer URL: https://github.com/apache/incubator-mxnet/pull/11429#issuecomment-401623044 @szha For dense weight with sparse grad, this PR pulls sparse grad instead of dense weight. Reduces gpu2gpu

[GitHub] eric-haibin-lin commented on issue #11429: Improve sparse pull performance for gluon trainer

2018-07-01 Thread GitBox
eric-haibin-lin commented on issue #11429: Improve sparse pull performance for gluon trainer URL: https://github.com/apache/incubator-mxnet/pull/11429#issuecomment-401623044 @szha For dense weight with sparse grad, this PR avoids pulls sparse grad instead of dense weight. Reduces gpu2gpu

[GitHub] nswamy closed issue #8090: JVM crash on Mac

2018-07-01 Thread GitBox
nswamy closed issue #8090: JVM crash on Mac URL: https://github.com/apache/incubator-mxnet/issues/8090 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the

[GitHub] nswamy commented on issue #8090: JVM crash on Mac

2018-07-01 Thread GitBox
nswamy commented on issue #8090: JVM crash on Mac URL: https://github.com/apache/incubator-mxnet/issues/8090#issuecomment-401623023 MXNet 0.11 is very old, could you please try upgrading to the latest 1.2.1 which is coming in the next day or two. We also have the Scala packages

[GitHub] eric-haibin-lin commented on a change in pull request #11429: Improve sparse pull performance for gluon trainer

2018-07-01 Thread GitBox
eric-haibin-lin commented on a change in pull request #11429: Improve sparse pull performance for gluon trainer URL: https://github.com/apache/incubator-mxnet/pull/11429#discussion_r199356565 ## File path: python/mxnet/gluon/trainer.py ## @@ -169,9 +190,9 @@ def

[GitHub] szha commented on a change in pull request #11483: Compile with LAPACK shared library

2018-07-01 Thread GitBox
szha commented on a change in pull request #11483: Compile with LAPACK shared library URL: https://github.com/apache/incubator-mxnet/pull/11483#discussion_r199352789 ## File path: Makefile ## @@ -157,9 +157,13 @@ endif ifeq ($(USE_LAPACK), 1) ifeq ($(USE_BLAS),$(filter

[GitHub] wkcn commented on issue #11423: contrib.BilinearResize2D has restriction on tensor's resolution (1000 x 1000)

2018-07-01 Thread GitBox
wkcn commented on issue #11423: contrib.BilinearResize2D has restriction on tensor's resolution (1000 x 1000) URL: https://github.com/apache/incubator-mxnet/issues/11423#issuecomment-401612537 @two-names Yes. I found several operators use set_range(1, 1000). It's necessary to

[GitHub] two-names commented on issue #11423: contrib.BilinearResize2D has restriction on tensor's resolution (1000 x 1000)

2018-07-01 Thread GitBox
two-names commented on issue #11423: contrib.BilinearResize2D has restriction on tensor's resolution (1000 x 1000) URL: https://github.com/apache/incubator-mxnet/issues/11423#issuecomment-401606130 @wkcn You mean declare new variables? I suppose it is necessary to expand the

[GitHub] wkcn commented on issue #11423: contrib.BilinearResize2D has restriction on tensor's resolution (1000 x 1000)

2018-07-01 Thread GitBox
wkcn commented on issue #11423: contrib.BilinearResize2D has restriction on tensor's resolution (1000 x 1000) URL: https://github.com/apache/incubator-mxnet/issues/11423#issuecomment-401604733 Is it better to add the minimum or maximum value for `set_range` in dmlc-core? e.g.

[incubator-mxnet] branch master updated: Add licenses to clojure package (#11512)

2018-07-01 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 37c8165 Add licenses to clojure

[GitHub] marcoabreu closed pull request #11512: [MXNET-616] Add licenses to files introduced by clojure package #11205

2018-07-01 Thread GitBox
marcoabreu closed pull request #11512: [MXNET-616] Add licenses to files introduced by clojure package #11205 URL: https://github.com/apache/incubator-mxnet/pull/11512 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the

[GitHub] pengzhao-intel commented on issue #10696: [MXNET-366]Extend MXNet Distributed Training by AllReduce

2018-07-01 Thread GitBox
pengzhao-intel commented on issue #10696: [MXNET-366]Extend MXNet Distributed Training by AllReduce URL: https://github.com/apache/incubator-mxnet/pull/10696#issuecomment-401604329 @threeleafzerg good catch! Could we improve the error message/doc to make the end user know what's wrong

[GitHub] threeleafzerg edited a comment on issue #10696: [MXNET-366]Extend MXNet Distributed Training by AllReduce

2018-07-01 Thread GitBox
threeleafzerg edited a comment on issue #10696: [MXNET-366]Extend MXNet Distributed Training by AllReduce URL: https://github.com/apache/incubator-mxnet/pull/10696#issuecomment-401604031 @ctcyang Your running command is not correct to launch kvstore with type allreduce It should

[GitHub] threeleafzerg commented on issue #10696: [MXNET-366]Extend MXNet Distributed Training by AllReduce

2018-07-01 Thread GitBox
threeleafzerg commented on issue #10696: [MXNET-366]Extend MXNet Distributed Training by AllReduce URL: https://github.com/apache/incubator-mxnet/pull/10696#issuecomment-401604031 @ctcyang Your running command is not correct to launch kvstore with type allreduce It should be:

[incubator-mxnet-site] branch asf-site updated: Bump the publish timestamp.

2018-07-01 Thread zhasheng
This is an automated email from the ASF dual-hosted git repository. zhasheng pushed a commit to branch asf-site in repository https://gitbox.apache.org/repos/asf/incubator-mxnet-site.git The following commit(s) were added to refs/heads/asf-site by this push: new bb6352f Bump the publish

[GitHub] gigasquid commented on a change in pull request #11205: Clojure Contrib Package

2018-07-01 Thread GitBox
gigasquid commented on a change in pull request #11205: Clojure Contrib Package URL: https://github.com/apache/incubator-mxnet/pull/11205#discussion_r199345706 ## File path: contrib/clojure-package/src/org/apache/clojure_mxnet/gen/ndarray.clj ## @@ -0,0 +1,2312 @@ +(ns

[GitHub] marcoabreu commented on issue #11512: [MXNET-616] Add licenses to files introduced by clojure package #11205

2018-07-01 Thread GitBox
marcoabreu commented on issue #11512: [MXNET-616] Add licenses to files introduced by clojure package #11205 URL: https://github.com/apache/incubator-mxnet/pull/11512#issuecomment-401599723 Thank you! It'd be great if you could do it as part of a separate PR.

[GitHub] gigasquid commented on a change in pull request #11205: Clojure Contrib Package

2018-07-01 Thread GitBox
gigasquid commented on a change in pull request #11205: Clojure Contrib Package URL: https://github.com/apache/incubator-mxnet/pull/11205#discussion_r199345684 ## File path: contrib/clojure-package/project.clj ## @@ -0,0 +1,54 @@ +;; +;; Licensed to the Apache Software

[GitHub] hqucms commented on issue #11493: Fix MXPredReshape in the c_predict_api

2018-07-01 Thread GitBox
hqucms commented on issue #11493: Fix MXPredReshape in the c_predict_api URL: https://github.com/apache/incubator-mxnet/pull/11493#issuecomment-401598819 @szha Any suggestions where the tests should be added? I am not very familiar with the organization of the test units.

[GitHub] marcoabreu opened a new pull request #11512: [MXNET-616] Add licenses to files introduced by clojure package #11205

2018-07-01 Thread GitBox
marcoabreu opened a new pull request #11512: [MXNET-616] Add licenses to files introduced by clojure package #11205 URL: https://github.com/apache/incubator-mxnet/pull/11512 ## Description ## Add licenses to files introduced by clojure package #11205 ## Checklist ## ###

[incubator-mxnet-site] branch asf-site updated: Bump the publish timestamp.

2018-07-01 Thread zhasheng
This is an automated email from the ASF dual-hosted git repository. zhasheng pushed a commit to branch asf-site in repository https://gitbox.apache.org/repos/asf/incubator-mxnet-site.git The following commit(s) were added to refs/heads/asf-site by this push: new ebdd143 Bump the publish

[GitHub] ctcyang edited a comment on issue #10696: [MXNET-366]Extend MXNet Distributed Training by AllReduce

2018-07-01 Thread GitBox
ctcyang edited a comment on issue #10696: [MXNET-366]Extend MXNet Distributed Training by AllReduce URL: https://github.com/apache/incubator-mxnet/pull/10696#issuecomment-401586011 I tried training with synthetic data with 8 machines x 8 GPUs (64 GPUs total) in float32 using:

[GitHub] ctcyang edited a comment on issue #10696: [MXNET-366]Extend MXNet Distributed Training by AllReduce

2018-07-01 Thread GitBox
ctcyang edited a comment on issue #10696: [MXNET-366]Extend MXNet Distributed Training by AllReduce URL: https://github.com/apache/incubator-mxnet/pull/10696#issuecomment-401586011 I tried training with synthetic data with 8 machines x 8 GPUs (64 GPUs total) in float32 using:

[GitHub] ctcyang edited a comment on issue #10696: [MXNET-366]Extend MXNet Distributed Training by AllReduce

2018-07-01 Thread GitBox
ctcyang edited a comment on issue #10696: [MXNET-366]Extend MXNet Distributed Training by AllReduce URL: https://github.com/apache/incubator-mxnet/pull/10696#issuecomment-401586011 I tried training with synthetic data with 8 machines x 8 GPUs (64 GPUs total) in float32 using:

[GitHub] yukang2017 commented on issue #11508: 【Question】Are there any examples for gradients clipping in gluon?

2018-07-01 Thread GitBox
yukang2017 commented on issue #11508: 【Question】Are there any examples for gradients clipping in gluon? URL: https://github.com/apache/incubator-mxnet/issues/11508#issuecomment-401586149 thanks This is an automated message

[GitHub] yukang2017 closed issue #11508: 【Question】Are there any examples for gradients clipping in gluon?

2018-07-01 Thread GitBox
yukang2017 closed issue #11508: 【Question】Are there any examples for gradients clipping in gluon? URL: https://github.com/apache/incubator-mxnet/issues/11508 This is an automated message from the Apache Git Service. To

[GitHub] ctcyang edited a comment on issue #10696: [MXNET-366]Extend MXNet Distributed Training by AllReduce

2018-07-01 Thread GitBox
ctcyang edited a comment on issue #10696: [MXNET-366]Extend MXNet Distributed Training by AllReduce URL: https://github.com/apache/incubator-mxnet/pull/10696#issuecomment-401586011 I tried training with synthetic data with 8 machines x 8 GPUs (64 GPUs total) in float32 using:

[GitHub] ctcyang edited a comment on issue #10696: [MXNET-366]Extend MXNet Distributed Training by AllReduce

2018-07-01 Thread GitBox
ctcyang edited a comment on issue #10696: [MXNET-366]Extend MXNet Distributed Training by AllReduce URL: https://github.com/apache/incubator-mxnet/pull/10696#issuecomment-401586011 I tried training with synthetic data with 8 machines x 8 GPUs (64 GPUs total) in float32 using:

[GitHub] ctcyang edited a comment on issue #10696: [MXNET-366]Extend MXNet Distributed Training by AllReduce

2018-07-01 Thread GitBox
ctcyang edited a comment on issue #10696: [MXNET-366]Extend MXNet Distributed Training by AllReduce URL: https://github.com/apache/incubator-mxnet/pull/10696#issuecomment-401586011 I tried training with synthetic data with 8 machines x 8 GPUs (64 GPUs total) in float32 using:

[GitHub] ctcyang commented on issue #10696: [MXNET-366]Extend MXNet Distributed Training by AllReduce

2018-07-01 Thread GitBox
ctcyang commented on issue #10696: [MXNET-366]Extend MXNet Distributed Training by AllReduce URL: https://github.com/apache/incubator-mxnet/pull/10696#issuecomment-401586011 I tried training with synthetic data with 8 machines x 8 GPUs (64 GPUs total) in float32 using:

[GitHub] iblis17 commented on issue #10149: WIP: import Julia binding

2018-07-01 Thread GitBox
iblis17 commented on issue #10149: WIP: import Julia binding URL: https://github.com/apache/incubator-mxnet/pull/10149#issuecomment-401585930 Jenkins error ``` Step 17/35 : ADD ../../julia/REQUIRE /work/julia/ ADD failed: Forbidden path outside the build context: