[GitHub] edmBernard commented on issue #8525: how to make the IOS library?

2017-12-12 Thread GitBox
edmBernard commented on issue #8525: how to make the IOS library? URL: https://github.com/apache/incubator-mxnet/issues/8525#issuecomment-351309677 @nihilityworld Have you finally achieved to compile amalgamation on iOS ? Thi

[GitHub] rahul003 commented on issue #9049: Allow user to pass specific environment variables for server/worker/both through launch.py

2017-12-12 Thread GitBox
rahul003 commented on issue #9049: Allow user to pass specific environment variables for server/worker/both through launch.py URL: https://github.com/apache/incubator-mxnet/pull/9049#issuecomment-351301961 This build won't pass till the dmlc PR gets merged

[GitHub] lhb8125 opened a new issue #9052: Build warning

2017-12-12 Thread GitBox
lhb8125 opened a new issue #9052: Build warning URL: https://github.com/apache/incubator-mxnet/issues/9052 I downloaded the source package mxnet-0.12.0, but I got a warning that the variable "i" was defined at line 218 and line 226 in file kvstore_local.h. I think you should better fix i

[incubator-mxnet] branch master updated: Add cscope filenames to gitignore (#9019)

2017-12-12 Thread jxie
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 73fa65f Add cscope filenames to gitignor

[GitHub] piiswrong closed pull request #9019: Add cscope filenames to gitignore

2017-12-12 Thread GitBox
piiswrong closed pull request #9019: Add cscope filenames to gitignore URL: https://github.com/apache/incubator-mxnet/pull/9019 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] dwSun commented on issue #8894: Mobilenet

2017-12-12 Thread GitBox
dwSun commented on issue #8894: Mobilenet URL: https://github.com/apache/incubator-mxnet/pull/8894#issuecomment-351293248 With alpha=1, this change is fully compatible with previously trained model. The symbol loaded by mx.model.load_checkpoint is slightly different with the one retu

[GitHub] nihilityworld closed issue #8525: how to make the IOS library?

2017-12-12 Thread GitBox
nihilityworld closed issue #8525: how to make the IOS library? URL: https://github.com/apache/incubator-mxnet/issues/8525 This is an automated message from the Apache Git Service. To respond to the message, please log on GitH

[GitHub] eric-haibin-lin commented on issue #9027: Usability improvement for cnn_chinese_text_classification

2017-12-12 Thread GitBox
eric-haibin-lin commented on issue #9027: Usability improvement for cnn_chinese_text_classification URL: https://github.com/apache/incubator-mxnet/pull/9027#issuecomment-351285677 Also removed the deprecated code example. Thi

[GitHub] nihilityworld opened a new issue #9051: [Android,v0.11.0] The result of mxnet_predict is wrong! AND how to compile the amalgamation in Android Studio?

2017-12-12 Thread GitBox
nihilityworld opened a new issue #9051: [Android,v0.11.0] The result of mxnet_predict is wrong! AND how to compile the amalgamation in Android Studio? URL: https://github.com/apache/incubator-mxnet/issues/9051 @piiswrong I have compile the amalgamation in IOS and ANDROID. The params and th

[GitHub] szha commented on issue #8671: Discussion and troubleshooting on PyPI (pip) installation

2017-12-12 Thread GitBox
szha commented on issue #8671: Discussion and troubleshooting on PyPI (pip) installation URL: https://github.com/apache/incubator-mxnet/issues/8671#issuecomment-351283896 #9048 This is an automated message from the Apache G

[GitHub] iblis17 opened a new issue #9050: got segfault from lenet with stn example

2017-12-12 Thread GitBox
iblis17 opened a new issue #9050: got segfault from lenet with stn example URL: https://github.com/apache/incubator-mxnet/issues/9050 Hi, We encounter segfault with stn. Here is the original issue https://github.com/dmlc/MXNet.jl/issues/369. TL;DR: Segfault happened in CPU

[GitHub] szha commented on issue #9048: pip package relies on outdated libgfortran.so version

2017-12-12 Thread GitBox
szha commented on issue #9048: pip package relies on outdated libgfortran.so version URL: https://github.com/apache/incubator-mxnet/issues/9048#issuecomment-351277879 I'm evaluating ways to allow pip users to benefit from the change as well without static-linking. ---

[GitHub] piiswrong commented on issue #9048: pip package relies on outdated libgfortran.so version

2017-12-12 Thread GitBox
piiswrong commented on issue #9048: pip package relies on outdated libgfortran.so version URL: https://github.com/apache/incubator-mxnet/issues/9048#issuecomment-351277646 @szha Can we do a conda build? This is an automated

[GitHub] piiswrong commented on issue #8972: Profiling enhancements, python API, vtune and chrome tracing objects, etc.

2017-12-12 Thread GitBox
piiswrong commented on issue #8972: Profiling enhancements, python API, vtune and chrome tracing objects, etc. URL: https://github.com/apache/incubator-mxnet/pull/8972#issuecomment-351272867 Can you use a dmlc::Parameter for profiler config parsing? Similar to how operator parameters are p

[GitHub] piiswrong closed issue #9000: Flaky test OOM: test_optimizers:test_sgd

2017-12-12 Thread GitBox
piiswrong closed issue #9000: Flaky test OOM: test_optimizers:test_sgd URL: https://github.com/apache/incubator-mxnet/issues/9000 This is an automated message from the Apache Git Service. To respond to the message, please log

[incubator-mxnet] branch master updated: Disable test_optimizers:test_sgd (Temp fix #9000) (#9001)

2017-12-12 Thread jxie
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 9e2bdeb Disable test_optimizers:test_sgd

[GitHub] piiswrong closed pull request #9001: Disable test_optimizers:test_sgd (Temp fix #9000)

2017-12-12 Thread GitBox
piiswrong closed pull request #9001: Disable test_optimizers:test_sgd (Temp fix #9000) URL: https://github.com/apache/incubator-mxnet/pull/9001 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 t

[GitHub] leezu commented on issue #8280: v0.12 autograd regression

2017-12-12 Thread GitBox
leezu commented on issue #8280: v0.12 autograd regression URL: https://github.com/apache/incubator-mxnet/issues/8280#issuecomment-351269852 The issue seems to be fixed in v1. Perhaps by #8701? Anyways, thanks! This is an auto

[GitHub] leezu closed issue #8280: v0.12 autograd regression

2017-12-12 Thread GitBox
leezu closed issue #8280: v0.12 autograd regression URL: https://github.com/apache/incubator-mxnet/issues/8280 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use

[GitHub] huabinhuang1994 closed issue #9031: cuda error:Check failed: e == cudaSuccess || e == cudaErrorCudartUnloading CUDA: unknown error

2017-12-12 Thread GitBox
huabinhuang1994 closed issue #9031: cuda error:Check failed: e == cudaSuccess || e == cudaErrorCudartUnloading CUDA: unknown error URL: https://github.com/apache/incubator-mxnet/issues/9031 This is an automated message from

[GitHub] huabinhuang1994 commented on issue #9031: cuda error:Check failed: e == cudaSuccess || e == cudaErrorCudartUnloading CUDA: unknown error

2017-12-12 Thread GitBox
huabinhuang1994 commented on issue #9031: cuda error:Check failed: e == cudaSuccess || e == cudaErrorCudartUnloading CUDA: unknown error URL: https://github.com/apache/incubator-mxnet/issues/9031#issuecomment-351269912 I found that it's because my nvidia driver is broken ,I re-installed i

[GitHub] piiswrong closed pull request #9045: Remove defunct NLP tutorial page

2017-12-12 Thread GitBox
piiswrong closed pull request #9045: Remove defunct NLP tutorial page URL: https://github.com/apache/incubator-mxnet/pull/9045 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign p

[incubator-mxnet] branch master updated: Remove defunct NLP tutorial page (#9045)

2017-12-12 Thread jxie
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 6d1f07d Remove defunct NLP tutorial page

[GitHub] formath commented on issue #9023: Multiple featureid in one field in wide&deep example

2017-12-12 Thread GitBox
formath commented on issue #9023: Multiple featureid in one field in wide&deep example URL: https://github.com/apache/incubator-mxnet/issues/9023#issuecomment-351269223 `libsvm` format like `fieldid:featureid` will generate `CSRNDArray` in which index are `fieldid` but we want to use `fea

[GitHub] piiswrong closed pull request #9039: update autograd.Function docstring for usage

2017-12-12 Thread GitBox
piiswrong closed pull request #9039: update autograd.Function docstring for usage URL: https://github.com/apache/incubator-mxnet/pull/9039 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 i

[incubator-mxnet] branch master updated: update autograd.Function docstring for usage (#9039)

2017-12-12 Thread jxie
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 8f79651 update autograd.Function docstri

[GitHub] madjam commented on issue #9033: Reorganize the tutorials index page

2017-12-12 Thread GitBox
madjam commented on issue #9033: Reorganize the tutorials index page URL: https://github.com/apache/incubator-mxnet/pull/9033#issuecomment-351268163 Looks good to me. This is an automated message from the Apache Git Service. T

[GitHub] formath opened a new issue #9023: Multiple featureid in one field in wide&deep example

2017-12-12 Thread GitBox
formath opened a new issue #9023: Multiple featureid in one field in wide&deep example URL: https://github.com/apache/incubator-mxnet/issues/9023 Hi @ZiyueHuang Now in the example `sparse/wide_deep`, it assumes one `featureid` in one `field`. How about multiple `featureid` in one `fiel

[GitHub] formath commented on issue #9023: Multiple featureid in one field in wide&deep example

2017-12-12 Thread GitBox
formath commented on issue #9023: Multiple featureid in one field in wide&deep example URL: https://github.com/apache/incubator-mxnet/issues/9023#issuecomment-350709313 I have known how to do it. thanks This is an automated

[GitHub] rahul003 opened a new pull request #9049: Allow user to pass specific environment variables for server/worker/both through launch.py

2017-12-12 Thread GitBox
rahul003 opened a new pull request #9049: Allow user to pass specific environment variables for server/worker/both through launch.py URL: https://github.com/apache/incubator-mxnet/pull/9049 ## Description ## Sometimes user might want to pass a specific environment variable when launchin

[GitHub] leezu opened a new issue #9048: pip package relies on deprecated libgfortran.so version

2017-12-12 Thread GitBox
leezu opened a new issue #9048: pip package relies on deprecated libgfortran.so version URL: https://github.com/apache/incubator-mxnet/issues/9048 Since at least gcc 7.2 released [2017-08-14] libgfortran.so in version 4 (i.e. libgfortran.so.4) is provided. The mxnet pip package however lin

[GitHub] janelu9 opened a new issue #9047: how to resolve the problem of IO?can mxnet train data on hadoop?

2017-12-12 Thread GitBox
janelu9 opened a new issue #9047: how to resolve the problem of IO?can mxnet train data on hadoop? URL: https://github.com/apache/incubator-mxnet/issues/9047 Though I have a piece GPU of Tesla P40, the train speed is so slow because of IO --

[GitHub] sandeep-krishnamurthy closed pull request #9030: Fix Gan

2017-12-12 Thread GitBox
sandeep-krishnamurthy closed pull request #9030: Fix Gan URL: https://github.com/apache/incubator-mxnet/pull/9030 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request (

[incubator-mxnet] branch master updated: Fix Gan tutorials with symbolic interface (#9030)

2017-12-12 Thread skm
This is an automated email from the ASF dual-hosted git repository. skm 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 ca79cfc Fix Gan tutorials with symbolic i

[GitHub] pracheer commented on issue #9030: Fix Gan

2017-12-12 Thread GitBox
pracheer commented on issue #9030: Fix Gan URL: https://github.com/apache/incubator-mxnet/pull/9030#issuecomment-351261860 Looks good to me! This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] Soonhwan-Kwon commented on issue #8787: add CapsNet example

2017-12-12 Thread GitBox
Soonhwan-Kwon commented on issue #8787: add CapsNet example URL: https://github.com/apache/incubator-mxnet/pull/8787#issuecomment-351261857 @piiswrong Thank you for review. I rebased to the master. This is an automated message

[GitHub] yufengwhy commented on issue #9020: pooling of dcnn

2017-12-12 Thread GitBox
yufengwhy commented on issue #9020: pooling of dcnn URL: https://github.com/apache/incubator-mxnet/issues/9020#issuecomment-351258271 Apologize for my thoughtlessness, line 119 is actually also global pooling, because the pooling size == the input size. But if (global_pool=True) is given,

[GitHub] yufengwhy commented on issue #9020: pooling of dcnn

2017-12-12 Thread GitBox
yufengwhy commented on issue #9020: pooling of dcnn URL: https://github.com/apache/incubator-mxnet/issues/9020#issuecomment-351258271 Apologize for my thoughtlessness, line 119 is actually also global pooling, because the pooling size == the input size. But if (global_pool=True) is given,

[GitHub] yufengwhy commented on issue #9020: pooling of dcnn

2017-12-12 Thread GitBox
yufengwhy commented on issue #9020: pooling of dcnn URL: https://github.com/apache/incubator-mxnet/issues/9020#issuecomment-351260533 1d convolution is row-wise convolution over 2d input (embedding_dimension x num_words), which is useful in NLP or recommender. There is actually a theano-i

[GitHub] yufengwhy commented on issue #9020: pooling of dcnn

2017-12-12 Thread GitBox
yufengwhy commented on issue #9020: pooling of dcnn URL: https://github.com/apache/incubator-mxnet/issues/9020#issuecomment-351260533 1d convolution is row-wise convolution over 2d input (embedding_dimension x num_words), which is useful in NLP or recommender. There is actually a theano-i

[GitHub] anirudh2290 commented on issue #9008: Usability improvement mxnet vae

2017-12-12 Thread GitBox
anirudh2290 commented on issue #9008: Usability improvement mxnet vae URL: https://github.com/apache/incubator-mxnet/pull/9008#issuecomment-351260080 @madjam I have made the required changes! This is an automated message from

[GitHub] yufengwhy commented on issue #9020: pooling of dcnn

2017-12-12 Thread GitBox
yufengwhy commented on issue #9020: pooling of dcnn URL: https://github.com/apache/incubator-mxnet/issues/9020#issuecomment-351258271 Apologize for my thoughtlessness, line 119 is actually also global pooling, because the pooling size == the input size. But if (global_pool=True) is given,

[GitHub] sergeykolychev commented on issue #8972: Profiling enhancements, python API, vtune and chrome tracing objects, etc.

2017-12-12 Thread GitBox
sergeykolychev commented on issue #8972: Profiling enhancements, python API, vtune and chrome tracing objects, etc. URL: https://github.com/apache/incubator-mxnet/pull/8972#issuecomment-351258030 @cjolivier01 Perl changes look good. Thanks. --

[GitHub] anirudh2290 commented on a change in pull request #8938: Add operator for dot(dns, csr) = csr

2017-12-12 Thread GitBox
anirudh2290 commented on a change in pull request #8938: Add operator for dot(dns, csr) = csr URL: https://github.com/apache/incubator-mxnet/pull/8938#discussion_r156546512 ## File path: src/operator/tensor/dot-inl.h ## @@ -811,6 +891,94 @@ inline void DotCsrRspRspImpl(con

[GitHub] anirudh2290 commented on a change in pull request #8938: Add operator for dot(dns, csr) = csr

2017-12-12 Thread GitBox
anirudh2290 commented on a change in pull request #8938: Add operator for dot(dns, csr) = csr URL: https://github.com/apache/incubator-mxnet/pull/8938#discussion_r156546520 ## File path: src/operator/tensor/dot-inl.h ## @@ -228,8 +231,16 @@ inline bool DotForwardInferStora

[GitHub] Laurawly commented on issue #8915: NVLink communication pattern updated

2017-12-12 Thread GitBox
Laurawly commented on issue #8915: NVLink communication pattern updated URL: https://github.com/apache/incubator-mxnet/pull/8915#issuecomment-351253691 @rahul003 Yeah, I'll do an update on ReduceCompressed function as well. Thanks for reminding that. --

[GitHub] Laurawly commented on issue #8915: NVLink communication pattern updated

2017-12-12 Thread GitBox
Laurawly commented on issue #8915: NVLink communication pattern updated URL: https://github.com/apache/incubator-mxnet/pull/8915#issuecomment-351253691 @rahul003 Yeah, I'll do an update on ReduceCompressed function as well. T

[GitHub] cjolivier01 commented on issue #8972: Profiling enhancements, python API, vtune and chrome tracing objects, etc.

2017-12-12 Thread GitBox
cjolivier01 commented on issue #8972: Profiling enhancements, python API, vtune and chrome tracing objects, etc. URL: https://github.com/apache/incubator-mxnet/pull/8972#issuecomment-351253046 @sergeykolychev Can you check Perl changes, please? -

[GitHub] cjolivier01 commented on issue #8972: Profiling enhancements, python API, vtune and chrome tracing objects, etc.

2017-12-12 Thread GitBox
cjolivier01 commented on issue #8972: Profiling enhancements, python API, vtune and chrome tracing objects, etc. URL: https://github.com/apache/incubator-mxnet/pull/8972#issuecomment-351253046 @sergeykolychev Can you check Perl changes, please? https://github.com/apache/incubator-mx

[GitHub] cjolivier01 commented on a change in pull request #8972: Profiling enhancements, python API, vtune and chrome tracing objects, etc.

2017-12-12 Thread GitBox
cjolivier01 commented on a change in pull request #8972: Profiling enhancements, python API, vtune and chrome tracing objects, etc. URL: https://github.com/apache/incubator-mxnet/pull/8972#discussion_r156544303 ## File path: include/mxnet/c_api.h ## @@ -227,10 +227,138 @@

[GitHub] yuruofeifei commented on issue #8964: Fix example/module folder and remove duplicate examples

2017-12-12 Thread GitBox
yuruofeifei commented on issue #8964: Fix example/module folder and remove duplicate examples URL: https://github.com/apache/incubator-mxnet/pull/8964#issuecomment-351251352 Rebased... This is an automated message from the Ap

[GitHub] larroy commented on a change in pull request #9016: Add googletest as a 3rdparty library

2017-12-12 Thread GitBox
larroy commented on a change in pull request #9016: Add googletest as a 3rdparty library URL: https://github.com/apache/incubator-mxnet/pull/9016#discussion_r156542580 ## File path: .gitmodules ## @@ -1,9 +1,9 @@ [submodule "mshadow"] path = mshadow - url =

[GitHub] javelinjs commented on issue #8245: Use argmax instead of argmax_channel in Accuracy to keep dimention

2017-12-12 Thread GitBox
javelinjs commented on issue #8245: Use argmax instead of argmax_channel in Accuracy to keep dimention URL: https://github.com/apache/incubator-mxnet/pull/8245#issuecomment-351249142 @benqua This is an automated message from

[GitHub] mbaijal commented on a change in pull request #9046: Fix example example/reinforcement-learning/a3c

2017-12-12 Thread GitBox
mbaijal commented on a change in pull request #9046: Fix example example/reinforcement-learning/a3c URL: https://github.com/apache/incubator-mxnet/pull/9046#discussion_r156541743 ## File path: example/reinforcement-learning/a3c/README.md ## @@ -7,6 +7,12 @@ The algorithm

[GitHub] javelinjs commented on issue #8958: [WIP] fix sgld optimizer and its related example

2017-12-12 Thread GitBox
javelinjs commented on issue #8958: [WIP] fix sgld optimizer and its related example URL: https://github.com/apache/incubator-mxnet/pull/8958#issuecomment-351248661 @piiswrong Yes. I'm working on it. This is an automated mess

[GitHub] mbaijal commented on a change in pull request #9046: Fix example example/reinforcement-learning/a3c

2017-12-12 Thread GitBox
mbaijal commented on a change in pull request #9046: Fix example example/reinforcement-learning/a3c URL: https://github.com/apache/incubator-mxnet/pull/9046#discussion_r156541473 ## File path: example/reinforcement-learning/a3c/a3c.py ## @@ -139,7 +144,7 @@ def train():

[GitHub] mbaijal opened a new pull request #9046: Fix example example/reinforcement-learning/a3c

2017-12-12 Thread GitBox
mbaijal opened a new pull request #9046: Fix example example/reinforcement-learning/a3c URL: https://github.com/apache/incubator-mxnet/pull/9046 ## Description ## - This example was not compatible with python3. I made necessary changes. - I also updated the README with the Pre-r

[GitHub] rahul003 closed pull request #9037: Fix memory leaks and some other fixes identified by coverity

2017-12-12 Thread GitBox
rahul003 closed pull request #9037: Fix memory leaks and some other fixes identified by coverity URL: https://github.com/apache/incubator-mxnet/pull/9037 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 provena

[GitHub] BiranLi closed issue #8659: Some problems about support for Volta GPU architecture(V100 and CUDA9/CUDNNV7).

2017-12-12 Thread GitBox
BiranLi closed issue #8659: Some problems about support for Volta GPU architecture(V100 and CUDA9/CUDNNV7). URL: https://github.com/apache/incubator-mxnet/issues/8659 This is an automated message from the Apache Git Service.

[GitHub] sandeep-krishnamurthy commented on issue #9030: Fix Gan

2017-12-12 Thread GitBox
sandeep-krishnamurthy commented on issue #9030: Fix Gan URL: https://github.com/apache/incubator-mxnet/pull/9030#issuecomment-351245420 Thank you @pracheer and @madjam for the detailed review and feedback. Made the suggested corrections.

[GitHub] cjolivier01 commented on issue #9016: Add googletest as a 3rdparty library

2017-12-12 Thread GitBox
cjolivier01 commented on issue #9016: Add googletest as a 3rdparty library URL: https://github.com/apache/incubator-mxnet/pull/9016#issuecomment-351242804 +1 to use as a submodule. This way our version can stay more current. T

[GitHub] larroy commented on issue #9033: Reorganize the tutorials index page

2017-12-12 Thread GitBox
larroy commented on issue #9033: Reorganize the tutorials index page URL: https://github.com/apache/incubator-mxnet/pull/9033#issuecomment-351242421 I think it can be merged as it is, it's not blocked by ci This is an automate

[GitHub] larroy commented on issue #9033: Reorganize the tutorials index page

2017-12-12 Thread GitBox
larroy commented on issue #9033: Reorganize the tutorials index page URL: https://github.com/apache/incubator-mxnet/pull/9033#issuecomment-351242440 @piiswrong This is an automated message from the Apache Git Service. To

[GitHub] marcoabreu commented on issue #9001: Disable test_optimizers:test_sgd (Temp fix #9000)

2017-12-12 Thread GitBox
marcoabreu commented on issue #9001: Disable test_optimizers:test_sgd (Temp fix #9000) URL: https://github.com/apache/incubator-mxnet/pull/9001#issuecomment-351240266 @piiswrong ready to merge This is an automated message fro

[GitHub] indhub commented on issue #9033: Reorganize the tutorials index page

2017-12-12 Thread GitBox
indhub commented on issue #9033: Reorganize the tutorials index page URL: https://github.com/apache/incubator-mxnet/pull/9033#issuecomment-351239490 @larroy There is a lot of difference between the Jenkins file in this branch and the one in master. Could you please let me know where the mis

[GitHub] cjolivier01 commented on a change in pull request #8972: Profiling enhancements, python API, vtune and chrome tracing objects, etc.

2017-12-12 Thread GitBox
cjolivier01 commented on a change in pull request #8972: Profiling enhancements, python API, vtune and chrome tracing objects, etc. URL: https://github.com/apache/incubator-mxnet/pull/8972#discussion_r156530707 ## File path: python/mxnet/profiler.py ## @@ -56,3 +53,235 @@

[GitHub] larroy commented on issue #9033: Reorganize the tutorials index page

2017-12-12 Thread GitBox
larroy commented on issue #9033: Reorganize the tutorials index page URL: https://github.com/apache/incubator-mxnet/pull/9033#issuecomment-351234313 The build failed because of a Jenkinsfile difference, but if a commiter can merge this without CI then please go ahead. -

[GitHub] cjolivier01 commented on a change in pull request #8972: Profiling enhancements, python API, vtune and chrome tracing objects, etc.

2017-12-12 Thread GitBox
cjolivier01 commented on a change in pull request #8972: Profiling enhancements, python API, vtune and chrome tracing objects, etc. URL: https://github.com/apache/incubator-mxnet/pull/8972#discussion_r156530352 ## File path: python/mxnet/profiler.py ## @@ -56,3 +53,235 @@

[GitHub] larroy commented on issue #9033: Reorganize the tutorials index page

2017-12-12 Thread GitBox
larroy commented on issue #9033: Reorganize the tutorials index page URL: https://github.com/apache/incubator-mxnet/pull/9033#issuecomment-351234313 The build failed beause of a Jenkinsfile difference, but if a commiter can merge this without CI then please go ahead. --

[GitHub] simoncorstonoliver commented on issue #9045: Remove defunct NLP tutorial page

2017-12-12 Thread GitBox
simoncorstonoliver commented on issue #9045: Remove defunct NLP tutorial page URL: https://github.com/apache/incubator-mxnet/pull/9045#issuecomment-351234245 RNNs are covered at length in the Straight Dope book. I don't think we need to write another tutorial. -

[GitHub] larroy commented on a change in pull request #9035: [WIP] Build improvements, for users and for Jenkins

2017-12-12 Thread GitBox
larroy commented on a change in pull request #9035: [WIP] Build improvements, for users and for Jenkins URL: https://github.com/apache/incubator-mxnet/pull/9035#discussion_r156529917 ## File path: build.py ## @@ -0,0 +1,167 @@ +#!/usr/bin/env python +# -*- coding: utf-8 -*

[GitHub] eric-haibin-lin commented on issue #9042: Avoiding weight sharing among certain layers in BucketingModule?

2017-12-12 Thread GitBox
eric-haibin-lin commented on issue #9042: Avoiding weight sharing among certain layers in BucketingModule? URL: https://github.com/apache/incubator-mxnet/issues/9042#issuecomment-351233681 You can make `sym_gen(default_bucket_key)` returns a "master network" that contains all these FC lay

[GitHub] cjolivier01 commented on a change in pull request #8972: Profiling enhancements, python API, vtune and chrome tracing objects, etc.

2017-12-12 Thread GitBox
cjolivier01 commented on a change in pull request #8972: Profiling enhancements, python API, vtune and chrome tracing objects, etc. URL: https://github.com/apache/incubator-mxnet/pull/8972#discussion_r156529905 ## File path: include/mxnet/c_api.h ## @@ -227,10 +227,138 @@

[GitHub] pracheer commented on issue #9045: Remove defunct NLP tutorial page

2017-12-12 Thread GitBox
pracheer commented on issue #9045: Remove defunct NLP tutorial page URL: https://github.com/apache/incubator-mxnet/pull/9045#issuecomment-351233712 Would it make more sense to write a basic tutorial on rnns using MXNet as part of this PR itself?

[GitHub] eric-haibin-lin commented on issue #9042: Avoiding weight sharing among certain layers in BucketingModule?

2017-12-12 Thread GitBox
eric-haibin-lin commented on issue #9042: Avoiding weight sharing among certain layers in BucketingModule? URL: https://github.com/apache/incubator-mxnet/issues/9042#issuecomment-351233681 You can make `sym_gen(default_bucket_key)` returns a "master network" that contains all these FC lay

[GitHub] eric-haibin-lin commented on issue #9042: Avoiding weight sharing among certain layers in BucketingModule?

2017-12-12 Thread GitBox
eric-haibin-lin commented on issue #9042: Avoiding weight sharing among certain layers in BucketingModule? URL: https://github.com/apache/incubator-mxnet/issues/9042#issuecomment-351233681 You can make `sym_gen(default_bucket_key)` returns a "master network" that contains both these two F

[GitHub] simoncorstonoliver opened a new pull request #9045: Remove defunct NLP tutorial page

2017-12-12 Thread GitBox
simoncorstonoliver opened a new pull request #9045: Remove defunct NLP tutorial page URL: https://github.com/apache/incubator-mxnet/pull/9045 ## Description ## The tutorial page that purports to be about RNNs has no content apart from two broken links to source code. Also the tut

[GitHub] szha closed pull request #9043: Adding .png to the list of allowed extensions in the im2rec.py

2017-12-12 Thread GitBox
szha closed pull request #9043: Adding .png to the list of allowed extensions in the im2rec.py URL: https://github.com/apache/incubator-mxnet/pull/9043 This is an automated message from the Apache Git Service. To respond to

[incubator-mxnet] branch piiswrong-patch-2 deleted (was 4c1f05b)

2017-12-12 Thread zhasheng
This is an automated email from the ASF dual-hosted git repository. zhasheng pushed a change to branch piiswrong-patch-2 in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. was 4c1f05b Adding .png to the list of allowed extensions in the im2rec.py The revisions that wer

[incubator-mxnet] branch master updated: Adding .png to the list of allowed extensions in the im2rec.py (#9043)

2017-12-12 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 087a906 Adding .png to the list of a

[GitHub] Laurawly commented on a change in pull request #8915: NVLink communication pattern updated

2017-12-12 Thread GitBox
Laurawly commented on a change in pull request #8915: NVLink communication pattern updated URL: https://github.com/apache/incubator-mxnet/pull/8915#discussion_r156526811 ## File path: src/kvstore/comm.h ## @@ -670,31 +715,75 @@ class CommDevice : public Comm { using Ke

[GitHub] szha commented on a change in pull request #9016: Add googletest as a 3rdparty library

2017-12-12 Thread GitBox
szha commented on a change in pull request #9016: Add googletest as a 3rdparty library URL: https://github.com/apache/incubator-mxnet/pull/9016#discussion_r156526353 ## File path: .gitmodules ## @@ -1,9 +1,9 @@ [submodule "mshadow"] path = mshadow - url = ht

[GitHub] chsin opened a new issue #9044: [scala] run_gan_mnist.sh fails

2017-12-12 Thread GitBox
chsin opened a new issue #9044: [scala] run_gan_mnist.sh fails URL: https://github.com/apache/incubator-mxnet/issues/9044 ## Description Was able to built libmxnet.so, and successfully build the Scala package, but run_gan_mnist.sh fails. ## Environment info (Required) I had to

[GitHub] larroy commented on issue #9016: Add googletest as a 3rdparty library

2017-12-12 Thread GitBox
larroy commented on issue #9016: Add googletest as a 3rdparty library URL: https://github.com/apache/incubator-mxnet/pull/9016#issuecomment-351226080 We discussed in the list to add it as a third party The library is very small, why is it better to have it as a submodule? -

[GitHub] indhub commented on issue #9033: Reorganize the tutorials index page

2017-12-12 Thread GitBox
indhub commented on issue #9033: Reorganize the tutorials index page URL: https://github.com/apache/incubator-mxnet/pull/9033#issuecomment-351224919 @larroy Why is the change to jenkinsfile required? This is an automated mess

[GitHub] zhreshold commented on a change in pull request #8547: Revised validating process of a bounding box in ImageDetIter

2017-12-12 Thread GitBox
zhreshold commented on a change in pull request #8547: Revised validating process of a bounding box in ImageDetIter URL: https://github.com/apache/incubator-mxnet/pull/8547#discussion_r156521729 ## File path: python/mxnet/image/detection.py ## @@ -726,10 +726,23 @@ def _pa

[GitHub] zhreshold commented on a change in pull request #8547: Revised validating process of a bounding box in ImageDetIter

2017-12-12 Thread GitBox
zhreshold commented on a change in pull request #8547: Revised validating process of a bounding box in ImageDetIter URL: https://github.com/apache/incubator-mxnet/pull/8547#discussion_r156521772 ## File path: python/mxnet/image/detection.py ## @@ -726,10 +726,23 @@ def _pa

[GitHub] zhreshold commented on a change in pull request #8547: Revised validating process of a bounding box in ImageDetIter

2017-12-12 Thread GitBox
zhreshold commented on a change in pull request #8547: Revised validating process of a bounding box in ImageDetIter URL: https://github.com/apache/incubator-mxnet/pull/8547#discussion_r156522021 ## File path: python/mxnet/image/detection.py ## @@ -726,10 +726,23 @@ def _pa

[GitHub] indhub commented on issue #9033: Reorganize the tutorials index page

2017-12-12 Thread GitBox
indhub commented on issue #9033: Reorganize the tutorials index page URL: https://github.com/apache/incubator-mxnet/pull/9033#issuecomment-351224605 @madjam This is an automated message from the Apache Git Service. To respond

[GitHub] zheng-da commented on a change in pull request #8302: Refactor operators & MKLDNN

2017-12-12 Thread GitBox
zheng-da commented on a change in pull request #8302: Refactor operators & MKLDNN URL: https://github.com/apache/incubator-mxnet/pull/8302#discussion_r156520457 ## File path: src/ndarray/ndarray.cc ## @@ -64,17 +158,147 @@ nnvm::Symbol NDArray::get_autograd_symbol() const

[GitHub] zheng-da commented on a change in pull request #8302: Refactor operators & MKLDNN

2017-12-12 Thread GitBox
zheng-da commented on a change in pull request #8302: Refactor operators & MKLDNN URL: https://github.com/apache/incubator-mxnet/pull/8302#discussion_r156519807 ## File path: src/ndarray/ndarray.cc ## @@ -64,17 +158,147 @@ nnvm::Symbol NDArray::get_autograd_symbol() const

[GitHub] piiswrong commented on a change in pull request #8972: Profiling enhancements, python API, vtune and chrome tracing objects, etc.

2017-12-12 Thread GitBox
piiswrong commented on a change in pull request #8972: Profiling enhancements, python API, vtune and chrome tracing objects, etc. URL: https://github.com/apache/incubator-mxnet/pull/8972#discussion_r156519270 ## File path: python/mxnet/profiler.py ## @@ -56,3 +53,235 @@ de

[GitHub] zheng-da commented on a change in pull request #8302: Refactor operators & MKLDNN

2017-12-12 Thread GitBox
zheng-da commented on a change in pull request #8302: Refactor operators & MKLDNN URL: https://github.com/apache/incubator-mxnet/pull/8302#discussion_r156519245 ## File path: src/ndarray/ndarray.cc ## @@ -64,17 +158,147 @@ nnvm::Symbol NDArray::get_autograd_symbol() const

[GitHub] piiswrong commented on a change in pull request #8972: Profiling enhancements, python API, vtune and chrome tracing objects, etc.

2017-12-12 Thread GitBox
piiswrong commented on a change in pull request #8972: Profiling enhancements, python API, vtune and chrome tracing objects, etc. URL: https://github.com/apache/incubator-mxnet/pull/8972#discussion_r156519228 ## File path: include/mxnet/c_api.h ## @@ -227,10 +227,138 @@ MX

[incubator-mxnet] branch master updated: fix small memory leak of sparse embedding (#9025)

2017-12-12 Thread jxie
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 167871a fix small memory leak of sparse

[GitHub] piiswrong closed pull request #9025: fix small memory leak of sparse embedding

2017-12-12 Thread GitBox
piiswrong closed pull request #9025: fix small memory leak of sparse embedding URL: https://github.com/apache/incubator-mxnet/pull/9025 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a

[GitHub] piiswrong commented on issue #9040: add zero-grad for rounding ops

2017-12-12 Thread GitBox
piiswrong commented on issue #9040: add zero-grad for rounding ops URL: https://github.com/apache/incubator-mxnet/pull/9040#issuecomment-351221009 Are these zero for other frameworks? This is an automated message from the Apac

[GitHub] marcoabreu commented on issue #9001: Disable test_optimizers:test_sgd (Temp fix #9000)

2017-12-12 Thread GitBox
marcoabreu commented on issue #9001: Disable test_optimizers:test_sgd (Temp fix #9000) URL: https://github.com/apache/incubator-mxnet/pull/9001#issuecomment-351220891 Rebuilding This is an automated message from the Apache Gi

[GitHub] piiswrong closed pull request #9021: Remove redundant or unused imports

2017-12-12 Thread GitBox
piiswrong closed pull request #9021: Remove redundant or unused imports URL: https://github.com/apache/incubator-mxnet/pull/9021 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

[incubator-mxnet] branch master updated: Remove redundant or unused imports (#9021)

2017-12-12 Thread jxie
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 db15049 Remove redundant or unused impor

  1   2   3   >