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

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

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

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

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

2018-03-02 Thread GitBox
cjolivier01 commented on a change in pull request #9958: Parallelization for ROIpooling OP URL: https://github.com/apache/incubator-mxnet/pull/9958#discussion_r171917655 ## File path: src/operator/roi_pooling.cc ## @@ -98,30 +109,26 @@ inline void ROIPoolForward(const

[GitHub] cjolivier01 closed pull request #9955: [DO NOT MERGE] Cython CI check

2018-03-02 Thread GitBox
cjolivier01 closed pull request #9955: [DO NOT MERGE] Cython CI check URL: https://github.com/apache/incubator-mxnet/pull/9955 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

[GitHub] cjolivier01 commented on issue #9955: [DO NOT MERGE] Cython CI check

2018-03-02 Thread GitBox
cjolivier01 commented on issue #9955: [DO NOT MERGE] Cython CI check URL: https://github.com/apache/incubator-mxnet/pull/9955#issuecomment-370004346 closing to stop pushes from generating builds This is an automated message

[GitHub] tqchen commented on a change in pull request #9946: [DO NOT MERGE] Start CI docker revamp

2018-03-02 Thread GitBox
tqchen commented on a change in pull request #9946: [DO NOT MERGE] Start CI docker revamp URL: https://github.com/apache/incubator-mxnet/pull/9946#discussion_r171903273 ## File path: tests/ci_build/Dockerfile.cpu_clang ## @@ -1,21 +0,0 @@ -FROM ubuntu:16.04 - -COPY

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

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

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

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

[GitHub] KishoreKarunakaran commented on issue #9952: Latest Windows pre-built binary for R?(MXNet Version 1.1.0)

2018-03-02 Thread GitBox
KishoreKarunakaran commented on issue #9952: Latest Windows pre-built binary for R?(MXNet Version 1.1.0) URL: https://github.com/apache/incubator-mxnet/issues/9952#issuecomment-370012506 is there latest prebuilt mxnet-cu90 binary for win10 (i.e. Mxnet 1.1.0.) So far I was only able to

[GitHub] anirudhacharya opened a new pull request #9963: Onnx Module to import onnx models into mxnet

2018-03-02 Thread GitBox
anirudhacharya opened a new pull request #9963: Onnx Module to import onnx models into mxnet URL: https://github.com/apache/incubator-mxnet/pull/9963 ## Description ## This module tries to import onnx models into mxnet. ## Checklist ## ### Essentials ### - [-] Passed code

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

2018-03-02 Thread GitBox
anirudhacharya commented on a change in pull request #9963: Onnx Module to import onnx models into mxnet URL: https://github.com/apache/incubator-mxnet/pull/9963#discussion_r171928159 ## File path: python/mxnet/contrib/onnx/_import/__init__.py ## @@ -0,0 +1,46 @@ +#

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

2018-03-02 Thread GitBox
reminisce commented on a change in pull request #9963: Onnx Module to import onnx models into mxnet URL: https://github.com/apache/incubator-mxnet/pull/9963#discussion_r171925586 ## File path: python/mxnet/contrib/onnx/_import/__init__.py ## @@ -0,0 +1,46 @@ +# Licensed

[GitHub] zhanghang1989 commented on a change in pull request #9931: Add axes support to Dropout for variational dropout in NLP

2018-03-02 Thread GitBox
zhanghang1989 commented on a change in pull request #9931: Add axes support to Dropout for variational dropout in NLP URL: https://github.com/apache/incubator-mxnet/pull/9931#discussion_r171929232 ## File path: src/operator/nn/dropout.cc ## @@ -93,10 +93,16 @@ Example::

[GitHub] marcoabreu closed pull request #9964: Fix R-package version

2018-03-02 Thread GitBox
marcoabreu closed pull request #9964: Fix R-package version URL: https://github.com/apache/incubator-mxnet/pull/9964 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

[GitHub] zhanghang1989 commented on a change in pull request #9931: Add axes support to Dropout for variational dropout in NLP

2018-03-02 Thread GitBox
zhanghang1989 commented on a change in pull request #9931: Add axes support to Dropout for variational dropout in NLP URL: https://github.com/apache/incubator-mxnet/pull/9931#discussion_r171937954 ## File path: src/operator/nn/dropout-inl.h ## @@ -178,30 +184,17 @@ class

[GitHub] marcoabreu commented on a change in pull request #9946: [DO NOT MERGE] Start CI docker revamp

2018-03-02 Thread GitBox
marcoabreu commented on a change in pull request #9946: [DO NOT MERGE] Start CI docker revamp URL: https://github.com/apache/incubator-mxnet/pull/9946#discussion_r171937930 ## File path: tests/ci_build/Dockerfile.cpu_clang ## @@ -1,21 +0,0 @@ -FROM ubuntu:16.04 - -COPY

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

