[GitHub] ArmageddonKnight commented on issue #11364: [MXNET-490] Added OpenLSTMRNN together with benchmarks and Tensorboard callback routines.

2018-10-31 Thread GitBox
ArmageddonKnight commented on issue #11364: [MXNET-490] Added OpenLSTMRNN together with benchmarks and Tensorboard callback routines. URL: https://github.com/apache/incubator-mxnet/pull/11364#issuecomment-434936117 @ankkhedia Thank you so much for your reminder. I have addressed her

[GitHub] ArmageddonKnight commented on a change in pull request #11364: [MXNET-490] Added OpenLSTMRNN together with benchmarks and Tensorboard callback routines.

2018-10-31 Thread GitBox
ArmageddonKnight commented on a change in pull request #11364: [MXNET-490] Added OpenLSTMRNN together with benchmarks and Tensorboard callback routines. URL: https://github.com/apache/incubator-mxnet/pull/11364#discussion_r229943760 ## File path:

[GitHub] ArmageddonKnight commented on a change in pull request #11364: [MXNET-490] Added OpenLSTMRNN together with benchmarks and Tensorboard callback routines.

2018-10-31 Thread GitBox
ArmageddonKnight commented on a change in pull request #11364: [MXNET-490] Added OpenLSTMRNN together with benchmarks and Tensorboard callback routines. URL: https://github.com/apache/incubator-mxnet/pull/11364#discussion_r229943448 ## File path:

[GitHub] ArmageddonKnight commented on a change in pull request #11364: [MXNET-490] Added OpenLSTMRNN together with benchmarks and Tensorboard callback routines.

2018-10-31 Thread GitBox
ArmageddonKnight commented on a change in pull request #11364: [MXNET-490] Added OpenLSTMRNN together with benchmarks and Tensorboard callback routines. URL: https://github.com/apache/incubator-mxnet/pull/11364#discussion_r229943448 ## File path:

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

2018-10-31 Thread GitBox
yzhliu 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-434932500 @terrytangyuan @gigasquid Thanks for comment. I think we all agree that we should not print the entire data.

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

2018-10-31 Thread GitBox
leezu commented on issue #13059: Refactor L2_normalization URL: https://github.com/apache/incubator-mxnet/pull/13059#issuecomment-434923748 Thanks for your PR! Your current implementation will tend to underflow for small values and overflow for large values. How about using a stable

[GitHub] wshuail opened a new issue #13074: Bug in bipartite_matching

2018-10-31 Thread GitBox
wshuail opened a new issue #13074: Bug in bipartite_matching URL: https://github.com/apache/incubator-mxnet/issues/13074 Hi when I use SSD for my own dataset, I found a bug in mx.sym.contrib.bipartite_matching for mxnet 1.3.x. it's easy to reproduce with the code as below. But maybe

[GitHub] wkcn commented on issue #13070: Cannot create large ndarray from numpy

2018-10-31 Thread GitBox
wkcn commented on issue #13070: Cannot create large ndarray from numpy URL: https://github.com/apache/incubator-mxnet/issues/13070#issuecomment-434922718 Similar Issue: #13036 This is an automated message from the Apache

[GitHub] yash0307 opened a new issue #13073: 3D version of mxnert.ndarray.UpSampling

2018-10-31 Thread GitBox
yash0307 opened a new issue #13073: 3D version of mxnert.ndarray.UpSampling URL: https://github.com/apache/incubator-mxnet/issues/13073 mxnert.ndarray.UpSampling checks if the data is in form: (batch_size, channels, x,y) (that is, number of dimensions=4), however, I want to do UpSampling

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

2018-10-31 Thread GitBox
wkcn commented on issue #13036: fix operators to support large arrays. URL: https://github.com/apache/incubator-mxnet/issues/13036#issuecomment-434916277 @zheng-da Maybe we can try to use `int32_t` for small for-loop, and `int64_t` for large for-loop.

[GitHub] lanking520 commented on issue #13046: [MXNET-1155] Add scala packageTest utility

