[GitHub] cjolivier01 commented on a change in pull request #10090: [MXNET-86] Revert to pre-profile-changes copy code

cjolivier01 commented on a change in pull request #10090: [MXNET-86] Revert to pre-profile-changes copy code URL: https://github.com/apache/incubator-mxnet/pull/10090#discussion_r174295660 ## File path: src/ndarray/ndarray_function.cc ## @@ -26,35 +26,24 @@ #include

[GitHub] sxjscience commented on a change in pull request #10090: [MXNET-86] Revert to pre-profile-changes copy code

sxjscience commented on a change in pull request #10090: [MXNET-86] Revert to pre-profile-changes copy code URL: https://github.com/apache/incubator-mxnet/pull/10090#discussion_r174294274 ## File path: src/ndarray/ndarray_function.cc ## @@ -26,35 +26,24 @@ #include

[GitHub] cjolivier01 commented on a change in pull request #10089: enable all activations in MKLDNN.

cjolivier01 commented on a change in pull request #10089: enable all activations in MKLDNN. URL: https://github.com/apache/incubator-mxnet/pull/10089#discussion_r174294065 ## File path: src/operator/nn/mkldnn/mkldnn_act.cc ## @@ -45,11 +45,9 @@ namespace op { bool

[GitHub] marcoabreu commented on issue #10089: enable all activations in MKLDNN.

marcoabreu commented on issue #10089: enable all activations in MKLDNN. URL: https://github.com/apache/incubator-mxnet/pull/10089#issuecomment-372827762 How have the precision problems been resolved? Is there a test? This is

[GitHub] marcoabreu commented on a change in pull request #10089: enable all activations in MKLDNN.

marcoabreu commented on a change in pull request #10089: enable all activations in MKLDNN. URL: https://github.com/apache/incubator-mxnet/pull/10089#discussion_r174293369 ## File path: src/operator/nn/mkldnn/mkldnn_act.cc ## @@ -45,11 +45,9 @@ namespace op { bool

[GitHub] marcoabreu commented on issue #10090: [MXNET-86] Revert to pre-profile-changes copy code

marcoabreu commented on issue #10090: [MXNET-86] Revert to pre-profile-changes copy code URL: https://github.com/apache/incubator-mxnet/pull/10090#issuecomment-372827316 Would you mind creating a test with the MVE the user provided to ensure this does not happen again?

[GitHub] cjolivier01 commented on issue #10042: [MXNET-86] Gluon dataloader crash on speech recognition training

cjolivier01 commented on issue #10042: [MXNET-86] Gluon dataloader crash on speech recognition training URL: https://github.com/apache/incubator-mxnet/issues/10042#issuecomment-372827323 I think that would be a separate issue. This one so far is just the "stuck" fix.

[GitHub] marcoabreu commented on issue #10090: [MXNET-86] Revert to pre-profile-changes copy code

marcoabreu commented on issue #10090: [MXNET-86] Revert to pre-profile-changes copy code URL: https://github.com/apache/incubator-mxnet/pull/10090#issuecomment-372827316 Would you mind creating a test with the MWE the user provided to ensure this does not happen again?

[GitHub] cjolivier01 commented on issue #10042: [MXNET-86] Gluon dataloader crash on speech recognition training

cjolivier01 commented on issue #10042: [MXNET-86] Gluon dataloader crash on speech recognition training URL: https://github.com/apache/incubator-mxnet/issues/10042#issuecomment-372827323 I think that would be a separate issue.

[GitHub] marcoabreu commented on issue #9919: Update PR Template

marcoabreu commented on issue #9919: Update PR Template URL: https://github.com/apache/incubator-mxnet/pull/9919#issuecomment-372826823 Could we also add an item to remind people linking the JIRA issue? This is an automated

[GitHub] cjolivier01 opened a new pull request #10090: Revert to pre-profile-changes copy code

cjolivier01 opened a new pull request #10090: Revert to pre-profile-changes copy code URL: https://github.com/apache/incubator-mxnet/pull/10090 ## Description ## Fix for: https://github.com/apache/incubator-mxnet/issues/10042 For whatever reason, the new approach isn't

[GitHub] marcoabreu commented on a change in pull request #9928: host doc on s3

marcoabreu commented on a change in pull request #9928: host doc on s3 URL: https://github.com/apache/incubator-mxnet/pull/9928#discussion_r174286591 ## File path: tests/ci_build/deploy/ci_deploy_doc.sh ## @@ -0,0 +1,35 @@ +#!/usr/bin/env bash + +# Licensed to the Apache

