[GitHub] indhub commented on issue #9033: Rename the section titled 'MXNet' in tutorials page to 'Low Level Interface'

2017-12-12 Thread GitBox
indhub commented on issue #9033: Rename the section titled 'MXNet' in tutorials page to 'Low Level Interface' URL: https://github.com/apache/incubator-mxnet/pull/9033#issuecomment-351163010 @larroy Website is created from 1.0.0. This needs to be fixed in 1.0.0 to fix the website. Please

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

2017-12-12 Thread GitBox
eric-haibin-lin 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_r156471898 ## File path: src/operator/tensor/dot-inl.h ## @@ -811,6 +891,94 @@ inline void

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

2017-12-12 Thread GitBox
eric-haibin-lin 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_r156473118 ## File path: src/operator/tensor/dot-inl.h ## @@ -228,8 +231,16 @@ inline bool

[GitHub] leleamol commented on issue #8957: Adding the steps to build MXNet with NCCL

2017-12-12 Thread GitBox
leleamol commented on issue #8957: Adding the steps to build MXNet with NCCL URL: https://github.com/apache/incubator-mxnet/pull/8957#issuecomment-351174177 @ptrendx I have added the "Passed" statement in the test_nccl.py

[GitHub] pracheer commented on a change in pull request #9033: Rename the section titled 'MXNet' in tutorials page to 'Low Level Interface'

2017-12-12 Thread GitBox
pracheer commented on a change in pull request #9033: Rename the section titled 'MXNet' in tutorials page to 'Low Level Interface' URL: https://github.com/apache/incubator-mxnet/pull/9033#discussion_r156483296 ## File path: docs/tutorials/index.md ## @@ -27,9 +27,9 @@

[GitHub] larroy commented on issue #9035: [WIP] Build improvements, for users and for Jenkins

2017-12-12 Thread GitBox
larroy commented on issue #9035: [WIP] Build improvements, for users and for Jenkins URL: https://github.com/apache/incubator-mxnet/pull/9035#issuecomment-351130522 I actually was thinking of having a build.json file with all the flags that can be easily modified.

[GitHub] jerrin92 commented on issue #8874: mxnet installation from source: C++ linkage error on HPC

2017-12-12 Thread GitBox
jerrin92 commented on issue #8874: mxnet installation from source: C++ linkage error on HPC URL: https://github.com/apache/incubator-mxnet/issues/8874#issuecomment-351112576 Copied below is the config.mk

[GitHub] jerrin92 commented on issue #8874: mxnet installation from source: C++ linkage error on HPC

2017-12-12 Thread GitBox
jerrin92 commented on issue #8874: mxnet installation from source: C++ linkage error on HPC URL: https://github.com/apache/incubator-mxnet/issues/8874#issuecomment-351112576 ``` Copied below is the config.mk

[GitHub] marcoabreu commented on issue #9035: [WIP] Build improvements, for users and for Jenkins

2017-12-12 Thread GitBox
marcoabreu commented on issue #9035: [WIP] Build improvements, for users and for Jenkins URL: https://github.com/apache/incubator-mxnet/pull/9035#issuecomment-351122282 I would like to keep the parameters in the jenkinsfile. Please make it possible that they can be passed into your python

[GitHub] larroy commented on issue #9035: [WIP] Build improvements, for users and for Jenkins

2017-12-12 Thread GitBox
larroy commented on issue #9035: [WIP] Build improvements, for users and for Jenkins URL: https://github.com/apache/incubator-mxnet/pull/9035#issuecomment-351128643 Why do you want to keep the parameters in the Jenkinsfile? From what I have seen in the past from great test engineers it

[GitHub] marcoabreu commented on issue #8919: [Vision] add test cases for flip, normalize, to_tensor

2017-12-12 Thread GitBox
marcoabreu commented on issue #8919: [Vision] add test cases for flip, normalize, to_tensor URL: https://github.com/apache/incubator-mxnet/pull/8919#issuecomment-351151366 Hello @javelinjs, please rebase your PR. This is an

