[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

[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

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

[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] 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] 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] 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] 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] 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-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] 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] 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] 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] 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] 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] 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] 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] 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] 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 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] 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] 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] 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] 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] 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 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 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-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] 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 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] 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] 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] 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 #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] 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] 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] 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-370094358 @chowkamlee81 Have you solved the problem? This looks li

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

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

[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] 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). I

[GitHub] anirudh2290 commented on issue #9972: Flaky test on Windows: test_random.test_random_seed_setting

2018-03-02 Thread GitBox
anirudh2290 commented on issue #9972: Flaky test on Windows: test_random.test_random_seed_setting URL: https://github.com/apache/incubator-mxnet/issues/9972#issuecomment-370093423 @marcoabreu can you please add the flaky test tag ?

[GitHub] anirudh2290 opened a new issue #9972: Flaky test on Windows: test_random.test_random_seed_setting

2018-03-02 Thread GitBox
anirudh2290 opened a new issue #9972: Flaky test on Windows: test_random.test_random_seed_setting URL: https://github.com/apache/incubator-mxnet/issues/9972 Note: Providing complete information in the most concise form is the best way to get help. This issue template serves as the checkli

[GitHub] sxjscience commented on issue #9969: Expose MaxUnpooling operator

2018-03-02 Thread GitBox
sxjscience commented on issue #9969: Expose MaxUnpooling operator URL: https://github.com/apache/incubator-mxnet/issues/9969#issuecomment-370092590 @ehsanmok Thanks for reporting this! Adding Unpooling layer is on our schedule. Please stay tuned. I'll close this because it's a duplicate of

[GitHub] sxjscience closed issue #9969: Expose MaxUnpooling operator

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

[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 https://issues.apache.org/jira/browse

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

[GitHub] ehsanmok opened a new issue #9970: Incomplete UpSamping doc

2018-03-02 Thread GitBox
ehsanmok opened a new issue #9970: Incomplete UpSamping doc URL: https://github.com/apache/incubator-mxnet/issues/9970 Despite the doc provided [here](https://github.com/apache/incubator-mxnet/blob/master/src/operator/nn/upsampling-inl.h#L78) there's no details provided in its [Python API

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

[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). This

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

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

[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_r171984279 ## File path: R-package/R/viz.graph.R ## @@ -143,7 +142,7 @@ graph.viz <- function(symbol, shape=NULL, direction="T

[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_r171984322 ## File path: R-package/R/viz.graph.R ## @@ -143,7 +142,7 @@ graph.viz <- function(symbol, shape=NULL, directio

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

[GitHub] reminisce commented on issue #9897: for custom callbacks in module fit, training script can not finish

2018-03-02 Thread GitBox
reminisce commented on issue #9897: for custom callbacks in module fit, training script can not finish URL: https://github.com/apache/incubator-mxnet/issues/9897#issuecomment-370070772 It looks like one logging thread got stuck. The dmlc/tensorboard was adapted from TensorFlow. There is

[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 https://issues.apache.org/jira/browse

[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] 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_r171971745 ## File path: R-package/R/viz.graph.R ## @@ -143,7 +142,7 @@ graph.viz <- function(symbol, shape=NULL, directio

[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] 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, directio

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

[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 (#99

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

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

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

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

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

[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 https://github.com/rich-iannone/DiagrammeR/commit/41a70fbcf5ea40f881c26515e839d07ebd726df5#diff-7347fe5a0f184f79e

[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 @@ +# Licens

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

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

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

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

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

[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 +1

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

[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 +606,8

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

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

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

  1   2   >