[GitHub] whu-lyh commented on issue #12086: distribute training on local machines not AWS cloud

whu-lyh commented on issue #12086: distribute training on local machines not AWS cloud URL: https://github.com/apache/incubator-mxnet/issues/12086#issuecomment-412492874 @rahul003 @rahul003 @rahul003 This is an automated

[GitHub] whu-lyh commented on issue #12086: distribute training on local machines not AWS cloud

whu-lyh commented on issue #12086: distribute training on local machines not AWS cloud URL: https://github.com/apache/incubator-mxnet/issues/12086#issuecomment-412492783 Anybody here?,The question i asked and described before in five days ago has confused me so many days. and i still

[GitHub] taliesinb commented on a change in pull request #12064: Allow stop of arange to be inferred from dims.

taliesinb commented on a change in pull request #12064: Allow stop of arange to be inferred from dims. URL: https://github.com/apache/incubator-mxnet/pull/12064#discussion_r209578617 ## File path: src/operator/tensor/init_op.h ## @@ -471,6 +473,11 @@ inline bool

[GitHub] KellenSunderland commented on issue #8270: 10 Undefined names in Python code

KellenSunderland commented on issue #8270: 10 Undefined names in Python code URL: https://github.com/apache/incubator-mxnet/issues/8270#issuecomment-412489332 @vandanavk Thanks for the catch! Should be addressed in https://github.com/apache/incubator-mxnet/pull/12147