2018-10-31 Thread GitBox
lanking520 commented on issue #13046: [MXNET-1155] Add scala packageTest utility URL: https://github.com/apache/incubator-mxnet/pull/13046#issuecomment-434914574 One general question, is that possible to use one pom file to trigger all the tests instead of defining a scala test target in

[GitHub] huangzhiyuan commented on issue #12980: Add reshape op supported by MKL-DNN

2018-10-31 Thread GitBox
huangzhiyuan commented on issue #12980: Add reshape op supported by MKL-DNN URL: https://github.com/apache/incubator-mxnet/pull/12980#issuecomment-434914352 @ankkhedia please take a view, ths :) This is an automated message

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

2018-10-31 Thread GitBox
ZhennanQin commented on issue #13059: Refactor L2_normalization URL: https://github.com/apache/incubator-mxnet/pull/13059#issuecomment-434912864 > The original version implemented in mshadow expression template works both for cpu and gpu; If you are only intereseted in an accelerated cpu

[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-434903903 Thanks @aaronmarkham - All your feedback has now been incorporated :)

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

2018-10-31 Thread GitBox
RogerChern commented on issue #13059: Refactor L2_normalization URL: https://github.com/apache/incubator-mxnet/pull/13059#issuecomment-434903200 The original version implemented in mshadow expression template works both for cpu and gpu; If you are only intereseted in an accelerated cpu

[GitHub] gigasquid commented on a change in pull request #12881: Improve the Clojure Package README to Make it Easier to Get Started

2018-10-31 Thread GitBox
gigasquid commented on a change in pull request #12881: Improve the Clojure Package README to Make it Easier to Get Started URL: https://github.com/apache/incubator-mxnet/pull/12881#discussion_r229917802 ## File path: contrib/clojure-package/README.md ## @@ -80,75 +83,66

[GitHub] gigasquid commented on a change in pull request #12881: Improve the Clojure Package README to Make it Easier to Get Started

2018-10-31 Thread GitBox
gigasquid commented on a change in pull request #12881: Improve the Clojure Package README to Make it Easier to Get Started URL: https://github.com/apache/incubator-mxnet/pull/12881#discussion_r229917802 ## File path: contrib/clojure-package/README.md ## @@ -80,75 +83,66

[GitHub] gigasquid commented on a change in pull request #12881: Improve the Clojure Package README to Make it Easier to Get Started

2018-10-31 Thread GitBox
gigasquid commented on a change in pull request #12881: Improve the Clojure Package README to Make it Easier to Get Started URL: https://github.com/apache/incubator-mxnet/pull/12881#discussion_r229917533 ## File path: contrib/clojure-package/README.md ## @@ -80,75 +83,66

[GitHub] janelu9 opened a new issue #13072: Does mxnet have a crf-loglikehood ?

2018-10-31 Thread GitBox
janelu9 opened a new issue #13072: Does mxnet have a crf-loglikehood ? URL: https://github.com/apache/incubator-mxnet/issues/13072 This is an automated message from the Apache Git Service. To respond to the message, please

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

2018-10-31 Thread GitBox
gigasquid edited a comment 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-434901131 Another reference is from

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

2018-10-31 Thread GitBox
gigasquid 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-434901131 Another reference is from https://docs.scipy.org/doc/numpy/reference/generated/numpy.set_printoptions.html

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

2018-10-31 Thread GitBox
azai91 commented on issue #12953: Update MKL-DNN dependency URL: https://github.com/apache/incubator-mxnet/pull/12953#issuecomment-434897409 just adding this PR fixes the issue described here: https://github.com/intel/mkl-dnn/issues/106 we have people facing this problem so this PR

[GitHub] ddavydenko commented on issue #13071: [WIP] add Java API docs generation

2018-10-31 Thread GitBox
ddavydenko commented on issue #13071: [WIP] add Java API docs generation URL: https://github.com/apache/incubator-mxnet/pull/13071#issuecomment-434896411 Yup, looks good now. And yes, I don't see Java API mixed in with Scala API in either docs, which is desired behavior.