2018-03-02 Thread GitBox
anirudhacharya commented on a change in pull request #9963: Onnx Module to import onnx models into mxnet URL: https://github.com/apache/incubator-mxnet/pull/9963#discussion_r171937836 ## File path: python/mxnet/contrib/onnx/_import/op_translations/negative.py ## @@ -0,0

[GitHub] lanking520 commented on a change in pull request #9954: Update on OSX_setup.md

2018-03-02 Thread GitBox
lanking520 commented on a change in pull request #9954: Update on OSX_setup.md URL: https://github.com/apache/incubator-mxnet/pull/9954#discussion_r171939790 ## File path: docs/install/osx_setup.md ## @@ -65,14 +65,19 @@ Install the dependencies, required for MXNet, with

[GitHub] AndreGuerra123 opened a new issue #9967: Error in output.shape[[output.names]]

2018-03-02 Thread GitBox
AndreGuerra123 opened a new issue #9967: Error in output.shape[[output.names]] URL: https://github.com/apache/incubator-mxnet/issues/9967 Note: Providing complete information in the most concise form is the best way to get help. This issue template serves as the checklist for essential

[GitHub] zhanghang1989 commented on a change in pull request #9931: Add axes support to Dropout for variational dropout in NLP

2018-03-02 Thread GitBox
zhanghang1989 commented on a change in pull request #9931: Add axes support to Dropout for variational dropout in NLP URL: https://github.com/apache/incubator-mxnet/pull/9931#discussion_r171929974 ## File path: src/operator/nn/dropout-inl.h ## @@ -228,11 +301,27 @@ class

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

2018-03-02 Thread GitBox
piiswrong commented on a change in pull request #9963: Onnx Module to import onnx models into mxnet URL: https://github.com/apache/incubator-mxnet/pull/9963#discussion_r171934166 ## File path: python/mxnet/contrib/onnx/_import/op_translations/_add.py ## @@ -0,0 +1,25 @@

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

2018-03-02 Thread GitBox
piiswrong commented on a change in pull request #9963: Onnx Module to import onnx models into mxnet URL: https://github.com/apache/incubator-mxnet/pull/9963#discussion_r171934166 ## File path: python/mxnet/contrib/onnx/_import/op_translations/_add.py ## @@ -0,0 +1,25 @@

[GitHub] marcoabreu commented on issue #9965: Fix R build crash in CI

2018-03-02 Thread GitBox
marcoabreu commented on issue #9965: Fix R build crash in CI URL: https://github.com/apache/incubator-mxnet/pull/9965#issuecomment-370021909 Thanks a lot! @apache/mxnet-committers please merge this ASAP in order to unblock CI. Afterwards, please retrigger failed PRs.

[GitHub] Roshrini commented on issue #9954: Update on OSX_setup.md

2018-03-02 Thread GitBox
Roshrini commented on issue #9954: Update on OSX_setup.md URL: https://github.com/apache/incubator-mxnet/pull/9954#issuecomment-370023660 @aaronmarkham This is an automated message from the Apache Git Service. To respond to

[GitHub] cjolivier01 commented on a change in pull request #9931: Add axes support to Dropout for variational dropout in NLP

2018-03-02 Thread GitBox
cjolivier01 commented on a change in pull request #9931: Add axes support to Dropout for variational dropout in NLP URL: https://github.com/apache/incubator-mxnet/pull/9931#discussion_r171937353 ## File path: src/operator/nn/dropout-inl.h ## @@ -178,30 +184,17 @@ class

[GitHub] marcoabreu commented on a change in pull request #9918: Update mkldnn to the newest & Add clang build test with mkldnn.

2018-03-02 Thread GitBox
marcoabreu commented on a change in pull request #9918: Update mkldnn to the newest & Add clang build test with mkldnn. URL: https://github.com/apache/incubator-mxnet/pull/9918#discussion_r171937193 ## File path: src/operator/nn/mkldnn/mkldnn_base.cc ## @@ -237,13 +237,14

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

2018-03-02 Thread GitBox
Roshrini commented on a change in pull request #9963: Onnx Module to import onnx models into mxnet URL: https://github.com/apache/incubator-mxnet/pull/9963#discussion_r171944650 ## File path: python/mxnet/contrib/onnx/_import/op_translations/negative.py ## @@ -0,0 +1,20

[GitHub] spidyDev opened a new issue #9966: pad operator only supports 4D or 5D tensors. Is there a plan to add support for lower dimension tensors?