[GitHub] nswamy commented on a change in pull request #9678: [MXNET-50] Scala Inference APIs

nswamy commented on a change in pull request #9678: [MXNET-50] Scala Inference APIs URL: https://github.com/apache/incubator-mxnet/pull/9678#discussion_r174285487 ## File path: scala-package/infer/src/main/scala/ml/dmlc/mxnet/infer/Predictor.scala ## @@ -0,0 +1,188 @@

[GitHub] sxjscience commented on issue #10042: [MXNET-86] Gluon dataloader crash on speech recognition training

sxjscience commented on issue #10042: [MXNET-86] Gluon dataloader crash on speech recognition training URL: https://github.com/apache/incubator-mxnet/issues/10042#issuecomment-372817252 @cjolivier01 Before the commit it will not freeze and will print the data instead.

[GitHub] Jerryzcn commented on issue #10042: [MXNET-86] Gluon dataloader crash on speech recognition training

Jerryzcn commented on issue #10042: [MXNET-86] Gluon dataloader crash on speech recognition training URL: https://github.com/apache/incubator-mxnet/issues/10042#issuecomment-372817360 there seems to be other issues as well, after training for 1 day or so i got segfault. This does not

[GitHub] sxjscience commented on issue #10042: [MXNET-86] Gluon dataloader crash on speech recognition training

sxjscience commented on issue #10042: [MXNET-86] Gluon dataloader crash on speech recognition training URL: https://github.com/apache/incubator-mxnet/issues/10042#issuecomment-372817252 @cjolivier01 Before the commit it will not freeze and print the data.

[GitHub] Jerryzcn commented on issue #10042: [MXNET-86] Gluon dataloader crash on speech recognition training

Jerryzcn commented on issue #10042: [MXNET-86] Gluon dataloader crash on speech recognition training URL: https://github.com/apache/incubator-mxnet/issues/10042#issuecomment-372816606 The error output only happens when you train on the actual data. When you use the test script, it will

[GitHub] Jerryzcn commented on issue #10042: [MXNET-86] Gluon dataloader crash on speech recognition training

Jerryzcn commented on issue #10042: [MXNET-86] Gluon dataloader crash on speech recognition training URL: https://github.com/apache/incubator-mxnet/issues/10042#issuecomment-372816606 The error output only happens when you train on the actual data. However, when use the test script, it

[GitHub] Jerryzcn commented on issue #10042: [MXNET-86] Gluon dataloader crash on speech recognition training

Jerryzcn commented on issue #10042: [MXNET-86] Gluon dataloader crash on speech recognition training URL: https://github.com/apache/incubator-mxnet/issues/10042#issuecomment-372816606 The error output only happens when you train on the actual data. When use the test script, it will

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

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

[GitHub] eric-haibin-lin commented on a change in pull request #9928: host doc on s3

eric-haibin-lin commented on a change in pull request #9928: host doc on s3 URL: https://github.com/apache/incubator-mxnet/pull/9928#discussion_r174279994 ## File path: tests/ci_build/deploy/ci_deploy_doc.sh ## @@ -0,0 +1,35 @@ +#!/usr/bin/env bash + +# Licensed to the

[GitHub] cjolivier01 commented on issue #10042: [MXNET-86] Gluon dataloader crash on speech recognition training

cjolivier01 commented on issue #10042: [MXNET-86] Gluon dataloader crash on speech recognition training URL: https://github.com/apache/incubator-mxnet/issues/10042#issuecomment-372804704 On which epoch/batch does it occur?

[GitHub] cjolivier01 commented on issue #10042: [MXNET-86] Gluon dataloader crash on speech recognition training

cjolivier01 commented on issue #10042: [MXNET-86] Gluon dataloader crash on speech recognition training URL: https://github.com/apache/incubator-mxnet/issues/10042#issuecomment-372801771 This script just freezes for me without doing anything. In Connection._recv, it seems., What is it

[GitHub] cjolivier01 commented on issue #10042: [MXNET-86] Gluon dataloader crash on speech recognition training

cjolivier01 commented on issue #10042: [MXNET-86] Gluon dataloader crash on speech recognition training URL: https://github.com/apache/incubator-mxnet/issues/10042#issuecomment-372804704 On which epoch/batch does it occur?

[GitHub] cjolivier01 commented on issue #10042: [MXNET-86] Gluon dataloader crash on speech recognition training

cjolivier01 commented on issue #10042: [MXNET-86] Gluon dataloader crash on speech recognition training URL: https://github.com/apache/incubator-mxnet/issues/10042#issuecomment-372801771 This script just freezes for me without doing anything. In Connection._recv, it seems., What is it