[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 6fa0403 Bump the publish

[GitHub] aaronmarkham commented on issue #13071: [WIP] add Java API docs generation

2018-10-31 Thread GitBox
aaronmarkham commented on issue #13071: [WIP] add Java API docs generation URL: https://github.com/apache/incubator-mxnet/pull/13071#issuecomment-434895770 @ddavydenko Thanks for the check. I updated the preview urls for Scala. It doesn't seem to have the javaapi in it... in the index or

[GitHub] ddavydenko edited a comment on issue #13071: [WIP] add Java API docs generation

2018-10-31 Thread GitBox
ddavydenko edited a comment on issue #13071: [WIP] add Java API docs generation URL: https://github.com/apache/incubator-mxnet/pull/13071#issuecomment-434894681 Even though URLs are different - they both seems to render javadoc documentation for Java API. I think Scala URL should render

[GitHub] ddavydenko commented on issue #13071: [WIP] add Java API docs generation

2018-10-31 Thread GitBox
ddavydenko commented on issue #13071: [WIP] add Java API docs generation URL: https://github.com/apache/incubator-mxnet/pull/13071#issuecomment-434894681 Even though URLs are different - they both seems to render javadoc documentation for Java API. I think Scala URL should render existing

[GitHub] aaronmarkham opened a new pull request #13071: [WIP] add Java API docs generation

2018-10-31 Thread GitBox
aaronmarkham opened a new pull request #13071: [WIP] add Java API docs generation URL: https://github.com/apache/incubator-mxnet/pull/13071 ## Description ## This PR splits out the Java API from the Scala API docs. Currently WIP as I want to see what CI makes of it. My tests seem

[GitHub] ankkhedia commented on issue #11713: test_operator.test_deformable_psroipooling has fixed seed that can mask flakiness

2018-10-31 Thread GitBox
ankkhedia commented on issue #11713: test_operator.test_deformable_psroipooling has fixed seed that can mask flakiness URL: https://github.com/apache/incubator-mxnet/issues/11713#issuecomment-434888950 @Oh233 Did you get a chance to look into the fix as the PR #12178 didn't help fixing

[GitHub] nikudyshko commented on issue #12966: Strange behaviour/possible bug with mxnet::cpp::Symbol::Variable name argument

2018-10-31 Thread GitBox
nikudyshko commented on issue #12966: Strange behaviour/possible bug with mxnet::cpp::Symbol::Variable name argument URL: https://github.com/apache/incubator-mxnet/issues/12966#issuecomment-434888309 @ddavydenko Sorry for my late answer, was AFK for few days. I'm developing application,

[GitHub] ankkhedia commented on issue #12063: MxNet Pre-trained Prediction Doc Not Working

2018-10-31 Thread GitBox
ankkhedia commented on issue #12063: MxNet Pre-trained Prediction Doc Not Working URL: https://github.com/apache/incubator-mxnet/issues/12063#issuecomment-434887945 @prathik-naidu Did you get a chance to look into results after applying normalization?

[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-434886842 @wongjoel you can do an empty commit to retrigger CI This is

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

2018-10-31 Thread GitBox
wongjoel commented on issue #13052: Fix incorrect variable name in tutorial code URL: https://github.com/apache/incubator-mxnet/pull/13052#issuecomment-434886493 @ankkhedia Thanks - I have no idea why there is a CI failure for `GPU: CMake MKLDNN` - the commit should have only changed the

[GitHub] ankkhedia commented on issue #12815: incubator-mxnet/3rdparty/mshadow/mshadow/./base.h:162:14: fatal error: cblas.h: No such file or directory

2018-10-31 Thread GitBox
ankkhedia commented on issue #12815: incubator-mxnet/3rdparty/mshadow/mshadow/./base.h:162:14: fatal error: cblas.h: No such file or directory URL: https://github.com/apache/incubator-mxnet/issues/12815#issuecomment-434886043 @sahbiept Did the solutions suggested above worked for you?

[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-434885018 Closing the PR as this file does not exist anymore and the corrected link is no more referenced in the

[GitHub] sandeep-krishnamurthy closed pull request #11481: # updated the link as the previous one was corrupted

2018-10-31 Thread GitBox
sandeep-krishnamurthy closed pull request #11481: # updated the link as the previous one was corrupted URL: https://github.com/apache/incubator-mxnet/pull/11481 This is an automated message from the Apache Git Service. To

[GitHub] zachgk commented on a change in pull request #13038: [MXNET-918] Introduce Random module / Refact code generation

2018-10-31 Thread GitBox
zachgk commented on a change in pull request #13038: [MXNET-918] Introduce Random module / Refact code generation URL: https://github.com/apache/incubator-mxnet/pull/13038#discussion_r229897662 ## File path:

[GitHub] zachgk commented on a change in pull request #13038: [MXNET-918] Introduce Random module / Refact code generation

2018-10-31 Thread GitBox
zachgk commented on a change in pull request #13038: [MXNET-918] Introduce Random module / Refact code generation URL: https://github.com/apache/incubator-mxnet/pull/13038#discussion_r229901924 ## File path: scala-package/macros/src/main/scala/org/apache/mxnet/SymbolMacro.scala

[GitHub] zachgk commented on a change in pull request #13038: [MXNET-918] Introduce Random module / Refact code generation

2018-10-31 Thread GitBox
zachgk commented on a change in pull request #13038: [MXNET-918] Introduce Random module / Refact code generation URL: https://github.com/apache/incubator-mxnet/pull/13038#discussion_r229901549 ## File path: scala-package/macros/src/main/scala/org/apache/mxnet/NDArrayMacro.scala

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

2018-10-31 Thread GitBox
zheng-da commented on issue #13036: fix operators to support large arrays. URL: https://github.com/apache/incubator-mxnet/issues/13036#issuecomment-434882294 @wkcn in cpu, it shouldn't be a problem. I heard concerns on GPUs. Potentially, we can use int64_t for CPU and int for GPU.

[GitHub] wkcn edited a comment on issue #12972: [MXNET-1174]Support np.longlong on Linux and Fix issue #12843

2018-10-31 Thread GitBox
wkcn edited a comment on issue #12972: [MXNET-1174]Support np.longlong on Linux and Fix issue #12843 URL: https://github.com/apache/incubator-mxnet/pull/12972#issuecomment-434880656 @eric-haibin-lin it is related to NumPy. If c_longlong is 128 bits, I think that

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

2018-10-31 Thread GitBox
zheng-da commented on issue #13036: fix operators to support large arrays. URL: https://github.com/apache/incubator-mxnet/issues/13036#issuecomment-434882013 i'm fixing some of the operators. but we need a systematic fix. The problem is everywhere. i'll provide a temp fix for some of the

[GitHub] wkcn commented on issue #12972: [MXNET-1174]Support np.longlong on Linux and Fix issue #12843

2018-10-31 Thread GitBox
wkcn commented on issue #12972: [MXNET-1174]Support np.longlong on Linux and Fix issue #12843 URL: https://github.com/apache/incubator-mxnet/pull/12972#issuecomment-434880656 @eric-haibin-lin it is related to NumPy. If c_longlong is 128 bits, I think that

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

2018-10-31 Thread GitBox
wkcn commented on issue #13036: fix operators to support large arrays. URL: https://github.com/apache/incubator-mxnet/issues/13036#issuecomment-434879773 @apeforest will it drop the performance down If changing the type of iteration to int64_t? In PyTorch, the type of iteration is

[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

[incubator-mxnet] branch master updated: removed unused header (#13066)

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

[GitHub] szha closed pull request #13066: Remove unused header from engine.h

2018-10-31 Thread GitBox
szha closed pull request #13066: Remove unused header from engine.h URL: https://github.com/apache/incubator-mxnet/pull/13066 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] eric-haibin-lin opened a new issue #13070: Cannot create large ndarray from numpy

2018-10-31 Thread GitBox
eric-haibin-lin opened a new issue #13070: Cannot create large ndarray from numpy URL: https://github.com/apache/incubator-mxnet/issues/13070 ``` >>> import mxnet as mx >>> import numpy as np >>> a = np.empty((50,)) >>> a array([0., 0., 0., ..., 0., 0., 0.]) >>>

[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] 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] 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] Ishitori commented on issue #13041: Updated recommenders example

2018-10-31 Thread GitBox
Ishitori commented on issue #13041: Updated recommenders example URL: https://github.com/apache/incubator-mxnet/pull/13041#issuecomment-434869454 Cannot comment on notebook - too big, so pasting comments here: **demo1-MF**: 1) ratings =

[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] eric-haibin-lin commented on issue #12972: [MXNET-1174]Support np.longlong on Linux and Fix issue #12843

2018-10-31 Thread GitBox
eric-haibin-lin commented on issue #12972: [MXNET-1174]Support np.longlong on Linux and Fix issue #12843 URL: https://github.com/apache/incubator-mxnet/pull/12972#issuecomment-434861063 I'm a bit concerned that by c standard longlong requires at least 64 bit and there's no guarantee it's

[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

[GitHub] ThomasDelteil edited a comment on issue #12964: Update multi-task learning example

2018-10-31 Thread GitBox
ThomasDelteil edited a comment on issue #12964: Update multi-task learning example URL: https://github.com/apache/incubator-mxnet/pull/12964#issuecomment-434858891 @aaronmarkham I am doing a first pass on the examples where I do one of: - fix - delete - replace with Gluon

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

2018-10-31 Thread GitBox
ThomasDelteil commented on issue #12964: Update multi-task learning example URL: https://github.com/apache/incubator-mxnet/pull/12964#issuecomment-434858891 @aaronmarkham I am doing a first pass on the examples where I do one of: - fix - delete - replace with Gluon depending on

[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] 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] ankkhedia commented on issue #12933: Update autoencoder example

2018-10-31 Thread GitBox
ankkhedia commented on issue #12933: Update autoencoder example URL: https://github.com/apache/incubator-mxnet/pull/12933#issuecomment-434856682 @aaronmarkham Another example to be reviewed :) This is an automated message

[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] 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-434853072 JIRA task created: https://issues.apache.org/jira/browse/MXNET-1185

[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-434852976 @wkcn is right. We need to chang int to index_t. I am busy with other tasks now and can only come to this in one week. Let me know

[GitHub] ankkhedia commented on issue #13065: [MXNET-793] Virtual testing with Qemu, refinement and extract test results to root MXNet folder

2018-10-31 Thread GitBox
ankkhedia commented on issue #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#issuecomment-434851620 @larroy Thanks for your contribution! @mxnet-label-bot[

[GitHub] ankkhedia commented on issue #13066: Remove unused header from engine.h

2018-10-31 Thread GitBox
ankkhedia commented on issue #13066: Remove unused header from engine.h URL: https://github.com/apache/incubator-mxnet/pull/13066#issuecomment-434851180 @apeforest Thanks for taking up the task simplify dependencies. @mxnet-label-bot [pr-awaiting-review, backend]

[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] 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] anirudhacharya commented on issue #13007: Image API docs error

2018-10-31 Thread GitBox
anirudhacharya commented on issue #13007: Image API docs error URL: https://github.com/apache/incubator-mxnet/issues/13007#issuecomment-434845151 click on mxnet.image and you will see it goes to the wrong place. This is an

[GitHub] yajiedesign commented on issue #13040: Import and GPU init extremely slow on Windows

2018-10-31 Thread GitBox
yajiedesign commented on issue #13040: Import and GPU init extremely slow on Windows URL: https://github.com/apache/incubator-mxnet/issues/13040#issuecomment-434843640 Titan V must SM_70 arch.wait update This is an

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

2018-10-31 Thread GitBox
vandanavk commented on issue #13067: ONNX export: Set np.array type to float32 URL: https://github.com/apache/incubator-mxnet/pull/13067#issuecomment-434843500 @ankkhedia @anirudhacharya @Roshrini @zhreshold for review This

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

2018-10-31 Thread GitBox
harshp8l edited a comment 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

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

2018-10-31 Thread GitBox
harshp8l edited a comment 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

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

2018-10-31 Thread GitBox
harshp8l edited a comment 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

[GitHub] apeforest commented on a change in pull request #13048: Updated CONTRIBUTERS.md to include label-bot

2018-10-31 Thread GitBox
apeforest commented on a change in pull request #13048: Updated CONTRIBUTERS.md to include label-bot URL: https://github.com/apache/incubator-mxnet/pull/13048#discussion_r229863010 ## File path: CONTRIBUTORS.md ## @@ -58,6 +58,13 @@ who are willing to help maintaining and

[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] 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] zachgk commented on a change in pull request #13046: [MXNET-1155] Add scala packageTest utility

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

[GitHub] apeforest opened a new pull request #13066: Remove unused header from engine.h

2018-10-31 Thread GitBox
apeforest opened a new pull request #13066: Remove unused header from engine.h URL: https://github.com/apache/incubator-mxnet/pull/13066 ## Description ## There are very complicated dependencies in the current mxnet include folder. This is an initial step to reduce unnecessary

[GitHub] ankkhedia commented on issue #12467: Add PolyScheduler which mimics the caffe polynomial lr scheduler

2018-10-31 Thread GitBox
ankkhedia commented on issue #12467: Add PolyScheduler which mimics the caffe polynomial lr scheduler URL: https://github.com/apache/incubator-mxnet/pull/12467#issuecomment-434837242 @bzcheeseman Sorry to hear that :( @leleamol Could you please help @bzcheeseman if you know of any

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

2018-10-31 Thread GitBox
larroy commented on issue #13048: Updated CONTRIBUTERS.md to include label-bot URL: https://github.com/apache/incubator-mxnet/pull/13048#issuecomment-434836940 It's hilarious that it can't prevent spelling mistakes on the title of the PR. ;-) Why are we committing under the label bot

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

[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 f958d39 Bump the publish

[GitHub] ankkhedia commented on issue #12380: Manually track num_max_thread

2018-10-31 Thread GitBox
ankkhedia commented on issue #12380: Manually track num_max_thread URL: https://github.com/apache/incubator-mxnet/pull/12380#issuecomment-434831878 @azai91 Could you please follow up on the comments given above? This is an

[GitHub] ankkhedia commented on issue #12331: Moves f16c autodetection to its own cmake module

2018-10-31 Thread GitBox
ankkhedia commented on issue #12331: Moves f16c autodetection to its own cmake module URL: https://github.com/apache/incubator-mxnet/pull/12331#issuecomment-434830874 @szha @marcoabreu @nswamy @cjolivier01 Request you to review this PR as it is blocking #10564

[GitHub] samskalicky commented on issue #12904: support upper triangular matrices in linalg

2018-10-31 Thread GitBox
samskalicky commented on issue #12904: support upper triangular matrices in linalg URL: https://github.com/apache/incubator-mxnet/pull/12904#issuecomment-434828260 added some suggestions on cleaning up the names throughout to match the new documentation. This is necessary for maintenance

[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_r229846687 ## File path: src/operator/tensor/la_op_inline.h ## @@ -493,11 +519,11 @@ struct

[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_r229846545 ## File path: src/operator/tensor/la_op_inline.h ## @@ -458,14 +483,15 @@ struct

[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_r229846402 ## File path: src/operator/tensor/la_op_inline.h ## @@ -430,14 +447,22 @@ struct

[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_r229846100 ## File path: src/operator/tensor/la_op_inline.h ## @@ -403,15 +417,18 @@ struct

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

  1   2   >