2018-03-02 Thread GitBox
spidyDev opened a new issue #9966: pad operator only supports 4D or 5D tensors. Is there a plan to add support for lower dimension tensors? URL: https://github.com/apache/incubator-mxnet/issues/9966 Note: Providing complete information in the most concise form is the best way to get help.

[GitHub] cjolivier01 commented on a change in pull request #9931: Add axes support to Dropout for variational dropout in NLP

2018-03-02 Thread GitBox
cjolivier01 commented on a change in pull request #9931: Add axes support to Dropout for variational dropout in NLP URL: https://github.com/apache/incubator-mxnet/pull/9931#discussion_r171930460 ## File path: src/operator/nn/dropout-inl.h ## @@ -178,30 +184,17 @@ class

[GitHub] Roshrini commented on a change in pull request #9954: Update on OSX_setup.md

2018-03-02 Thread GitBox
Roshrini commented on a change in pull request #9954: Update on OSX_setup.md URL: https://github.com/apache/incubator-mxnet/pull/9954#discussion_r171938333 ## File path: docs/install/osx_setup.md ## @@ -65,14 +65,19 @@ Install the dependencies, required for MXNet, with the

[GitHub] anirudhacharya commented on issue #9963: Onnx Module to import onnx models into mxnet

2018-03-02 Thread GitBox
anirudhacharya commented on issue #9963: Onnx Module to import onnx models into mxnet URL: https://github.com/apache/incubator-mxnet/pull/9963#issuecomment-370029582 @awslabs/aws-deep-learning-sdk This is an automated

[GitHub] anirudhacharya commented on issue #9963: Onnx Module to import onnx models into mxnet

2018-03-02 Thread GitBox
anirudhacharya commented on issue #9963: Onnx Module to import onnx models into mxnet URL: https://github.com/apache/incubator-mxnet/pull/9963#issuecomment-370029582 @awslabs/aws-deep-learning-sdk This is an automated

[GitHub] jeremiedb opened a new pull request #9965: Fix R build crash in CI

2018-03-02 Thread GitBox
jeremiedb opened a new pull request #9965: Fix R build crash in CI URL: https://github.com/apache/incubator-mxnet/pull/9965 Fix for #9957 Resolves the deprecated function in DiagrammeR package used in graph.viz. Graph nodes are now of fixed size rather than adapting to wrap the

[GitHub] marcoabreu commented on issue #9964: Fix R-package version

2018-03-02 Thread GitBox
marcoabreu commented on issue #9964: Fix R-package version URL: https://github.com/apache/incubator-mxnet/pull/9964#issuecomment-370021496 Obsolete as of https://github.com/apache/incubator-mxnet/pull/9965 This is an

[GitHub] cjolivier01 commented on a change in pull request #9931: Add axes support to Dropout for variational dropout in NLP

2018-03-02 Thread GitBox
cjolivier01 commented on a change in pull request #9931: Add axes support to Dropout for variational dropout in NLP URL: https://github.com/apache/incubator-mxnet/pull/9931#discussion_r171937542 ## File path: src/operator/nn/dropout-inl.h ## @@ -228,11 +301,27 @@ class

[GitHub] piiswrong commented on a change in pull request #9961: Unary ops norm_logcdf, norm_derivlogcdf for log CDF of standard normal

2018-03-02 Thread GitBox
piiswrong commented on a change in pull request #9961: Unary ops norm_logcdf, norm_derivlogcdf for log CDF of standard normal URL: https://github.com/apache/incubator-mxnet/pull/9961#discussion_r171936985 ## File path: docs/api/python/symbol/symbol.md ## @@ -607,6 +607,8

[GitHub] piiswrong commented on a change in pull request #9961: Unary ops norm_logcdf, norm_derivlogcdf for log CDF of standard normal

2018-03-02 Thread GitBox
piiswrong commented on a change in pull request #9961: Unary ops norm_logcdf, norm_derivlogcdf for log CDF of standard normal URL: https://github.com/apache/incubator-mxnet/pull/9961#discussion_r171936965 ## File path: docs/api/python/ndarray/ndarray.md ## @@ -606,6

[GitHub] cgraywang commented on issue #9572: Return a NaN when using operator ( ** ) on Windows version with GPU

2018-03-02 Thread GitBox
cgraywang commented on issue #9572: Return a NaN when using operator ( ** ) on Windows version with GPU URL: https://github.com/apache/incubator-mxnet/issues/9572#issuecomment-370025922 @Feywell @zhreshold Is the issue still there? I tested on linux gpu and cpu with both mx version 1.0.0

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

2018-03-02 Thread GitBox
anirudhacharya commented on a change in pull request #9963: Onnx Module to import onnx models into mxnet URL: https://github.com/apache/incubator-mxnet/pull/9963#discussion_r171939279 ## File path: python/mxnet/contrib/onnx/_import/op_translations/_add.py ## @@ -0,0 +1,25