[GitHub] cjolivier01 commented on issue #10075: [MXNET-83] Fix CMake build issue with MKL.

cjolivier01 commented on issue #10075: [MXNET-83] Fix CMake build issue with MKL. URL: https://github.com/apache/incubator-mxnet/pull/10075#issuecomment-372797327 I am not sure if the mkl flags in cmake still follow those, but I don't think these changes affect that behavior (allowable

[GitHub] cjolivier01 commented on a change in pull request #10081: [MXNET-82] [WIP] Sparse op tutorial for developers

cjolivier01 commented on a change in pull request #10081: [MXNET-82] [WIP] Sparse op tutorial for developers URL: https://github.com/apache/incubator-mxnet/pull/10081#discussion_r174262825 ## File path: docs/how_to/add_sparse_op_in_backend.md ## @@ -0,0 +1,457 @@ +# TODO

[GitHub] anirudh2290 commented on issue #10075: [MXNET-83] Fix CMake build issue with MKL.

anirudh2290 commented on issue #10075: [MXNET-83] Fix CMake build issue with MKL. URL: https://github.com/apache/incubator-mxnet/pull/10075#issuecomment-372787414 @pengzhao-intel thank you for the explanation. Is this information documented somewhere on the Intel or mxnet docs ?

[GitHub] haojin2 commented on a change in pull request #10078: [MXNET-92] Support float16 in L2Normalization operator

haojin2 commented on a change in pull request #10078: [MXNET-92] Support float16 in L2Normalization operator URL: https://github.com/apache/incubator-mxnet/pull/10078#discussion_r174252581 ## File path: tests/python/unittest/test_operator.py ## @@ -2396,21 +2396,22 @@ def

[GitHub] haojin2 commented on a change in pull request #10078: [MXNET-92] Support float16 in L2Normalization operator

haojin2 commented on a change in pull request #10078: [MXNET-92] Support float16 in L2Normalization operator URL: https://github.com/apache/incubator-mxnet/pull/10078#discussion_r174252529 ## File path: src/operator/l2_normalization-inl.h ## @@ -235,6 +247,19 @@ class

[GitHub] anirudh2290 commented on a change in pull request #10081: [MXNET-82] [WIP] Sparse op tutorial for developers

anirudh2290 commented on a change in pull request #10081: [MXNET-82] [WIP] Sparse op tutorial for developers URL: https://github.com/apache/incubator-mxnet/pull/10081#discussion_r174248908 ## File path: docs/how_to/add_sparse_op_in_backend.md ## @@ -0,0 +1,457 @@ +# TODO

[GitHub] cjolivier01 commented on a change in pull request #10078: [MXNET-92] Support float16 in L2Normalization operator

cjolivier01 commented on a change in pull request #10078: [MXNET-92] Support float16 in L2Normalization operator URL: https://github.com/apache/incubator-mxnet/pull/10078#discussion_r174248453 ## File path: src/operator/l2_normalization.cc ## @@ -26,13 +26,22 @@

[GitHub] anirudh2290 commented on a change in pull request #10078: [MXNET-92] Support float16 in L2Normalization operator

anirudh2290 commented on a change in pull request #10078: [MXNET-92] Support float16 in L2Normalization operator URL: https://github.com/apache/incubator-mxnet/pull/10078#discussion_r174246995 ## File path: tests/python/unittest/test_operator.py ## @@ -2396,21 +2396,22 @@

[GitHub] nswamy commented on a change in pull request #9678: [MXNET-50] Scala Inference APIs

nswamy commented on a change in pull request #9678: [MXNET-50] Scala Inference APIs URL: https://github.com/apache/incubator-mxnet/pull/9678#discussion_r174247041 ## File path: scala-package/infer/src/main/scala/ml/dmlc/mxnet/infer/Classifier.scala ## @@ -0,0 +1,167 @@

[GitHub] szha closed issue #9865: Confusing behavior of some evaluation metrics

szha closed issue #9865: Confusing behavior of some evaluation metrics URL: https://github.com/apache/incubator-mxnet/issues/9865 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] yzhliu commented on a change in pull request #9678: [MXNET-50] Scala Inference APIs

yzhliu commented on a change in pull request #9678: [MXNET-50] Scala Inference APIs URL: https://github.com/apache/incubator-mxnet/pull/9678#discussion_r174218993 ## File path: scala-package/infer/src/main/scala/ml/dmlc/mxnet/infer/Classifier.scala ## @@ -0,0 +1,167 @@

