[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 req

[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-370094839 @chowkamlee81 There is no need to provide an example. I'

[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 setu

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

2018-03-02 Thread GitBox
yajiedesign 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-370098120 sorry for later,The problem has come back, and I'll try to find out.

[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, out_height=5

[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. CMAKE_GENERATOR_TOOLSET

[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 wo

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

2018-03-02 Thread GitBox
cjolivier01 commented on issue #9940: remove set CMAKE_GENERATOR_TOOLSET URL: https://github.com/apache/incubator-mxnet/pull/9940#issuecomment-370101619 i don?t understand the statement about status quo? with your change, is it still possible to build the ?old way? without the generator?

[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] 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-370102668 ok,I've updated it again.. at present: 1.if cmake is lower than 3.9.0, auto choose "old way?. 2.if cmake greater than equal to or

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

2018-03-02 Thread GitBox
cjolivier01 commented on issue #9940: remove set CMAKE_GENERATOR_TOOLSET URL: https://github.com/apache/incubator-mxnet/pull/9940#issuecomment-370102721 We must have the ability to turn it off for any version of cmake, because it causes problems for some use cases such as running unit tests

[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-370102783 i am updated . This is an automated message from the Apache Git Service.

[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] 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_r172000299 ## File path: CMakeLists.txt ## @@ -44,16 +44,16 @@ if(USE_CUDA AND NOT USE_OLDCMAKECUDA) OR

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

2018-03-02 Thread GitBox
HaoLiuHust 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-370104619 the error is "HybridBlock input must be (nested) list of Symbol or NDArray, " \ "but got %s of ty

[GitHub] eric-haibin-lin commented on issue #9919: Update PR Template

2018-03-02 Thread GitBox
eric-haibin-lin commented on issue #9919: Update PR Template URL: https://github.com/apache/incubator-mxnet/pull/9919#issuecomment-370107887 Done This is an automated message from the Apache Git Service. To respond to the mess

[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, direction="TD

[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] jeremiedb opened a new pull request #9975: revert DiagrammeR compatibility to 0.9.0

2018-03-02 Thread GitBox
jeremiedb opened a new pull request #9975: revert DiagrammeR compatibility to 0.9.0 URL: https://github.com/apache/incubator-mxnet/pull/9975 Follow comments to maintain compatibility with 0.9.0: #9965 `attr_theme = NULL` is the only material change along the DESCRIPTION requirement. Ot

[GitHub] wkcn opened a new issue #9976: The assignment problem about NDArray

2018-03-02 Thread GitBox
wkcn opened a new issue #9976: The assignment problem about NDArray URL: https://github.com/apache/incubator-mxnet/issues/9976 ## Description If `a` is a multi-dims array, it is not available to use `a[0] = a[1]` to assign a row. However, it's available to use `a[0, :] = a[1]` or `a[0

[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 ## @@ -553

[GitHub] Jerryzcn opened a new pull request #9977: Cpu lstm inference

2018-03-02 Thread GitBox
Jerryzcn opened a new pull request #9977: Cpu lstm inference URL: https://github.com/apache/incubator-mxnet/pull/9977 ## Description ## (Brief description on what this PR is about) CPU LSTM inference kernel. This is around 9.5x faster than gluon LSTM cell. Verified on speech

[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 ## @@ -553

[GitHub] CoinCheung opened a new issue #9978: Error with random generator

2018-03-02 Thread GitBox
CoinCheung opened a new issue #9978: Error with random generator URL: https://github.com/apache/incubator-mxnet/issues/9978 ## Description Once I use DataLoader, the nd.random seed will be fixed. ## Environment info (Required) ``` --Python Info--

[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 autograd.rec

[GitHub] eric-haibin-lin opened a new issue #9980: [Feature Request] broadcast_mul(csr, dense)

2018-03-02 Thread GitBox
eric-haibin-lin opened a new issue #9980: [Feature Request] broadcast_mul(csr, dense) URL: https://github.com/apache/incubator-mxnet/issues/9980 Let's say we have a MxN CSR matrix, it's quite common to normalize the CSR matrix `A` by a length M vector or a length N vector `B`. How

[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 infer

[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 se

[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 se

[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 ``` print(mx.nd.random.normal(0,1,shape=(2,2),dtype=np.float

[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] 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 some

[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 (or

[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 refactor

[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 mor

[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 foreig

[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. https://github.com/apa

[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 fr

<    1   2