[GitHub] [incubator-mxnet] anirudh2290 commented on issue #16654: Multithreaded Inference Support

2020-01-16 Thread GitBox
anirudh2290 commented on issue #16654: Multithreaded Inference Support URL: https://github.com/apache/incubator-mxnet/pull/16654#issuecomment-575030986 > On the other hand, I am not sure if we can accept this cached op without bulking and subgraph support. Subgraph is used for multiple

[GitHub] [incubator-mxnet] leezu commented on a change in pull request #17293: [Build] Add a reasonable default for CMAKE_CUDA_COMPILER in *nix

2020-01-16 Thread GitBox
leezu commented on a change in pull request #17293: [Build] Add a reasonable default for CMAKE_CUDA_COMPILER in *nix URL: https://github.com/apache/incubator-mxnet/pull/17293#discussion_r367339823 ## File path: CMakeLists.txt ## @@ -84,6 +84,10 @@ message(STATUS "CMake

[GitHub] [incubator-mxnet] IHateMint opened a new pull request #17339: Typo Change

2020-01-16 Thread GitBox
IHateMint opened a new pull request #17339: Typo Change URL: https://github.com/apache/incubator-mxnet/pull/17339 ## Description ## Minor Typo Change ## Checklist ## ### Essentials ### Please feel free to remove inapplicable items for your PR. - [ ] The PR title starts

[GitHub] [incubator-mxnet] TaoLv commented on issue #17338: Unable to build mxnet with USE_OPENMP=OFF with MKLDNN backend

2020-01-16 Thread GitBox
TaoLv commented on issue #17338: Unable to build mxnet with USE_OPENMP=OFF with MKLDNN backend URL: https://github.com/apache/incubator-mxnet/issues/17338#issuecomment-575073921 @cyrusbehr Could you please try to apply below change to the CMakeList file? ``` diff --git

[GitHub] [incubator-mxnet] TaoLv commented on issue #17329: License issue with 1.6.0.rc1

2020-01-16 Thread GitBox
TaoLv commented on issue #17329: License issue with 1.6.0.rc1 URL: https://github.com/apache/incubator-mxnet/issues/17329#issuecomment-575078886 @roywei Regarding [3], [4], do you think we need list the copyrights/license in MXNet top LICENSE file or ask MKL-DNN itself to change? I

[GitHub] [incubator-mxnet] ChaokunChang edited a comment on issue #17299: Error when building mxnet with USE_DIST_KVSTORE

2020-01-16 Thread GitBox
ChaokunChang edited a comment on issue #17299: Error when building mxnet with USE_DIST_KVSTORE URL: https://github.com/apache/incubator-mxnet/issues/17299#issuecomment-575139822 > Looking at your output, you can see > > ``` > -- Could NOT find ZMQ (missing: ZMQ_LIBRARY

[GitHub] [incubator-mxnet] ZheyuYe edited a comment on issue #17327: np.ndarray indexing after hybridize

2020-01-16 Thread GitBox
ZheyuYe edited a comment on issue #17327: np.ndarray indexing after hybridize URL: https://github.com/apache/incubator-mxnet/issues/17327#issuecomment-575180346 @sxjscience It would be great if hybridized indexing could be supported as the last line in the code snippet. ```

[GitHub] [incubator-mxnet] leezu edited a comment on issue #17299: Error when building mxnet with USE_DIST_KVSTORE

2020-01-16 Thread GitBox
leezu edited a comment on issue #17299: Error when building mxnet with USE_DIST_KVSTORE URL: https://github.com/apache/incubator-mxnet/issues/17299#issuecomment-575143725 > Thank you a lot. Do you have any suggestions to disable Conda protobuf without uninstalling? (I didn't use conda

[GitHub] [incubator-mxnet] leezu commented on issue #17299: Error when building mxnet with USE_DIST_KVSTORE

2020-01-16 Thread GitBox
leezu commented on issue #17299: Error when building mxnet with USE_DIST_KVSTORE URL: https://github.com/apache/incubator-mxnet/issues/17299#issuecomment-575143725 > Thank you a lot. Do you have any suggestions to disable Conda protobuf without uninstalling? (I didn't use conda

[GitHub] [incubator-mxnet] TaoLv commented on issue #17338: Unable to build mxnet with USE_OPENMP=OFF with MKLDNN backend

2020-01-16 Thread GitBox
TaoLv commented on issue #17338: Unable to build mxnet with USE_OPENMP=OFF with MKLDNN backend URL: https://github.com/apache/incubator-mxnet/issues/17338#issuecomment-575150538 @leezu Thanks. We need check USE_OPENMP in cmake as what we did for makefile:

[GitHub] [incubator-mxnet] ZheyuYe commented on issue #17327: np.ndarray indexing after hybridize

2020-01-16 Thread GitBox
ZheyuYe commented on issue #17327: np.ndarray indexing after hybridize URL: https://github.com/apache/incubator-mxnet/issues/17327#issuecomment-575180346 @sxjscience ``` import numpy as np import mxnet as mx import numpy.testing as npt sequence =

[GitHub] [incubator-mxnet] aaronmarkham merged pull request #17339: Typo Change

2020-01-16 Thread GitBox
aaronmarkham merged pull request #17339: Typo Change URL: https://github.com/apache/incubator-mxnet/pull/17339 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [incubator-mxnet] ChaokunChang commented on issue #17299: Error when building mxnet with USE_DIST_KVSTORE