[GitHub] marcoabreu commented on issue #9957: R Package Fails to Build since #9882

2018-03-02 Thread GitBox
marcoabreu commented on issue #9957: R Package Fails to Build since #9882 URL: https://github.com/apache/incubator-mxnet/issues/9957#issuecomment-370021070 Hello @jeremiedb, I have submitted a temporary fix at https://github.com/apache/incubator-mxnet/pull/9964. Could you upgrade

[GitHub] marcoabreu opened a new pull request #9964: Fix R-package version

2018-03-02 Thread GitBox
marcoabreu opened a new pull request #9964: Fix R-package version URL: https://github.com/apache/incubator-mxnet/pull/9964 DiagrammeR has been upgraded to version 1.0, resulting in breaking changes. This PR pins the used version as a temporary fix.

[GitHub] marcoabreu commented on issue #9952: Latest Windows pre-built binary for R?(MXNet Version 1.1.0)

2018-03-02 Thread GitBox
marcoabreu commented on issue #9952: Latest Windows pre-built binary for R?(MXNet Version 1.1.0) URL: https://github.com/apache/incubator-mxnet/issues/9952#issuecomment-370024373 @yzhliu This is an automated message from

[GitHub] zheng-da commented on issue #9874: ResNet-50 is slower on Volta since #8302

2018-03-02 Thread GitBox
zheng-da commented on issue #9874: ResNet-50 is slower on Volta since #8302 URL: https://github.com/apache/incubator-mxnet/issues/9874#issuecomment-370032059 I think I may know what is the potential cause of this problem. I'll fix it next week.

[GitHub] zhanghang1989 commented on a change in pull request #9931: Add axes support to Dropout for variational dropout in NLP

2018-03-02 Thread GitBox
zhanghang1989 commented on a change in pull request #9931: Add axes support to Dropout for variational dropout in NLP URL: https://github.com/apache/incubator-mxnet/pull/9931#discussion_r171931760 ## File path: src/operator/nn/dropout-inl.h ## @@ -178,30 +184,17 @@ class

[GitHub] zhanghang1989 commented on a change in pull request #9931: Add axes support to Dropout for variational dropout in NLP

2018-03-02 Thread GitBox
zhanghang1989 commented on a change in pull request #9931: Add axes support to Dropout for variational dropout in NLP URL: https://github.com/apache/incubator-mxnet/pull/9931#discussion_r171931913 ## File path: src/operator/nn/dropout-inl.h ## @@ -178,30 +184,17 @@ class

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

2018-03-02 Thread GitBox
piiswrong commented on a change in pull request #9963: Onnx Module to import onnx models into mxnet URL: https://github.com/apache/incubator-mxnet/pull/9963#discussion_r171933621 ## File path: python/mxnet/contrib/onnx/_import/op_translations/negative.py ## @@ -0,0 +1,20

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

2018-03-02 Thread GitBox
anirudhacharya commented on a change in pull request #9963: Onnx Module to import onnx models into mxnet URL: https://github.com/apache/incubator-mxnet/pull/9963#discussion_r171939279 ## File path: python/mxnet/contrib/onnx/_import/op_translations/_add.py ## @@ -0,0 +1,25

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

2018-03-02 Thread GitBox
anirudhacharya commented on a change in pull request #9963: Onnx Module to import onnx models into mxnet URL: https://github.com/apache/incubator-mxnet/pull/9963#discussion_r171940657 ## File path: python/mxnet/contrib/onnx/_import/__init__.py ## @@ -0,0 +1,46 @@ +#

[GitHub] anirudhacharya commented on issue #9963: Onnx Module to import onnx models into mxnet

2018-03-02 Thread GitBox
anirudhacharya commented on issue #9963: Onnx Module to import onnx models into mxnet URL: https://github.com/apache/incubator-mxnet/pull/9963#issuecomment-370027006 For all the test cases to pass for this module, we will need changes in the CI process. It needs Onnx and protobuf in the

[GitHub] szha commented on issue #9957: R Package Fails to Build since #9882

2018-03-02 Thread GitBox
szha commented on issue #9957: R Package Fails to Build since #9882 URL: https://github.com/apache/incubator-mxnet/issues/9957#issuecomment-370028936 This seems to be the cause

[GitHub] piiswrong commented on issue #9962: how to group 2 or more outputs in gluon just like sym.Group()

2018-03-02 Thread GitBox
piiswrong commented on issue #9962: how to group 2 or more outputs in gluon just like sym.Group() URL: https://github.com/apache/incubator-mxnet/issues/9962#issuecomment-370031470 returning a list should work. What's error?

[GitHub] piiswrong commented on issue #9962: how to group 2 or more outputs in gluon just like sym.Group()