[incubator-mxnet] branch master updated: Remove unused TensorRT code (#12147)

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 ee8755a Remove unused TensorRT

[GitHub] marcoabreu closed pull request #12147: [MXNET-703] Remove unused TensorRT code

marcoabreu closed pull request #12147: [MXNET-703] Remove unused TensorRT code URL: https://github.com/apache/incubator-mxnet/pull/12147 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

[GitHub] marcoabreu commented on issue #12146: Disable broken test test_io.test_CSVIter

marcoabreu commented on issue #12146: Disable broken test test_io.test_CSVIter URL: https://github.com/apache/incubator-mxnet/pull/12146#issuecomment-412484176 Thank you, I have changed the description and also edited the title of your issue.

[GitHub] sbodenstein commented on issue #11984: Generalized broadcast_like operator

sbodenstein commented on issue #11984: Generalized broadcast_like operator URL: https://github.com/apache/incubator-mxnet/pull/11984#issuecomment-412483709 @szha: could you review this PR? This is an automated message from

[GitHub] sbodenstein commented on a change in pull request #12035: Pad Operator Type Support

sbodenstein commented on a change in pull request #12035: Pad Operator Type Support URL: https://github.com/apache/incubator-mxnet/pull/12035#discussion_r209569552 ## File path: tests/python/unittest/test_operator.py ## @@ -2905,37 +2905,46 @@ def test_roipooling():

[GitHub] asitstands commented on issue #11268: A binary RBM example

asitstands commented on issue #11268: A binary RBM example URL: https://github.com/apache/incubator-mxnet/pull/11268#issuecomment-412480234 I updated the code. Now it shows the initial real images also. The sample image included in the README is the one provided by yifeim. Thanks @yifeim.

[GitHub] larroy edited a comment on issue #12146: Disable broken test test_io.test_CSVIter

larroy edited a comment on issue #12146: Disable broken test test_io.test_CSVIter URL: https://github.com/apache/incubator-mxnet/pull/12146#issuecomment-412478578 LGTM, I don't think the test is flaky. This is failing consistently and double free / corruption is a problem that has to be

[GitHub] larroy edited a comment on issue #12146: Disable broken test test_io.test_CSVIter

larroy edited a comment on issue #12146: Disable broken test test_io.test_CSVIter URL: https://github.com/apache/incubator-mxnet/pull/12146#issuecomment-412478578 LGTM, I don't think the test is flaky. This is failing consistently and double free / corruption is a problem that has to be

[GitHub] marcoabreu commented on issue #12148: Fix RAT license checker which is broken in trunk

marcoabreu commented on issue #12148: Fix RAT license checker which is broken in trunk URL: https://github.com/apache/incubator-mxnet/pull/12148#issuecomment-412478636 Could we split these efforts into separate PRs to have them with a single purpose instead of mixing concerns?

[GitHub] larroy commented on issue #12146: Disable broken test test_io.test_CSVIter

larroy commented on issue #12146: Disable broken test test_io.test_CSVIter URL: https://github.com/apache/incubator-mxnet/pull/12146#issuecomment-412478578 LGTM This is an automated message from the Apache Git Service. To

[GitHub] marcoabreu commented on issue #12149: Pin RAT checker version

marcoabreu commented on issue #12149: Pin RAT checker version URL: https://github.com/apache/incubator-mxnet/pull/12149#issuecomment-412478330 Duplicate of https://github.com/apache/incubator-mxnet/pull/12148 This is an

[GitHub] marcoabreu closed pull request #12149: Pin RAT checker version

marcoabreu closed pull request #12149: Pin RAT checker version URL: https://github.com/apache/incubator-mxnet/pull/12149 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] marcoabreu opened a new pull request #12149: Pin RAT checker version

marcoabreu opened a new pull request #12149: Pin RAT checker version URL: https://github.com/apache/incubator-mxnet/pull/12149 ## Description ## Pin the RAT version. The trunk currently seems to be broken and to avoided cases like this in future, we're pinning the version. ##

[GitHub] Faldict commented on issue #12142: Failed to import MXNet built with TensorRT

Faldict commented on issue #12142: Failed to import MXNet built with TensorRT URL: https://github.com/apache/incubator-mxnet/issues/12142#issuecomment-412477285 @KellenSunderland Thanks for your useful reply! I run `ldd libmxnet.so` and get the following information: ```

[GitHub] larroy opened a new pull request #12148: Fix RAT license checker which is broken in trunk

larroy opened a new pull request #12148: Fix RAT license checker which is broken in trunk URL: https://github.com/apache/incubator-mxnet/pull/12148 ## Description ## Fix RAT license checker which is broken in trunk Remove obsolete CI folder ## Checklist ## ###

[GitHub] lebeg commented on a change in pull request #11148: [MXNET-679] Refactor handling BLAS libraries with cmake

lebeg commented on a change in pull request #11148: [MXNET-679] Refactor handling BLAS libraries with cmake URL: https://github.com/apache/incubator-mxnet/pull/11148#discussion_r209539556 ## File path: cmake/ChooseBLAS.cmake ## @@ -0,0 +1,273 @@ +# Licensed to the Apache

[GitHub] lebeg commented on a change in pull request #11148: [MXNET-679] Refactor handling BLAS libraries with cmake

lebeg commented on a change in pull request #11148: [MXNET-679] Refactor handling BLAS libraries with cmake URL: https://github.com/apache/incubator-mxnet/pull/11148#discussion_r209539259 ## File path: cmake/ChooseBLAS.cmake ## @@ -0,0 +1,273 @@ +# Licensed to the Apache

[GitHub] ciyongch commented on issue #10205: [Operator] Accelerate the CPU side performance of topk

ciyongch commented on issue #10205: [Operator] Accelerate the CPU side performance of topk URL: https://github.com/apache/incubator-mxnet/issues/10205#issuecomment-412451725 Hi @tdomhan @mjpost @fhieber, the optimized `topk` has being merged into MXNet's master branch now :) I

[GitHub] KellenSunderland commented on issue #12147: [MXNET-703] Remove unused TensorRT code

KellenSunderland commented on issue #12147: [MXNET-703] Remove unused TensorRT code URL: https://github.com/apache/incubator-mxnet/pull/12147#issuecomment-412450408 As discussed off-line, hope it's ok if we remove this code @mkolod. We've got it in git history if needed. @cclauss

[GitHub] KellenSunderland commented on issue #12142: Failed to import MXNet built with TensorRT

KellenSunderland commented on issue #12142: Failed to import MXNet built with TensorRT URL: https://github.com/apache/incubator-mxnet/issues/12142#issuecomment-412449388 Hey @Faldict. The problem is that you don't have protobuf on your LD_LIBRARY_PATH. I'd recommend setting your path

[GitHub] KellenSunderland edited a comment on issue #12142: Failed to import MXNet built with TensorRT

KellenSunderland edited a comment on issue #12142: Failed to import MXNet built with TensorRT URL: https://github.com/apache/incubator-mxnet/issues/12142#issuecomment-412449388 Hey @Faldict. The problem is that you don't have protobuf on your LD_LIBRARY_PATH. I'd recommend setting your

[GitHub] cclauss commented on issue #12144: [MXNET-313] Undefined names: C --> ctypes

cclauss commented on issue #12144: [MXNET-313] Undefined names: C --> ctypes URL: https://github.com/apache/incubator-mxnet/pull/12144#issuecomment-412447784 Closed in favor of #12147 This is an automated message from the

[GitHub] cclauss closed pull request #12144: [MXNET-313] Undefined names: C --> ctypes

cclauss closed pull request #12144: [MXNET-313] Undefined names: C --> ctypes URL: https://github.com/apache/incubator-mxnet/pull/12144 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a

[GitHub] KellenSunderland opened a new pull request #12147: [MXNET-703] Remove unused TensorRT code

KellenSunderland opened a new pull request #12147: [MXNET-703] Remove unused TensorRT code URL: https://github.com/apache/incubator-mxnet/pull/12147 Removing some python code that isn't in the current TensorRT execution paths. This should make the code more readable and avoid potential

[GitHub] marcoabreu opened a new pull request #12146: Disable broken test test_io.test_CSVIter

marcoabreu opened a new pull request #12146: Disable broken test test_io.test_CSVIter URL: https://github.com/apache/incubator-mxnet/pull/12146 https://github.com/apache/incubator-mxnet/issues/12139 This is an automated

[GitHub] prathik-naidu commented on issue #12127: LSTM split0 Operator Error

prathik-naidu commented on issue #12127: LSTM split0 Operator Error URL: https://github.com/apache/incubator-mxnet/issues/12127#issuecomment-412445819 @lanking520 Sure, I also just put this up on discuss.mxnet.io This is an

[GitHub] marcoabreu commented on issue #12139: Flaky test test_io.test_CSVIter

marcoabreu commented on issue #12139: Flaky test test_io.test_CSVIter URL: https://github.com/apache/incubator-mxnet/issues/12139#issuecomment-412445253 http://jenkins.mxnet-ci.amazon-ml.com/blue/organizations/jenkins/incubator-mxnet/detail/master/1397/pipeline

[GitHub] KellenSunderland commented on issue #8270: 10 Undefined names in Python code

KellenSunderland commented on issue #8270: 10 Undefined names in Python code URL: https://github.com/apache/incubator-mxnet/issues/8270#issuecomment-412441700 @vandanavk Taking a look. This is an automated message from the

[GitHub] mdespriee commented on issue #12140: Impossible to provide arguments to random_normal in scala ?

mdespriee commented on issue #12140: Impossible to provide arguments to random_normal in scala ? URL: https://github.com/apache/incubator-mxnet/issues/12140#issuecomment-412439645 Actually, I tried that as well, and can't make it work either. ```scala object NormalTest extends App

[GitHub] yifeim commented on a change in pull request #11268: A binary RBM example

yifeim commented on a change in pull request #11268: A binary RBM example URL: https://github.com/apache/incubator-mxnet/pull/11268#discussion_r209519538 ## File path: example/restricted-boltzmann-machine/binary_rbm_gluon.py ## @@ -0,0 +1,139 @@ +# Licensed to the Apache

[GitHub] TS0713 commented on issue #12119: mxnet package in r (3.5.1) - It doesn't have lstm and gluon

TS0713 commented on issue #12119: mxnet package in r (3.5.1) - It doesn't have lstm and gluon URL: https://github.com/apache/incubator-mxnet/issues/12119#issuecomment-412435329 Hi, Thanks for your support. Indeed I have tried installing for R version below 3.5.0 , still I am not

[GitHub] asitstands commented on a change in pull request #11268: A binary RBM example

asitstands commented on a change in pull request #11268: A binary RBM example URL: https://github.com/apache/incubator-mxnet/pull/11268#discussion_r209515585 ## File path: example/restricted-boltzmann-machine/binary_rbm_gluon.py ## @@ -0,0 +1,139 @@ +# Licensed to the

[GitHub] fengshi2017 opened a new issue #12145: syncbatchnorm for 3d cnn

fengshi2017 opened a new issue #12145: syncbatchnorm for 3d cnn URL: https://github.com/apache/incubator-mxnet/issues/12145 Hello, during training I3d with videos, I simply replace gluon.nn.BatchNorm with gluon.nn.SyncBatchNorm but an error as following is reported. Could anyone explain

[GitHub] asitstands commented on a change in pull request #11268: A binary RBM example

asitstands commented on a change in pull request #11268: A binary RBM example URL: https://github.com/apache/incubator-mxnet/pull/11268#discussion_r209515585 ## File path: example/restricted-boltzmann-machine/binary_rbm_gluon.py ## @@ -0,0 +1,139 @@ +# Licensed to the

[GitHub] marcoabreu commented on issue #12142: Failed to import MXNet built with TensorRT

marcoabreu commented on issue #12142: Failed to import MXNet built with TensorRT URL: https://github.com/apache/incubator-mxnet/issues/12142#issuecomment-412432923 @KellenSunderland This is an automated message from the

[GitHub] marcoabreu commented on issue #12143: Use dict comprehension to avoid backslash

marcoabreu commented on issue #12143: Use dict comprehension to avoid backslash URL: https://github.com/apache/incubator-mxnet/pull/12143#issuecomment-412432709 Thanks for your contribution! What do you think about adding a small case to avoid this in future?

[GitHub] anirudhacharya commented on issue #12119: mxnet package in r (3.5.1) - It doesn't have lstm and gluon

anirudhacharya commented on issue #12119: mxnet package in r (3.5.1) - It doesn't have lstm and gluon URL: https://github.com/apache/incubator-mxnet/issues/12119#issuecomment-412430378 @TS0713 gluon is not yet supported in the R API. For lstm, you can check this tutorial for pointers on

[GitHub] cclauss opened a new pull request #12144: [MXNET-313] Undefined names: C --> ctypes

cclauss opened a new pull request #12144: [MXNET-313] Undefined names: C --> ctypes URL: https://github.com/apache/incubator-mxnet/pull/12144 Discovered via https://github.com/apache/incubator-mxnet/issues/8270#issuecomment-412399056 This could be a copy and paste error where the

[GitHub] yifeim commented on a change in pull request #11268: A binary RBM example

yifeim commented on a change in pull request #11268: A binary RBM example URL: https://github.com/apache/incubator-mxnet/pull/11268#discussion_r209505568 ## File path: example/restricted-boltzmann-machine/binary_rbm_gluon.py ## @@ -0,0 +1,139 @@ +# Licensed to the Apache

[GitHub] yifeim commented on a change in pull request #11268: A binary RBM example

yifeim commented on a change in pull request #11268: A binary RBM example URL: https://github.com/apache/incubator-mxnet/pull/11268#discussion_r209505070 ## File path: example/restricted-boltzmann-machine/binary_rbm_gluon.py ## @@ -0,0 +1,139 @@ +# Licensed to the Apache

[GitHub] yifeim commented on a change in pull request #11268: A binary RBM example

yifeim commented on a change in pull request #11268: A binary RBM example URL: https://github.com/apache/incubator-mxnet/pull/11268#discussion_r209504826 ## File path: example/restricted-boltzmann-machine/binary_rbm_gluon.py ## @@ -0,0 +1,139 @@ +# Licensed to the Apache

[GitHub] szha commented on a change in pull request #11268: A binary RBM example

szha commented on a change in pull request #11268: A binary RBM example URL: https://github.com/apache/incubator-mxnet/pull/11268#discussion_r209504713 ## File path: example/restricted-boltzmann-machine/binary_rbm_gluon.py ## @@ -0,0 +1,139 @@ +# Licensed to the Apache

[GitHub] szha commented on a change in pull request #12064: Allow stop of arange to be inferred from dims.

szha commented on a change in pull request #12064: Allow stop of arange to be inferred from dims. URL: https://github.com/apache/incubator-mxnet/pull/12064#discussion_r209504005 ## File path: src/operator/tensor/init_op.h ## @@ -471,6 +473,11 @@ inline bool

[GitHub] cclauss opened a new pull request #12143: Use dict comprehension to avoid backslash

cclauss opened a new pull request #12143: Use dict comprehension to avoid backslash URL: https://github.com/apache/incubator-mxnet/pull/12143 One whitespace character to the right of the backslash breaks the script because of a change that is invisible to the reader. ## Description

[GitHub] asitstands commented on a change in pull request #11268: A binary RBM example

asitstands commented on a change in pull request #11268: A binary RBM example URL: https://github.com/apache/incubator-mxnet/pull/11268#discussion_r209502861 ## File path: example/restricted-boltzmann-machine/binary_rbm_gluon.py ## @@ -0,0 +1,139 @@ +# Licensed to the

[GitHub] yifeim commented on a change in pull request #11268: A binary RBM example

yifeim commented on a change in pull request #11268: A binary RBM example URL: https://github.com/apache/incubator-mxnet/pull/11268#discussion_r209502873 ## File path: example/restricted-boltzmann-machine/binary_rbm_gluon.py ## @@ -0,0 +1,139 @@ +# Licensed to the Apache

[GitHub] anirudhacharya commented on issue #12132: [Feature Request] Implementation of L-BFGS optimizer

anirudhacharya commented on issue #12132: [Feature Request] Implementation of L-BFGS optimizer URL: https://github.com/apache/incubator-mxnet/issues/12132#issuecomment-412420766 I dont think MXNet has this optimizer, and optimizer requests are tracked here -

[GitHub] lanking520 commented on issue #12127: LSTM split0 Operator Error

lanking520 commented on issue #12127: LSTM split0 Operator Error URL: https://github.com/apache/incubator-mxnet/issues/12127#issuecomment-412420574 @prathik-naidu Thanks for your issue. Please also put this issue to the discuss.mxnet.io, there are more people working on different

[GitHub] lanking520 edited a comment on issue #12139: Centos7 build is broken in master

lanking520 edited a comment on issue #12139: Centos7 build is broken in master URL: https://github.com/apache/incubator-mxnet/issues/12139#issuecomment-412416804 @mxnet-label-bot please label this as [build, CI] same in here:

[GitHub] lanking520 commented on issue #12134: why MKL and MKL-DNN can't be used simultaneously in ChooseBlas.cmake

lanking520 commented on issue #12134: why MKL and MKL-DNN can't be used simultaneously in ChooseBlas.cmake URL: https://github.com/apache/incubator-mxnet/issues/12134#issuecomment-412419608 @mxnet-label-bot could you please label this as [build, cmake, mkldnn]

[GitHub] lanking520 commented on issue #12142: Failed to import MXNet built with TensorRT

lanking520 commented on issue #12142: Failed to import MXNet built with TensorRT URL: https://github.com/apache/incubator-mxnet/issues/12142#issuecomment-412419276 Hi @Faldict thanks for your reported issue. @haojin2 could you please take a look at here, I remember somebody is

[GitHub] lanking520 commented on issue #12141: C_API for C++ iterator only supports one data and one label for one sample

lanking520 commented on issue #12141: C_API for C++ iterator only supports one data and one label for one sample URL: https://github.com/apache/incubator-mxnet/issues/12141#issuecomment-412418004 @squidszyd thanks for your discovery. The C++ API is incomplete comparing to other languages

[GitHub] sandeep-krishnamurthy commented on a change in pull request #12137: [MXNET-696] Fix undefined name errors

sandeep-krishnamurthy commented on a change in pull request #12137: [MXNET-696] Fix undefined name errors URL: https://github.com/apache/incubator-mxnet/pull/12137#discussion_r209498946 ## File path: example/profiler/profiler_executor.py ## @@ -84,9 +84,7 @@ def

[GitHub] lanking520 commented on issue #12139: Centos7 build is broken in master

lanking520 commented on issue #12139: Centos7 build is broken in master URL: https://github.com/apache/incubator-mxnet/issues/12139#issuecomment-412416804 @mxnet-label-bot please label this as [build, CI] This is an

[GitHub] lanking520 commented on issue #12132: [Feature Request] Implementation of L-BFGS optimizer

lanking520 commented on issue #12132: [Feature Request] Implementation of L-BFGS optimizer URL: https://github.com/apache/incubator-mxnet/issues/12132#issuecomment-412416624 @jasonyu1996 thanks for your question and keep watching this thread. @eric-haibin-lin do we have something

[GitHub] lanking520 commented on issue #12140: Impossible to provide arguments to random_normal in scala ?

lanking520 commented on issue #12140: Impossible to provide arguments to random_normal in scala ? URL: https://github.com/apache/incubator-mxnet/issues/12140#issuecomment-412416142 Hi @mdespriee Nice try with Scala package! In your case, you can implement in the following ways:

<    1   2