[GitHub] ZiyueHuang commented on a change in pull request #9025: fix small memory leak of sparse embedding

2017-12-12 Thread GitBox
ZiyueHuang commented on a change in pull request #9025: fix small memory leak of sparse embedding URL: https://github.com/apache/incubator-mxnet/pull/9025#discussion_r156447279 ## File path: src/operator/tensor/indexing_op.cu ## @@ -91,6 +91,7 @@ void

[GitHub] javelinjs commented on issue #8919: [Vision] add test cases for flip, normalize, to_tensor

2017-12-12 Thread GitBox
javelinjs commented on issue #8919: [Vision] add test cases for flip, normalize, to_tensor URL: https://github.com/apache/incubator-mxnet/pull/8919#issuecomment-351152519 @marcoabreu You mean rebase from master? Is there anything related to these operators?

[GitHub] larroy opened a new pull request #9035: [WIP] Build improvements, for users and for Jenkins

2017-12-12 Thread GitBox
larroy opened a new pull request #9035: [WIP] Build improvements, for users and for Jenkins URL: https://github.com/apache/incubator-mxnet/pull/9035 ## Description ## Provide an easy to use build.py script to aglutinate coplex build logic which works on multiple platforms and does a

[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-351118175 @cjolivier01 right, it was voted and passed in the list, didn't you see?

[GitHub] larroy commented on issue #9035: [WIP] Build improvements, for users and for Jenkins

2017-12-12 Thread GitBox
larroy commented on issue #9035: [WIP] Build improvements, for users and for Jenkins URL: https://github.com/apache/incubator-mxnet/pull/9035#issuecomment-351120512 @marcoabreu @mbaijal @KellenSunderland This is an

[GitHub] larroy commented on issue #8872: Minor refactor: prevent string copying, list -> vector, shared_ptr by?

2017-12-12 Thread GitBox
larroy commented on issue #8872: Minor refactor: prevent string copying, list -> vector, shared_ptr by? URL: https://github.com/apache/incubator-mxnet/pull/8872#issuecomment-351120257 @piiswrong please merge? This is an

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

2017-12-12 Thread GitBox
rahul003 commented on issue #9001: Disable test_optimizers:test_sgd (Temp fix #9000) URL: https://github.com/apache/incubator-mxnet/pull/9001#issuecomment-351143893 Please consider merging. I'm running into this test failure for 3 of my PRs

[GitHub] marcoabreu commented on issue #8919: [Vision] add test cases for flip, normalize, to_tensor

2017-12-12 Thread GitBox
marcoabreu commented on issue #8919: [Vision] add test cases for flip, normalize, to_tensor URL: https://github.com/apache/incubator-mxnet/pull/8919#issuecomment-351153571 Ah my bad, I didn't see you're merging into a different branch. Nevermind :)

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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-351167375 @indhub This is an automated message from the Apache Git Service. To respond to the

[GitHub] rahul003 opened a new pull request #9037: Coverity fixes

2017-12-12 Thread GitBox
rahul003 opened a new pull request #9037: Coverity fixes URL: https://github.com/apache/incubator-mxnet/pull/9037 ## Description ## Fixing errors identified by static analysis tool Coverity. - Fixes memory leaks - Sets value to uninitialized variable - Adds a default case to

[GitHub] larroy commented on issue #9033: Rename the section titled 'MXNet' in tutorials page to 'Low Level Interface'

2017-12-12 Thread GitBox
larroy commented on issue #9033: Rename the section titled 'MXNet' in tutorials page to 'Low Level Interface' URL: https://github.com/apache/incubator-mxnet/pull/9033#issuecomment-351176655 Then you need to fix the jenkinsfile, a hyphen is missing, check diff against master.

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

[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

[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

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

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

[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

[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

[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

[incubator-mxnet] branch master updated: Update for MXNet 1.0. PEP8 fixes to code and misc improvements. Tested under Python 2.7 and Python 3.6 on Sagemaker. (#9036)

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 5284513 Update for MXNet 1.0. PEP8

[GitHub] eric-haibin-lin opened a new pull request #9038: Add truncated bptt RNN example using symbol & module API

2017-12-12 Thread GitBox
eric-haibin-lin opened a new pull request #9038: Add truncated bptt RNN example using symbol & module API URL: https://github.com/apache/incubator-mxnet/pull/9038 ## Description ## @szha @sxjscience I added the global_clip_norm functionality to a custom module wrapper in this

[GitHub] szha opened a new pull request #9039: update autograd.Function docstring for usage

2017-12-12 Thread GitBox
szha opened a new pull request #9039: update autograd.Function docstring for usage URL: https://github.com/apache/incubator-mxnet/pull/9039 ## Description ## Update docstring for autograd.Function to include usage. ## Checklist ## ### Essentials ### - [x] Passed code style

[GitHub] pracheer commented on a change in pull request #9030: Fix Gan

2017-12-12 Thread GitBox
pracheer commented on a change in pull request #9030: Fix Gan URL: https://github.com/apache/incubator-mxnet/pull/9030#discussion_r156484820 ## File path: docs/tutorials/unsupervised_learning/gan.md ## @@ -1,43 +1,43 @@ -# Generative Adversarial Networks +# Generative

[GitHub] pracheer commented on a change in pull request #9030: Fix Gan

2017-12-12 Thread GitBox
pracheer commented on a change in pull request #9030: Fix Gan URL: https://github.com/apache/incubator-mxnet/pull/9030#discussion_r156491258 ## File path: docs/tutorials/unsupervised_learning/gan.md ## @@ -86,25 +86,25 @@ Since the DCGAN that we're creating takes in a

[GitHub] pracheer commented on a change in pull request #9030: Fix Gan

2017-12-12 Thread GitBox
pracheer commented on a change in pull request #9030: Fix Gan URL: https://github.com/apache/incubator-mxnet/pull/9030#discussion_r156484051 ## File path: docs/tutorials/unsupervised_learning/gan.md ## @@ -1,43 +1,43 @@ -# Generative Adversarial Networks +# Generative

[GitHub] pracheer commented on a change in pull request #9030: Fix Gan

2017-12-12 Thread GitBox
pracheer commented on a change in pull request #9030: Fix Gan URL: https://github.com/apache/incubator-mxnet/pull/9030#discussion_r156492396 ## File path: docs/tutorials/unsupervised_learning/gan.md ## @@ -86,25 +86,25 @@ Since the DCGAN that we're creating takes in a

[GitHub] pracheer commented on a change in pull request #9030: Fix Gan

2017-12-12 Thread GitBox
pracheer commented on a change in pull request #9030: Fix Gan URL: https://github.com/apache/incubator-mxnet/pull/9030#discussion_r156490895 ## File path: docs/tutorials/unsupervised_learning/gan.md ## @@ -86,25 +86,25 @@ Since the DCGAN that we're creating takes in a

[GitHub] pracheer commented on a change in pull request #9030: Fix Gan

2017-12-12 Thread GitBox
pracheer commented on a change in pull request #9030: Fix Gan URL: https://github.com/apache/incubator-mxnet/pull/9030#discussion_r156488669 ## File path: docs/tutorials/unsupervised_learning/gan.md ## @@ -47,31 +47,31 @@ To complete this tutorial, you need: - Python 2.7,

[GitHub] pracheer commented on a change in pull request #9030: Fix Gan

2017-12-12 Thread GitBox
pracheer commented on a change in pull request #9030: Fix Gan URL: https://github.com/apache/incubator-mxnet/pull/9030#discussion_r156488038 ## File path: docs/tutorials/unsupervised_learning/gan.md ## @@ -1,43 +1,43 @@ -# Generative Adversarial Networks +# Generative

[GitHub] pracheer commented on a change in pull request #9027: Usability improvement for cnn_chinese_text_classification

2017-12-12 Thread GitBox
pracheer commented on a change in pull request #9027: Usability improvement for cnn_chinese_text_classification URL: https://github.com/apache/incubator-mxnet/pull/9027#discussion_r156494304 ## File path: example/cnn_chinese_text_classification/text_cnn.py ## @@ -20,20

[GitHub] larroy commented on issue #8874: mxnet installation from source: C++ linkage error on HPC

2017-12-12 Thread GitBox
larroy commented on issue #8874: mxnet installation from source: C++ linkage error on HPC URL: https://github.com/apache/incubator-mxnet/issues/8874#issuecomment-351193968 more compiler error context is required, which .cc file is failing to build? please paste more output.

[GitHub] sxjscience commented on issue #9038: Add truncated bptt RNN example using symbol & module API

2017-12-12 Thread GitBox
sxjscience commented on issue #9038: Add truncated bptt RNN example using symbol & module API URL: https://github.com/apache/incubator-mxnet/pull/9038#issuecomment-351195136 I think adding this wrapper for the PTB example is a good way. This avoids the complexities of changing the APIs in

[GitHub] sxjscience commented on issue #9038: Add truncated bptt RNN example using symbol & module API

2017-12-12 Thread GitBox
sxjscience commented on issue #9038: Add truncated bptt RNN example using symbol & module API URL: https://github.com/apache/incubator-mxnet/pull/9038#issuecomment-351195136 I think adding a new wrapper for the PTB example is a good way. This avoids the complexities of changing the APIs

[GitHub] pracheer commented on a change in pull request #9021: Remove redundant or unused imports

2017-12-12 Thread GitBox
pracheer commented on a change in pull request #9021: Remove redundant or unused imports URL: https://github.com/apache/incubator-mxnet/pull/9021#discussion_r156496761 ## File path: tests/python/unittest/test_ndarray.py ## @@ -15,15 +15,11 @@ # specific language

[GitHub] indhub commented on a change in pull request #9033: Rename the section titled 'MXNet' in tutorials page to 'Low Level Interface'

2017-12-12 Thread GitBox
indhub commented on a change in pull request #9033: Rename the section titled 'MXNet' in tutorials page to 'Low Level Interface' URL: https://github.com/apache/incubator-mxnet/pull/9033#discussion_r156497783 ## File path: docs/tutorials/index.md ## @@ -27,9 +27,9 @@ This

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

2017-12-12 Thread GitBox
piiswrong 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_r156499301 ## File path: Jenkinsfile ## @@ -228,31 +228,10 @@ try { 'Build CPU windows':{

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

2017-12-12 Thread GitBox
piiswrong 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_r156499753 ## File path: build.py ## @@ -0,0 +1,167 @@ +#!/usr/bin/env python +# -*- coding:

[GitHub] piiswrong closed pull request #8919: [Vision] add test cases for flip, normalize, to_tensor

2017-12-12 Thread GitBox
piiswrong closed pull request #8919: [Vision] add test cases for flip, normalize, to_tensor URL: https://github.com/apache/incubator-mxnet/pull/8919 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:

[incubator-mxnet] branch vision updated: [Vision] add test cases for flip, normalize, to_tensor (#8919)

2017-12-12 Thread jxie
This is an automated email from the ASF dual-hosted git repository. jxie pushed a commit to branch vision in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git The following commit(s) were added to refs/heads/vision by this push: new 7c92b98 [Vision] add test cases for

[GitHub] astonzhang commented on a change in pull request #9021: Remove redundant or unused imports

2017-12-12 Thread GitBox
astonzhang commented on a change in pull request #9021: Remove redundant or unused imports URL: https://github.com/apache/incubator-mxnet/pull/9021#discussion_r156500452 ## File path: tests/python/unittest/test_ndarray.py ## @@ -15,15 +15,11 @@ # specific language

[GitHub] szha opened a new pull request #9040: add zero-grad for rounding ops

2017-12-12 Thread GitBox
szha opened a new pull request #9040: add zero-grad for rounding ops URL: https://github.com/apache/incubator-mxnet/pull/9040 ## Description ## add zero gradients to rounding operators including 'rint', 'ceil', 'floor', 'trunc', and 'fix'. ## Checklist ## ### Essentials ###

[GitHub] eric-haibin-lin opened a new issue #9041: Scala API doc doesn't show available operators

2017-12-12 Thread GitBox
eric-haibin-lin opened a new issue #9041: Scala API doc doesn't show available operators URL: https://github.com/apache/incubator-mxnet/issues/9041 As I browse the scala [API doc](https://mxnet.incubator.apache.org/api/scala/docs/index.html#ml.dmlc.mxnet.NDArray), I don't see any

[GitHub] piiswrong closed pull request #9038: Add truncated bptt RNN example using symbol & module API

2017-12-12 Thread GitBox
piiswrong closed pull request #9038: Add truncated bptt RNN example using symbol & module API URL: https://github.com/apache/incubator-mxnet/pull/9038 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

[incubator-mxnet] branch master updated: Add truncated bptt RNN example using symbol & module API (#9038)

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 06ed4dc Add truncated bptt RNN example

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

2017-12-12 Thread GitBox
piiswrong commented on issue #9016: Add googletest as a 3rdparty library URL: https://github.com/apache/incubator-mxnet/pull/9016#issuecomment-351204543 please add a submodule. Don't copy paste the code This is an automated

[incubator-mxnet] branch master updated: Use backtrace from dmlc-core with demangled C++ symbols on segfault (#8851)

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 3901f5f Use backtrace from dmlc-core

[GitHub] piiswrong closed pull request #8851: Use backtrace from dmlc-core (dmlc::stack_trace())

2017-12-12 Thread GitBox
piiswrong closed pull request #8851: Use backtrace from dmlc-core (dmlc::stack_trace()) URL: https://github.com/apache/incubator-mxnet/pull/8851 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] madjam commented on a change in pull request #9008: Usability improvement mxnet vae

2017-12-12 Thread GitBox
madjam commented on a change in pull request #9008: Usability improvement mxnet vae URL: https://github.com/apache/incubator-mxnet/pull/9008#discussion_r156503316 ## File path: example/mxnet_adversarial_vae/README.md ## @@ -1,22 +1,55 @@ -VAE-GAN in MXNet +# VAE-GAN in

[GitHub] madjam commented on a change in pull request #9008: Usability improvement mxnet vae

2017-12-12 Thread GitBox
madjam commented on a change in pull request #9008: Usability improvement mxnet vae URL: https://github.com/apache/incubator-mxnet/pull/9008#discussion_r156504457 ## File path: example/mxnet_adversarial_vae/README.md ## @@ -1,22 +1,55 @@ -VAE-GAN in MXNet +# VAE-GAN in

[GitHub] madjam commented on a change in pull request #9008: Usability improvement mxnet vae

2017-12-12 Thread GitBox
madjam commented on a change in pull request #9008: Usability improvement mxnet vae URL: https://github.com/apache/incubator-mxnet/pull/9008#discussion_r156504507 ## File path: example/mxnet_adversarial_vae/README.md ## @@ -1,22 +1,55 @@ -VAE-GAN in MXNet +# VAE-GAN in

[GitHub] madjam commented on a change in pull request #9008: Usability improvement mxnet vae

2017-12-12 Thread GitBox
madjam commented on a change in pull request #9008: Usability improvement mxnet vae URL: https://github.com/apache/incubator-mxnet/pull/9008#discussion_r156504706 ## File path: example/mxnet_adversarial_vae/README.md ## @@ -1,22 +1,55 @@ -VAE-GAN in MXNet +# VAE-GAN in

[GitHub] madjam commented on a change in pull request #9008: Usability improvement mxnet vae

2017-12-12 Thread GitBox
madjam commented on a change in pull request #9008: Usability improvement mxnet vae URL: https://github.com/apache/incubator-mxnet/pull/9008#discussion_r156502867 ## File path: example/mxnet_adversarial_vae/README.md ## @@ -1,22 +1,55 @@ -VAE-GAN in MXNet +# VAE-GAN in

[GitHub] eric-haibin-lin commented on issue #9020: pooling of dcnn

2017-12-12 Thread GitBox
eric-haibin-lin commented on issue #9020: pooling of dcnn URL: https://github.com/apache/incubator-mxnet/issues/9020#issuecomment-351206480 Agreed that the original paper uses global pooling. Was it a bug in the example or it was by any chance "intentional simplification"? @qcl6355

[GitHub] piiswrong commented on issue #7856: add missing cuda libs to LDFLAGS

2017-12-12 Thread GitBox
piiswrong commented on issue #7856: add missing cuda libs to LDFLAGS URL: https://github.com/apache/incubator-mxnet/pull/7856#issuecomment-351207174 closing as this doesn't seem to be necessary This is an automated message

[GitHub] piiswrong closed pull request #7856: add missing cuda libs to LDFLAGS

2017-12-12 Thread GitBox
piiswrong closed pull request #7856: add missing cuda libs to LDFLAGS URL: https://github.com/apache/incubator-mxnet/pull/7856 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] piiswrong commented on issue #6496: Add hinge loss for svm

2017-12-12 Thread GitBox
piiswrong commented on issue #6496: Add hinge loss for svm URL: https://github.com/apache/incubator-mxnet/pull/6496#issuecomment-351207393 closing due to inactive. Please reopen after addressing comments This is an automated

[GitHub] piiswrong closed pull request #6496: Add hinge loss for svm

2017-12-12 Thread GitBox
piiswrong closed pull request #6496: Add hinge loss for svm URL: https://github.com/apache/incubator-mxnet/pull/6496 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull

[GitHub] piiswrong commented on issue #7938: instance norm and reflection padding

2017-12-12 Thread GitBox
piiswrong commented on issue #7938: instance norm and reflection padding URL: https://github.com/apache/incubator-mxnet/pull/7938#issuecomment-351207795 @szha The original author seems to have disappeared. Would you like to take over?

[GitHub] piiswrong commented on issue #7898: Stop the build if there are Python syntax errors

2017-12-12 Thread GitBox
piiswrong commented on issue #7898: Stop the build if there are Python syntax errors URL: https://github.com/apache/incubator-mxnet/pull/7898#issuecomment-351207891 closing since travis is not used anymore This is an

[GitHub] piiswrong commented on issue #7984: Update parse_log.py

2017-12-12 Thread GitBox
piiswrong commented on issue #7984: Update parse_log.py URL: https://github.com/apache/incubator-mxnet/pull/7984#issuecomment-351208407 @mli Could you have a look? This is an automated message from the Apache Git Service. To

[GitHub] piiswrong closed pull request #7984: Update parse_log.py

2017-12-12 Thread GitBox
piiswrong closed pull request #7984: Update parse_log.py URL: https://github.com/apache/incubator-mxnet/pull/7984 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

[GitHub] piiswrong commented on issue #8027: Optional reshape of predictions in Perplexity metric

2017-12-12 Thread GitBox
piiswrong commented on issue #8027: Optional reshape of predictions in Perplexity metric URL: https://github.com/apache/incubator-mxnet/pull/8027#issuecomment-351208732 @reminisce Could you take over? I think @zackchase complained about metric speed

[GitHub] piiswrong closed pull request #7944: Update softmax_output.cc

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

[GitHub] piiswrong commented on issue #7944: Update softmax_output.cc

2017-12-12 Thread GitBox
piiswrong commented on issue #7944: Update softmax_output.cc URL: https://github.com/apache/incubator-mxnet/pull/7944#issuecomment-351208883 closing due to inactive This is an automated message from the Apache Git Service. To

[GitHub] piiswrong commented on issue #7809: open dlpack in windows

2017-12-12 Thread GitBox
piiswrong commented on issue #7809: open dlpack in windows URL: https://github.com/apache/incubator-mxnet/pull/7809#issuecomment-351208946 closing due to inactive This is an automated message from the Apache Git Service. To

[GitHub] piiswrong closed pull request #7809: open dlpack in windows

2017-12-12 Thread GitBox
piiswrong closed pull request #7809: open dlpack in windows URL: https://github.com/apache/incubator-mxnet/pull/7809 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull

[GitHub] piiswrong closed pull request #6586: Implementation for concat relu operator

2017-12-12 Thread GitBox
piiswrong closed pull request #6586: Implementation for concat relu operator URL: https://github.com/apache/incubator-mxnet/pull/6586 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 #6586: Implementation for concat relu operator

2017-12-12 Thread GitBox
piiswrong commented on issue #6586: Implementation for concat relu operator URL: https://github.com/apache/incubator-mxnet/pull/6586#issuecomment-351209237 closing due to inactive This is an automated message from the Apache

[GitHub] piiswrong closed pull request #7772: Use memcopy instead of assigning each individual element

2017-12-12 Thread GitBox
piiswrong closed pull request #7772: Use memcopy instead of assigning each individual element URL: https://github.com/apache/incubator-mxnet/pull/7772 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of

[GitHub] 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_r156508822 ## File path: src/ndarray/ndarray.cc ## @@ -95,12 +319,36 @@ NDArray NDArray::ReshapeWithRecord(const TShape )

[GitHub] arshitgupta opened a new issue #9042: Avoiding weight sharing among certain layers in BucketingModule?

2017-12-12 Thread GitBox
arshitgupta opened a new issue #9042: Avoiding weight sharing among certain layers in BucketingModule? URL: https://github.com/apache/incubator-mxnet/issues/9042 I am using BucketingModule for training multiple small models/bots together. Here, the bucket key is bot_id. However, each bot

[GitHub] piiswrong commented on issue #8161: Creating sim-link to raw1394 file

2017-12-12 Thread GitBox
piiswrong commented on issue #8161: Creating sim-link to raw1394 file URL: https://github.com/apache/incubator-mxnet/pull/8161#issuecomment-351210144 closing due to outdated This is an automated message from the Apache Git

[GitHub] piiswrong closed pull request #8161: Creating sim-link to raw1394 file

2017-12-12 Thread GitBox
piiswrong closed pull request #8161: Creating sim-link to raw1394 file URL: https://github.com/apache/incubator-mxnet/pull/8161 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] 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_r156509390 ## File path: src/operator/nn/activation-inl.h ## @@ -61,158 +62,127 @@ struct ActivationParam : public

[GitHub] piiswrong commented on issue #8070: modify shell file of classification for gluon

2017-12-12 Thread GitBox
piiswrong commented on issue #8070: modify shell file of classification for gluon URL: https://github.com/apache/incubator-mxnet/pull/8070#issuecomment-351210443 closing due to outdated. @zhreshold Please take over if you think this is still relevant.

[GitHub] piiswrong closed pull request #8070: modify shell file of classification for gluon

2017-12-12 Thread GitBox
piiswrong closed pull request #8070: modify shell file of classification for gluon URL: https://github.com/apache/incubator-mxnet/pull/8070 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

[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_r156509390 ## File path: src/operator/nn/activation-inl.h ## @@ -61,158 +62,127 @@ struct ActivationParam : public

[GitHub] piiswrong commented on issue #7781: Implement Khatri-Rao operator

2017-12-12 Thread GitBox
piiswrong commented on issue #7781: Implement Khatri-Rao operator URL: https://github.com/apache/incubator-mxnet/pull/7781#issuecomment-351210702 @cswiercz Could you rebase to current master? This is an automated message from

  1   2   3   >