2018-03-02 Thread GitBox
piiswrong commented on issue #9962: how to group 2 or more outputs in gluon just like sym.Group() URL: https://github.com/apache/incubator-mxnet/issues/9962#issuecomment-370031470 returning a list should work. What's error? note that you need to unpack the list and only feed the output

[GitHub] yajiedesign commented on issue #9940: remove set CMAKE_GENERATOR_TOOLSET

2018-03-02 Thread GitBox
yajiedesign commented on issue #9940: remove set CMAKE_GENERATOR_TOOLSET URL: https://github.com/apache/incubator-mxnet/pull/9940#issuecomment-370098699 It can also maintain the status quo. It doesn't affect the compilation. Only CMAKE_GENERATOR_TOOLSET doesn't work.

[GitHub] zhanghang1989 commented on issue #9688: Adapt operators from PyTorch, will keep adding

2018-03-02 Thread GitBox
zhanghang1989 commented on issue #9688: Adapt operators from PyTorch, will keep adding URL: https://github.com/apache/incubator-mxnet/pull/9688#issuecomment-363525503 @chinakook ``` import mxnet as mx x1 = mx.nd.ones(shape=(2,3,4,4)) y1 = mx.nd.BilinearResize2D(x1,

[GitHub] jwfromm opened a new issue #9974: DataLoader with workers not compatible with ImageRecordDataset

2018-03-02 Thread GitBox
jwfromm opened a new issue #9974: DataLoader with workers not compatible with ImageRecordDataset URL: https://github.com/apache/incubator-mxnet/issues/9974 ## Description Using a DataLoader with a non-zero number of workers on a ImageRecordDataset crashes. Being able to have multiple

[GitHub] yajiedesign commented on issue #9940: remove set CMAKE_GENERATOR_TOOLSET

2018-03-02 Thread GitBox
yajiedesign commented on issue #9940: remove set CMAKE_GENERATOR_TOOLSET URL: https://github.com/apache/incubator-mxnet/pull/9940#issuecomment-370102121 If cmake is lower than 3.9.0, the ?old way? will be automatically enabled.

[GitHub] cjolivier01 commented on a change in pull request #9940: remove set CMAKE_GENERATOR_TOOLSET

2018-03-02 Thread GitBox
cjolivier01 commented on a change in pull request #9940: remove set CMAKE_GENERATOR_TOOLSET URL: https://github.com/apache/incubator-mxnet/pull/9940#discussion_r171999786 ## File path: CMakeLists.txt ## @@ -44,16 +44,16 @@ if(USE_CUDA AND NOT USE_OLDCMAKECUDA) OR

[GitHub] yajiedesign commented on a change in pull request #9940: remove set CMAKE_GENERATOR_TOOLSET

2018-03-02 Thread GitBox
yajiedesign commented on a change in pull request #9940: remove set CMAKE_GENERATOR_TOOLSET URL: https://github.com/apache/incubator-mxnet/pull/9940#discussion_r17200 ## File path: CMakeLists.txt ## @@ -44,16 +44,16 @@ if(USE_CUDA AND NOT USE_OLDCMAKECUDA) OR

[GitHub] jeremiedb commented on a change in pull request #9965: Fix R build crash in CI

2018-03-02 Thread GitBox
jeremiedb commented on a change in pull request #9965: Fix R build crash in CI URL: https://github.com/apache/incubator-mxnet/pull/9965#discussion_r172002855 ## File path: R-package/R/viz.graph.R ## @@ -143,7 +142,7 @@ graph.viz <- function(symbol, shape=NULL,

[GitHub] precedenceguo commented on a change in pull request #9939: add multi proposal operator (cpu version) and fix the bug in proposal op (gpu version)

2018-03-02 Thread GitBox
precedenceguo commented on a change in pull request #9939: add multi proposal operator (cpu version) and fix the bug in proposal op (gpu version) URL: https://github.com/apache/incubator-mxnet/pull/9939#discussion_r172002879 ## File path: src/operator/contrib/proposal.cu

[GitHub] wkcn commented on a change in pull request #9939: add multi proposal operator (cpu version) and fix the bug in proposal op (gpu version)

2018-03-02 Thread GitBox
wkcn commented on a change in pull request #9939: add multi proposal operator (cpu version) and fix the bug in proposal op (gpu version) URL: https://github.com/apache/incubator-mxnet/pull/9939#discussion_r172003236 ## File path: src/operator/contrib/proposal.cu ## @@

[GitHub] aidan-plenert-macdonald opened a new issue #9979: Computing the second derivative

