[GitHub] wkcn commented on issue #13058: AttributeError: 'str' object has no attribute 'rank'

2018-10-31 Thread GitBox
wkcn commented on issue #13058: AttributeError: 'str' object has no attribute 'rank' URL: https://github.com/apache/incubator-mxnet/issues/13058#issuecomment-434647448 It seems the SSD example doesn't support the old version of MXNet. Please upgrade MXNet. The latest version is

[GitHub] MoritzMaxeiner opened a new issue #13061: Incorrect ONNX export of SliceChannel

2018-10-31 Thread GitBox
MoritzMaxeiner opened a new issue #13061: Incorrect ONNX export of SliceChannel URL: https://github.com/apache/incubator-mxnet/issues/13061 ## Description The [operator

[GitHub] frankfliu commented on issue #13051: Tutorial: Incorrect variable name in tutorial code snippet

2018-10-31 Thread GitBox
frankfliu commented on issue #13051: Tutorial: Incorrect variable name in tutorial code snippet URL: https://github.com/apache/incubator-mxnet/issues/13051#issuecomment-434740541 @mxnet-label-bot [Doc] This is an automated

[GitHub] frankfliu commented on issue #13058: AttributeError: 'str' object has no attribute 'rank'

2018-10-31 Thread GitBox
frankfliu commented on issue #13058: AttributeError: 'str' object has no attribute 'rank' URL: https://github.com/apache/incubator-mxnet/issues/13058#issuecomment-434743647 @mxnet-label-bot [Python, Example] This is an

[incubator-mxnet-site] branch asf-site updated: Bump the publish timestamp.

2018-10-31 Thread zhasheng
This is an automated email from the ASF dual-hosted git repository. zhasheng pushed a commit to branch asf-site in repository https://gitbox.apache.org/repos/asf/incubator-mxnet-site.git The following commit(s) were added to refs/heads/asf-site by this push: new 5f836b9 Bump the publish

[GitHub] Chancebair opened a new pull request #13060: Disable flaky test test_prelu

2018-10-31 Thread GitBox
Chancebair opened a new pull request #13060: Disable flaky test test_prelu URL: https://github.com/apache/incubator-mxnet/pull/13060 https://github.com/apache/incubator-mxnet/issues/12885 This is an automated message from the

[GitHub] frankfliu commented on issue #13054: Multiple trainers within a single worker using a distributed KVStore

2018-10-31 Thread GitBox
frankfliu commented on issue #13054: Multiple trainers within a single worker using a distributed KVStore URL: https://github.com/apache/incubator-mxnet/issues/13054#issuecomment-434742603 @mxnet-label-bot [KVStore, Thread Safety]