2020-01-16 Thread GitBox
ChaokunChang commented on issue #17299: Error when building mxnet with USE_DIST_KVSTORE URL: https://github.com/apache/incubator-mxnet/issues/17299#issuecomment-575139822 > Looking at your output, you can see > > ``` > -- Could NOT find ZMQ (missing: ZMQ_LIBRARY

[GitHub] [incubator-mxnet] leezu edited a comment on issue #17338: Unable to build mxnet with USE_OPENMP=OFF with MKLDNN backend

2020-01-16 Thread GitBox
leezu edited a comment on issue #17338: Unable to build mxnet with USE_OPENMP=OFF with MKLDNN backend URL: https://github.com/apache/incubator-mxnet/issues/17338#issuecomment-575145811 @TaoLv this fixes the issue for me (though I didn't try linking with MKL). I'll reassign this issue to

[incubator-mxnet] branch master updated (04c3eec -> b9ab7ad)

2020-01-16 Thread lausen
This is an automated email from the ASF dual-hosted git repository. lausen pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. from 04c3eec grouping large vector tests based on their type and adding their nightly test function (#17306)

[incubator-mxnet] branch master updated (04c3eec -> b9ab7ad)

2020-01-16 Thread lausen
This is an automated email from the ASF dual-hosted git repository. lausen pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. from 04c3eec grouping large vector tests based on their type and adding their nightly test function (#17306)

[GitHub] [incubator-mxnet] leezu closed issue #17337: MXNET Installation for R-Package are wrong

2020-01-16 Thread GitBox
leezu closed issue #17337: MXNET Installation for R-Package are wrong URL: https://github.com/apache/incubator-mxnet/issues/17337 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [incubator-mxnet] leezu merged pull request #17340: Fix typo in "Install the MXNet Package for R"

2020-01-16 Thread GitBox
leezu merged pull request #17340: Fix typo in "Install the MXNet Package for R" URL: https://github.com/apache/incubator-mxnet/pull/17340 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [incubator-mxnet] leezu commented on issue #17299: Error when building mxnet with USE_DIST_KVSTORE

2020-01-16 Thread GitBox
leezu commented on issue #17299: Error when building mxnet with USE_DIST_KVSTORE URL: https://github.com/apache/incubator-mxnet/issues/17299#issuecomment-575184498 It means you are using the `(base)` conda environment. Maybe it's activated by default on the Deep Learning AMI. You

[GitHub] [incubator-mxnet] leezu commented on issue #17338: Unable to build mxnet with USE_OPENMP=OFF with MKLDNN backend

2020-01-16 Thread GitBox
leezu commented on issue #17338: Unable to build mxnet with USE_OPENMP=OFF with MKLDNN backend URL: https://github.com/apache/incubator-mxnet/issues/17338#issuecomment-575145811 @TaoLv this fixes the issue for me (though I didn't try linking with MKL). I'll reassign this issue to you, as

[GitHub] [incubator-mxnet] lkubin commented on issue #16794: Random rotation

2020-01-16 Thread GitBox
lkubin commented on issue #16794: Random rotation URL: https://github.com/apache/incubator-mxnet/pull/16794#issuecomment-575164731 @szha I opened this pull request 2 moths ago, I'm wondering if it is an useful feature to be added. Otherwise I'm going to close it.

[GitHub] [incubator-mxnet] kshitij12345 opened a new issue #17342: [bug] Indexed Assignment Incorrect

2020-01-16 Thread GitBox
kshitij12345 opened a new issue #17342: [bug] Indexed Assignment Incorrect URL: https://github.com/apache/incubator-mxnet/issues/17342 ## Description Incorrect output when assigning with mask_array ## To Reproduce ```python import mxnet as mx import numpy as np def

[GitHub] [incubator-mxnet] ChaokunChang commented on issue #17299: Error when building mxnet with USE_DIST_KVSTORE

2020-01-16 Thread GitBox
ChaokunChang commented on issue #17299: Error when building mxnet with USE_DIST_KVSTORE URL: https://github.com/apache/incubator-mxnet/issues/17299#issuecomment-575147222 > so I have never used `source activate mxnet_p3` command before (In fact I even didn't know what is it ...).

[GitHub] [incubator-mxnet] TaoLv opened a new issue #17341: Segfault of test_gluon_data.test_recordimage_dataset_with_data_loader_multiworker

2020-01-16 Thread GitBox
TaoLv opened a new issue #17341: Segfault of test_gluon_data.test_recordimage_dataset_with_data_loader_multiworker URL: https://github.com/apache/incubator-mxnet/issues/17341 ## Description Maybe not that flaky. I met the crash in my MKL-DNN upgrading PR (#17313) which seems to be not

[GitHub] [incubator-mxnet] kshitij12345 commented on a change in pull request #17325: Fix Flaky Test Higher Order Grad

2020-01-16 Thread GitBox
kshitij12345 commented on a change in pull request #17325: Fix Flaky Test Higher Order Grad URL: https://github.com/apache/incubator-mxnet/pull/17325#discussion_r367474680 ## File path: python/mxnet/test_utils.py ## @@ -102,6 +102,16 @@ def random_arrays(*shapes):

[incubator-mxnet] branch master updated (b9ab7ad -> eb27161)

2020-01-16 Thread aaronmarkham
This is an automated email from the ASF dual-hosted git repository. aaronmarkham pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. from b9ab7ad Fix typo in "Install the MXNet Package for R" (#17340) add eb27161 Typo Change

[GitHub] [incubator-mxnet] codecov-io commented on issue #17326: Fix IOT Devices section of Get Started page

2020-01-16 Thread GitBox
codecov-io commented on issue #17326: Fix IOT Devices section of Get Started page URL: https://github.com/apache/incubator-mxnet/pull/17326#issuecomment-575277424 # [Codecov](https://codecov.io/gh/apache/incubator-mxnet/pull/17326?src=pr=h1) Report > Merging

[GitHub] [incubator-mxnet] cyrusbehr edited a comment on issue #15275: How to run mxnet(C++) in single-thread mode?

2020-01-16 Thread GitBox
cyrusbehr edited a comment on issue #15275: How to run mxnet(C++) in single-thread mode? URL: https://github.com/apache/incubator-mxnet/issues/15275#issuecomment-574939560 @igor-byel I experienced the same issue you are facing, and here is my fix. The issue is likely due to the way you

[incubator-mxnet] branch master updated (eb27161 -> a296dad)

2020-01-16 Thread dickjc123
This is an automated email from the ASF dual-hosted git repository. dickjc123 pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. from eb27161 Typo Change (#17339) add a296dad Fix flakey test_ndarray.py:test_reduce (#17312) No new

[GitHub] [incubator-mxnet] leezu opened a new pull request #17343: CMake: Attempt using ccache by default if available

2020-01-16 Thread GitBox
leezu opened a new pull request #17343: CMake: Attempt using ccache by default if available URL: https://github.com/apache/incubator-mxnet/pull/17343 ## Description ## Attempt using ccache by default if available. Simplifies user's build commands from ``` cmake -GNinja \

[GitHub] [incubator-mxnet] ChaiBapchya commented on issue #17200: [WIP] loosen numpy version requirement

2020-01-16 Thread GitBox
ChaiBapchya commented on issue #17200: [WIP] loosen numpy version requirement URL: https://github.com/apache/incubator-mxnet/pull/17200#issuecomment-575321863 Wasn't this downgraded to prevent numpy.decorator issue? This is

[GitHub] [incubator-mxnet] ChaiBapchya edited a comment on issue #17200: [WIP] loosen numpy version requirement

2020-01-16 Thread GitBox
ChaiBapchya edited a comment on issue #17200: [WIP] loosen numpy version requirement URL: https://github.com/apache/incubator-mxnet/pull/17200#issuecomment-575321863 Wasn't numpy version restriction (<1.18) done to prevent numpy.decorator issue? @leezu @reminisce

[GitHub] [incubator-mxnet] larroy commented on issue #17293: [Build] Add a reasonable default for CMAKE_CUDA_COMPILER in *nix

2020-01-16 Thread GitBox
larroy commented on issue #17293: [Build] Add a reasonable default for CMAKE_CUDA_COMPILER in *nix URL: https://github.com/apache/incubator-mxnet/pull/17293#issuecomment-575329905 This is the output from pytorch build: ``` -- Found CUDA: /usr/local/cuda (found version "10.2")

[GitHub] [incubator-mxnet] leezu edited a comment on issue #17293: [Build] Add a reasonable default for CMAKE_CUDA_COMPILER in *nix

2020-01-16 Thread GitBox
leezu edited a comment on issue #17293: [Build] Add a reasonable default for CMAKE_CUDA_COMPILER in *nix URL: https://github.com/apache/incubator-mxnet/pull/17293#issuecomment-575337042 @larroy, yes it's required to check the environment variables (both `CUDACXX` and `PATH`) first before

[GitHub] [incubator-mxnet] leezu edited a comment on issue #17293: [Build] Add a reasonable default for CMAKE_CUDA_COMPILER in *nix

2020-01-16 Thread GitBox
leezu edited a comment on issue #17293: [Build] Add a reasonable default for CMAKE_CUDA_COMPILER in *nix URL: https://github.com/apache/incubator-mxnet/pull/17293#issuecomment-575337042 @larroy, yes it's required to check the environment variables (both `CUDACXX` and `PATH`) first before

[GitHub] [incubator-mxnet] apeforest edited a comment on issue #17292: Can't run horovod with latest nightly wheel

2020-01-16 Thread GitBox
apeforest edited a comment on issue #17292: Can't run horovod with latest nightly wheel URL: https://github.com/apache/incubator-mxnet/issues/17292#issuecomment-574848779 Thanks @stephenrawls for the analysis. Here are the causes of the problem: 1) Horovod uses MX_API_BEGIN()

[GitHub] [incubator-mxnet] apeforest edited a comment on issue #17292: Can't run horovod with latest nightly wheel

2020-01-16 Thread GitBox
apeforest edited a comment on issue #17292: Can't run horovod with latest nightly wheel URL: https://github.com/apache/incubator-mxnet/issues/17292#issuecomment-574848779 Thanks @stephenrawls for the analysis. Here are the causes of the problem: 1) Horovod uses MX_API_BEGIN()

[incubator-mxnet] branch master updated (a296dad -> 57b98de)

2020-01-16 Thread apeforest
This is an automated email from the ASF dual-hosted git repository. apeforest pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. from a296dad Fix flakey test_ndarray.py:test_reduce (#17312) add 57b98de Build.py improvement (#16976)

[GitHub] [incubator-mxnet] ChaiBapchya commented on issue #17200: [WIP] loosen numpy version requirement

2020-01-16 Thread GitBox
ChaiBapchya commented on issue #17200: [WIP] loosen numpy version requirement URL: https://github.com/apache/incubator-mxnet/pull/17200#issuecomment-575342644 Any idea which PR should be merged to get 1.5.x into working? I tried cherry-picking

[GitHub] [incubator-mxnet] haojin2 commented on issue #17298: [MXNET-1438] Adding SDML loss function

2020-01-16 Thread GitBox
haojin2 commented on issue #17298: [MXNET-1438] Adding SDML loss function URL: https://github.com/apache/incubator-mxnet/pull/17298#issuecomment-575348787 @anjishnu I'm only concerned about if there's any flakiness in such a test, to verify, please try `MXNET_TEST_COUNT=500 nosetests

[GitHub] [incubator-mxnet] DickJC123 merged pull request #17312: Fix flakey test_ndarray.py:test_reduce

2020-01-16 Thread GitBox
DickJC123 merged pull request #17312: Fix flakey test_ndarray.py:test_reduce URL: https://github.com/apache/incubator-mxnet/pull/17312 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [incubator-mxnet] leezu commented on a change in pull request #17297: Fix NCCL Cmake autodetect issue

2020-01-16 Thread GitBox
leezu commented on a change in pull request #17297: Fix NCCL Cmake autodetect issue URL: https://github.com/apache/incubator-mxnet/pull/17297#discussion_r367579230 ## File path: cmake/Modules/FindNCCL.cmake ## @@ -33,6 +33,23 @@ set(NCCL_ROOT_DIR "" CACHE PATH "Folder

[incubator-mxnet] branch master updated (eb27161 -> a296dad)

2020-01-16 Thread dickjc123
This is an automated email from the ASF dual-hosted git repository. dickjc123 pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. from eb27161 Typo Change (#17339) add a296dad Fix flakey test_ndarray.py:test_reduce (#17312) No new

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

2020-01-16 Thread aaronmarkham
This is an automated email from the ASF dual-hosted git repository. aaronmarkham 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 b47c44d Bump the

[GitHub] [incubator-mxnet] anjishnu commented on issue #17298: [MXNET-1438] Adding SDML loss function

2020-01-16 Thread GitBox
anjishnu commented on issue #17298: [MXNET-1438] Adding SDML loss function URL: https://github.com/apache/incubator-mxnet/pull/17298#issuecomment-575347534 @haojin2 Are there any serious concerns with the new unit test? This

[GitHub] [incubator-mxnet] ChaiBapchya opened a new pull request #17346: Fix reshape interoperability test (#17155)

2020-01-16 Thread GitBox
ChaiBapchya opened a new pull request #17346: Fix reshape interoperability test (#17155) URL: https://github.com/apache/incubator-mxnet/pull/17346 * fix reshape interoperability test * fix for scipy import ## Description ## To fix the numpy.decorator issue in

[GitHub] [incubator-mxnet] cyrusbehr commented on issue #17338: Unable to build mxnet with USE_OPENMP=OFF with MKLDNN backend

2020-01-16 Thread GitBox
cyrusbehr commented on issue #17338: Unable to build mxnet with USE_OPENMP=OFF with MKLDNN backend URL: https://github.com/apache/incubator-mxnet/issues/17338#issuecomment-575278640 @leezu @TaoLv Adding `set(MKLDNN_CPU_RUNTIME SEQ CACHE INTERNAL "" FORCE)` solved the problem.

[GitHub] [incubator-mxnet] ChaiBapchya commented on a change in pull request #17297: Fix NCCL Cmake autodetect issue

2020-01-16 Thread GitBox
ChaiBapchya commented on a change in pull request #17297: Fix NCCL Cmake autodetect issue URL: https://github.com/apache/incubator-mxnet/pull/17297#discussion_r367575646 ## File path: cmake/Modules/FindNCCL.cmake ## @@ -33,6 +33,23 @@ set(NCCL_ROOT_DIR "" CACHE PATH

[GitHub] [incubator-mxnet] ChaiBapchya commented on a change in pull request #17297: Fix NCCL Cmake autodetect issue

2020-01-16 Thread GitBox
ChaiBapchya commented on a change in pull request #17297: Fix NCCL Cmake autodetect issue URL: https://github.com/apache/incubator-mxnet/pull/17297#discussion_r367575646 ## File path: cmake/Modules/FindNCCL.cmake ## @@ -33,6 +33,23 @@ set(NCCL_ROOT_DIR "" CACHE PATH

[GitHub] [incubator-mxnet] ChaiBapchya opened a new pull request #17344: Cherrypick Fix reshape interoperability test (#17155)

2020-01-16 Thread GitBox
ChaiBapchya opened a new pull request #17344: Cherrypick Fix reshape interoperability test (#17155) URL: https://github.com/apache/incubator-mxnet/pull/17344 * fix reshape interoperability test * fix for scipy import ## Description ## (Brief description on what this PR is

[GitHub] [incubator-mxnet] ChaiBapchya closed pull request #17344: Cherrypick Fix reshape interoperability test (#17155)

2020-01-16 Thread GitBox
ChaiBapchya closed pull request #17344: Cherrypick Fix reshape interoperability test (#17155) URL: https://github.com/apache/incubator-mxnet/pull/17344 This is an automated message from the Apache Git Service. To respond to

[GitHub] [incubator-mxnet] larroy commented on a change in pull request #13456: Improve CCache handling

2020-01-16 Thread GitBox
larroy commented on a change in pull request #13456: Improve CCache handling URL: https://github.com/apache/incubator-mxnet/pull/13456#discussion_r367631941 ## File path: ci/docker/runtime_functions.sh ## @@ -637,9 +685,11 @@ build_ubuntu_gpu_cmake_mkldnn() {

[GitHub] [incubator-mxnet] leezu edited a comment on issue #17293: [Build] Add a reasonable default for CMAKE_CUDA_COMPILER in *nix

2020-01-16 Thread GitBox
leezu edited a comment on issue #17293: [Build] Add a reasonable default for CMAKE_CUDA_COMPILER in *nix URL: https://github.com/apache/incubator-mxnet/pull/17293#issuecomment-575337042 @larroy, yes it's required to check the environment variables (both `CUDACXX` and `PATH`) first before

[GitHub] [incubator-mxnet] leezu commented on issue #17293: [Build] Add a reasonable default for CMAKE_CUDA_COMPILER in *nix

2020-01-16 Thread GitBox
leezu commented on issue #17293: [Build] Add a reasonable default for CMAKE_CUDA_COMPILER in *nix URL: https://github.com/apache/incubator-mxnet/pull/17293#issuecomment-575337042 @larroy, yes it's required to check the environment variables (both `CUDACXX` and `PATH` first before "falling

[GitHub] [incubator-mxnet] leezu edited a comment on issue #17293: [Build] Add a reasonable default for CMAKE_CUDA_COMPILER in *nix

2020-01-16 Thread GitBox
leezu edited a comment on issue #17293: [Build] Add a reasonable default for CMAKE_CUDA_COMPILER in *nix URL: https://github.com/apache/incubator-mxnet/pull/17293#issuecomment-575337042 @larroy, yes it's required to check the environment variables (both `CUDACXX` and `PATH`) first before

[GitHub] [incubator-mxnet] leezu edited a comment on issue #17293: [Build] Add a reasonable default for CMAKE_CUDA_COMPILER in *nix

2020-01-16 Thread GitBox
leezu edited a comment on issue #17293: [Build] Add a reasonable default for CMAKE_CUDA_COMPILER in *nix URL: https://github.com/apache/incubator-mxnet/pull/17293#issuecomment-575337042 @larroy, yes it's required to check the environment variables (both `CUDACXX` and `PATH`) first before

[GitHub] [incubator-mxnet] larroy commented on issue #16976: Build.py improvement

2020-01-16 Thread GitBox
larroy commented on issue #16976: Build.py improvement URL: https://github.com/apache/incubator-mxnet/pull/16976#issuecomment-575340365 @mxnet-label-bot add [pr-awaiting-merge] This is an automated message from the Apache Git

[GitHub] [incubator-mxnet] ChaiBapchya commented on a change in pull request #17297: Fix NCCL Cmake autodetect issue

2020-01-16 Thread GitBox
ChaiBapchya commented on a change in pull request #17297: Fix NCCL Cmake autodetect issue URL: https://github.com/apache/incubator-mxnet/pull/17297#discussion_r367578134 ## File path: cmake/Modules/FindNCCL.cmake ## @@ -33,6 +33,23 @@ set(NCCL_ROOT_DIR "" CACHE PATH

[GitHub] [incubator-mxnet] haojin2 merged pull request #17288: fix lstm layer with projection save params (#17266)

2020-01-16 Thread GitBox
haojin2 merged pull request #17288: fix lstm layer with projection save params (#17266) URL: https://github.com/apache/incubator-mxnet/pull/17288 This is an automated message from the Apache Git Service. To respond to the

[incubator-mxnet] branch v1.6.x updated: fix lstm layer with projection save params (#17266) (#17288)

2020-01-16 Thread haoj
This is an automated email from the ASF dual-hosted git repository. haoj pushed a commit to branch v1.6.x in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git The following commit(s) were added to refs/heads/v1.6.x by this push: new a003132 fix lstm layer with projection

[GitHub] [incubator-mxnet] roywei commented on issue #17329: License issue with 1.6.0.rc1

2020-01-16 Thread GitBox
roywei commented on issue #17329: License issue with 1.6.0.rc1 URL: https://github.com/apache/incubator-mxnet/issues/17329#issuecomment-575294965 > @roywei Regarding [3], [4], do you think we need list the copyrights/license in MXNet top LICENSE file or ask MKL-DNN itself to change? I

[GitHub] [incubator-mxnet] leezu commented on a change in pull request #13456: Improve CCache handling

2020-01-16 Thread GitBox
leezu commented on a change in pull request #13456: Improve CCache handling URL: https://github.com/apache/incubator-mxnet/pull/13456#discussion_r367593222 ## File path: ci/docker/runtime_functions.sh ## @@ -637,9 +685,11 @@ build_ubuntu_gpu_cmake_mkldnn() {

[GitHub] [incubator-mxnet] roywei edited a comment on issue #17329: License issue with 1.6.0.rc1

2020-01-16 Thread GitBox
roywei edited a comment on issue #17329: License issue with 1.6.0.rc1 URL: https://github.com/apache/incubator-mxnet/issues/17329#issuecomment-575294965 > @roywei Regarding [3], [4], do you think we need list the copyrights/license in MXNet top LICENSE file or ask MKL-DNN itself to

[GitHub] [incubator-mxnet] ChaiBapchya closed pull request #17345: Cherrypick Fix reshape interoperability test (#17155)

2020-01-16 Thread GitBox
ChaiBapchya closed pull request #17345: Cherrypick Fix reshape interoperability test (#17155) URL: https://github.com/apache/incubator-mxnet/pull/17345 This is an automated message from the Apache Git Service. To respond to

[GitHub] [incubator-mxnet] ChaiBapchya opened a new pull request #17345: Cherrypick Fix reshape interoperability test (#17155)

2020-01-16 Thread GitBox
ChaiBapchya opened a new pull request #17345: Cherrypick Fix reshape interoperability test (#17155) URL: https://github.com/apache/incubator-mxnet/pull/17345 * fix reshape interoperability test * fix for scipy import ## Description ## Cherrypick this PR into 1.5.x so that

[GitHub] [incubator-mxnet] larroy commented on issue #13456: Improve CCache handling

2020-01-16 Thread GitBox
larroy commented on issue #13456: Improve CCache handling URL: https://github.com/apache/incubator-mxnet/pull/13456#issuecomment-575330699 @josephevans please review, you said you got ccache to work with nvcc. This is an

[GitHub] [incubator-mxnet] apeforest merged pull request #16976: Build.py improvement

2020-01-16 Thread GitBox
apeforest merged pull request #16976: Build.py improvement URL: https://github.com/apache/incubator-mxnet/pull/16976 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [incubator-mxnet] larroy commented on issue #17293: [Build] Add a reasonable default for CMAKE_CUDA_COMPILER in *nix

2020-01-16 Thread GitBox
larroy commented on issue #17293: [Build] Add a reasonable default for CMAKE_CUDA_COMPILER in *nix URL: https://github.com/apache/incubator-mxnet/pull/17293#issuecomment-575346572 Thanks for the clarifications, make sense. I don't think LD_LIBRARY_PATH will be an issue in this case. As

[GitHub] [incubator-mxnet] ChaiBapchya commented on a change in pull request #17297: Fix NCCL Cmake autodetect issue

2020-01-16 Thread GitBox
ChaiBapchya commented on a change in pull request #17297: Fix NCCL Cmake autodetect issue URL: https://github.com/apache/incubator-mxnet/pull/17297#discussion_r367577696 ## File path: cmake/Modules/FindNCCL.cmake ## @@ -33,6 +33,23 @@ set(NCCL_ROOT_DIR "" CACHE PATH

[GitHub] [incubator-mxnet] marcoabreu commented on issue #16794: Random rotation

2020-01-16 Thread GitBox
marcoabreu commented on issue #16794: Random rotation URL: https://github.com/apache/incubator-mxnet/pull/16794#issuecomment-575334331 This is a very useful feature, thanks! @reminisce @eric-haibin-lin @apeforest could you review please?

[GitHub] [incubator-mxnet] leezu commented on issue #17200: [WIP] loosen numpy version requirement

2020-01-16 Thread GitBox
leezu commented on issue #17200: [WIP] loosen numpy version requirement URL: https://github.com/apache/incubator-mxnet/pull/17200#issuecomment-575327309 It was also done due to https://github.com/apache/incubator-mxnet/issues/17154 That issue was fixed and no other failing tests

[GitHub] [incubator-mxnet] marcoabreu commented on a change in pull request #13456: Improve CCache handling

2020-01-16 Thread GitBox
marcoabreu commented on a change in pull request #13456: Improve CCache handling URL: https://github.com/apache/incubator-mxnet/pull/13456#discussion_r367634124 ## File path: ci/docker/runtime_functions.sh ## @@ -637,9 +685,11 @@ build_ubuntu_gpu_cmake_mkldnn() {

[GitHub] [incubator-mxnet] apeforest commented on a change in pull request #17297: Fix NCCL Cmake autodetect issue

2020-01-16 Thread GitBox
apeforest commented on a change in pull request #17297: Fix NCCL Cmake autodetect issue URL: https://github.com/apache/incubator-mxnet/pull/17297#discussion_r367568453 ## File path: cmake/Modules/FindNCCL.cmake ## @@ -33,6 +33,23 @@ set(NCCL_ROOT_DIR "" CACHE PATH

[GitHub] [incubator-mxnet] apeforest commented on a change in pull request #17297: Fix NCCL Cmake autodetect issue

2020-01-16 Thread GitBox
apeforest commented on a change in pull request #17297: Fix NCCL Cmake autodetect issue URL: https://github.com/apache/incubator-mxnet/pull/17297#discussion_r367568504 ## File path: cmake/Modules/FindNCCL.cmake ## @@ -33,6 +33,23 @@ set(NCCL_ROOT_DIR "" CACHE PATH

[GitHub] [incubator-mxnet] larroy commented on issue #17293: [Build] Add a reasonable default for CMAKE_CUDA_COMPILER in *nix

2020-01-16 Thread GitBox
larroy commented on issue #17293: [Build] Add a reasonable default for CMAKE_CUDA_COMPILER in *nix URL: https://github.com/apache/incubator-mxnet/pull/17293#issuecomment-575328978 What do you suggest wrt https://cmake.org/cmake/help/v3.13/envvar/CUDACXX.html should we check if this is

[GitHub] [incubator-mxnet] larroy edited a comment on issue #17293: [Build] Add a reasonable default for CMAKE_CUDA_COMPILER in *nix

2020-01-16 Thread GitBox
larroy edited a comment on issue #17293: [Build] Add a reasonable default for CMAKE_CUDA_COMPILER in *nix URL: https://github.com/apache/incubator-mxnet/pull/17293#issuecomment-575328978 What do you suggest wrt https://cmake.org/cmake/help/v3.13/envvar/CUDACXX.html should we check if this

[incubator-mxnet] branch master updated (a296dad -> 57b98de)

2020-01-16 Thread apeforest
This is an automated email from the ASF dual-hosted git repository. apeforest pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. from a296dad Fix flakey test_ndarray.py:test_reduce (#17312) add 57b98de Build.py improvement (#16976)

[GitHub] [incubator-mxnet] leezu commented on issue #17293: [Build] Add a reasonable default for CMAKE_CUDA_COMPILER in *nix

2020-01-16 Thread GitBox
leezu commented on issue #17293: [Build] Add a reasonable default for CMAKE_CUDA_COMPILER in *nix URL: https://github.com/apache/incubator-mxnet/pull/17293#issuecomment-575355873 LD_LIBRARY_PATH will not cause issues in this case, but it's a related problem source. It will cause problems

[GitHub] [incubator-mxnet] szha commented on a change in pull request #17348: Fix Horovod build error due to missing exported symbols

2020-01-16 Thread GitBox
szha commented on a change in pull request #17348: Fix Horovod build error due to missing exported symbols URL: https://github.com/apache/incubator-mxnet/pull/17348#discussion_r367790272 ## File path: make/config/libmxnet.sym ## @@ -12,4 +12,4 @@ Java_org_apache_mxnet*

[GitHub] [incubator-mxnet] haojin2 commented on a change in pull request #17302: [numpy]add op random.logistic, random.gumbel

2020-01-16 Thread GitBox
haojin2 commented on a change in pull request #17302: [numpy]add op random.logistic, random.gumbel URL: https://github.com/apache/incubator-mxnet/pull/17302#discussion_r367799853 ## File path: src/operator/numpy/random/np_location_scale_op.h ## @@ -0,0 +1,510 @@ +/* + *

[GitHub] [incubator-mxnet] TaoLv opened a new pull request #17356: cmake: check USE_OPENMP for proper MKL-DNN build flag

2020-01-16 Thread GitBox
TaoLv opened a new pull request #17356: cmake: check USE_OPENMP for proper MKL-DNN build flag URL: https://github.com/apache/incubator-mxnet/pull/17356 ## Description ## Fix https://github.com/apache/incubator-mxnet/issues/17338 If `USE_OPENMP=OFF`, build MKL-DNN with

[GitHub] [incubator-mxnet] TaoLv commented on issue #17356: cmake: check USE_OPENMP for proper MKL-DNN build flag

2020-01-16 Thread GitBox
TaoLv commented on issue #17356: cmake: check USE_OPENMP for proper MKL-DNN build flag URL: https://github.com/apache/incubator-mxnet/pull/17356#issuecomment-575481956 @leezu @cyrusbehr @pengzhao-intel Please review~ This is

[GitHub] [incubator-mxnet] xidulu commented on a change in pull request #17302: [numpy]add op random.logistic, random.gumbel

2020-01-16 Thread GitBox
xidulu commented on a change in pull request #17302: [numpy]add op random.logistic, random.gumbel URL: https://github.com/apache/incubator-mxnet/pull/17302#discussion_r367796564 ## File path: python/mxnet/numpy/random.py ## @@ -200,6 +200,192 @@ def normal(loc=0.0,

[GitHub] [incubator-mxnet] szhengac commented on issue #17358: np.broadcast_to extension

2020-01-16 Thread GitBox
szhengac commented on issue #17358: np.broadcast_to extension URL: https://github.com/apache/incubator-mxnet/pull/17358#issuecomment-575507918 Great! This is an automated message from the Apache Git Service. To respond to the

[GitHub] [incubator-mxnet] Yiyan66 commented on a change in pull request #17302: [numpy]add op random.logistic, random.gumbel

2020-01-16 Thread GitBox
Yiyan66 commented on a change in pull request #17302: [numpy]add op random.logistic, random.gumbel URL: https://github.com/apache/incubator-mxnet/pull/17302#discussion_r367800505 ## File path: src/operator/numpy/random/np_location_scale_op.h ## @@ -0,0 +1,510 @@ +/* + *

[GitHub] [incubator-mxnet] TaoLv commented on issue #17352: Error in mkldnn unit test

2020-01-16 Thread GitBox
TaoLv commented on issue #17352: Error in mkldnn unit test URL: https://github.com/apache/incubator-mxnet/issues/17352#issuecomment-575466455 @juliusshufan This is an automated message from the Apache Git Service. To

[GitHub] [incubator-mxnet] TaoLv commented on issue #17352: Error in mkldnn unit test

2020-01-16 Thread GitBox
TaoLv commented on issue #17352: Error in mkldnn unit test URL: https://github.com/apache/incubator-mxnet/issues/17352#issuecomment-575466220 Yes, we're aware of this problem as in our local validation system, python tests are split into single run for each test. We didn't dive into the

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

2020-01-16 Thread aaronmarkham
This is an automated email from the ASF dual-hosted git repository. aaronmarkham 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 84774a3 Bump the

[GitHub] [incubator-mxnet] xidulu commented on a change in pull request #17302: [numpy]add op random.logistic, random.gumbel

2020-01-16 Thread GitBox
xidulu commented on a change in pull request #17302: [numpy]add op random.logistic, random.gumbel URL: https://github.com/apache/incubator-mxnet/pull/17302#discussion_r367798667 ## File path: src/operator/numpy/random/np_location_scale_op.h ## @@ -0,0 +1,510 @@ +/* + *

[GitHub] [incubator-mxnet] TaoLv opened a new pull request #17355: [Scripts] Update MKL to 2020.0

2020-01-16 Thread GitBox
TaoLv opened a new pull request #17355: [Scripts] Update MKL to 2020.0 URL: https://github.com/apache/incubator-mxnet/pull/17355 ## Description ## This PR updates the MKL installation script to 2020.0. ## Checklist ## ### Essentials ### Please feel free to remove

[GitHub] [incubator-mxnet] apeforest commented on a change in pull request #17349: Fix links and highlights in ubuntu_setup.md page

2020-01-16 Thread GitBox
apeforest commented on a change in pull request #17349: Fix links and highlights in ubuntu_setup.md page URL: https://github.com/apache/incubator-mxnet/pull/17349#discussion_r367775903 ## File path: docs/static_site/src/pages/get_started/ubuntu_setup.md ## @@ -181,7

[GitHub] [incubator-mxnet] kohillyang opened a new issue #17357: lr_scheduler does not work as expect when training from checkpoint.(Not sure it is a bug.)

2020-01-16 Thread GitBox
kohillyang opened a new issue #17357: lr_scheduler does not work as expect when training from checkpoint.(Not sure it is a bug.) URL: https://github.com/apache/incubator-mxnet/issues/17357 ## Description As I know, the optimizer decides the num_update according to its

[GitHub] [incubator-mxnet] haojin2 opened a new pull request #17358: np.broadcast_to extension

2020-01-16 Thread GitBox
haojin2 opened a new pull request #17358: np.broadcast_to extension URL: https://github.com/apache/incubator-mxnet/pull/17358 ## Description ## Extending `broadcast_to` to infer shape from input tensor, useful under symbolic mode. ## Checklist ## ### Essentials ### Please

[GitHub] [incubator-mxnet] xidulu commented on a change in pull request #17302: [numpy]add op random.logistic, random.gumbel

2020-01-16 Thread GitBox
xidulu commented on a change in pull request #17302: [numpy]add op random.logistic, random.gumbel URL: https://github.com/apache/incubator-mxnet/pull/17302#discussion_r367796729 ## File path: python/mxnet/numpy/random.py ## @@ -200,6 +200,192 @@ def normal(loc=0.0,

[GitHub] [incubator-mxnet] ChaiBapchya commented on issue #17343: CMake: Attempt using ccache by default if available

2020-01-16 Thread GitBox
ChaiBapchya commented on issue #17343: CMake: Attempt using ccache by default if available URL: https://github.com/apache/incubator-mxnet/pull/17343#issuecomment-575513974 Thanks! Usability +1 This is an automated message

[GitHub] [incubator-mxnet] ChaiBapchya removed a comment on issue #17343: CMake: Attempt using ccache by default if available

2020-01-16 Thread GitBox
ChaiBapchya removed a comment on issue #17343: CMake: Attempt using ccache by default if available URL: https://github.com/apache/incubator-mxnet/pull/17343#issuecomment-575513974 Thanks! Usability +1 This is an automated

[GitHub] [incubator-mxnet] ChaiBapchya commented on a change in pull request #17346: Fix reshape interoperability test (#17155)

2020-01-16 Thread GitBox
ChaiBapchya commented on a change in pull request #17346: Fix reshape interoperability test (#17155) URL: https://github.com/apache/incubator-mxnet/pull/17346#discussion_r367676509 ## File path: ci/docker/install/docs_requirements ## @@ -26,11 +26,11 @@ h5py==2.8.0rc1

[incubator-mxnet] branch master updated (57b98de -> a63390d)

2020-01-16 Thread apeforest
This is an automated email from the ASF dual-hosted git repository. apeforest pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. from 57b98de Build.py improvement (#16976) add a63390d Add RandomApply in gluon's transforms (#17242)

[incubator-mxnet] branch master updated (57b98de -> a63390d)

2020-01-16 Thread apeforest
This is an automated email from the ASF dual-hosted git repository. apeforest pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. from 57b98de Build.py improvement (#16976) add a63390d Add RandomApply in gluon's transforms (#17242)

[GitHub] [incubator-mxnet] leezu commented on issue #17347: Static build script error out

2020-01-16 Thread GitBox
leezu commented on issue #17347: Static build script error out URL: https://github.com/apache/incubator-mxnet/issues/17347#issuecomment-575392966 This is not related to our CMake build. This is an automated message from the

  1   2   >