2018-03-02 Thread GitBox
aidan-plenert-macdonald opened a new issue #9979: Computing the second derivative URL: https://github.com/apache/incubator-mxnet/issues/9979 I would like to compute the second derivative like, ```python from mxnet import nd, autograd x = nd.array([2.0]) with

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

2018-03-02 Thread GitBox
pengzhao-intel commented on issue #9939: add multi proposal operator (cpu version) and fix the bug in proposal op (gpu version) URL: https://github.com/apache/incubator-mxnet/pull/9939#issuecomment-370119051 @wkcn What's the size of your for-loop in the test-case? How can I

[GitHub] asitstands commented on issue #9978: Error with random generator

2018-03-02 Thread GitBox
asitstands commented on issue #9978: Error with random generator URL: https://github.com/apache/incubator-mxnet/issues/9978#issuecomment-370119319 The random number genertors in `mxnet.nd.random` and `mxnet.sym.random` have a fixed seed regardless of the use of `DataLoader`. You need to

[GitHub] asitstands commented on issue #9978: Error with random generator

2018-03-02 Thread GitBox
asitstands commented on issue #9978: Error with random generator URL: https://github.com/apache/incubator-mxnet/issues/9978#issuecomment-370119319 The random number genertors in `mxnet.nd.random` and `mxnet.sym.random` have a fixed seed regardless of the use of `DataLoader`. You need to

[GitHub] asitstands commented on issue #9978: Error with random generator

2018-03-02 Thread GitBox
asitstands commented on issue #9978: Error with random generator URL: https://github.com/apache/incubator-mxnet/issues/9978#issuecomment-370120684 ```python # test.py import mxnet as mx import numpy as np print(mx.nd.random.normal(0,1,shape=(2,2),dtype=np.float32,ctx=mx.gpu()))

[GitHub] asitstands commented on issue #9978: Error with random generator

2018-03-02 Thread GitBox
asitstands commented on issue #9978: Error with random generator URL: https://github.com/apache/incubator-mxnet/issues/9978#issuecomment-370120684 ```python # test.py import mxnet as mx import numpy as np print(mx.nd.random.normal(0,1,shape=(2,2),dtype=np.float32,ctx=mx.gpu()))

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

2018-03-02 Thread GitBox
reminisce commented on issue #9977: Cpu lstm inference URL: https://github.com/apache/incubator-mxnet/pull/9977#issuecomment-370122405 It's great to have cpu version implemented. We are deprecating operator implementation using the legacy interface. It's would be better if you can

[GitHub] pengzhao-intel commented on issue #9977: Cpu lstm inference

2018-03-02 Thread GitBox
pengzhao-intel commented on issue #9977: Cpu lstm inference URL: https://github.com/apache/incubator-mxnet/pull/9977#issuecomment-370118760 @Jerryzcn It's very great to explore the full CPU power for RNN cell. FYI, we have implemented the fused LSTM OP of CPU in local including

[GitHub] CoinCheung commented on issue #9978: Error with random generator

2018-03-02 Thread GitBox
CoinCheung commented on issue #9978: Error with random generator URL: https://github.com/apache/incubator-mxnet/issues/9978#issuecomment-370120074 @asitstands It does not seem so, when I run a single line in the terminal ```

[GitHub] orchidmajumder commented on issue #9962: how to group 2 or more outputs in gluon just like sym.Group()

2018-03-02 Thread GitBox
orchidmajumder commented on issue #9962: how to group 2 or more outputs in gluon just like sym.Group() URL: https://github.com/apache/incubator-mxnet/issues/9962#issuecomment-370121916 You are using imperative code in the hybrid block. Either you need to move it to make it static code

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

2018-03-02 Thread GitBox
Jerryzcn commented on issue #9977: Cpu lstm inference URL: https://github.com/apache/incubator-mxnet/pull/9977#issuecomment-370121842 @pengzhao-intel This is great, we can definitely collaborate. The reason I am sending this PR is for one of our own project. and we would like to have

[GitHub] reminisce commented on issue #9552: [REQUEST FOR REVIEW | DO NOT MERGE] Model Quantization with Calibration

2018-03-02 Thread GitBox
reminisce commented on issue #9552: [REQUEST FOR REVIEW | DO NOT MERGE] Model Quantization with Calibration URL: https://github.com/apache/incubator-mxnet/pull/9552#issuecomment-370122535 @wentingj Thanks for the benchmark results. It aligns with what we observed cudnn: int8 conv costs

[GitHub] eric-haibin-lin closed pull request #9651: Improve sparse adagrad update

2018-03-02 Thread GitBox
eric-haibin-lin closed pull request #9651: Improve sparse adagrad update URL: https://github.com/apache/incubator-mxnet/pull/9651 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: Improve sparse adagrad update (#9651)