[incubator-mxnet] branch master updated: Support single array input for metric (#9930)

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 dd1f21b Support single array input

[GitHub] yzhliu commented on a change in pull request #9678: [MXNET-50] Scala Inference APIs

yzhliu commented on a change in pull request #9678: [MXNET-50] Scala Inference APIs URL: https://github.com/apache/incubator-mxnet/pull/9678#discussion_r174218503 ## File path: scala-package/infer/src/main/scala/ml/dmlc/mxnet/infer/Classifier.scala ## @@ -0,0 +1,167 @@

[GitHub] yzhliu commented on a change in pull request #9678: [MXNET-50] Scala Inference APIs

yzhliu commented on a change in pull request #9678: [MXNET-50] Scala Inference APIs URL: https://github.com/apache/incubator-mxnet/pull/9678#discussion_r174244994 ## File path: scala-package/infer/src/main/scala/ml/dmlc/mxnet/infer/Predictor.scala ## @@ -0,0 +1,188 @@

[GitHub] yzhliu commented on a change in pull request #9678: [MXNET-50] Scala Inference APIs

yzhliu commented on a change in pull request #9678: [MXNET-50] Scala Inference APIs URL: https://github.com/apache/incubator-mxnet/pull/9678#discussion_r174241847 ## File path: scala-package/infer/src/main/scala/ml/dmlc/mxnet/infer/Classifier.scala ## @@ -0,0 +1,167 @@

[GitHub] szha closed pull request #9930: Support single array input for metric

szha closed pull request #9930: Support single array input for metric URL: https://github.com/apache/incubator-mxnet/pull/9930 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 #9996: add softsign operator to v1.1.0

cjolivier01 commented on issue #9996: add softsign operator to v1.1.0 URL: https://github.com/apache/incubator-mxnet/pull/9996#issuecomment-372775775 I have no idea. I just see an open PR without a JIRA. This is an automated

[GitHub] cjolivier01 commented on issue #10078: [MXNET-92] Support float16 in L2Normalization operator

cjolivier01 commented on issue #10078: [MXNET-92] Support float16 in L2Normalization operator URL: https://github.com/apache/incubator-mxnet/pull/10078#issuecomment-372773464 JIRA: https://issues.apache.org/jira/browse/MXNET-92

[GitHub] robbig2871 opened a new issue #10088: can not install

robbig2871 opened a new issue #10088: can not install URL: https://github.com/apache/incubator-mxnet/issues/10088 > cran <- getOption("repos") > cran["dmlc"] <- "https://apache-mxnet.s3-accelerate.dualstack.amazonaws.com/R/CRAN/; > options(repos = cran) > install.packages("mxnet")

[GitHub] reminisce commented on a change in pull request #10078: Support float16 in L2Normalization operator

reminisce commented on a change in pull request #10078: Support float16 in L2Normalization operator URL: https://github.com/apache/incubator-mxnet/pull/10078#discussion_r174235637 ## File path: src/operator/l2_normalization-inl.h ## @@ -235,6 +247,19 @@ class

[GitHub] sxjscience commented on a change in pull request #10000: [MXNET-80] Fix average pooling kernel size assignment error

sxjscience commented on a change in pull request #1: [MXNET-80] Fix average pooling kernel size assignment error URL: https://github.com/apache/incubator-mxnet/pull/1#discussion_r174234577 ## File path: src/operator/nn/pooling.cc ## @@ -98,9 +103,24 @@ static bool

[GitHub] sxjscience commented on a change in pull request #10000: [MXNET-80] Fix average pooling kernel size assignment error

sxjscience commented on a change in pull request #1: [MXNET-80] Fix average pooling kernel size assignment error URL: https://github.com/apache/incubator-mxnet/pull/1#discussion_r174234503 ## File path: src/operator/nn/pooling.cc ## @@ -98,9 +103,24 @@ static bool

[GitHub] sxjscience commented on a change in pull request #10000: [MXNET-80] Fix average pooling kernel size assignment error

sxjscience commented on a change in pull request #1: [MXNET-80] Fix average pooling kernel size assignment error URL: https://github.com/apache/incubator-mxnet/pull/1#discussion_r174233854 ## File path: src/operator/nn/pooling.cc ## @@ -98,9 +103,24 @@ static bool

[GitHub] marcoabreu commented on issue #9996: add softsign operator to v1.1.0

marcoabreu commented on issue #9996: add softsign operator to v1.1.0 URL: https://github.com/apache/incubator-mxnet/pull/9996#issuecomment-372764271 Are we backporting this operator now? > This is an automated

[GitHub] cjolivier01 commented on issue #10087: Flaky test on Ubuntu: test_operator_gpu.test_batchnorm_with_type

cjolivier01 commented on issue #10087: Flaky test on Ubuntu: test_operator_gpu.test_batchnorm_with_type URL: https://github.com/apache/incubator-mxnet/issues/10087#issuecomment-372764215 Great! Thanks! This is an automated

[GitHub] reminisce commented on issue #10087: Flaky test on Ubuntu: test_operator_gpu.test_batchnorm_with_type

reminisce commented on issue #10087: Flaky test on Ubuntu: test_operator_gpu.test_batchnorm_with_type URL: https://github.com/apache/incubator-mxnet/issues/10087#issuecomment-372763627 We are going to make the unit test stable. See here for comments and action items.

[GitHub] zheng-da commented on issue #10087: Flaky test on Ubuntu: test_operator_gpu.test_batchnorm_with_type

zheng-da commented on issue #10087: Flaky test on Ubuntu: test_operator_gpu.test_batchnorm_with_type URL: https://github.com/apache/incubator-mxnet/issues/10087#issuecomment-372762672 I have observed the problem before. https://github.com/apache/incubator-mxnet/issues/9916 It's some

[GitHub] cjolivier01 commented on issue #10087: Flaky test on Ubuntu: test_operator_gpu.test_batchnorm_with_type

cjolivier01 commented on issue #10087: Flaky test on Ubuntu: test_operator_gpu.test_batchnorm_with_type URL: https://github.com/apache/incubator-mxnet/issues/10087#issuecomment-372762525 For @marcoabreu do you know if this test used to fail intermittently before the MKL changes?

[GitHub] anirudhacharya commented on issue #10087: Flaky test on Ubuntu: test_operator_gpu.test_batchnorm_with_type

anirudhacharya commented on issue #10087: Flaky test on Ubuntu: test_operator_gpu.test_batchnorm_with_type URL: https://github.com/apache/incubator-mxnet/issues/10087#issuecomment-372761638 @cjolivier01 This failure was yesterday. I am not sure which MKL changes you are referring to. If

[GitHub] jmacglashan commented on issue #9822: gluon HybridBlock wrapper of constant nd.array, is it possible?

jmacglashan commented on issue #9822: gluon HybridBlock wrapper of constant nd.array, is it possible? URL: https://github.com/apache/incubator-mxnet/issues/9822#issuecomment-372761565 @feevos I haven't tried your code yet, but I think the issue is that your `hybrid_forward` does not have

[GitHub] jmacglashan commented on issue #9822: gluon HybridBlock wrapper of constant nd.array, is it possible?

jmacglashan commented on issue #9822: gluon HybridBlock wrapper of constant nd.array, is it possible? URL: https://github.com/apache/incubator-mxnet/issues/9822#issuecomment-372761565 @feevos I haven't tried your code yet, but I think the issue is that your `hybrid_forward` does not have

[GitHub] cjolivier01 commented on issue #10078: Support float16 in L2Normalization operator

cjolivier01 commented on issue #10078: Support float16 in L2Normalization operator URL: https://github.com/apache/incubator-mxnet/pull/10078#issuecomment-372761495 Please add a JIRA ticket This is an automated message from

[GitHub] cjolivier01 closed issue #8923: GTEST tests are neither built nor run by CI

cjolivier01 closed issue #8923: GTEST tests are neither built nor run by CI URL: https://github.com/apache/incubator-mxnet/issues/8923 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] cjolivier01 commented on issue #10087: Flaky test on Ubuntu: test_operator_gpu.test_batchnorm_with_type

cjolivier01 commented on issue #10087: Flaky test on Ubuntu: test_operator_gpu.test_batchnorm_with_type URL: https://github.com/apache/incubator-mxnet/issues/10087#issuecomment-372759257 Did this every used to fail before the MKL changes?

[GitHub] gigasquid commented on issue #8971: Clojure Library for mxnet

gigasquid commented on issue #8971: Clojure Library for mxnet URL: https://github.com/apache/incubator-mxnet/issues/8971#issuecomment-366543008 An update - I've ported over the MnistModule example to clojure - yay! ?

[GitHub] anirudhacharya commented on issue #9412: Flaky Tests Tracking Issue

anirudhacharya commented on issue #9412: Flaky Tests Tracking Issue URL: https://github.com/apache/incubator-mxnet/issues/9412#issuecomment-372752027 One more issue to be tracked - https://github.com/apache/incubator-mxnet/issues/10087

[GitHub] anirudhacharya opened a new issue #10087: Flaky test on Ubuntu: test_operator_gpu.test_batchnorm_with_type

anirudhacharya opened a new issue #10087: Flaky test on Ubuntu: test_operator_gpu.test_batchnorm_with_type URL: https://github.com/apache/incubator-mxnet/issues/10087 ## Description Flaky test on ubuntu_gpu for the test - test_operator_gpu.test_batchnorm_with_type It is a precision

[GitHub] cjolivier01 commented on issue #10042: Gluon dataloader crash on speech recognition training

cjolivier01 commented on issue #10042: Gluon dataloader crash on speech recognition training URL: https://github.com/apache/incubator-mxnet/issues/10042#issuecomment-372750505 Created JIRA work item: https://issues.apache.org/jira/browse/MXNET-86

[GitHub] yzhliu commented on issue #8971: Clojure Library for mxnet

yzhliu commented on issue #8971: Clojure Library for mxnet URL: https://github.com/apache/incubator-mxnet/issues/8971#issuecomment-372748901 @gigasquid I do not have a mac with GPU, would be good if you can check why doesn't it work - not an urgent issue.

[GitHub] szha closed pull request #10083: [TENSOR] Fix DLTensor conversion for int64

szha closed pull request #10083: [TENSOR] Fix DLTensor conversion for int64 URL: https://github.com/apache/incubator-mxnet/pull/10083 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: [TENSOR] Fix DLTensor conversion for int64 (#10083)

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 a9c1717 [TENSOR] Fix DLTensor

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

marcoabreu commented on a change in pull request #9963: [MXNET-34] Onnx Module to import onnx models into mxnet URL: https://github.com/apache/incubator-mxnet/pull/9963#discussion_r174206700 ## File path: tests/python-pytest/onnx/onnx_test.py ## @@ -0,0 +1,133 @@ +#

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

marcoabreu commented on a change in pull request #9963: [MXNET-34] Onnx Module to import onnx models into mxnet URL: https://github.com/apache/incubator-mxnet/pull/9963#discussion_r174206407 ## File path: tests/python-pytest/onnx/onnx_test.py ## @@ -0,0 +1,133 @@ +#

[GitHub] haojin2 commented on a change in pull request #10078: Support float16 in L2Normalization operator

haojin2 commented on a change in pull request #10078: Support float16 in L2Normalization operator URL: https://github.com/apache/incubator-mxnet/pull/10078#discussion_r174202873 ## File path: src/operator/l2_normalization.cc ## @@ -26,13 +26,22 @@ namespace mxnet {

[GitHub] sxjscience commented on issue #10083: [TENSOR] Fix DLTensor conversion for int64

sxjscience commented on issue #10083: [TENSOR] Fix DLTensor conversion for int64 URL: https://github.com/apache/incubator-mxnet/pull/10083#issuecomment-372725480 I need to revise the gradient test of layer norm This is an

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

marcoabreu commented on a change in pull request #9963: [MXNET-34] Onnx Module to import onnx models into mxnet URL: https://github.com/apache/incubator-mxnet/pull/9963#discussion_r174191440 ## File path: tests/python-pytest/onnx/onnx_test.py ## @@ -0,0 +1,138 @@ +#

[GitHub] tqchen commented on issue #10083: [TENSOR] Fix DLTensor conversion for int64

tqchen commented on issue #10083: [TENSOR] Fix DLTensor conversion for int64 URL: https://github.com/apache/incubator-mxnet/pull/10083#issuecomment-372719428 cc @sxjscience who introduces the test, here is the old error log

[GitHub] tqchen commented on issue #10083: [TENSOR] Fix DLTensor conversion for int64

tqchen commented on issue #10083: [TENSOR] Fix DLTensor conversion for int64 URL: https://github.com/apache/incubator-mxnet/pull/10083#issuecomment-372719428 cc @sxjscience who introduces the test This is an automated message

[GitHub] tqchen commented on issue #10083: [TENSOR] Fix DLTensor conversion for int64

tqchen commented on issue #10083: [TENSOR] Fix DLTensor conversion for int64 URL: https://github.com/apache/incubator-mxnet/pull/10083#issuecomment-372718542 layer norm testcase is not related to changes introduced here. This

[GitHub] marcoabreu opened a new issue #10086: test_autograd.test_unary_func @ Python3: MKLDNN-CPU

marcoabreu opened a new issue #10086: test_autograd.test_unary_func @ Python3: MKLDNN-CPU URL: https://github.com/apache/incubator-mxnet/issues/10086 http://jenkins.mxnet-ci.amazon-ml.com/blue/organizations/jenkins/incubator-mxnet/detail/master/474/pipeline/485/ ``` [INFO]

[GitHub] kobenaxie commented on a change in pull request #10074: Add vocabulary and embedding

kobenaxie commented on a change in pull request #10074: Add vocabulary and embedding URL: https://github.com/apache/incubator-mxnet/pull/10074#discussion_r174160853 ## File path: python/mxnet/text/embedding.py ## @@ -38,8 +38,12 @@ def register(embedding_cls):

[GitHub] pengzhao-intel commented on issue #8532: mxnet-mkl (v0.12.0) crash when using (conda-installed) numpy with MKL

pengzhao-intel commented on issue #8532: mxnet-mkl (v0.12.0) crash when using (conda-installed) numpy with MKL URL: https://github.com/apache/incubator-mxnet/issues/8532#issuecomment-372671112 @fhieber we have tried the latest release the problem is gone. Would you mind to try again?

[GitHub] pengzhao-intel commented on issue #8532: mxnet-mkl (v0.12.0) crash when using (conda-installed) numpy with MKL

pengzhao-intel commented on issue #8532: mxnet-mkl (v0.12.0) crash when using (conda-installed) numpy with MKL URL: https://github.com/apache/incubator-mxnet/issues/8532#issuecomment-372671112 @fhieber we have tried the latest release the problem is gone. Would you mind to try again?

[GitHub] CoinCheung commented on issue #10000: [MXNET-80] Fix average pooling kernel size assignment error

CoinCheung commented on issue #1: [MXNET-80] Fix average pooling kernel size assignment error URL: https://github.com/apache/incubator-mxnet/pull/1#issuecomment-372624274 I tried but failed. So what should be the correct behavior when kernel.ndim() is 0? @sxjscience

[incubator-mxnet] branch master updated: Adding back comments to index.md that cause nightly test to fail (#10058)

This is an automated email from the ASF dual-hosted git repository. jxie 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 0bcf334 Adding back comments to

[GitHub] piiswrong closed pull request #10058: Adding back comments to index.md that cause nightly test to fail

piiswrong closed pull request #10058: Adding back comments to index.md that cause nightly test to fail URL: https://github.com/apache/incubator-mxnet/pull/10058 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

[GitHub] piiswrong commented on issue #10074: Add vocabulary and embedding

piiswrong commented on issue #10074: Add vocabulary and embedding URL: https://github.com/apache/incubator-mxnet/pull/10074#issuecomment-372623377 This should be in gluon. If we put vocab and embedding in mxnet.text and textdataset in gluon it's going to be really confusing

[GitHub] piiswrong commented on a change in pull request #10078: Support float16 in L2Normalization operator

piiswrong commented on a change in pull request #10078: Support float16 in L2Normalization operator URL: https://github.com/apache/incubator-mxnet/pull/10078#discussion_r174083179 ## File path: src/operator/l2_normalization.cc ## @@ -26,13 +26,22 @@ namespace mxnet {

[GitHub] marcoabreu commented on a change in pull request #9552: [REQUEST FOR REVIEW | DO NOT MERGE] Model Quantization with Calibration

marcoabreu commented on a change in pull request #9552: [REQUEST FOR REVIEW | DO NOT MERGE] Model Quantization with Calibration URL: https://github.com/apache/incubator-mxnet/pull/9552#discussion_r174076331 ## File path: tests/ci_build/Dockerfile.build_cuda8_cudnn7 ## @@

[GitHub] marcoabreu commented on a change in pull request #9552: [REQUEST FOR REVIEW | DO NOT MERGE] Model Quantization with Calibration

marcoabreu commented on a change in pull request #9552: [REQUEST FOR REVIEW | DO NOT MERGE] Model Quantization with Calibration URL: https://github.com/apache/incubator-mxnet/pull/9552#discussion_r174076331 ## File path: tests/ci_build/Dockerfile.build_cuda8_cudnn7 ## @@

[GitHub] AndreGuerra123 commented on issue #9967: Error in output.shape[[output.names]]

AndreGuerra123 commented on issue #9967: Error in output.shape[[output.names]] URL: https://github.com/apache/incubator-mxnet/issues/9967#issuecomment-372597352 The number of observations is the same. (Edited above) What do you mean as a label passed as the y argument? I have a vector of

[GitHub] jinhuang415 commented on issue #9993: cmake cannot build mxnet

jinhuang415 commented on issue #9993: cmake cannot build mxnet URL: https://github.com/apache/incubator-mxnet/issues/9993#issuecomment-372580279 @jacky4323 your issue should be the same with https://github.com/apache/incubator-mxnet/issues/10072, would you try to patch the related

[GitHub] jinhuang415 commented on issue #9993: cmake cannot build mxnet

jinhuang415 commented on issue #9993: cmake cannot build mxnet URL: https://github.com/apache/incubator-mxnet/issues/9993#issuecomment-372580279 @jacky4323 your issue should be the same with https://github.com/apache/incubator-mxnet/issues/10072, would you try to patch the related PR in

[GitHub] sxjscience commented on issue #10000: [MXNET-80] Fix average pooling kernel size assignment error

sxjscience commented on issue #1: [MXNET-80] Fix average pooling kernel size assignment error URL: https://github.com/apache/incubator-mxnet/pull/1#issuecomment-372573233 I think it has not handled the case when kernel.ndim()=0 Get Outlook for iOS

[GitHub] CoinCheung commented on issue #10000: [MXNET-80] Fix average pooling kernel size assignment error

CoinCheung commented on issue #1: [MXNET-80] Fix average pooling kernel size assignment error URL: https://github.com/apache/incubator-mxnet/pull/1#issuecomment-372572675 @sxjscience But I did not see logic error in this function. From the printed error message, I see no

[GitHub] KellenSunderland commented on issue #10075: Fix CMake build issue with MKL.

KellenSunderland commented on issue #10075: Fix CMake build issue with MKL. URL: https://github.com/apache/incubator-mxnet/pull/10075#issuecomment-372569505 @pengzhao-intel Would also be great if you could review the PR. Does it look ok from your point of view?

[GitHub] chinakook commented on issue #10085: Ordering operators do not support kNullOp

chinakook commented on issue #10085: Ordering operators do not support kNullOp URL: https://github.com/apache/incubator-mxnet/issues/10085#issuecomment-372569521 @sxjscience thanks. This is an automated message from the

[GitHub] reminisce commented on a change in pull request #9552: [REQUEST FOR REVIEW | DO NOT MERGE] Model Quantization with Calibration

reminisce commented on a change in pull request #9552: [REQUEST FOR REVIEW | DO NOT MERGE] Model Quantization with Calibration URL: https://github.com/apache/incubator-mxnet/pull/9552#discussion_r174032235 ## File path: tests/ci_build/Dockerfile.build_cuda8_cudnn7 ## @@

[GitHub] sxjscience commented on issue #10000: [MXNET-80] Fix average pooling kernel size assignment error

sxjscience commented on issue #1: [MXNET-80] Fix average pooling kernel size assignment error URL: https://github.com/apache/incubator-mxnet/pull/1#issuecomment-372566536 We may also need to revise the shape assignment logic:

[GitHub] pengzhao-intel commented on issue #10075: Fix CMake build issue with MKL.

pengzhao-intel commented on issue #10075: Fix CMake build issue with MKL. URL: https://github.com/apache/incubator-mxnet/pull/10075#issuecomment-372565487 > Not sure. At this point, I really don't know what's going on with the mkl build, what works with what, does mkldnn use mklml or mkl?

[GitHub] sxjscience commented on issue #10085: Ordering operators do not support kNullOp

sxjscience commented on issue #10085: Ordering operators do not support kNullOp URL: https://github.com/apache/incubator-mxnet/issues/10085#issuecomment-372565456 @chinakook You can get both values using the following code: ```python values, inds = mx.sym.topk(a, axis=-1, k=5,

[GitHub] chinakook commented on issue #10085: Ordering operators do not support kNullOp

chinakook commented on issue #10085: Ordering operators do not support kNullOp URL: https://github.com/apache/incubator-mxnet/issues/10085#issuecomment-372564930 Yes, It's only support indices only. Thanks for your help!

[GitHub] sxjscience commented on issue #10085: Ordering operators do not support kNullOp

sxjscience commented on issue #10085: Ordering operators do not support kNullOp URL: https://github.com/apache/incubator-mxnet/issues/10085#issuecomment-372563935 @chinakook I'll fix it tomorrow. This is an automated message

[GitHub] sxjscience commented on issue #10085: Ordering operators do not support kNullOp

sxjscience commented on issue #10085: Ordering operators do not support kNullOp URL: https://github.com/apache/incubator-mxnet/issues/10085#issuecomment-372563598 The error also occurs in other ordering operators like Sort, ArgSort.

[GitHub] pengzhao-intel commented on issue #8974: MXNET compatibility with MKL libraries bundled in Microsoft R Open

pengzhao-intel commented on issue #8974: MXNET compatibility with MKL libraries bundled in Microsoft R Open URL: https://github.com/apache/incubator-mxnet/issues/8974#issuecomment-372562963 @mjmg The MKLML has been replaced by MKL-DNN. Could you try again?

<    1   2   3   >