[incubator-mxnet] branch master updated: Disable flaky test test_prelu (#13060)

2018-10-31 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 8209712 Disable flaky test

[GitHub] marcoabreu commented on issue #13050: Disabled flaky test test_operator.test_prelu

2018-10-31 Thread GitBox
marcoabreu commented on issue #13050: Disabled flaky test test_operator.test_prelu URL: https://github.com/apache/incubator-mxnet/pull/13050#issuecomment-434730074 Duplicate of #13060 This is an automated message from the

[GitHub] marcoabreu closed pull request #13050: Disabled flaky test test_operator.test_prelu

2018-10-31 Thread GitBox
marcoabreu closed pull request #13050: Disabled flaky test test_operator.test_prelu URL: https://github.com/apache/incubator-mxnet/pull/13050 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

[GitHub] marcoabreu closed pull request #13060: Disable flaky test test_prelu

2018-10-31 Thread GitBox
marcoabreu closed pull request #13060: Disable flaky test test_prelu URL: https://github.com/apache/incubator-mxnet/pull/13060 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] ZhennanQin opened a new pull request #13059: Refactor L2_normalization

2018-10-31 Thread GitBox
ZhennanQin opened a new pull request #13059: Refactor L2_normalization URL: https://github.com/apache/incubator-mxnet/pull/13059 ## Description ## Refactor L2_normalization for speed up. Before this change, on skx-8180, L2_normalization will take 8372 ms in ssd-vgg16 for bs=1,

[GitHub] MoritzMaxeiner commented on issue #11594: [FeatureRequest]MXNet Split operator to support variable length splits

2018-10-31 Thread GitBox
MoritzMaxeiner commented on issue #11594: [FeatureRequest]MXNet Split operator to support variable length splits URL: https://github.com/apache/incubator-mxnet/issues/11594#issuecomment-434681043 @nswamy Why was this feature request closed? The requested feature is not available in 1.2

[GitHub] juliusshufan commented on issue #12893: Dummy-data based benchmarking script for Gluon RNN-API

2018-10-31 Thread GitBox
juliusshufan commented on issue #12893: Dummy-data based benchmarking script for Gluon RNN-API URL: https://github.com/apache/incubator-mxnet/pull/12893#issuecomment-434713947 @sandeep-krishnamurthy Thanks for your time on code review, may I know if you get a chance to review my further

[GitHub] frankfliu commented on issue #13061: Incorrect ONNX export of SliceChannel

2018-10-31 Thread GitBox
frankfliu commented on issue #13061: Incorrect ONNX export of SliceChannel URL: https://github.com/apache/incubator-mxnet/issues/13061#issuecomment-434744206 @mxnet-label-bot [Bug, ONNX] This is an automated message from the

[GitHub] ChaiBapchya commented on issue #13050: Disabled flaky test test_operator.test_prelu

2018-10-31 Thread GitBox
ChaiBapchya commented on issue #13050: Disabled flaky test test_operator.test_prelu URL: https://github.com/apache/incubator-mxnet/pull/13050#issuecomment-434747499 @marcoabreu I had opened my PR 15 hours ago with the same change as #13060 can I know why mine was considered duplicate and

[GitHub] frankfliu commented on issue #13053: Something wrong of form .rec of my own data

2018-10-31 Thread GitBox
frankfliu commented on issue #13053: Something wrong of form .rec of my own data URL: https://github.com/apache/incubator-mxnet/issues/13053#issuecomment-434741396 @mxnet-label-bot [Question] This is an automated message

[GitHub] gigasquid commented on issue #12881: Improve the Clojure Package README to Make it Easier to Get Started

2018-10-31 Thread GitBox
gigasquid commented on issue #12881: Improve the Clojure Package README to Make it Easier to Get Started URL: https://github.com/apache/incubator-mxnet/pull/12881#issuecomment-434728486 Thanks again @ChaiBapchya - I'll take a look to make sure your feedback is addressed with the latest

[GitHub] apeforest commented on issue #13036: fix operators to support large arrays.

2018-10-31 Thread GitBox
apeforest commented on issue #13036: fix operators to support large arrays. URL: https://github.com/apache/incubator-mxnet/issues/13036#issuecomment-434737244 Will take a look This is an automated message from the Apache Git

[GitHub] gigasquid commented on issue #12881: Improve the Clojure Package README to Make it Easier to Get Started

2018-10-31 Thread GitBox
gigasquid commented on issue #12881: Improve the Clojure Package README to Make it Easier to Get Started URL: https://github.com/apache/incubator-mxnet/pull/12881#issuecomment-434727824 Thanks @daveliepmann and @kohr-h for all your work  Thanks @aaronmarkham for your feedback.

[GitHub] apeforest opened a new pull request #13062: [WIP] Add header files required by horovod

2018-10-31 Thread GitBox
apeforest opened a new pull request #13062: [WIP] Add header files required by horovod URL: https://github.com/apache/incubator-mxnet/pull/13062 ## Description ## Add header files required by MXNet Horovod integration. ## Comments ## - Still need to decide where is the best

[GitHub] anirudh2290 commented on issue #13050: Disabled flaky test test_operator.test_prelu

2018-10-31 Thread GitBox
anirudh2290 commented on issue #13050: Disabled flaky test test_operator.test_prelu URL: https://github.com/apache/incubator-mxnet/pull/13050#issuecomment-434757101 Its a fair question. It could be an honest mistake and @marcoabreu probably missed this PR.

[GitHub] lanking520 commented on issue #12690: [MXNET-1000] get Ndarray real value and form it from a NDArray

2018-10-31 Thread GitBox
lanking520 commented on issue #12690: [MXNET-1000] get Ndarray real value and form it from a NDArray URL: https://github.com/apache/incubator-mxnet/pull/12690#issuecomment-434764135 @gigasquid Could you please provide some opinions? Other two Scala committers from the

[GitHub] ankkhedia commented on issue #13052: Fix incorrect variable name in tutorial code

2018-10-31 Thread GitBox
ankkhedia commented on issue #13052: Fix incorrect variable name in tutorial code URL: https://github.com/apache/incubator-mxnet/pull/13052#issuecomment-434776785 @wongjoel Thanks for your contribution! Request you to look into the CI failures before code can be reviewed :)

[GitHub] sandeep-krishnamurthy commented on issue #12964: Update multi-task learning example

2018-10-31 Thread GitBox
sandeep-krishnamurthy commented on issue #12964: Update multi-task learning example URL: https://github.com/apache/incubator-mxnet/pull/12964#issuecomment-434793131 @aaronmarkham - FYI. New example. Can you take a look at this?

[GitHub] ankkhedia commented on issue #13062: [WIP] Add header files required by horovod

2018-10-31 Thread GitBox
ankkhedia commented on issue #13062: [WIP] Add header files required by horovod URL: https://github.com/apache/incubator-mxnet/pull/13062#issuecomment-434796700 @apeforest Thanks for your contribution! @mxnet-label-bot [pr-work-in-progress]

[GitHub] ChaiBapchya commented on issue #10323: test_gluon_model_zoo_gpu.test_training out of memory

2018-10-31 Thread GitBox
ChaiBapchya commented on issue #10323: test_gluon_model_zoo_gpu.test_training out of memory URL: https://github.com/apache/incubator-mxnet/issues/10323#issuecomment-434796900 @marcoabreu could you reopen this since it's still flaky?

[GitHub] vandanavk commented on issue #12924: Fix ONNX export of keepdims param

2018-10-31 Thread GitBox
vandanavk commented on issue #12924: Fix ONNX export of keepdims param URL: https://github.com/apache/incubator-mxnet/pull/12924#issuecomment-434800201 @zhreshold for review/merge This is an automated message from the Apache

[GitHub] kohr-h commented on issue #12881: Improve the Clojure Package README to Make it Easier to Get Started

2018-10-31 Thread GitBox
kohr-h commented on issue #12881: Improve the Clojure Package README to Make it Easier to Get Started URL: https://github.com/apache/incubator-mxnet/pull/12881#issuecomment-434800554 > Thanks @daveliepmann and @kohr-h for all your work smile_cat My pleasure! I'll have a go at

[GitHub] ankkhedia commented on issue #12974: Improve clojure tutorial

2018-10-31 Thread GitBox
ankkhedia commented on issue #12974: Improve clojure tutorial URL: https://github.com/apache/incubator-mxnet/pull/12974#issuecomment-434802038 @gigasquid The Travis CI build failures doesnot hold back PRs from getting merged if approved. They are just for informational purpose as of now.

[GitHub] gigasquid commented on issue #12974: Improve clojure tutorial

2018-10-31 Thread GitBox
gigasquid commented on issue #12974: Improve clojure tutorial URL: https://github.com/apache/incubator-mxnet/pull/12974#issuecomment-434804646 Thanks again @daveliepmann  This is an automated message from the Apache Git

[GitHub] gigasquid closed pull request #12974: Improve clojure tutorial

2018-10-31 Thread GitBox
gigasquid closed pull request #12974: Improve clojure tutorial URL: https://github.com/apache/incubator-mxnet/pull/12974 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

[incubator-mxnet] branch master updated: Improve clojure tutorial (#12974)

2018-10-31 Thread cmeier
This is an automated email from the ASF dual-hosted git repository. cmeier 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 0844d4b Improve clojure tutorial

[GitHub] gigasquid commented on issue #12974: Improve clojure tutorial

2018-10-31 Thread GitBox
gigasquid commented on issue #12974: Improve clojure tutorial URL: https://github.com/apache/incubator-mxnet/pull/12974#issuecomment-434804448 @ankkhedia - Thanks for the clarification. I'm so conditioned to seeing red as a stop :)

[GitHub] anirudhacharya commented on issue #13061: Incorrect ONNX export of SliceChannel

2018-10-31 Thread GitBox
anirudhacharya commented on issue #13061: Incorrect ONNX export of SliceChannel URL: https://github.com/apache/incubator-mxnet/issues/13061#issuecomment-434766156 @vandanavk This is an automated message from the Apache Git

[GitHub] apeforest commented on issue #13062: [WIP] Add header files required by horovod

2018-10-31 Thread GitBox
apeforest commented on issue #13062: [WIP] Add header files required by horovod URL: https://github.com/apache/incubator-mxnet/pull/13062#issuecomment-434777582 @szha Your review and advice are very much appreciated. This is

[GitHub] KellenSunderland commented on issue #12469: [MXNET-703] Use static libs for TensorRT

2018-10-31 Thread GitBox
KellenSunderland commented on issue #12469: [MXNET-703] Use static libs for TensorRT URL: https://github.com/apache/incubator-mxnet/pull/12469#issuecomment-434784436 @ankkhedia Looks like the build errors relate to the old findalgo function. My best guess is that this PR (and others

[GitHub] sandeep-krishnamurthy commented on issue #12356: [MXNET-860] Use modernized ranged loops where possible

2018-10-31 Thread GitBox
sandeep-krishnamurthy commented on issue #12356: [MXNET-860] Use modernized ranged loops where possible URL: https://github.com/apache/incubator-mxnet/pull/12356#issuecomment-434791236 @anirudh2290 - Can you please take a look at this and merge if it looks good?

[GitHub] vandanavk commented on issue #12878: ONNX export: Cleanup

2018-10-31 Thread GitBox
vandanavk commented on issue #12878: ONNX export: Cleanup URL: https://github.com/apache/incubator-mxnet/pull/12878#issuecomment-434794364 @zhreshold @Roshrini @anirudhacharya addressed all review comments so far. this PR is ready for review again

[GitHub] sandeep-krishnamurthy commented on issue #11481: # updated the link as the previous one was corrupted

2018-10-31 Thread GitBox
sandeep-krishnamurthy commented on issue #11481: # updated the link as the previous one was corrupted URL: https://github.com/apache/incubator-mxnet/pull/11481#issuecomment-434794390 @ankkhedia - I suggest to take over the changes in this PR, as it looks like stale, but with imp change,

[incubator-mxnet] branch master updated (8209712 -> 49e6a7e)

2018-10-31 Thread anirudh2290
This is an automated email from the ASF dual-hosted git repository. anirudh2290 pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. from 8209712 Disable flaky test test_prelu (#13060) add 49e6a7e la_op_inline.h to la_op-inl.h for

[GitHub] sandeep-krishnamurthy commented on issue #11945: python/setup.py modified to accept a package name argument.

2018-10-31 Thread GitBox
sandeep-krishnamurthy commented on issue #11945: python/setup.py modified to accept a package name argument. URL: https://github.com/apache/incubator-mxnet/pull/11945#issuecomment-434796103 @aaronmarkham - Are your comments addressed? @szha - Will this have any impact on PyPi?

[GitHub] szha opened a new issue #13064: cuDNN in CI needs upgrade

2018-10-31 Thread GitBox
szha opened a new issue #13064: cuDNN in CI needs upgrade URL: https://github.com/apache/incubator-mxnet/issues/13064 Current CI is running with [cudnn older than

[GitHub] apeforest edited a comment on issue #13048: Updated CONTRIBUTERS.md to include label-bot

2018-10-31 Thread GitBox
apeforest edited a comment on issue #13048: Updated CONTRIBUTERS.md to include label-bot URL: https://github.com/apache/incubator-mxnet/pull/13048#issuecomment-434758935 Also, I would prefer that we make sure label bot can append labels before exposing this super agent to the public.

[GitHub] jacky4323 commented on issue #12532: caffeOp mxnet build failed

2018-10-31 Thread GitBox
jacky4323 commented on issue #12532: caffeOp mxnet build failed URL: https://github.com/apache/incubator-mxnet/issues/12532#issuecomment-434768777 @safrooze Hi, now I use an old mxnet package build with caffe, it works now!! However, In my caffe operator which is only implemented

[GitHub] KellenSunderland commented on a change in pull request #12953: Update MKL-DNN dependency

2018-10-31 Thread GitBox
KellenSunderland commented on a change in pull request #12953: Update MKL-DNN dependency URL: https://github.com/apache/incubator-mxnet/pull/12953#discussion_r229801527 ## File path: tests/cpp/operator/mkldnn_test.cc ## @@ -100,9 +100,9 @@ static void VerifyDefMem(const

[GitHub] sandeep-krishnamurthy commented on issue #12999: [Issue #11912] throw mxnet exceptions when decoding invalid images.

2018-10-31 Thread GitBox
sandeep-krishnamurthy commented on issue #12999: [Issue #11912] throw mxnet exceptions when decoding invalid images. URL: https://github.com/apache/incubator-mxnet/pull/12999#issuecomment-434795057 @anirudh2290 This is an

[GitHub] ankkhedia commented on issue #13059: Refactor L2_normalization

2018-10-31 Thread GitBox
ankkhedia commented on issue #13059: Refactor L2_normalization URL: https://github.com/apache/incubator-mxnet/pull/13059#issuecomment-434797934 @ZhennanQin Thanks for the contrbution! Request you to look into CI failures before the PR can be reviewed. @mxnet-label-bot

[GitHub] aaronmarkham commented on issue #11945: python/setup.py modified to accept a package name argument.

2018-10-31 Thread GitBox
aaronmarkham commented on issue #11945: python/setup.py modified to accept a package name argument. URL: https://github.com/apache/incubator-mxnet/pull/11945#issuecomment-434803882 Shouldn't the info in this PR's description be added to

[incubator-mxnet-site] branch asf-site updated: Bump the publish timestamp.

2018-10-31 Thread zhasheng
This is an automated email from the ASF dual-hosted git repository. zhasheng pushed a commit to branch asf-site in repository https://gitbox.apache.org/repos/asf/incubator-mxnet-site.git The following commit(s) were added to refs/heads/asf-site by this push: new 599c090 Bump the publish

[GitHub] lebeg commented on issue #13064: cuDNN in CI needs upgrade

2018-10-31 Thread GitBox
lebeg commented on issue #13064: cuDNN in CI needs upgrade URL: https://github.com/apache/incubator-mxnet/issues/13064#issuecomment-434788932 Thanks @szha for the issue, I think this should be resolved with https://github.com/apache/incubator-mxnet/pull/12850 once it would be ready to be

[GitHub] sandeep-krishnamurthy closed issue #12870: What is the license for Saving and Loading Gluon Models?

2018-10-31 Thread GitBox
sandeep-krishnamurthy closed issue #12870: What is the license for Saving and Loading Gluon Models? URL: https://github.com/apache/incubator-mxnet/issues/12870 This is an automated message from the Apache Git Service. To

[GitHub] sandeep-krishnamurthy closed issue #12310: Flaky test: test_ndarray.test_order

2018-10-31 Thread GitBox
sandeep-krishnamurthy closed issue #12310: Flaky test: test_ndarray.test_order URL: https://github.com/apache/incubator-mxnet/issues/12310 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] aaronmarkham commented on issue #13007: Image API docs error

2018-10-31 Thread GitBox
aaronmarkham commented on issue #13007: Image API docs error URL: https://github.com/apache/incubator-mxnet/issues/13007#issuecomment-434796553 @anirudhacharya What are the right destinations? They seem correct to me. This

[GitHub] leleamol commented on issue #12576: Can anybody provide a kv-store (distributed training) example with cpp package?

2018-10-31 Thread GitBox
leleamol commented on issue #12576: Can anybody provide a kv-store (distributed training) example with cpp package? URL: https://github.com/apache/incubator-mxnet/issues/12576#issuecomment-434803402 @xiaolin-cheng Thanks for providing the information regarding your use case. The

[GitHub] larroy opened a new pull request #13065: [MXNET-793] Virtual testing with Qemu, refinement and extract test results to root MXNet folder

2018-10-31 Thread GitBox
larroy opened a new pull request #13065: [MXNET-793] Virtual testing with Qemu, refinement and extract test results to root MXNet folder URL: https://github.com/apache/incubator-mxnet/pull/13065 ## Description ## This PR refines the scripts to execute tests under QEMU and

[GitHub] apeforest commented on issue #13048: Updated CONTRIBUTERS.md to include label-bot

2018-10-31 Thread GitBox
apeforest commented on issue #13048: Updated CONTRIBUTERS.md to include label-bot URL: https://github.com/apache/incubator-mxnet/pull/13048#issuecomment-434758935 Also, I would prefer to make sure label bot can append labels before exposing this super agent to the public.

[GitHub] ankkhedia commented on issue #13056: Gluon LSTM Projection

2018-10-31 Thread GitBox
ankkhedia commented on issue #13056: Gluon LSTM Projection URL: https://github.com/apache/incubator-mxnet/pull/13056#issuecomment-434777672 @szha Thanks for your contribution! Requesting you to look into CI failures before the code be reviewed. @mxnet-label-bot

[GitHub] rahul003 commented on issue #13062: [WIP] Add header files required by horovod

2018-10-31 Thread GitBox
rahul003 commented on issue #13062: [WIP] Add header files required by horovod URL: https://github.com/apache/incubator-mxnet/pull/13062#issuecomment-434795440 Do we need the headers for the submodules too? This is an

[GitHub] rahul003 edited a comment on issue #13062: [WIP] Add header files required by horovod

2018-10-31 Thread GitBox
rahul003 edited a comment on issue #13062: [WIP] Add header files required by horovod URL: https://github.com/apache/incubator-mxnet/pull/13062#issuecomment-434795440 Do we need the headers for the submodules too? That would also become really hard to maintain.

[GitHub] wkcn commented on issue #13058: AttributeError: 'str' object has no attribute 'rank'

2018-10-31 Thread GitBox
wkcn commented on issue #13058: AttributeError: 'str' object has no attribute 'rank' URL: https://github.com/apache/incubator-mxnet/issues/13058#issuecomment-434756505 @TonyTangYu I recommend to use the SSD example in gluon-cv github.com/dmlc/gluon-cv, which could reach the baseline.

[GitHub] terrytangyuan commented on issue #12690: [MXNET-1000] get Ndarray real value and form it from a NDArray

2018-10-31 Thread GitBox
terrytangyuan commented on issue #12690: [MXNET-1000] get Ndarray real value and form it from a NDArray URL: https://github.com/apache/incubator-mxnet/pull/12690#issuecomment-434799956 I agree that we should not print the entire data. Rather we should let users to specify how much to

[GitHub] ankkhedia commented on issue #13041: Updated recommenders example

2018-10-31 Thread GitBox
ankkhedia commented on issue #13041: Updated recommenders example URL: https://github.com/apache/incubator-mxnet/pull/13041#issuecomment-434800138 @aaronmarkham @thomelane @safrooze Could you please take a look ? This is an

[GitHub] ankkhedia commented on issue #11481: # updated the link as the previous one was corrupted

2018-10-31 Thread GitBox
ankkhedia commented on issue #11481: # updated the link as the previous one was corrupted URL: https://github.com/apache/incubator-mxnet/pull/11481#issuecomment-434802675 @sandeep-krishnamurthy Sure thing! I will take a look.

[GitHub] TonyTangYu commented on issue #13058: AttributeError: 'str' object has no attribute 'rank'

2018-10-31 Thread GitBox
TonyTangYu commented on issue #13058: AttributeError: 'str' object has no attribute 'rank' URL: https://github.com/apache/incubator-mxnet/issues/13058#issuecomment-434751158 @wkcn Thank you for your response. I upgraded the version of MXNet. It works now. Thank you!

[GitHub] Wei-1 opened a new issue #13063: Scala-Package CI Missing

2018-10-31 Thread GitBox
Wei-1 opened a new issue #13063: Scala-Package CI Missing URL: https://github.com/apache/incubator-mxnet/issues/13063 ## Description On the Scala package page: https://github.com/apache/incubator-mxnet/tree/master/scala-package The badge for Travis-CI is not working correctly.

[GitHub] KellenSunderland commented on issue #12953: Update MKL-DNN dependency

2018-10-31 Thread GitBox
KellenSunderland commented on issue #12953: Update MKL-DNN dependency URL: https://github.com/apache/incubator-mxnet/pull/12953#issuecomment-434783078 @TaoLv Thanks for the info. No worries from my point of view if the PR still has some work to do. I think this should be up to whatever

[GitHub] frankfliu commented on issue #13063: Scala-Package CI Missing

2018-10-31 Thread GitBox
frankfliu commented on issue #13063: Scala-Package CI Missing URL: https://github.com/apache/incubator-mxnet/issues/13063#issuecomment-434786601 @mxnet-label-bot [Scala, Doc] This is an automated message from the Apache Git

[GitHub] sandeep-krishnamurthy commented on issue #12893: Dummy-data based benchmarking script for Gluon RNN-API

2018-10-31 Thread GitBox
sandeep-krishnamurthy commented on issue #12893: Dummy-data based benchmarking script for Gluon RNN-API URL: https://github.com/apache/incubator-mxnet/pull/12893#issuecomment-434790387 @juliusshufan - Thanks for addressing the comments. @szha - I am not sure if this will be useful

[GitHub] sandeep-krishnamurthy commented on issue #12904: support upper triangular matrices in linalg

2018-10-31 Thread GitBox
sandeep-krishnamurthy commented on issue #12904: support upper triangular matrices in linalg URL: https://github.com/apache/incubator-mxnet/pull/12904#issuecomment-434793438 @samskalicky - FYI. Can you please take a look?

[GitHub] anirudh2290 closed pull request #13045: la_op_inline.h to la_op-inl.h for consistency

2018-10-31 Thread GitBox
anirudh2290 closed pull request #13045: la_op_inline.h to la_op-inl.h for consistency URL: https://github.com/apache/incubator-mxnet/pull/13045 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

[GitHub] sandeep-krishnamurthy closed issue #11639: some big numbers can not be assigned correctly with ndarray, it is a big bug???

2018-10-31 Thread GitBox
sandeep-krishnamurthy closed issue #11639: some big numbers can not be assigned correctly with ndarray, it is a big bug??? URL: https://github.com/apache/incubator-mxnet/issues/11639 This is an automated message from the

[GitHub] aaronmarkham commented on issue #12964: Update multi-task learning example

2018-10-31 Thread GitBox
aaronmarkham commented on issue #12964: Update multi-task learning example URL: https://github.com/apache/incubator-mxnet/pull/12964#issuecomment-434800780 Curious why this can't be in /tutorials and have the main notebook source (of truth) be in markdown?

[GitHub] kohr-h commented on issue #12881: Improve the Clojure Package README to Make it Easier to Get Started

2018-10-31 Thread GitBox
kohr-h commented on issue #12881: Improve the Clojure Package README to Make it Easier to Get Started URL: https://github.com/apache/incubator-mxnet/pull/12881#issuecomment-434800935 BTW, the test fail is a timeout by Travis (cap at about 50 minutes per build as it seems).

[GitHub] ankkhedia edited a comment on issue #12881: Improve the Clojure Package README to Make it Easier to Get Started

2018-10-31 Thread GitBox
ankkhedia edited a comment on issue #12881: Improve the Clojure Package README to Make it Easier to Get Started URL: https://github.com/apache/incubator-mxnet/pull/12881#issuecomment-434819442 @kohr-h Travis build failure is not a blocker for PR merge as of now. It is for informational

[GitHub] ankkhedia commented on issue #12881: Improve the Clojure Package README to Make it Easier to Get Started

2018-10-31 Thread GitBox
ankkhedia commented on issue #12881: Improve the Clojure Package README to Make it Easier to Get Started URL: https://github.com/apache/incubator-mxnet/pull/12881#issuecomment-434819442 Hi @kohr-h Travis build failure is not a blocker for PR merge as of now. It is for informational

[GitHub] samskalicky commented on a change in pull request #12904: support upper triangular matrices in linalg

2018-10-31 Thread GitBox
samskalicky commented on a change in pull request #12904: support upper triangular matrices in linalg URL: https://github.com/apache/incubator-mxnet/pull/12904#discussion_r229843925 ## File path: src/operator/tensor/la_op.h ## @@ -81,10 +81,21 @@ struct LaMatrixMultParam

[GitHub] samskalicky commented on a change in pull request #12904: support upper triangular matrices in linalg

2018-10-31 Thread GitBox
samskalicky commented on a change in pull request #12904: support upper triangular matrices in linalg URL: https://github.com/apache/incubator-mxnet/pull/12904#discussion_r229842788 ## File path: src/operator/tensor/la_op.cc ## @@ -227,10 +231,11 @@

[GitHub] MyraBaba commented on issue #12869: Python trained model gives different features set in C++

2018-10-31 Thread GitBox
MyraBaba commented on issue #12869: Python trained model gives different features set in C++ URL: https://github.com/apache/incubator-mxnet/issues/12869#issuecomment-434823928 Hi @leleamol I am trying to integrate and testing your new inference. do you know the

[GitHub] ankkhedia commented on issue #11778: [MXNET-483] C++ tests for mkldnn convolution/deconvolution operator

2018-10-31 Thread GitBox
ankkhedia commented on issue #11778: [MXNET-483] C++ tests for mkldnn convolution/deconvolution operator URL: https://github.com/apache/incubator-mxnet/pull/11778#issuecomment-434825701 @mseth10 @zheng-da ping! Could you please take a look?

[GitHub] larroy commented on a change in pull request #13046: [MXNET-1155] Add scala packageTest utility

2018-10-31 Thread GitBox
larroy commented on a change in pull request #13046: [MXNET-1155] Add scala packageTest utility URL: https://github.com/apache/incubator-mxnet/pull/13046#discussion_r229855913 ## File path: Makefile ## @@ -602,6 +602,14 @@ scalaclean: (cd $(ROOTDIR)/scala-package;

[GitHub] vandanavk opened a new pull request #13067: ONNX export: Set np.array type to float32

2018-10-31 Thread GitBox
vandanavk opened a new pull request #13067: ONNX export: Set np.array type to float32 URL: https://github.com/apache/incubator-mxnet/pull/13067 ## Description ## np.array calls set default dtype to float64 which is not supported by ONNX. Setting these to float32. Fixes

[GitHub] ThomasDelteil commented on issue #12933: Update autoencoder example

2018-10-31 Thread GitBox
ThomasDelteil commented on issue #12933: Update autoencoder example URL: https://github.com/apache/incubator-mxnet/pull/12933#issuecomment-434857453 @ankkhedia @thomelane updated This is an automated message from the Apache

[GitHub] ThomasDelteil commented on issue #12961: Update module example

2018-10-31 Thread GitBox
ThomasDelteil commented on issue #12961: Update module example URL: https://github.com/apache/incubator-mxnet/pull/12961#issuecomment-434858420 @sandeep-krishnamurthy is it good to merge? This is an automated message from the

[GitHub] leleamol edited a comment on issue #12869: Python trained model gives different features set in C++

2018-10-31 Thread GitBox
leleamol edited a comment on issue #12869: Python trained model gives different features set in C++ URL: https://github.com/apache/incubator-mxnet/issues/12869#issuecomment-434870012 @MyraBaba According to the

[GitHub] leleamol commented on issue #12869: Python trained model gives different features set in C++

2018-10-31 Thread GitBox
leleamol commented on issue #12869: Python trained model gives different features set in C++ URL: https://github.com/apache/incubator-mxnet/issues/12869#issuecomment-434870012 @MyraBaba According to the

[GitHub] aaronmarkham commented on issue #12781: Fixed issue #12745

2018-10-31 Thread GitBox
aaronmarkham commented on issue #12781: Fixed issue #12745 URL: https://github.com/apache/incubator-mxnet/pull/12781#issuecomment-434817807 Preview: http://mxnet-ci-doc.s3-accelerate.dualstack.amazonaws.com/PR-12781/6/index.html Unfortunately, what I'm seeing on the install page, is

[GitHub] samskalicky commented on a change in pull request #12904: support upper triangular matrices in linalg

2018-10-31 Thread GitBox
samskalicky commented on a change in pull request #12904: support upper triangular matrices in linalg URL: https://github.com/apache/incubator-mxnet/pull/12904#discussion_r229844162 ## File path: src/operator/tensor/la_op.h ## @@ -93,6 +104,9 @@ struct

[GitHub] apeforest commented on issue #13062: [WIP] Add header files required by horovod

2018-10-31 Thread GitBox
apeforest commented on issue #13062: [WIP] Add header files required by horovod URL: https://github.com/apache/incubator-mxnet/pull/13062#issuecomment-434824580 @rahul003 Yes, because mxnet/engine.h --> mxnet/base.h --> All these submodules.

[GitHub] harshp8l commented on issue #13048: Updated CONTRIBUTERS.md to include label-bot

2018-10-31 Thread GitBox
harshp8l commented on issue #13048: Updated CONTRIBUTERS.md to include label-bot URL: https://github.com/apache/incubator-mxnet/pull/13048#issuecomment-434843088 The mxnet-label-bot is committing so that it has a commit under its name on the repo and when it is being referenced (i.e.

[GitHub] ThomasDelteil opened a new pull request #13068: Updates to several examples

2018-10-31 Thread GitBox
ThomasDelteil opened a new pull request #13068: Updates to several examples URL: https://github.com/apache/incubator-mxnet/pull/13068 ## Description ## Minor upgrade to several examples: - reinforcement-learning/ddpg - [ ] deprecated because of dependency on RLLAB

[GitHub] ankkhedia commented on issue #13067: ONNX export: Set np.array type to float32

2018-10-31 Thread GitBox
ankkhedia commented on issue #13067: ONNX export: Set np.array type to float32 URL: https://github.com/apache/incubator-mxnet/pull/13067#issuecomment-434849664 @vandanavk Thanks for the contribution! need this to fix models in ONNX Model Zoo. @mxnet-label-bot [pr-awaiting-review,

[GitHub] yajiedesign opened a new pull request #13069: add Volta and Turing arch

2018-10-31 Thread GitBox
yajiedesign opened a new pull request #13069: add Volta and Turing arch URL: https://github.com/apache/incubator-mxnet/pull/13069 ## Description ## add Volta and Turing arch Volta sm70 Turing sm75 ## Checklist ## ### Essentials ### Please feel free to remove

[GitHub] HyperZealot edited a comment on issue #12997: A better take forward kernel for CPU

2018-10-31 Thread GitBox
HyperZealot edited a comment on issue #12997: A better take forward kernel for CPU URL: https://github.com/apache/incubator-mxnet/pull/12997#issuecomment-434862551 I don't think the workloads provided by @rongzha1 is suitable for determining the memory bandwidth, based on the following

[GitHub] HyperZealot commented on issue #12997: A better take forward kernel for CPU

2018-10-31 Thread GitBox
HyperZealot commented on issue #12997: A better take forward kernel for CPU URL: https://github.com/apache/incubator-mxnet/pull/12997#issuecomment-434862551 I don't think the workloads provided by @rongzha1 is suitable for determining the memory bandwidth, based on the following arguments:

[GitHub] ankkhedia edited a comment on issue #13069: add Volta and Turing arch

2018-10-31 Thread GitBox
ankkhedia edited a comment on issue #13069: add Volta and Turing arch URL: https://github.com/apache/incubator-mxnet/pull/13069#issuecomment-434870156 @yajiedesign Thanks for your contribution! Request you to provide more description on the PR so that it is easier for people to

[GitHub] ankkhedia commented on issue #13069: add Volta and Turing arch

2018-10-31 Thread GitBox
ankkhedia commented on issue #13069: add Volta and Turing arch URL: https://github.com/apache/incubator-mxnet/pull/13069#issuecomment-434870156 @yajiedesign Thanks for your contribution! Request you to provide more description on the PR so that it is easier for people to undertsand the

[GitHub] ThomasDelteil commented on issue #13041: Updated recommenders example

2018-10-31 Thread GitBox
ThomasDelteil commented on issue #13041: Updated recommenders example URL: https://github.com/apache/incubator-mxnet/pull/13041#issuecomment-434871980 @Ishitori thanks, looks like my last minute changes broke everything :) Doing one last full run and will push the changes

[GitHub] Ishitori commented on issue #13068: Updates to several examples

2018-10-31 Thread GitBox
Ishitori commented on issue #13068: Updates to several examples URL: https://github.com/apache/incubator-mxnet/pull/13068#issuecomment-434875740 Looks good to me This is an automated message from the Apache Git Service. To

[GitHub] ThomasDelteil commented on issue #12542: [MXNET-949] Module API to Gluon API tutorial

2018-10-31 Thread GitBox
ThomasDelteil commented on issue #12542: [MXNET-949] Module API to Gluon API tutorial URL: https://github.com/apache/incubator-mxnet/pull/12542#issuecomment-434860204 @roywei going to go through the comments this afternoon and push an update

  1   2   >