2018-03-02 Thread haibin
This is an automated email from the ASF dual-hosted git repository. haibin pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git The following commit(s) were added to refs/heads/master by this push: new fc9e70b Improve sparse adagrad update

[GitHub] reminisce commented on issue #9976: The assignment problem about NDArray

2018-03-02 Thread GitBox
reminisce commented on issue #9976: The assignment problem about NDArray URL: https://github.com/apache/incubator-mxnet/issues/9976#issuecomment-370125976 This is a bug of `CopyFromTo` function. `a[0]` and `a[1]` share the same `var` and thus the copy is skipped.

[GitHub] wkcn commented on issue #9976: The assignment problem about NDArray

2018-03-02 Thread GitBox
wkcn commented on issue #9976: The assignment problem about NDArray URL: https://github.com/apache/incubator-mxnet/issues/9976#issuecomment-370127367 @reminisce Thank you! I will try to fix it. This is an automated message

[incubator-mxnet] branch master updated: Fix R build crash in CI (#9965)

2018-03-02 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 bce00aa Fix R build crash in CI

[GitHub] mseeger commented on issue #9961: Unary ops norm_logcdf, norm_derivlogcdf for log CDF of standard normal

2018-03-02 Thread GitBox
mseeger commented on issue #9961: Unary ops norm_logcdf, norm_derivlogcdf for log CDF of standard normal URL: https://github.com/apache/incubator-mxnet/pull/9961#issuecomment-370063432 I can change names to logcdf_normal, derivlogcdf_normal.

[GitHub] cjolivier01 opened a new pull request #9955: [DO NOT MERGE] Cython CI check

2018-03-02 Thread GitBox
cjolivier01 opened a new pull request #9955: [DO NOT MERGE] Cython CI check URL: https://github.com/apache/incubator-mxnet/pull/9955 ## Description ## This is an early WIP, so this PR will most likely get closed and recreated several times

[GitHub] marcoabreu commented on issue #9951: Fix test_cast

2018-03-02 Thread GitBox
marcoabreu commented on issue #9951: Fix test_cast URL: https://github.com/apache/incubator-mxnet/pull/9951#issuecomment-370079755 Yes, we got some flaky tests unfortunately. This is an automated message from the Apache Git

[GitHub] anirudhacharya commented on issue #9892: [WIP] Serde Module for Import/Export of models between Onnx and Mxnet

2018-03-02 Thread GitBox
anirudhacharya commented on issue #9892: [WIP] Serde Module for Import/Export of models between Onnx and Mxnet URL: https://github.com/apache/incubator-mxnet/pull/9892#issuecomment-369754064 will raise another PR. closing this PR https://github.com/apache/incubator-mxnet/pull/9963

[GitHub] cjolivier01 opened a new pull request #9955: [DO NOT MERGE] Cython CI check

2018-03-02 Thread GitBox
cjolivier01 opened a new pull request #9955: [DO NOT MERGE] Cython CI check URL: https://github.com/apache/incubator-mxnet/pull/9955 ## Description ## This is an early WIP, so this PR will most likely get closed and recreated several times

[GitHub] sxjscience commented on issue #9970: Incomplete UpSamping doc

2018-03-02 Thread GitBox
sxjscience commented on issue #9970: Incomplete UpSamping doc URL: https://github.com/apache/incubator-mxnet/issues/9970#issuecomment-370093851 Thanks for reporting this! @zhanghang1989 has written a new `BilinearResize` operator (See https://github.com/apache/incubator-mxnet/pull/9688).

[GitHub] sxjscience commented on issue #9970: Incomplete UpSamping doc

2018-03-02 Thread GitBox
sxjscience commented on issue #9970: Incomplete UpSamping doc URL: https://github.com/apache/incubator-mxnet/issues/9970#issuecomment-370094068 I'll close this first because the issue is the same as in https://github.com/apache/incubator-mxnet/issues/9138.

[GitHub] sxjscience closed issue #9970: Incomplete UpSamping doc

2018-03-02 Thread GitBox
sxjscience closed issue #9970: Incomplete UpSamping doc URL: https://github.com/apache/incubator-mxnet/issues/9970 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and

[GitHub] szha closed pull request #9965: Fix R build crash in CI

2018-03-02 Thread GitBox
szha closed pull request #9965: Fix R build crash in CI URL: https://github.com/apache/incubator-mxnet/pull/9965 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] marcoabreu commented on issue #9965: Fix R build crash in CI

2018-03-02 Thread GitBox
marcoabreu commented on issue #9965: Fix R build crash in CI URL: https://github.com/apache/incubator-mxnet/pull/9965#issuecomment-370057641 Thanks sheng Sheng Zha schrieb am Fr., 2. M?rz 2018, 22:23: > Merged #9965

[GitHub] marcoabreu commented on a change in pull request #9965: Fix R build crash in CI

2018-03-02 Thread GitBox
marcoabreu commented on a change in pull request #9965: Fix R build crash in CI URL: https://github.com/apache/incubator-mxnet/pull/9965#discussion_r171983600 ## File path: R-package/R/viz.graph.R ## @@ -143,7 +142,7 @@ graph.viz <- function(symbol, shape=NULL,

[GitHub] ehsanmok opened a new issue #9969: Expose MaxUnpooling operator

2018-03-02 Thread GitBox
ehsanmok opened a new issue #9969: Expose MaxUnpooling operator URL: https://github.com/apache/incubator-mxnet/issues/9969 [MaxUnpooling] operator API is not available despite its implementation [here](https://github.com/apache/incubator-mxnet/blob/master/src/operator/nn/pool.h#L388).

[GitHub] cjolivier01 closed pull request #9955: [DO NOT MERGE] Cython CI check

2018-03-02 Thread GitBox
cjolivier01 closed pull request #9955: [DO NOT MERGE] Cython CI check URL: https://github.com/apache/incubator-mxnet/pull/9955 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

[GitHub] marcoabreu closed pull request #9968: Fix typo 'maked' => made

2018-03-02 Thread GitBox
marcoabreu closed pull request #9968: Fix typo 'maked' => made URL: https://github.com/apache/incubator-mxnet/pull/9968 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

[GitHub] sxjscience commented on issue #8866: src/operator/./bilinear_sampler-inl.h:105: Have not implemented the data req combinations! gdata_req=0 ggrid_req=1

2018-03-02 Thread GitBox
sxjscience commented on issue #8866: src/operator/./bilinear_sampler-inl.h:105: Have not implemented the data req combinations! gdata_req=0 ggrid_req=1 URL: https://github.com/apache/incubator-mxnet/issues/8866#issuecomment-370094358 @chowkamlee81 Have you solved the problem? This looks

[incubator-mxnet] branch master updated: Fix typo (#9968)

2018-03-02 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 62b9d48 Fix typo (#9968) 62b9d48

[GitHub] marcoabreu commented on issue #9971: onnx-mxnet pip package prevents gpu use

2018-03-02 Thread GitBox
marcoabreu commented on issue #9971: onnx-mxnet pip package prevents gpu use URL: https://github.com/apache/incubator-mxnet/issues/9971#issuecomment-370094281 @KellenSunderland This is an automated message from the Apache

[GitHub] aaronmarkham opened a new issue #9973: update Scala tutorial - using IntelliJ IDE

2018-03-02 Thread GitBox
aaronmarkham opened a new issue #9973: update Scala tutorial - using IntelliJ IDE URL: https://github.com/apache/incubator-mxnet/issues/9973 ## Description http://mxnet.incubator.apache.org/tutorials/scala/mxnet_scala_on_intellij.html 1. Dated info - update and verify actual

[GitHub] terrytangyuan commented on a change in pull request #9965: Fix R build crash in CI

2018-03-02 Thread GitBox
terrytangyuan commented on a change in pull request #9965: Fix R build crash in CI URL: https://github.com/apache/incubator-mxnet/pull/9965#discussion_r171971463 ## File path: R-package/R/viz.graph.R ## @@ -143,7 +142,7 @@ graph.viz <- function(symbol, shape=NULL,

[GitHub] mseeger commented on a change in pull request #9961: Unary ops norm_logcdf, norm_derivlogcdf for log CDF of standard normal

2018-03-02 Thread GitBox
mseeger commented on a change in pull request #9961: Unary ops norm_logcdf, norm_derivlogcdf for log CDF of standard normal URL: https://github.com/apache/incubator-mxnet/pull/9961#discussion_r171971715 ## File path: docs/api/python/ndarray/ndarray.md ## @@ -606,6 +606,8

[GitHub] ThomasDelteil opened a new pull request #9968: Fix typo 'maked' => made

2018-03-02 Thread GitBox
ThomasDelteil opened a new pull request #9968: Fix typo 'maked' => made URL: https://github.com/apache/incubator-mxnet/pull/9968 Typo maked -> made ## Description ## (Brief description on what this PR is about) ## Checklist ## ### Essentials ### - [ ] Passed code

[GitHub] jwfromm opened a new issue #9971: onnx-mxnet pip package prevents gpu use

2018-03-02 Thread GitBox
jwfromm opened a new issue #9971: onnx-mxnet pip package prevents gpu use URL: https://github.com/apache/incubator-mxnet/issues/9971 Using pip to install mxnet-cu90 and onnx-mxnet causes pip to reinstall mxnet, removing GPU support. Changing the order the packages are installed does not

  1   2   >