[GitHub] anirudh2290 commented on issue #9681: Better Exception Handling for Operators

[GitBox]

anirudh2290 commented on issue #9681: Better Exception Handling for Operators
URL: https://github.com/apache/incubator-mxnet/pull/9681#issuecomment-364031160
 
 
   After discussion with @piiswrong , we came to a conclusion that in an 
execution graph once an exception is thrown, the same exception should not be 
thrown again. For example:
   
   ```
   x = None
   y = None
   try:
x, y = op1() # Fails
x.asnumpy() # Throw exception
   except:
handle_exc(x, y)
   y.asnumpy() # Should execute fine
   z = op2(x) # Should execute fine, expectation is that user modified the 
value of x from garbage, when handling exception
   ```
   
   You can see that op1 throws an exception and it may end up writing garbage 
values to x and y.  The line x.asnumpy() throws exception. Once this is done, 
user may handle the exception or keep the garbage values as it is. Any 
consequent usage of x or op2 should not throw the same exception, since user is 
not expecting and it is already handled.
   
   One challenge during the implementation was that dereferencing exception_ptr 
in C++ will cause undefined behavior. So there is no way to modify state of the 
exception_object that exception_ptr points to, just by using exception_ptr 
itself. To workaround this limitation, we are holding the exception_ptr itself 
in a shared_ptr object. 
   
   We decided to remove global_exception thrown in WaitForAll, since it adds 
unnecessary complexity and is not really used much except during benchmarking.


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[GitHub] freshcoderman commented on issue #9271: Windows: Python prompt gets stuck on exit()

[GitBox]

freshcoderman commented on issue #9271: Windows: Python prompt gets stuck on 
exit()
URL: 
https://github.com/apache/incubator-mxnet/issues/9271#issuecomment-364029154
 
 
   +1
   same issue on Windows7, running mxnet '1.0.0' with both python2 and python3 
as long as I do "import mxnet as mx"
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[GitHub] qingzhouzhen opened a new issue #9743: from tensorboard import SummaryWriter wrong

[GitBox]

qingzhouzhen opened a new issue #9743: from tensorboard import SummaryWriter 
wrong
URL: https://github.com/apache/incubator-mxnet/issues/9743
 
 
   this file 
```https://github.com/apache/incubator-mxnet/blob/master/python/mxnet/contrib/tensorboard.py```
 does not work any more,
   because new tensorboard does not have a Class named SummaryWrite in writer.py
   how could I use tensorboard now?


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[GitHub] eric-haibin-lin commented on issue #9705: Added unittest for benchmarking metric performance

[GitBox]

eric-haibin-lin commented on issue #9705: Added unittest for benchmarking 
metric performance
URL: https://github.com/apache/incubator-mxnet/pull/9705#issuecomment-364017775
 
 
   Wouldn't nightly test be a better place for performance tests like this? 
This unit test doesn't verify the correctness at all. 


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[GitHub] feevos commented on issue #9742: Feature request: please translate zh.gluon.ai in English

[GitBox]

feevos commented on issue #9742: Feature request: please translate zh.gluon.ai 
in English
URL: 
https://github.com/apache/incubator-mxnet/issues/9742#issuecomment-364008539
 
 
   Thank you @astonzhang !! @TaoLv  there are significant differences  
(according to my humble opinion -e.g. in Computer Vision). 


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[GitHub] TaoLv commented on issue #9742: Feature request: please translate zh.gluon.ai in English

[GitBox]

TaoLv commented on issue #9742: Feature request: please translate zh.gluon.ai 
in English
URL: 
https://github.com/apache/incubator-mxnet/issues/9742#issuecomment-364006535
 
 
   It says there is a English version here: http://gluon.mxnet.io/. But maybe 
there are some differences between these two versions.


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[GitHub] szha commented on issue #9652: Fallout from simplifying/fixing im2rec.py commandline boolean options.

[GitBox]

szha commented on issue #9652: Fallout from simplifying/fixing im2rec.py 
commandline boolean options.
URL: https://github.com/apache/incubator-mxnet/pull/9652#issuecomment-364006303
 
 
   Related: #8880 


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[incubator-mxnet] branch master updated: Fallout from simplifying/fixing im2rec.py commandline boolean options. (#9652)

[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 3761f2f  Fallout from simplifying/fixing im2rec.py commandline boolean 
options. (#9652)
3761f2f is described below

commit 3761f2f36e7e335f17e5c1dcbfe215de27f73fd5
Author: Pedro Larroy <928489+lar...@users.noreply.github.com>
AuthorDate: Thu Feb 8 06:24:50 2018 +0100

Fallout from simplifying/fixing im2rec.py commandline boolean options. 
(#9652)

Fix documentation referring to outdated usages of im2rec.py
---
 docs/faq/finetune.md| 4 ++--
 docs/tutorials/basic/data.md| 4 ++--
 docs/tutorials/basic/image_io.md| 4 ++--
 docs/tutorials/vision/large_scale_classification.md | 4 ++--
 example/image-classification/README.md  | 2 +-
 tools/im2rec.py | 4 ++--
 6 files changed, 11 insertions(+), 11 deletions(-)

diff --git a/docs/faq/finetune.md b/docs/faq/finetune.md
index 533c3ca..04244d1 100644
--- a/docs/faq/finetune.md
+++ b/docs/faq/finetune.md
@@ -61,8 +61,8 @@ for i in 256_ObjectCategories/*; do
 done
 done
 
-python ~/mxnet/tools/im2rec.py --list True --recursive True 
caltech-256-60-train caltech_256_train_60/
-python ~/mxnet/tools/im2rec.py --list True --recursive True caltech-256-60-val 
256_ObjectCategories/
+python ~/mxnet/tools/im2rec.py --list --recursive caltech-256-60-train 
caltech_256_train_60/
+python ~/mxnet/tools/im2rec.py --list --recursive caltech-256-60-val 
256_ObjectCategories/
 python ~/mxnet/tools/im2rec.py --resize 256 --quality 90 --num-thread 16 
caltech-256-60-val 256_ObjectCategories/
 python ~/mxnet/tools/im2rec.py --resize 256 --quality 90 --num-thread 16 
caltech-256-60-train caltech_256_train_60/
 ```
diff --git a/docs/tutorials/basic/data.md b/docs/tutorials/basic/data.md
index 66479d5..1a88242 100644
--- a/docs/tutorials/basic/data.md
+++ b/docs/tutorials/basic/data.md
@@ -392,7 +392,7 @@ Now let's convert them into record io format using the 
`im2rec.py` utility scrip
 First, we need to make a list that contains all the image files and their 
categories:
 
 ```python
-os.system('python %s/tools/im2rec.py --list=1 --recursive=1 --shuffle=1 
--test-ratio=0.2 data/caltech 
data/101_ObjectCategories'%os.environ['MXNET_HOME'])
+os.system('python %s/tools/im2rec.py --list --recursive --test-ratio=0.2 
data/caltech data/101_ObjectCategories'%os.environ['MXNET_HOME'])
 ```
 
 The resulting list file (./data/caltech_train.lst) is in the format 
`index\t(one or more label)\tpath`. In this case, there is only one label for 
each image but you can modify the list to add in more for multi-label training.
@@ -401,7 +401,7 @@ Then we can use this list to create our record io file:
 
 
 ```python
-os.system("python %s/tools/im2rec.py --num-thread=4 --pass-through=1 
data/caltech data/101_ObjectCategories"%os.environ['MXNET_HOME'])
+os.system("python %s/tools/im2rec.py --num-thread=4 --pass-through 
data/caltech data/101_ObjectCategories"%os.environ['MXNET_HOME'])
 ```
 
 The record io files are now saved at here (./data)
diff --git a/docs/tutorials/basic/image_io.md b/docs/tutorials/basic/image_io.md
index e643425..8d60ee8 100644
--- a/docs/tutorials/basic/image_io.md
+++ b/docs/tutorials/basic/image_io.md
@@ -55,7 +55,7 @@ contains all the image files and their categories:
 
 ```python
 assert(MXNET_HOME != '/scratch/mxnet'), "Please update your MXNet location"
-os.system('python %s/tools/im2rec.py --list=1 --recursive=1 --shuffle=1 
--test-ratio=0.2 data/caltech data/101_ObjectCategories'%MXNET_HOME)
+os.system('python %s/tools/im2rec.py --list --recursive --test-ratio=0.2 
data/caltech data/101_ObjectCategories'%MXNET_HOME)
 ```
 
 The resulting [list file](./data/caltech_train.lst) is in the format
@@ -66,7 +66,7 @@ Then we can use this list to create our record io file:
 
 
 ```python
-os.system("python %s/tools/im2rec.py --num-thread=4 --pass-through=1 
data/caltech data/101_ObjectCategories"%MXNET_HOME)
+os.system("python %s/tools/im2rec.py --num-thread=4 --pass-through 
data/caltech data/101_ObjectCategories"%MXNET_HOME)
 ```
 
 The record io files are now saved in the "data" directory.
diff --git a/docs/tutorials/vision/large_scale_classification.md 
b/docs/tutorials/vision/large_scale_classification.md
index 17701e6..089f55b 100644
--- a/docs/tutorials/vision/large_scale_classification.md
+++ b/docs/tutorials/vision/large_scale_classification.md
@@ -105,7 +105,7 @@ To create the recordIO files, we first create a list of 
images we want in the re
 
 ```
 mkdir -p train_meta
-python ${MXNET}/tools/im2rec.py --list True --chunks 8 --recursive True \
+python ${MXNET}/tools/im2rec.py --list --chunks 8 --recursive \
 train_meta/${NAME} ${ROOT}
 ```
 
@@ -127,7 +127,7 @@ We do similar 

[GitHub] szha closed pull request #9652: Fallout from simplifying/fixing im2rec.py commandline boolean options.

[GitBox]

szha closed pull request #9652: Fallout from simplifying/fixing im2rec.py 
commandline boolean options.
URL: https://github.com/apache/incubator-mxnet/pull/9652
 
 
   

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 (from a fork), the diff is supplied
below (as it won't show otherwise due to GitHub magic):

diff --git a/docs/faq/finetune.md b/docs/faq/finetune.md
index 2c6c7e3402..8650f3ef59 100644
--- a/docs/faq/finetune.md
+++ b/docs/faq/finetune.md
@@ -61,8 +61,8 @@ for i in 256_ObjectCategories/*; do
 done
 done
 
-python ~/mxnet/tools/im2rec.py --list True --recursive True 
caltech-256-60-train caltech_256_train_60/
-python ~/mxnet/tools/im2rec.py --list True --recursive True caltech-256-60-val 
256_ObjectCategories/
+python ~/mxnet/tools/im2rec.py --list --recursive caltech-256-60-train 
caltech_256_train_60/
+python ~/mxnet/tools/im2rec.py --list --recursive caltech-256-60-val 
256_ObjectCategories/
 python ~/mxnet/tools/im2rec.py --resize 256 --quality 90 --num-thread 16 
caltech-256-60-val 256_ObjectCategories/
 python ~/mxnet/tools/im2rec.py --resize 256 --quality 90 --num-thread 16 
caltech-256-60-train caltech_256_train_60/
 ```
diff --git a/docs/tutorials/basic/data.md b/docs/tutorials/basic/data.md
index 66479d5acc..1a88242592 100644
--- a/docs/tutorials/basic/data.md
+++ b/docs/tutorials/basic/data.md
@@ -392,7 +392,7 @@ Now let's convert them into record io format using the 
`im2rec.py` utility scrip
 First, we need to make a list that contains all the image files and their 
categories:
 
 ```python
-os.system('python %s/tools/im2rec.py --list=1 --recursive=1 --shuffle=1 
--test-ratio=0.2 data/caltech 
data/101_ObjectCategories'%os.environ['MXNET_HOME'])
+os.system('python %s/tools/im2rec.py --list --recursive --test-ratio=0.2 
data/caltech data/101_ObjectCategories'%os.environ['MXNET_HOME'])
 ```
 
 The resulting list file (./data/caltech_train.lst) is in the format 
`index\t(one or more label)\tpath`. In this case, there is only one label for 
each image but you can modify the list to add in more for multi-label training.
@@ -401,7 +401,7 @@ Then we can use this list to create our record io file:
 
 
 ```python
-os.system("python %s/tools/im2rec.py --num-thread=4 --pass-through=1 
data/caltech data/101_ObjectCategories"%os.environ['MXNET_HOME'])
+os.system("python %s/tools/im2rec.py --num-thread=4 --pass-through 
data/caltech data/101_ObjectCategories"%os.environ['MXNET_HOME'])
 ```
 
 The record io files are now saved at here (./data)
diff --git a/docs/tutorials/basic/image_io.md b/docs/tutorials/basic/image_io.md
index e6434257b7..8d60ee8fc0 100644
--- a/docs/tutorials/basic/image_io.md
+++ b/docs/tutorials/basic/image_io.md
@@ -55,7 +55,7 @@ contains all the image files and their categories:
 
 ```python
 assert(MXNET_HOME != '/scratch/mxnet'), "Please update your MXNet location"
-os.system('python %s/tools/im2rec.py --list=1 --recursive=1 --shuffle=1 
--test-ratio=0.2 data/caltech data/101_ObjectCategories'%MXNET_HOME)
+os.system('python %s/tools/im2rec.py --list --recursive --test-ratio=0.2 
data/caltech data/101_ObjectCategories'%MXNET_HOME)
 ```
 
 The resulting [list file](./data/caltech_train.lst) is in the format
@@ -66,7 +66,7 @@ Then we can use this list to create our record io file:
 
 
 ```python
-os.system("python %s/tools/im2rec.py --num-thread=4 --pass-through=1 
data/caltech data/101_ObjectCategories"%MXNET_HOME)
+os.system("python %s/tools/im2rec.py --num-thread=4 --pass-through 
data/caltech data/101_ObjectCategories"%MXNET_HOME)
 ```
 
 The record io files are now saved in the "data" directory.
diff --git a/docs/tutorials/vision/large_scale_classification.md 
b/docs/tutorials/vision/large_scale_classification.md
index 17701e6047..089f55b871 100644
--- a/docs/tutorials/vision/large_scale_classification.md
+++ b/docs/tutorials/vision/large_scale_classification.md
@@ -105,7 +105,7 @@ To create the recordIO files, we first create a list of 
images we want in the re
 
 ```
 mkdir -p train_meta
-python ${MXNET}/tools/im2rec.py --list True --chunks 8 --recursive True \
+python ${MXNET}/tools/im2rec.py --list --chunks 8 --recursive \
 train_meta/${NAME} ${ROOT}
 ```
 
@@ -127,7 +127,7 @@ We do similar preprocessing for the validation set.
 
 ```
 mkdir -p val_meta
-python ${MXNET}/tools/im2rec.py --list True --recursive True \
+python ${MXNET}/tools/im2rec.py --list --recursive \
 val_meta/${NAME} ${VAL_ROOT}
 python ${MXNET}/tools/im2rec.py --resize 480 --quality 90 \
 --num-thread 16 val_meta/${NAME} ${VAL_ROOT}
diff --git a/example/image-classification/README.md 
b/example/image-classification/README.md
index 296760590f..d0ffa3f72a 100644
--- a/example/image-classification/README.md
+++ b/example/image-classification/README.md
@@ -68,7 +68,7 @@ We first prepare two `.lst` files, which consist of the 
labels and image paths
 can be used for 

[GitHub] astonzhang commented on issue #9742: Feature request: please translate zh.gluon.ai in English

[GitBox]

astonzhang commented on issue #9742: Feature request: please translate 
zh.gluon.ai in English
URL: 
https://github.com/apache/incubator-mxnet/issues/9742#issuecomment-364006201
 
 
   Thanks. Will do.


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[GitHub] jeremiedb commented on issue #9737: dot product backwards in R

[GitBox]

jeremiedb commented on issue #9737: dot product backwards in R
URL: 
https://github.com/apache/incubator-mxnet/issues/9737#issuecomment-364005355
 
 
   Has you guessed it, axis order interpretation has to be reversed when 
working with MxnetR. Dimensions has to be interpreted from right to left, and 
axis are 0-index.  
   
   If considering a R matrix of shape [3, 1] when looknig at the symbolic or 
ndarray documentation for Python, axis 0 dim size is 1 (which typically matches 
batch size), and axis 1dim size is 3 (typically corresponding to features). 


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[GitHub] feevos opened a new issue #9742: Feature request: please translate zh.gluon.ai in English

[GitBox]

feevos opened a new issue #9742: Feature request: please translate zh.gluon.ai 
in English
URL: https://github.com/apache/incubator-mxnet/issues/9742
 
 
   Dear all, 
   
   there is a great deal of information in the [Chinese version of the gluon 
page](https://zh.gluon.ai/), google translator loses various English numbers 
when they are intermixed with Chinese characters and makes reading problematic 
(sometimes messes up code parts as well). Could please someone translate? 
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[GitHub] szha closed pull request #9739: fix FileNotFoundError error in diagnose.py

[GitBox]

szha closed pull request #9739: fix FileNotFoundError error in diagnose.py
URL: https://github.com/apache/incubator-mxnet/pull/9739
 
 
   

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 (from a fork), the diff is supplied
below (as it won't show otherwise due to GitHub magic):

diff --git a/tools/diagnose.py b/tools/diagnose.py
index adb621d1e4..0f878f918e 100644
--- a/tools/diagnose.py
+++ b/tools/diagnose.py
@@ -110,7 +110,7 @@ def check_mxnet():
 print('Commit Hash   :', ch)
 except ImportError:
 print('No MXNet installed.')
-except FileNotFoundError:
+except IOError:
 print('Hashtag not found. Not installed from pre-built package.')
 except Exception as e:
 import traceback


 


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[incubator-mxnet] branch master updated: fix FileNotFoundError error (#9739)

[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 d4645ce  fix FileNotFoundError error (#9739)
d4645ce is described below

commit d4645ce9c92f6841f6966244f026c84674166bc3
Author: chsin 
AuthorDate: Wed Feb 7 21:44:01 2018 -0500

fix FileNotFoundError error (#9739)

FileNotFoundError not available in python2.7, use IOError instead 
(https://stackoverflow.com/questions/21367320/searching-for-equivalent-of-filenotfounderror-in-python-2)
---
 tools/diagnose.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tools/diagnose.py b/tools/diagnose.py
index adb621d..0f878f9 100644
--- a/tools/diagnose.py
+++ b/tools/diagnose.py
@@ -110,7 +110,7 @@ def check_mxnet():
 print('Commit Hash   :', ch)
 except ImportError:
 print('No MXNet installed.')
-except FileNotFoundError:
+except IOError:
 print('Hashtag not found. Not installed from pre-built package.')
 except Exception as e:
 import traceback

-- 
To stop receiving notification emails like this one, please contact
zhash...@apache.org.

[GitHub] safrooze commented on issue #9705: Added unittest for benchmarking metric performance

[GitBox]

safrooze commented on issue #9705: Added unittest for benchmarking metric 
performance
URL: https://github.com/apache/incubator-mxnet/pull/9705#issuecomment-363977854
 
 
   OK I think I addressed all the feedback:
   * random is seeded
   * nd.wait_all() used before starting timing and before ending timing
   * Added batch-size values of 16, 64, 256, and 1024
   * Datasize varies by number of output channels to keep total runtime down to 
a few minutes
   
   The modified code output looks like this:
   ```
   Metric Data-Ctx  Label-Ctx   Data Size   Batch Size Output Dim   
  Elapsed Time
   
--
   acccpu(0)cpu(0)  131072  16 128  
  1.0015
   acccpu(0)gpu(0)  131072  16 128  
  1.682
   accgpu(0)cpu(0)  131072  16 128  
  2.6263
   accgpu(0)gpu(0)  131072  16 128  
  3.3028
   
--
   acccpu(0)cpu(0)  131072  64 128  
  0.42843
   acccpu(0)gpu(0)  131072  64 128  
  0.568
   accgpu(0)cpu(0)  131072  64 128  
  0.78586
   accgpu(0)gpu(0)  131072  64 128  
  0.94317
   
--
   acccpu(0)cpu(0)  131072  256128  
  0.19074
   acccpu(0)gpu(0)  131072  256128  
  0.24228
   accgpu(0)cpu(0)  131072  256128  
  0.21548
   accgpu(0)gpu(0)  131072  256128  
  0.25075
   
--
   acccpu(0)cpu(0)  131072  1024   128  
  0.1303
   acccpu(0)gpu(0)  131072  1024   128  
  0.14127
   accgpu(0)cpu(0)  131072  1024   128  
  0.055079
   accgpu(0)gpu(0)  131072  1024   128  
  0.065515
   
--
   ```


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[GitHub] safrooze commented on issue #9705: Added unittest for benchmarking metric performance

[GitBox]

safrooze commented on issue #9705: Added unittest for benchmarking metric 
performance
URL: https://github.com/apache/incubator-mxnet/pull/9705#issuecomment-363977854
 
 
   OK I think I addressed all the feedback:
   * random is seeded
   * nd.wait_all() used before starting timing and before ending timing
   * Added batch-size values of 16, 64, 256, and 1024
   * Datasize varies by number of output channels to keep total runtime down to 
a few minutes
   
   The modified code output looks like this:
   ```
   Metric Data-Ctx  Label-Ctx   Data Size   Batch Size Output Dim   
  Elapsed Time
   
--
   acccpu(0)cpu(0)  131072  16 128  
  1.0015
   acccpu(0)gpu(0)  131072  16 128  
  1.682
   accgpu(0)cpu(0)  131072  16 128  
  2.6263
   accgpu(0)gpu(0)  131072  16 128  
  3.3028
   
--
   acccpu(0)cpu(0)  131072  64 128  
  0.42843
   acccpu(0)gpu(0)  131072  64 128  
  0.568
   accgpu(0)cpu(0)  131072  64 128  
  0.78586
   accgpu(0)gpu(0)  131072  64 128  
  0.94317
   
--
   ```


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[GitHub] chsin commented on issue #7725: accuracy of cpp example is a constant value when training, no matter how many epochs trained!

[GitBox]

chsin commented on issue #7725: accuracy of cpp example is a constant value 
when training, no matter how many epochs trained!
URL: 
https://github.com/apache/incubator-mxnet/issues/7725#issuecomment-363976557
 
 
   @goodtogood if you pull master and rebuild, do you have the same issue? 
Apparently, the fix was merged: 
https://github.com/apache/incubator-mxnet/pull/8551. I'm curious bc I've just 
built master, but have the same problem with lenet.cpp


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[GitHub] HuangZhanPeng commented on issue #9729: how to use the transforms

[GitBox]

HuangZhanPeng commented on issue #9729: how to use the transforms 
URL: 
https://github.com/apache/incubator-mxnet/issues/9729#issuecomment-363971683
 
 
   thank you very much


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[GitHub] larroy opened a new pull request #9741: CI environment refinements

[GitBox]

larroy opened a new pull request #9741: CI environment refinements
URL: https://github.com/apache/incubator-mxnet/pull/9741
 
 
   ## Description ##
   Don't install nvidia driver
   Don't install gtest 
   set -x and -e  so scripts fail when command fail.
   
   ## Checklist ##
   ### Essentials ###
   - [ ] Passed code style checking (`make lint`)
   - [ ] Changes are complete (i.e. I finished coding on this PR)
   - [ ] All changes have test coverage:
   - Unit tests are added for small changes to verify correctness (e.g. adding 
a new operator)
   - Nightly tests are added for complicated/long-running ones (e.g. changing 
distributed kvstore)
   - Build tests will be added for build configuration changes (e.g. adding a 
new build option with NCCL)
   - [ ] Code is well-documented: 
   - For user-facing API changes, API doc string has been updated. 
   - For new C++ functions in header files, their functionalities and arguments 
are documented. 
   - For new examples, README.md is added to explain the what the example does, 
the source of the dataset, expected performance on test set and reference to 
the original paper if applicable
   - [ ] To the my best knowledge, examples are either not affected by this 
change, or have been fixed to be compatible with this change
   
   ### Changes ###
   - [ ] Feature1, tests, (and when applicable, API doc)
   - [ ] Feature2, tests, (and when applicable, API doc)
   
   ## Comments ##
   - If this change is a backward incompatible change, why must this change be 
made.
   - Interesting edge cases to note here
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[GitHub] szha commented on issue #9739: fix FileNotFoundError error in diagnose.py

[GitBox]

szha commented on issue #9739: fix FileNotFoundError error in diagnose.py
URL: https://github.com/apache/incubator-mxnet/pull/9739#issuecomment-363970915
 
 
   Thanks for fixing. cc'd @zhreshold 


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[GitHub] szha commented on issue #9739: fix FileNotFoundError error in diagnose.py

[GitBox]

szha commented on issue #9739: fix FileNotFoundError error in diagnose.py
URL: https://github.com/apache/incubator-mxnet/pull/9739#issuecomment-363970915
 
 
   Thanks for fixing. @zhreshold 


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[GitHub] szha commented on a change in pull request #9492: fix print_summary bug and add groups of convolution

[GitBox]

szha commented on a change in pull request #9492: fix print_summary bug and add 
groups of convolution
URL: https://github.com/apache/incubator-mxnet/pull/9492#discussion_r166808910
 
 

 ##
 File path: python/mxnet/visualization.py
 ##
 @@ -134,17 +134,23 @@ def print_layer_summary(node, out_shape):
 pre_filter = pre_filter + int(shape[0])
 cur_param = 0
 if op == 'Convolution':
-if ("no_bias" in node["attrs"]) and int(node["attrs"]["no_bias"]):
-cur_param = pre_filter * int(node["attrs"]["num_filter"])
+if ("no_bias" in node["attrs"]) and node["attrs"]["no_bias"] == 
'True':
+num_group = int(node["attrs"]["num_group"]) if \
+   ("num_group" in node["attrs"]) else 1
+cur_param = (pre_filter * int(node["attrs"]["num_filter"])) \
+   // num_group
 for k in _str2tuple(node["attrs"]["kernel"]):
 cur_param *= int(k)
 else:
-cur_param = pre_filter * int(node["attrs"]["num_filter"])
+num_group = int(node["attrs"]["num_group"]) if \
+   ("num_group" in node["attrs"]) else 1
+cur_param = (pre_filter * int(node["attrs"]["num_filter"])) \
+   // num_group
 for k in _str2tuple(node["attrs"]["kernel"]):
 cur_param *= int(k)
 cur_param += int(node["attrs"]["num_filter"])
 elif op == 'FullyConnected':
-if ("no_bias" in node["attrs"]) and int(node["attrs"]["no_bias"]):
+if ("no_bias" in node["attrs"]) and node["attrs"]["no_bias"] == 
'True':
 
 Review comment:
   The json dump should have made booleans without the parenthesis so that the 
json's type is used, i.e.
   ```
   "attrs": {
 "flatten": "True",
 "no_bias": "False",
 "num_hidden": "1000"
   }
   # should have been
   "attrs": {
 "flatten": True,
 "no_bias": False,
 "num_hidden": 1000
   }
   ```
   
   @piiswrong would it be OK to change json export to properly use json boolean?


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[GitHub] szha commented on a change in pull request #9492: fix print_summary bug and add groups of convolution

[GitBox]

szha commented on a change in pull request #9492: fix print_summary bug and add 
groups of convolution
URL: https://github.com/apache/incubator-mxnet/pull/9492#discussion_r166808910
 
 

 ##
 File path: python/mxnet/visualization.py
 ##
 @@ -134,17 +134,23 @@ def print_layer_summary(node, out_shape):
 pre_filter = pre_filter + int(shape[0])
 cur_param = 0
 if op == 'Convolution':
-if ("no_bias" in node["attrs"]) and int(node["attrs"]["no_bias"]):
-cur_param = pre_filter * int(node["attrs"]["num_filter"])
+if ("no_bias" in node["attrs"]) and node["attrs"]["no_bias"] == 
'True':
+num_group = int(node["attrs"]["num_group"]) if \
+   ("num_group" in node["attrs"]) else 1
+cur_param = (pre_filter * int(node["attrs"]["num_filter"])) \
+   // num_group
 for k in _str2tuple(node["attrs"]["kernel"]):
 cur_param *= int(k)
 else:
-cur_param = pre_filter * int(node["attrs"]["num_filter"])
+num_group = int(node["attrs"]["num_group"]) if \
+   ("num_group" in node["attrs"]) else 1
+cur_param = (pre_filter * int(node["attrs"]["num_filter"])) \
+   // num_group
 for k in _str2tuple(node["attrs"]["kernel"]):
 cur_param *= int(k)
 cur_param += int(node["attrs"]["num_filter"])
 elif op == 'FullyConnected':
-if ("no_bias" in node["attrs"]) and int(node["attrs"]["no_bias"]):
+if ("no_bias" in node["attrs"]) and node["attrs"]["no_bias"] == 
'True':
 
 Review comment:
   The json dump should have made booleans without the parenthesis so that the 
json's type is used, i.e.
   ```
   "attrs": {
 "flatten": "True",
 "no_bias": "False",
 "num_hidden": "1000"
   }
   # should have been
   "attrs": {
 "flatten": True,
 "no_bias": False,
 "num_hidden": "1000"
   }
   ```
   
   @piiswrong would it be OK to change json export to properly use json boolean?


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[GitHub] szha commented on a change in pull request #9492: fix print_summary bug and add groups of convolution

[GitBox]

szha commented on a change in pull request #9492: fix print_summary bug and add 
groups of convolution
URL: https://github.com/apache/incubator-mxnet/pull/9492#discussion_r166808910
 
 

 ##
 File path: python/mxnet/visualization.py
 ##
 @@ -134,17 +134,23 @@ def print_layer_summary(node, out_shape):
 pre_filter = pre_filter + int(shape[0])
 cur_param = 0
 if op == 'Convolution':
-if ("no_bias" in node["attrs"]) and int(node["attrs"]["no_bias"]):
-cur_param = pre_filter * int(node["attrs"]["num_filter"])
+if ("no_bias" in node["attrs"]) and node["attrs"]["no_bias"] == 
'True':
+num_group = int(node["attrs"]["num_group"]) if \
+   ("num_group" in node["attrs"]) else 1
+cur_param = (pre_filter * int(node["attrs"]["num_filter"])) \
+   // num_group
 for k in _str2tuple(node["attrs"]["kernel"]):
 cur_param *= int(k)
 else:
-cur_param = pre_filter * int(node["attrs"]["num_filter"])
+num_group = int(node["attrs"]["num_group"]) if \
+   ("num_group" in node["attrs"]) else 1
+cur_param = (pre_filter * int(node["attrs"]["num_filter"])) \
+   // num_group
 for k in _str2tuple(node["attrs"]["kernel"]):
 cur_param *= int(k)
 cur_param += int(node["attrs"]["num_filter"])
 elif op == 'FullyConnected':
-if ("no_bias" in node["attrs"]) and int(node["attrs"]["no_bias"]):
+if ("no_bias" in node["attrs"]) and node["attrs"]["no_bias"] == 
'True':
 
 Review comment:
   The json dump should have made booleans without the parenthesis so that the 
json boolean type is used, i.e.
   ```
   "attrs": {
 "flatten": "True",
 "no_bias": "False",
 "num_hidden": "1000"
   }
   # should have been
   "attrs": {
 "flatten": True,
 "no_bias": False,
 "num_hidden": "1000"
   }
   ```
   
   @piiswrong would it be OK to change json export to properly use json boolean?


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[GitHub] zheng-da opened a new pull request #9740: add axis support and gradient for L2norm

[GitBox]

zheng-da opened a new pull request #9740: add axis support and gradient for 
L2norm
URL: https://github.com/apache/incubator-mxnet/pull/9740
 
 
   ## Description ##
   This PR adds axis support and compute gradient for L2norm.
   
   ## Checklist ##
   ### Essentials ###
   - [ ] Passed code style checking (`make lint`)
   - [ ] Changes are complete (i.e. I finished coding on this PR)
   - [ ] All changes have test coverage:
   - Unit tests are added for small changes to verify correctness (e.g. adding 
a new operator)
   - Nightly tests are added for complicated/long-running ones (e.g. changing 
distributed kvstore)
   - Build tests will be added for build configuration changes (e.g. adding a 
new build option with NCCL)
   - [ ] Code is well-documented: 
   - For user-facing API changes, API doc string has been updated. 
   - For new C++ functions in header files, their functionalities and arguments 
are documented. 
   - For new examples, README.md is added to explain the what the example does, 
the source of the dataset, expected performance on test set and reference to 
the original paper if applicable
   - [ ] To the my best knowledge, examples are either not affected by this 
change, or have been fixed to be compatible with this change
   
   ### Changes ###
   - add axis support, which adds new parameters to norm.
   - compute gradient of L2 norm.


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[GitHub] szha commented on a change in pull request #9492: fix print_summary bug and add groups of convolution

[GitBox]

szha commented on a change in pull request #9492: fix print_summary bug and add 
groups of convolution
URL: https://github.com/apache/incubator-mxnet/pull/9492#discussion_r166807999
 
 

 ##
 File path: python/mxnet/visualization.py
 ##
 @@ -134,17 +134,23 @@ def print_layer_summary(node, out_shape):
 pre_filter = pre_filter + int(shape[0])
 cur_param = 0
 if op == 'Convolution':
-if ("no_bias" in node["attrs"]) and int(node["attrs"]["no_bias"]):
-cur_param = pre_filter * int(node["attrs"]["num_filter"])
+if ("no_bias" in node["attrs"]) and node["attrs"]["no_bias"] == 
'True':
+num_group = int(node["attrs"]["num_group"]) if \
+   ("num_group" in node["attrs"]) else 1
+cur_param = (pre_filter * int(node["attrs"]["num_filter"])) \
+   // num_group
 for k in _str2tuple(node["attrs"]["kernel"]):
 cur_param *= int(k)
 else:
-cur_param = pre_filter * int(node["attrs"]["num_filter"])
+num_group = int(node["attrs"]["num_group"]) if \
+   ("num_group" in node["attrs"]) else 1
+cur_param = (pre_filter * int(node["attrs"]["num_filter"])) \
+   // num_group
 for k in _str2tuple(node["attrs"]["kernel"]):
 cur_param *= int(k)
 cur_param += int(node["attrs"]["num_filter"])
 elif op == 'FullyConnected':
-if ("no_bias" in node["attrs"]) and int(node["attrs"]["no_bias"]):
+if ("no_bias" in node["attrs"]) and node["attrs"]["no_bias"] == 
'True':
 
 Review comment:
   yes


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[GitHub] chsin opened a new pull request #9739: fix FileNotFoundError error in diagnose.py

[GitBox]

chsin opened a new pull request #9739: fix FileNotFoundError error in 
diagnose.py
URL: https://github.com/apache/incubator-mxnet/pull/9739
 
 
   FileNotFoundError not available in python2.7, use IOError instead.
   
   ## Description ##
   
   What happens with old code:
   ```
   --Python Info--
   ('Version  :', '2.7.13')
   ('Compiler :', 'GCC 4.8.2 20140120 (Red Hat 4.8.2-15)')
   ('Build:', ('default', 'Dec 19 2017 13:31:54'))
   ('Arch :', ('64bit', 'ELF'))
   Pip Info---
   ('Version  :', '9.0.1')
   ('Directory:', 
'/scratch/cahsin/pythondir27/lib/python2.7/site-packages/pip')
   --MXNet Info---
   ('Version  :', '1.0.0')
   ('Directory:', '/scratch/cahsin/repo/mxnet/python/mxnet')
   Traceback (most recent call last):
 File "/scratch/cahsin/repo/mxnet/tools/diagnose.py", line 171, in 
   check_mxnet()
 File "/scratch/cahsin/repo/mxnet/tools/diagnose.py", line 113, in 
check_mxnet
   except FileNotFoundError:
   NameError: global name 'FileNotFoundError' is not defined
   ```
   
   ## Checklist ##
   ### Essentials ###
   - [x] Changes are complete (i.e. I finished coding on this PR)
   - [x] To the my best knowledge, examples are either not affected by this 
change, or have been fixed to be compatible with this change


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[GitHub] pengzhao-intel commented on issue #9730: Check padding size for global pooling

[GitBox]

pengzhao-intel commented on issue #9730: Check padding size for global pooling
URL: https://github.com/apache/incubator-mxnet/pull/9730#issuecomment-363963004
 
 
   @szha @zheng-da please help take a review 


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[GitHub] szha commented on issue #8546: What's the equivalent of np.where()?

[GitBox]

szha commented on issue #8546: What's the equivalent of np.where()?
URL: 
https://github.com/apache/incubator-mxnet/issues/8546#issuecomment-363959753
 
 
   @apache/mxnet-committers: This issue has been inactive for the past 90 days. 
It has no label and needs triage.
   
   For general "how-to" questions, our [user forum](https://discuss.mxnet.io/) 
(and [Chinese version](https://discuss.gluon.ai/)) is a good place to get help.


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[GitHub] szha commented on issue #8593: Runtime errors during forward in a custom gluon.Block

[GitBox]

szha commented on issue #8593: Runtime errors during forward in a custom 
gluon.Block
URL: 
https://github.com/apache/incubator-mxnet/issues/8593#issuecomment-363959759
 
 
   @apache/mxnet-committers: This issue has been inactive for the past 90 days. 
It has no label and needs triage.
   
   For general "how-to" questions, our [user forum](https://discuss.mxnet.io/) 
(and [Chinese version](https://discuss.gluon.ai/)) is a good place to get help.


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[GitHub] szha commented on issue #7933: CoreML conversion with finetuned model

[GitBox]

szha commented on issue #7933: CoreML conversion with finetuned model 
URL: 
https://github.com/apache/incubator-mxnet/issues/7933#issuecomment-363959748
 
 
   @apache/mxnet-committers: This issue has been inactive for the past 90 days. 
It has no label and needs triage.
   
   For general "how-to" questions, our [user forum](https://discuss.mxnet.io/) 
(and [Chinese version](https://discuss.gluon.ai/)) is a good place to get help.


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[GitHub] eric-haibin-lin closed pull request #9701: [REVIEW REQUIRED] Revert PR #9484 & add additional dependency licenses to LICENSE file

[GitBox]

eric-haibin-lin closed pull request #9701: [REVIEW REQUIRED] Revert PR #9484 & 
add additional dependency licenses to LICENSE file
URL: https://github.com/apache/incubator-mxnet/pull/9701
 
 
   

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 (from a fork), the diff is supplied
below (as it won't show otherwise due to GitHub magic):

diff --git a/LICENSE b/LICENSE
index d3b3d6f9dd..e7d50c3772 100644
--- a/LICENSE
+++ b/LICENSE
@@ -201,43 +201,145 @@
See the License for the specific language governing permissions and
limitations under the License.
 
-===
+
==
 Apache MXNET (incubating) Subcomponents:
 
-The Apache MXNET (incubating) project contains subcomponents with separate
-copyright notices and license terms. Your use of the source code for the 
these
+The Apache MXNET (incubating) project contains subcomponents with separate 
copyright
+notices and license terms. Your use of the source code for the these
 subcomponents is subject to the terms and conditions of the following
-licenses -
-
-
-1. Apache-2.0 license as above, wherever applicable
-
-
-
-2. MIT license wherever applicable
-
-Permission is hereby granted, free of charge, to any person obtaining a
-copy of this software and associated documentation files (the "Software"),
-to deal in the Software without restriction, including without limitation
-the rights to use, copy, modify, merge, publish, distribute, sublicense,
-and/or sell copies of the Software, and to permit persons to whom the
-Software is furnished to do so, subject to the following conditions:
-
-The above copyright notice and this permission notice shall be included
-in all copies or substantial portions of the Software.
-
-THE SOFTWARE IS PROVIDED *AS IS*, WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
-IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
-FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL
-THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR
-OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE,
-ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR
-OTHER DEALINGS IN THE SOFTWARE.
-
-
-
-3. BSD License wherever applicable
-
+licenses.
+
+
===
+Apache-2.0 licenses
+
===
+
+The following components are provided under an Apache 2.0 license.
+
+1. MXNet Cpp-package - For details, /cpp-package/LICENSE
+2. MXNet rcnn - For details, see, example/rcnn/LICENSE
+3. scala-package - For details, see, scala-package/LICENSE
+4. Warp-CTC - For details, see, src/operator/contrib/ctc_include/LICENSE
+5. dlpack - For details, see, dlpack/LICENSE
+6. dmlc-core - For details, see, dmlc-core/LICENSE
+7. mshadow - For details, see, mshadow/LICENSE
+8. nnvm/dmlc-core - For details, see, nnvm/dmlc-core/LICENSE
+9. nnvm - For details, see, nnvm/LICENSE
+10. nnvm-fusion - For details, see, nnvm/plugin/nnvm-fusion/LICENSE
+11. ps-lite - For details, see, ps-lite/LICENSE
+12. nnvm/tvm - For details, see, nnvm/tvm/LICENSE
+13. googlemock scripts/generator - For details, see, 
3rdparty/googletest/googlemock/scripts/generator/LICENSE
+
+
+
===
+MIT licenses
+
===
+
+1. Fast R-CNN  - For details, see example/rcnn/LICENSE
+2. Faster R-CNN - For details, see example/rcnn/LICENSE
+3. tree_lstm - For details, see example/gluon/tree_lstm/LICENSE
+4. OpenMP - For details, see 3rdparty/openmp/LICENSE.txt
+5. HalideIR - For details, see nnvm/tvm/HalideIR/LICENSE
+
+
+
===
+NVIDIA Licenses
+
===
+
+1. Moderngpu
+For details, see, 

[incubator-mxnet] branch v1.1.0 updated: [REVIEW REQUIRED] Revert PR #9484 & add additional dependency licenses to LICENSE file (#9701)

[haibin]

This is an automated email from the ASF dual-hosted git repository.

haibin pushed a commit to branch v1.1.0
in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git


The following commit(s) were added to refs/heads/v1.1.0 by this push:
 new 6deba93  [REVIEW REQUIRED] Revert PR #9484 & add additional dependency 
licenses to LICENSE file (#9701)
6deba93 is described below

commit 6deba93e3f6a118c11eee8aa678bd0235b15748a
Author: mbaijal <30911248+mbai...@users.noreply.github.com>
AuthorDate: Wed Feb 7 16:17:31 2018 -0800

[REVIEW REQUIRED] Revert PR #9484 & add additional dependency licenses to 
LICENSE file (#9701)

* Revert "[Review Required] Fixing Licenses: Cleaning up the Top Level 
LICENSE file (#9484)"

This reverts commit 8930d96b265560a797c5554a9617f607cea7740f.

* Some more LICENSE fixes

* Adding some more packages to the LICENSE file

* Adding dependencies of dependencies
---
 LICENSE | 312 +---
 1 file changed, 278 insertions(+), 34 deletions(-)

diff --git a/LICENSE b/LICENSE
index d3b3d6f..e7d50c3 100644
--- a/LICENSE
+++ b/LICENSE
@@ -201,43 +201,145 @@
See the License for the specific language governing permissions and
limitations under the License.
 
-===
+
==
 Apache MXNET (incubating) Subcomponents:
 
-The Apache MXNET (incubating) project contains subcomponents with separate
-copyright notices and license terms. Your use of the source code for the 
these
+The Apache MXNET (incubating) project contains subcomponents with separate 
copyright
+notices and license terms. Your use of the source code for the these
 subcomponents is subject to the terms and conditions of the following
-licenses -
-
-
-1. Apache-2.0 license as above, wherever applicable
-
-
-
-2. MIT license wherever applicable
-
-Permission is hereby granted, free of charge, to any person obtaining a
-copy of this software and associated documentation files (the "Software"),
-to deal in the Software without restriction, including without limitation
-the rights to use, copy, modify, merge, publish, distribute, sublicense,
-and/or sell copies of the Software, and to permit persons to whom the
-Software is furnished to do so, subject to the following conditions:
-
-The above copyright notice and this permission notice shall be included
-in all copies or substantial portions of the Software.
-
-THE SOFTWARE IS PROVIDED *AS IS*, WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
-IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
-FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL
-THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR
-OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE,
-ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR
-OTHER DEALINGS IN THE SOFTWARE.
-
-
-
-3. BSD License wherever applicable
-
+licenses.
+
+
===
+Apache-2.0 licenses
+
===
+
+The following components are provided under an Apache 2.0 license.
+
+1. MXNet Cpp-package - For details, /cpp-package/LICENSE
+2. MXNet rcnn - For details, see, example/rcnn/LICENSE
+3. scala-package - For details, see, scala-package/LICENSE
+4. Warp-CTC - For details, see, src/operator/contrib/ctc_include/LICENSE
+5. dlpack - For details, see, dlpack/LICENSE
+6. dmlc-core - For details, see, dmlc-core/LICENSE
+7. mshadow - For details, see, mshadow/LICENSE
+8. nnvm/dmlc-core - For details, see, nnvm/dmlc-core/LICENSE
+9. nnvm - For details, see, nnvm/LICENSE
+10. nnvm-fusion - For details, see, nnvm/plugin/nnvm-fusion/LICENSE
+11. ps-lite - For details, see, ps-lite/LICENSE
+12. nnvm/tvm - For details, see, nnvm/tvm/LICENSE
+13. googlemock scripts/generator - For details, see, 
3rdparty/googletest/googlemock/scripts/generator/LICENSE
+
+
+
===
+MIT licenses
+

[GitHub] zhanghang1989 commented on issue #9738: Add Support for int64

[GitBox]

zhanghang1989 commented on issue #9738: Add Support for int64
URL: https://github.com/apache/incubator-mxnet/pull/9738#issuecomment-363956820
 
 
   @piiswrong added a macro.


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[GitHub] mbaijal commented on issue #9701: [REVIEW REQUIRED] Revert PR #9484 & add additional dependency licenses to LICENSE file

[GitBox]

mbaijal commented on issue #9701: [REVIEW REQUIRED] Revert PR #9484 & add 
additional dependency licenses to LICENSE file
URL: https://github.com/apache/incubator-mxnet/pull/9701#issuecomment-363952968
 
 
   Thanks Henri! 
   @eric-haibin-lin Can you please merge this PR.


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[GitHub] hyandell commented on issue #9701: [REVIEW REQUIRED] Revert PR #9484 & add additional dependency licenses to LICENSE file

[GitBox]

hyandell commented on issue #9701: [REVIEW REQUIRED] Revert PR #9484 & add 
additional dependency licenses to LICENSE file
URL: https://github.com/apache/incubator-mxnet/pull/9701#issuecomment-363952750
 
 
   Looks good to me. +1 to merge. 


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[GitHub] piiswrong commented on issue #9738: Add Support for int64

[GitBox]

piiswrong commented on issue #9738: Add Support for int64
URL: https://github.com/apache/incubator-mxnet/pull/9738#issuecomment-363952242
 
 
   Also need to add int8.
   Could you turn
   ```
.add_enum("float32", mshadow::kFloat32)
.add_enum("float64", mshadow::kFloat64)
 .add_enum("float16", mshadow::kFloat16)
 .add_enum("uint8", mshadow::kUint8)
 .add_enum("int32", mshadow::kInt32)
.add_enum("int64", mshadow::kInt64)
   ```
   into a macro so we don't need to make the change every where each time?


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[GitHub] szha closed pull request #9723: Specify lint-versions and fix docker build issues due to nvidia-384

[GitBox]

szha closed pull request #9723: Specify lint-versions and fix docker build 
issues due to nvidia-384
URL: https://github.com/apache/incubator-mxnet/pull/9723
 
 
   

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 (from a fork), the diff is supplied
below (as it won't show otherwise due to GitHub magic):

diff --git a/tests/ci_build/Dockerfile.lint b/tests/ci_build/Dockerfile.lint
index a72b3f8886..7fde1f5daa 100644
--- a/tests/ci_build/Dockerfile.lint
+++ b/tests/ci_build/Dockerfile.lint
@@ -1,6 +1,5 @@
 # For lint test
 FROM ubuntu:16.04
 
-# Sudo is not present on ubuntu16.04
 RUN apt-get update && apt-get install -y python-pip sudo
-RUN pip install cpplint pylint
+RUN pip install cpplint==1.3.0 pylint==1.8.2
diff --git a/tests/ci_build/install/ubuntu_install_nvidia.sh 
b/tests/ci_build/install/ubuntu_install_nvidia.sh
index 71fde8e800..e1a3e4f4e7 100755
--- a/tests/ci_build/install/ubuntu_install_nvidia.sh
+++ b/tests/ci_build/install/ubuntu_install_nvidia.sh
@@ -28,4 +28,4 @@ add-apt-repository -y ppa:graphics-drivers
 # Retrieve ppa:graphics-drivers and install nvidia-drivers.
 # Note: DEBIAN_FRONTEND required to skip the interactive setup steps
 apt update && \
-DEBIAN_FRONTEND=noninteractive apt install -y nvidia-384
+DEBIAN_FRONTEND=noninteractive apt install -y --no-install-recommends 
cuda-8-0


 


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[GitHub] zhanghang1989 opened a new pull request #9738: Add Support for int64

[GitBox]

zhanghang1989 opened a new pull request #9738: Add Support for int64
URL: https://github.com/apache/incubator-mxnet/pull/9738
 
 
   ## Description ##
   Address the issue https://github.com/apache/incubator-mxnet/issues/9536
   
   ## Checklist ##
   ### Essentials ###
   - [ ] Passed code style checking (`make lint`)
   - [ ] Changes are complete (i.e. I finished coding on this PR)
   - [ ] All changes have test coverage:
   - Unit tests are added for small changes to verify correctness (e.g. adding 
a new operator)
   - Nightly tests are added for complicated/long-running ones (e.g. changing 
distributed kvstore)
   - Build tests will be added for build configuration changes (e.g. adding a 
new build option with NCCL)
   - [ ] Code is well-documented: 
   - For user-facing API changes, API doc string has been updated. 
   - For new C++ functions in header files, their functionalities and arguments 
are documented. 
   - For new examples, README.md is added to explain the what the example does, 
the source of the dataset, expected performance on test set and reference to 
the original paper if applicable
   - [ ] To the my best knowledge, examples are either not affected by this 
change, or have been fixed to be compatible with this change
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[incubator-mxnet] branch v1.1.0 updated: Add my key (#9736)

[liuyizhi]

This is an automated email from the ASF dual-hosted git repository.

liuyizhi pushed a commit to branch v1.1.0
in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git


The following commit(s) were added to refs/heads/v1.1.0 by this push:
 new ead9b1a  Add my key (#9736)
ead9b1a is described below

commit ead9b1af1f48d6bf4430ab96c60a085caa46088a
Author: Chris Olivier 
AuthorDate: Wed Feb 7 14:34:39 2018 -0800

Add my key (#9736)
---
 KEYS | 112 +--
 1 file changed, 55 insertions(+), 57 deletions(-)

diff --git a/KEYS b/KEYS
index 77710ad..fa88584 100644
--- a/KEYS
+++ b/KEYS
@@ -304,66 +304,64 @@ 
IjljtjhIMhMLB5rf8BPCZ6og5fKqUF5LOp8DujG2DGa9ZhYWTzOO/UGZP60qGTot
 GZZVNUU0hQYfulYDY5E8fJ4Olzpf5OE=
 =WmLB
 -END PGP PUBLIC KEY BLOCK-
-pub   rsa4096 2017-11-21 [SC]
-  331E9A5ED727FADD429B2894F2F1EAB589EBCFB1
-uid   [ultimate] Haibin Lin 
-sig 3F2F1EAB589EBCFB1 2017-11-21  Haibin Lin 
-sub   rsa4096 2017-11-21 [E]
-sig  F2F1EAB589EBCFB1 2017-11-21  Haibin Lin 
+pub   4096R/C622DF82 2017-11-29
+uid  Chris Olivier (CODE SIGNING KEY) 
+sig 3C622DF82 2017-11-29  Chris Olivier (CODE SIGNING KEY) 

+sub   4096R/A70D0AA4 2017-11-29
+sig  C622DF82 2017-11-29  Chris Olivier (CODE SIGNING KEY) 

 
 -BEGIN PGP PUBLIC KEY BLOCK-
+Version: GnuPG v1
 
-mQINBFoTp3YBEACiGa++rsTjQal+33xADuWxzN9L8bTkMu4uFJqYvyNP2z1Q0fcM
-DFjLJcvsc3ODSlkDGlkrtFpYlqkBTFERABU19TcAQ5FYFu1uULUybtHm55h6OKAm
-1qfSRcKvdidDRytf7XAnhK/jvjtY71EQZUz2OtvKj0p93C22JcaJasKjHEF+8Jv0
-1rvV4BsZcY3hl9ORbv+nvBB6PX6zkpfhh0edVl50yzJEM34dtBZ1CTVlcJhIj0yo
-LEZkt+zKEz5C3/D5OgM2DoclUInAvPeIGXvOgoQi9he4YjMppC3fmcA9O+sJ8XFh
-dqNxcI+ddcvg84g4ntC2iJb8OOX75xkkoIsJXhZgwxBbdnwINNY6Eqqyx2lMvGRI
-BLTSxLKsfX/mCmW9mwNrKxfrBIb107ldxwfo+13/Vh45nIlhM0yxfhlukHmYEHp+
-G+T+aD67t0HHZHr27M2x0qTdKkRoI+7xYTUvu+OmObJej48UDhi4GMAjQ61TeLm1
-OyetyMoKpB+Cah1n0O5j6nDPRJBS9OPi361DIZRhlg4IkrbIP5MHs+Zvof8O04xq
-GRfYAqEhT6rP98TidpHVhFEV3CrDLVDJLZ3Vqglj2iyNOjEjF1GJJBaFWUoXhKPs
-WVZMfgpkaXRwng6r6ieRmmt/Ci//JV6ztkwKk7e0OQJBqbwA0A7lqx7j2QARAQAB
-tCVIYWliaW4gTGluIDxsaW5oYWliaW4uZXJpY0BnbWFpbC5jb20+iQJOBBMBCAA4
-FiEEMx6aXtcn+t1CmyiU8vHqtYnrz7EFAloTp3YCGwMFCwkIBwIGFQgJCgsCBBYC
-AwECHgECF4AACgkQ8vHqtYnrz7GFWA//Z6YTxtlZSHFlqkAFFOsDtV3DghSC8zJe
-LRm508fZn53e9a3fUvT9U1sUfW8DI69GRK+IBkvP5hcmMb1U4N3MxzX4YC/13wMY
-3BtUbCIpD8uBJOtuC7fPAH//Ij/4wv4Fp1/3WL6y04+mJIayMyKqmc3nBLD0rVWC
-AHEsPR7tiDDMltrzxMNHIJCDaiClJzKiCrQ4owKBOnY2TU/E64xyk5IwAczz2lCY
-712h6+q2mO7F672Yt6b6pqmugnFqWdqUj9dx1V9x//4y/k0DefF7G/1Lk1lh4Eyo
-aUx3jve/74Y87ICW1AhR2/TvdfWbsAkPyfy98k1SLR/9BulSIXIFeduxaFl7M3D8
-98aB5pqO8tPl2BFUJwh/uywDx0994MjQ8Xvrjmb9WJOAx9OyokivVCvmqJOkBzve
-Fk/4KUHTFTGQCoXbbBlIQTC9hBd8c1S4t0gFGbcjlqTvr/ZnTdpSgbzZ/96/SVRm
-dYOgjjpkrBOZgJPwsmmRQ2MufeZUtmkFSqdIRLGBNTefsMDDCGvyNeR/XCgM5Zfy
-39PX/GHFKgq5Ei2ywEyZOGLCK5MwA12fMExYoedazFFjv6ApGpz+j831A2z/crEo
-bRpVvd+rFzGnCKDq5viUD7cRzIPLVltYCNEayEgWta4KI+00/ayaaT6sM7N7oM32
-r01Wv02FvdG5Ag0EWhOndgEQAPiiTvmo9fZNW/5IxL7kDR6u9FEmEb2EZI+KxzbN
-RYYY0IPsnA8TY9Rzj9D7xV8Vmf2Pd5SUyCtVwLfBKhadLh755NeehNXWIbW802gH
-bvbykL/Zcn98oiLOVfK/Op/6MVpDuGXZ6CpDbQDSn6ne6/CWQnoz1+Wo+wbs1TOy
-AhO6xKa20NtGIZrfZD01dSzRC5DMJD3GK1j6HdVUz5piwiTsGvGRJ3ZLfObdlHGn
-CTMA39Jb8zQ0QtWPsOre0Nz2JQ53awMBaUhan5MeoOYp6ccsgD1BigyxmKb8iIDN
-NM/Iwi0Ib5L4AiGh6fQFf0WF8p74yIn1WgFcWxJXR1ZzvMDDHXqq97SQtbr9FKhu
-xrceh/92Ga4ruAJRCbMtmOTUP4APTeT4csANdgJxtW+I4QAp01BQSl75pB2QDlam
-+tqePQDboAGc78Ck6096wML0ZMKDDxXPrI67uppuM02FYuJ41ZQjOytigeoGS88g
-ByZwPcFIT+5XgtNC0BH7U9VIkiap5U00lykzEjcRjrZTtKqHdeFPbSEpv1QfIcLG
-Ra439g9acRHX82sVzhzZk5uu9QKyDN1EpuWoLOaOrICHcMSC7GkVXS8+/7TX0vAN
-vn/51fb+tHJekGfaPhsPuIbSba2kmUy8sSS/6JJHkJ1aEFigAPbwUbZTqNlb4IRm
-FBVBABEBAAGJAjYEGAEIACAWIQQzHppe1yf63UKbKJTy8eq1ievPsQUCWhOndgIb
-DAAKCRDy8eq1ievPsbrpEACQ8HqAvq3NuiM00WyHla7VtghCWVEmRozbYc4dR7u+
-sTQrVgbLfgR5zeSWCMHpEcaN/RS58O/i1Dk0DLHTu3NrarzrkEPlHwIgJQ7orxFD
-YW3Z2Ytk40uKex4ou/8VzvXTpj1u8d/GHgGdvChBmtw5FaMgc8PBi4FnlIS5cAGU
-1ca1RwMX0WpFsp9HgrQLVxgkDs/m7oRSmC5GvPDIpb5S9QFzJKYKTJxSfXXO6hCk
-FGAGHWjVC26a/wSUtZQfb3G9sYZJuKUOwr4tpz1y6Ronc34cZYi1FlKWJuz01w4s
-4PKjFG/wbYSd+QLfftyyVPMLdY+wCwc8O59QqKx5Rj8HQLxIwSL3chhmdAHCmejM
-zKCpkFyLOc6+Wjet6hD6X3EsjIee1AAy22D24EaLJsju9zR/khJFS4K76aQX7dYN
-aB3C7S5HGxvYGSqfnn4eBaEzrSOde7HEcqYpYKxS+jB1c4X4W91NSTsqDd0QJMVF
-35eKfhWj+X6jWIC+48kfzypXdOCnPbto7wrr40yYCHw3XSXj40H5dWSsWEZVmS+s
-Dzz6zy9maHVyXa/rNsL7OjqimtKad65r/wfSFPPIcR1jJfP4GMNHV0TYqxdyDaXg
-iEVpHzOV7gd75fJbOvoNxNZj20Yj5sg8OCwbv8PxLXEcBFs7hhjQMhVRsjpNYzAR
-Iw==
-=rMlc
+mQINBFoe7TkBEACjmUNxxx+nvLv7+NO+J/dkzR+/imuMPp6Op1wP6dXWQ+gj5n9V
+ILMaGQcFp6LLaoFZQgGjFVsM6FZ4d92yT9pIbXg0VNCGGfmnQEQEwE3ZWdeQeOdz
+tbAqOiX/ZSw6C7gsUivMKBbMY3iJ46dgE2eqYD7aRyOYbP2mgX4mkmez/05bM48n
+8cuLj6dZgt8DduUwabYhrm2/xpSpaHnYKqwE3E0bhek0EEZ9Plmo6c+qW0v01iw0

[GitHub] danithaca commented on issue #9735: Feature request: Re-write FastText in MXNet

[GitBox]

danithaca commented on issue #9735: Feature request: Re-write FastText in MXNet
URL: 
https://github.com/apache/incubator-mxnet/issues/9735#issuecomment-363942797
 
 
   To us, efficiency is nice but perhaps not as important as the ability to do 
it inside MXNET. If we can have something simple for v1, then we can gradually 
build on top of it, and maybe contribute some code back.


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[GitHub] piiswrong commented on issue #9729: how to use the transforms

[GitBox]

piiswrong commented on issue #9729: how to use the transforms 
URL: 
https://github.com/apache/incubator-mxnet/issues/9729#issuecomment-363936749
 
 
   `train_data = 
gluon.data.vision.MNIST(train=True).transform_first(transforms.ToTensor())`


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[incubator-mxnet] branch master updated: Add my key (#9736)

[liuyizhi]

This is an automated email from the ASF dual-hosted git repository.

liuyizhi 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 a51cdb1  Add my key (#9736)
a51cdb1 is described below

commit a51cdb1bdeec521618d491f136621f7382588416
Author: Chris Olivier 
AuthorDate: Wed Feb 7 14:34:39 2018 -0800

Add my key (#9736)
---
 KEYS | 112 +--
 1 file changed, 55 insertions(+), 57 deletions(-)

diff --git a/KEYS b/KEYS
index 77710ad..fa88584 100644
--- a/KEYS
+++ b/KEYS
@@ -304,66 +304,64 @@ 
IjljtjhIMhMLB5rf8BPCZ6og5fKqUF5LOp8DujG2DGa9ZhYWTzOO/UGZP60qGTot
 GZZVNUU0hQYfulYDY5E8fJ4Olzpf5OE=
 =WmLB
 -END PGP PUBLIC KEY BLOCK-
-pub   rsa4096 2017-11-21 [SC]
-  331E9A5ED727FADD429B2894F2F1EAB589EBCFB1
-uid   [ultimate] Haibin Lin 
-sig 3F2F1EAB589EBCFB1 2017-11-21  Haibin Lin 
-sub   rsa4096 2017-11-21 [E]
-sig  F2F1EAB589EBCFB1 2017-11-21  Haibin Lin 
+pub   4096R/C622DF82 2017-11-29
+uid  Chris Olivier (CODE SIGNING KEY) 
+sig 3C622DF82 2017-11-29  Chris Olivier (CODE SIGNING KEY) 

+sub   4096R/A70D0AA4 2017-11-29
+sig  C622DF82 2017-11-29  Chris Olivier (CODE SIGNING KEY) 

 
 -BEGIN PGP PUBLIC KEY BLOCK-
+Version: GnuPG v1
 
-mQINBFoTp3YBEACiGa++rsTjQal+33xADuWxzN9L8bTkMu4uFJqYvyNP2z1Q0fcM
-DFjLJcvsc3ODSlkDGlkrtFpYlqkBTFERABU19TcAQ5FYFu1uULUybtHm55h6OKAm
-1qfSRcKvdidDRytf7XAnhK/jvjtY71EQZUz2OtvKj0p93C22JcaJasKjHEF+8Jv0
-1rvV4BsZcY3hl9ORbv+nvBB6PX6zkpfhh0edVl50yzJEM34dtBZ1CTVlcJhIj0yo
-LEZkt+zKEz5C3/D5OgM2DoclUInAvPeIGXvOgoQi9he4YjMppC3fmcA9O+sJ8XFh
-dqNxcI+ddcvg84g4ntC2iJb8OOX75xkkoIsJXhZgwxBbdnwINNY6Eqqyx2lMvGRI
-BLTSxLKsfX/mCmW9mwNrKxfrBIb107ldxwfo+13/Vh45nIlhM0yxfhlukHmYEHp+
-G+T+aD67t0HHZHr27M2x0qTdKkRoI+7xYTUvu+OmObJej48UDhi4GMAjQ61TeLm1
-OyetyMoKpB+Cah1n0O5j6nDPRJBS9OPi361DIZRhlg4IkrbIP5MHs+Zvof8O04xq
-GRfYAqEhT6rP98TidpHVhFEV3CrDLVDJLZ3Vqglj2iyNOjEjF1GJJBaFWUoXhKPs
-WVZMfgpkaXRwng6r6ieRmmt/Ci//JV6ztkwKk7e0OQJBqbwA0A7lqx7j2QARAQAB
-tCVIYWliaW4gTGluIDxsaW5oYWliaW4uZXJpY0BnbWFpbC5jb20+iQJOBBMBCAA4
-FiEEMx6aXtcn+t1CmyiU8vHqtYnrz7EFAloTp3YCGwMFCwkIBwIGFQgJCgsCBBYC
-AwECHgECF4AACgkQ8vHqtYnrz7GFWA//Z6YTxtlZSHFlqkAFFOsDtV3DghSC8zJe
-LRm508fZn53e9a3fUvT9U1sUfW8DI69GRK+IBkvP5hcmMb1U4N3MxzX4YC/13wMY
-3BtUbCIpD8uBJOtuC7fPAH//Ij/4wv4Fp1/3WL6y04+mJIayMyKqmc3nBLD0rVWC
-AHEsPR7tiDDMltrzxMNHIJCDaiClJzKiCrQ4owKBOnY2TU/E64xyk5IwAczz2lCY
-712h6+q2mO7F672Yt6b6pqmugnFqWdqUj9dx1V9x//4y/k0DefF7G/1Lk1lh4Eyo
-aUx3jve/74Y87ICW1AhR2/TvdfWbsAkPyfy98k1SLR/9BulSIXIFeduxaFl7M3D8
-98aB5pqO8tPl2BFUJwh/uywDx0994MjQ8Xvrjmb9WJOAx9OyokivVCvmqJOkBzve
-Fk/4KUHTFTGQCoXbbBlIQTC9hBd8c1S4t0gFGbcjlqTvr/ZnTdpSgbzZ/96/SVRm
-dYOgjjpkrBOZgJPwsmmRQ2MufeZUtmkFSqdIRLGBNTefsMDDCGvyNeR/XCgM5Zfy
-39PX/GHFKgq5Ei2ywEyZOGLCK5MwA12fMExYoedazFFjv6ApGpz+j831A2z/crEo
-bRpVvd+rFzGnCKDq5viUD7cRzIPLVltYCNEayEgWta4KI+00/ayaaT6sM7N7oM32
-r01Wv02FvdG5Ag0EWhOndgEQAPiiTvmo9fZNW/5IxL7kDR6u9FEmEb2EZI+KxzbN
-RYYY0IPsnA8TY9Rzj9D7xV8Vmf2Pd5SUyCtVwLfBKhadLh755NeehNXWIbW802gH
-bvbykL/Zcn98oiLOVfK/Op/6MVpDuGXZ6CpDbQDSn6ne6/CWQnoz1+Wo+wbs1TOy
-AhO6xKa20NtGIZrfZD01dSzRC5DMJD3GK1j6HdVUz5piwiTsGvGRJ3ZLfObdlHGn
-CTMA39Jb8zQ0QtWPsOre0Nz2JQ53awMBaUhan5MeoOYp6ccsgD1BigyxmKb8iIDN
-NM/Iwi0Ib5L4AiGh6fQFf0WF8p74yIn1WgFcWxJXR1ZzvMDDHXqq97SQtbr9FKhu
-xrceh/92Ga4ruAJRCbMtmOTUP4APTeT4csANdgJxtW+I4QAp01BQSl75pB2QDlam
-+tqePQDboAGc78Ck6096wML0ZMKDDxXPrI67uppuM02FYuJ41ZQjOytigeoGS88g
-ByZwPcFIT+5XgtNC0BH7U9VIkiap5U00lykzEjcRjrZTtKqHdeFPbSEpv1QfIcLG
-Ra439g9acRHX82sVzhzZk5uu9QKyDN1EpuWoLOaOrICHcMSC7GkVXS8+/7TX0vAN
-vn/51fb+tHJekGfaPhsPuIbSba2kmUy8sSS/6JJHkJ1aEFigAPbwUbZTqNlb4IRm
-FBVBABEBAAGJAjYEGAEIACAWIQQzHppe1yf63UKbKJTy8eq1ievPsQUCWhOndgIb
-DAAKCRDy8eq1ievPsbrpEACQ8HqAvq3NuiM00WyHla7VtghCWVEmRozbYc4dR7u+
-sTQrVgbLfgR5zeSWCMHpEcaN/RS58O/i1Dk0DLHTu3NrarzrkEPlHwIgJQ7orxFD
-YW3Z2Ytk40uKex4ou/8VzvXTpj1u8d/GHgGdvChBmtw5FaMgc8PBi4FnlIS5cAGU
-1ca1RwMX0WpFsp9HgrQLVxgkDs/m7oRSmC5GvPDIpb5S9QFzJKYKTJxSfXXO6hCk
-FGAGHWjVC26a/wSUtZQfb3G9sYZJuKUOwr4tpz1y6Ronc34cZYi1FlKWJuz01w4s
-4PKjFG/wbYSd+QLfftyyVPMLdY+wCwc8O59QqKx5Rj8HQLxIwSL3chhmdAHCmejM
-zKCpkFyLOc6+Wjet6hD6X3EsjIee1AAy22D24EaLJsju9zR/khJFS4K76aQX7dYN
-aB3C7S5HGxvYGSqfnn4eBaEzrSOde7HEcqYpYKxS+jB1c4X4W91NSTsqDd0QJMVF
-35eKfhWj+X6jWIC+48kfzypXdOCnPbto7wrr40yYCHw3XSXj40H5dWSsWEZVmS+s
-Dzz6zy9maHVyXa/rNsL7OjqimtKad65r/wfSFPPIcR1jJfP4GMNHV0TYqxdyDaXg
-iEVpHzOV7gd75fJbOvoNxNZj20Yj5sg8OCwbv8PxLXEcBFs7hhjQMhVRsjpNYzAR
-Iw==
-=rMlc
+mQINBFoe7TkBEACjmUNxxx+nvLv7+NO+J/dkzR+/imuMPp6Op1wP6dXWQ+gj5n9V
+ILMaGQcFp6LLaoFZQgGjFVsM6FZ4d92yT9pIbXg0VNCGGfmnQEQEwE3ZWdeQeOdz
+tbAqOiX/ZSw6C7gsUivMKBbMY3iJ46dgE2eqYD7aRyOYbP2mgX4mkmez/05bM48n
+8cuLj6dZgt8DduUwabYhrm2/xpSpaHnYKqwE3E0bhek0EEZ9Plmo6c+qW0v01iw0

[GitHub] yzhliu closed pull request #9736: Remove Haibin's unused gmail key and added my apache.org key in its place

[GitBox]

yzhliu closed pull request #9736: Remove Haibin's unused gmail key and added my 
apache.org key in its place
URL: https://github.com/apache/incubator-mxnet/pull/9736
 
 
   

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 (from a fork), the diff is supplied
below (as it won't show otherwise due to GitHub magic):

diff --git a/KEYS b/KEYS
index 77710ad5c4..fa885848d9 100644
--- a/KEYS
+++ b/KEYS
@@ -304,66 +304,64 @@ 
IjljtjhIMhMLB5rf8BPCZ6og5fKqUF5LOp8DujG2DGa9ZhYWTzOO/UGZP60qGTot
 GZZVNUU0hQYfulYDY5E8fJ4Olzpf5OE=
 =WmLB
 -END PGP PUBLIC KEY BLOCK-
-pub   rsa4096 2017-11-21 [SC]
-  331E9A5ED727FADD429B2894F2F1EAB589EBCFB1
-uid   [ultimate] Haibin Lin 
-sig 3F2F1EAB589EBCFB1 2017-11-21  Haibin Lin 
-sub   rsa4096 2017-11-21 [E]
-sig  F2F1EAB589EBCFB1 2017-11-21  Haibin Lin 
+pub   4096R/C622DF82 2017-11-29
+uid  Chris Olivier (CODE SIGNING KEY) 
+sig 3C622DF82 2017-11-29  Chris Olivier (CODE SIGNING KEY) 

+sub   4096R/A70D0AA4 2017-11-29
+sig  C622DF82 2017-11-29  Chris Olivier (CODE SIGNING KEY) 

 
 -BEGIN PGP PUBLIC KEY BLOCK-
+Version: GnuPG v1
 
-mQINBFoTp3YBEACiGa++rsTjQal+33xADuWxzN9L8bTkMu4uFJqYvyNP2z1Q0fcM
-DFjLJcvsc3ODSlkDGlkrtFpYlqkBTFERABU19TcAQ5FYFu1uULUybtHm55h6OKAm
-1qfSRcKvdidDRytf7XAnhK/jvjtY71EQZUz2OtvKj0p93C22JcaJasKjHEF+8Jv0
-1rvV4BsZcY3hl9ORbv+nvBB6PX6zkpfhh0edVl50yzJEM34dtBZ1CTVlcJhIj0yo
-LEZkt+zKEz5C3/D5OgM2DoclUInAvPeIGXvOgoQi9he4YjMppC3fmcA9O+sJ8XFh
-dqNxcI+ddcvg84g4ntC2iJb8OOX75xkkoIsJXhZgwxBbdnwINNY6Eqqyx2lMvGRI
-BLTSxLKsfX/mCmW9mwNrKxfrBIb107ldxwfo+13/Vh45nIlhM0yxfhlukHmYEHp+
-G+T+aD67t0HHZHr27M2x0qTdKkRoI+7xYTUvu+OmObJej48UDhi4GMAjQ61TeLm1
-OyetyMoKpB+Cah1n0O5j6nDPRJBS9OPi361DIZRhlg4IkrbIP5MHs+Zvof8O04xq
-GRfYAqEhT6rP98TidpHVhFEV3CrDLVDJLZ3Vqglj2iyNOjEjF1GJJBaFWUoXhKPs
-WVZMfgpkaXRwng6r6ieRmmt/Ci//JV6ztkwKk7e0OQJBqbwA0A7lqx7j2QARAQAB
-tCVIYWliaW4gTGluIDxsaW5oYWliaW4uZXJpY0BnbWFpbC5jb20+iQJOBBMBCAA4
-FiEEMx6aXtcn+t1CmyiU8vHqtYnrz7EFAloTp3YCGwMFCwkIBwIGFQgJCgsCBBYC
-AwECHgECF4AACgkQ8vHqtYnrz7GFWA//Z6YTxtlZSHFlqkAFFOsDtV3DghSC8zJe
-LRm508fZn53e9a3fUvT9U1sUfW8DI69GRK+IBkvP5hcmMb1U4N3MxzX4YC/13wMY
-3BtUbCIpD8uBJOtuC7fPAH//Ij/4wv4Fp1/3WL6y04+mJIayMyKqmc3nBLD0rVWC
-AHEsPR7tiDDMltrzxMNHIJCDaiClJzKiCrQ4owKBOnY2TU/E64xyk5IwAczz2lCY
-712h6+q2mO7F672Yt6b6pqmugnFqWdqUj9dx1V9x//4y/k0DefF7G/1Lk1lh4Eyo
-aUx3jve/74Y87ICW1AhR2/TvdfWbsAkPyfy98k1SLR/9BulSIXIFeduxaFl7M3D8
-98aB5pqO8tPl2BFUJwh/uywDx0994MjQ8Xvrjmb9WJOAx9OyokivVCvmqJOkBzve
-Fk/4KUHTFTGQCoXbbBlIQTC9hBd8c1S4t0gFGbcjlqTvr/ZnTdpSgbzZ/96/SVRm
-dYOgjjpkrBOZgJPwsmmRQ2MufeZUtmkFSqdIRLGBNTefsMDDCGvyNeR/XCgM5Zfy
-39PX/GHFKgq5Ei2ywEyZOGLCK5MwA12fMExYoedazFFjv6ApGpz+j831A2z/crEo
-bRpVvd+rFzGnCKDq5viUD7cRzIPLVltYCNEayEgWta4KI+00/ayaaT6sM7N7oM32
-r01Wv02FvdG5Ag0EWhOndgEQAPiiTvmo9fZNW/5IxL7kDR6u9FEmEb2EZI+KxzbN
-RYYY0IPsnA8TY9Rzj9D7xV8Vmf2Pd5SUyCtVwLfBKhadLh755NeehNXWIbW802gH
-bvbykL/Zcn98oiLOVfK/Op/6MVpDuGXZ6CpDbQDSn6ne6/CWQnoz1+Wo+wbs1TOy
-AhO6xKa20NtGIZrfZD01dSzRC5DMJD3GK1j6HdVUz5piwiTsGvGRJ3ZLfObdlHGn
-CTMA39Jb8zQ0QtWPsOre0Nz2JQ53awMBaUhan5MeoOYp6ccsgD1BigyxmKb8iIDN
-NM/Iwi0Ib5L4AiGh6fQFf0WF8p74yIn1WgFcWxJXR1ZzvMDDHXqq97SQtbr9FKhu
-xrceh/92Ga4ruAJRCbMtmOTUP4APTeT4csANdgJxtW+I4QAp01BQSl75pB2QDlam
-+tqePQDboAGc78Ck6096wML0ZMKDDxXPrI67uppuM02FYuJ41ZQjOytigeoGS88g
-ByZwPcFIT+5XgtNC0BH7U9VIkiap5U00lykzEjcRjrZTtKqHdeFPbSEpv1QfIcLG
-Ra439g9acRHX82sVzhzZk5uu9QKyDN1EpuWoLOaOrICHcMSC7GkVXS8+/7TX0vAN
-vn/51fb+tHJekGfaPhsPuIbSba2kmUy8sSS/6JJHkJ1aEFigAPbwUbZTqNlb4IRm
-FBVBABEBAAGJAjYEGAEIACAWIQQzHppe1yf63UKbKJTy8eq1ievPsQUCWhOndgIb
-DAAKCRDy8eq1ievPsbrpEACQ8HqAvq3NuiM00WyHla7VtghCWVEmRozbYc4dR7u+
-sTQrVgbLfgR5zeSWCMHpEcaN/RS58O/i1Dk0DLHTu3NrarzrkEPlHwIgJQ7orxFD
-YW3Z2Ytk40uKex4ou/8VzvXTpj1u8d/GHgGdvChBmtw5FaMgc8PBi4FnlIS5cAGU
-1ca1RwMX0WpFsp9HgrQLVxgkDs/m7oRSmC5GvPDIpb5S9QFzJKYKTJxSfXXO6hCk
-FGAGHWjVC26a/wSUtZQfb3G9sYZJuKUOwr4tpz1y6Ronc34cZYi1FlKWJuz01w4s
-4PKjFG/wbYSd+QLfftyyVPMLdY+wCwc8O59QqKx5Rj8HQLxIwSL3chhmdAHCmejM
-zKCpkFyLOc6+Wjet6hD6X3EsjIee1AAy22D24EaLJsju9zR/khJFS4K76aQX7dYN
-aB3C7S5HGxvYGSqfnn4eBaEzrSOde7HEcqYpYKxS+jB1c4X4W91NSTsqDd0QJMVF
-35eKfhWj+X6jWIC+48kfzypXdOCnPbto7wrr40yYCHw3XSXj40H5dWSsWEZVmS+s
-Dzz6zy9maHVyXa/rNsL7OjqimtKad65r/wfSFPPIcR1jJfP4GMNHV0TYqxdyDaXg
-iEVpHzOV7gd75fJbOvoNxNZj20Yj5sg8OCwbv8PxLXEcBFs7hhjQMhVRsjpNYzAR
-Iw==
-=rMlc
+mQINBFoe7TkBEACjmUNxxx+nvLv7+NO+J/dkzR+/imuMPp6Op1wP6dXWQ+gj5n9V
+ILMaGQcFp6LLaoFZQgGjFVsM6FZ4d92yT9pIbXg0VNCGGfmnQEQEwE3ZWdeQeOdz
+tbAqOiX/ZSw6C7gsUivMKBbMY3iJ46dgE2eqYD7aRyOYbP2mgX4mkmez/05bM48n
+8cuLj6dZgt8DduUwabYhrm2/xpSpaHnYKqwE3E0bhek0EEZ9Plmo6c+qW0v01iw0
+Y9EJQgA1Ulayh8Ub4JX1ZlqP/zOCmNB1tnNS7Y93sEDVZ942T/f+xX2Myk71dq4j
++Fm+q9V62Xu+zrHgdsMj8G84bsR8iGRmJHI0ZpjRN3e8Uvh+r4+h4UhFiSO3MP8G
+zX7BJvckOp0/zRKeMuq4UFBL+6lgYJWiN5KyG41v4zNiuuoNL9DE9brIRfwwrFxX

[GitHub] Sinnombre opened a new issue #9737: dot product backwards in R

[GitBox]

Sinnombre opened a new issue #9737: dot product backwards in R
URL: https://github.com/apache/incubator-mxnet/issues/9737
 
 
   ## Description
   When using R, the routine mx.nd.dot processes the inputs in the wrong order. 
See example below. A 1x3 vector dot a 3x1 vector should be a scalar, and a 3x1 
dot 1x3 should be 3x3, not the other way around. I'm guessing this has to do 
with R being column major instead of row major.
   
   ## Environment info (Required)
   > sessionInfo()
   R version 3.4.3 (2017-11-30)
   Platform: x86_64-w64-mingw32/x64 (64-bit)
   Running under: Windows >= 8 x64 (build 9200)
   
   Matrix products: default
   
   locale:
   [1] LC_COLLATE=English_United States.1252 
   [2] LC_CTYPE=English_United States.1252   
   [3] LC_MONETARY=English_United States.1252
   [4] LC_NUMERIC=C  
   [5] LC_TIME=English_United States.1252
   
   attached base packages:
   [1] stats graphics  grDevices utils datasets 
   [6] methods   base 
   
   other attached packages:
   [1] mxnet_0.10.1
   
   loaded via a namespace (and not attached):
[1] Rcpp_0.12.14   compiler_3.4.3
[3] RColorBrewer_1.1-2 influenceR_0.1.0  
[5] plyr_1.8.4 bindr_0.1 
[7] viridis_0.5.0  tools_3.4.3   
[9] digest_0.6.12  jsonlite_1.5  
   [11] viridisLite_0.3.0  tibble_1.3.4  
   [13] gtable_0.2.0   rgexf_0.15.3  
   [15] pkgconfig_2.0.1rlang_0.1.4   
   [17] igraph_1.1.2   rstudioapi_0.7
   [19] bindrcpp_0.2   gridExtra_2.3 
   [21] downloader_0.4 DiagrammeR_0.9.2  
   [23] dplyr_0.7.4stringr_1.2.0 
   [25] htmlwidgets_1.0hms_0.4.1 
   [27] grid_3.4.3 glue_1.2.0
   [29] R6_2.2.2   Rook_1.1-1
   [31] XML_3.98-1.9   readr_1.1.1   
   [33] purrr_0.2.4tidyr_0.8.0   
   [35] ggplot2_2.2.1  magrittr_1.5  
   [37] codetools_0.2-15   scales_0.5.0  
   [39] htmltools_0.3.6assertthat_0.2.0  
   [41] colorspace_1.3-2   brew_1.0-6
   [43] stringi_1.1.6  visNetwork_2.0.3  
   [45] lazyeval_0.2.1 munsell_0.4.3 
   
   ## Error Message:
   N/A
   
   ## Minimum reproducible example
   > a <- mx.nd.ones(c(1,3), ctx = mx.cpu())
   > b <- mx.nd.ones(c(3,1), ctx = mx.cpu())
   > a
[,1] [,2] [,3]
   [1,]111
   > b
[,1]
   [1,]1
   [2,]1
   [3,]1
   > mx.nd.dot(a,b)
[,1] [,2] [,3]
   [1,]111
   [2,]111
   [3,]111
   > mx.nd.dot(b,a)
[,1]
   [1,]3
   
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[GitHub] szha commented on issue #9662: Gluon PReLU, ELU, SELU, Swish

[GitBox]

szha commented on issue #9662: Gluon PReLU, ELU, SELU, Swish
URL: https://github.com/apache/incubator-mxnet/pull/9662#issuecomment-363929433
 
 
   @piiswrong, I addressed latest comments. Let me know if any further change 
is needed.


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[GitHub] szha commented on issue #9735: Feature request: Re-write FastText in MXNet

[GitBox]

szha commented on issue #9735: Feature request: Re-write FastText in MXNet
URL: 
https://github.com/apache/incubator-mxnet/issues/9735#issuecomment-363929115
 
 
   This is definitely worth exploring. It will take some work to match the 
efficiency with the optimized implementation in facebook's fasttext. 
@astonzhang 


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[GitHub] cjolivier01 opened a new pull request #9736: Remove Haibin's unused gmail key and added my apache.org key in its place

[GitBox]

cjolivier01 opened a new pull request #9736: Remove Haibin's unused gmail key 
and added my apache.org key in its place
URL: https://github.com/apache/incubator-mxnet/pull/9736
 
 
   ## Description ##
   
   Remove Haibin's unused gmail key and added my apache.org key in its place
   
   ## Checklist ##
   ### Essentials ###
   - [ ] Passed code style checking (`make lint`)
   - [ ] Changes are complete (i.e. I finished coding on this PR)
   - [ ] All changes have test coverage:
   - Unit tests are added for small changes to verify correctness (e.g. adding 
a new operator)
   - Nightly tests are added for complicated/long-running ones (e.g. changing 
distributed kvstore)
   - Build tests will be added for build configuration changes (e.g. adding a 
new build option with NCCL)
   - [ ] Code is well-documented: 
   - For user-facing API changes, API doc string has been updated. 
   - For new C++ functions in header files, their functionalities and arguments 
are documented. 
   - For new examples, README.md is added to explain the what the example does, 
the source of the dataset, expected performance on test set and reference to 
the original paper if applicable
   - [ ] To the my best knowledge, examples are either not affected by this 
change, or have been fixed to be compatible with this change
   
   ### Changes ###
   - [ ] Feature1, tests, (and when applicable, API doc)
   - [ ] Feature2, tests, (and when applicable, API doc)
   
   ## Comments ##
   - If this change is a backward incompatible change, why must this change be 
made.
   - Interesting edge cases to note here
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[GitHub] danithaca opened a new issue #9735: Feature request: Re-write FastText in MXNet

[GitBox]

danithaca opened a new issue #9735: Feature request: Re-write FastText in MXNet
URL: https://github.com/apache/incubator-mxnet/issues/9735
 
 
   Can someone rewrite FastText in MXNet? I saw mxnet now has pre-trained 
FastText embeddings (#9551). But it would be really cool to compute embeddings 
directly from MXNET and then do supervised learning altogether. That way we 
don't have to prepare data twice for both FastText and MXNet.
   
   Someone reimplemented FastText in TensorFlow: 
   https://github.com/apcode/tensorflow_fasttext


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[GitHub] chsin commented on issue #8126: Not able to train a neural network [XOR added]

[GitBox]

chsin commented on issue #8126: Not able to train a neural network [XOR added]
URL: 
https://github.com/apache/incubator-mxnet/issues/8126#issuecomment-363900630
 
 
   I'm having the same problem, but for the lenet.cpp example using just the 
CPU (example in 
https://github.com/apache/incubator-mxnet/blob/master/cpp-package/example/lenet.cpp).
 I built v1.0.0 Mxnet with MKL, but that shouldn't be the issue since you have 
the same problem. Does lenet.cpp fail to learn for you too? 


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[GitHub] chsin commented on issue #8126: Not able to train a neural network [XOR added]

[GitBox]

chsin commented on issue #8126: Not able to train a neural network [XOR added]
URL: 
https://github.com/apache/incubator-mxnet/issues/8126#issuecomment-363900630
 
 
   I'm having the same problem, bu for the lenet.cpp example using just the CPU 
(example in 
https://github.com/apache/incubator-mxnet/blob/master/cpp-package/example/lenet.cpp).
 I built v1.0.0 Mxnet with MKL, but that shouldn't be the issue since you have 
the same problem. Does lenet.cpp fail to learn for you too? 


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[GitHub] chsin commented on issue #8126: Not able to train a neural network [XOR added]

[GitBox]

chsin commented on issue #8126: Not able to train a neural network [XOR added]
URL: 
https://github.com/apache/incubator-mxnet/issues/8126#issuecomment-363900630
 
 
   I'm having the same problem, but one for the lenet.cpp example using just 
the CPU (example in 
https://github.com/apache/incubator-mxnet/blob/master/cpp-package/example/lenet.cpp).
 I built v1.0.0 mxnet.so with MKL, but that shouldn't be the issue since you 
have the same problem. Does lenet.cpp fail to learn for you too? 


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[GitHub] chsin commented on issue #8126: Not able to train a neural network [XOR added]

[GitBox]

chsin commented on issue #8126: Not able to train a neural network [XOR added]
URL: 
https://github.com/apache/incubator-mxnet/issues/8126#issuecomment-363900630
 
 
   I'm having the same problem, bu for the lenet.cpp example using just the CPU 
(example in 
https://github.com/apache/incubator-mxnet/blob/master/cpp-package/example/lenet.cpp).
 I built v1.0.0 mxnet.so with MKL, but that shouldn't be the issue since you 
have the same problem. Does lenet.cpp fail to learn for you too? 


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[GitHub] piiswrong commented on issue #9727: Refactored storage handling

[GitBox]

piiswrong commented on issue #9727: Refactored storage handling
URL: https://github.com/apache/incubator-mxnet/pull/9727#issuecomment-363893822
 
 
   @cjolivier01 
   Since the current implementation is working OK, we need to make sure the 
refactor does not have visible impact on performance before accepting it.


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[GitHub] piiswrong commented on issue #9727: Refactored storage handling

[GitBox]

piiswrong commented on issue #9727: Refactored storage handling
URL: https://github.com/apache/incubator-mxnet/pull/9727#issuecomment-363893822
 
 
   @cjolivier01 
   Since the current implementation is working OK, we need to make sure the 
refactor does not have visible impact on performance before merging it.


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[GitHub] lebeg commented on issue #9727: Refactored storage handling

[GitBox]

lebeg commented on issue #9727: Refactored storage handling
URL: https://github.com/apache/incubator-mxnet/pull/9727#issuecomment-363878445
 
 
   > What's the benefit of using shared_ptr for storage? 
   
   The ownership of memory storage::Handles are pointing to with their dptr is 
protected from extinction and managed by the shared_ptr.
   
   > Chunks are already in shared_ptr and never copied anyway.
   
   There are other places which use storage::Handle and mostly as just a 
pointer to some allocated memory without access protection.
   
   > How much performance is lost by refactoring to shared_ptr and 
std::function?
   
   * There is an additional reference count stored on the heap next to each 
Handle
   * 1 indirection when constructing a TBlob


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[incubator-mxnet] branch master updated: Improve take documentation. (#9717)

[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 48dcbf2  Improve take documentation. (#9717)
48dcbf2 is described below

commit 48dcbf23ab7d0c1e103a8419ed4e72b1f9d43fa2
Author: Pedro Larroy <928489+lar...@users.noreply.github.com>
AuthorDate: Wed Feb 7 19:58:46 2018 +0100

Improve take documentation. (#9717)
---
 src/operator/tensor/indexing_op.cc | 6 +-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/src/operator/tensor/indexing_op.cc 
b/src/operator/tensor/indexing_op.cc
index b90a95d..cce4537 100644
--- a/src/operator/tensor/indexing_op.cc
+++ b/src/operator/tensor/indexing_op.cc
@@ -352,12 +352,16 @@ will have shape ``(i0, i1, d1, d2)``, computed by::
- `mode`- Only `clip` mode is supported for now.
 
 Examples::
+  x = [4.  5.  6.]
+
+  // Trivial case, take the second element along the first axis.
+  take(x, [1]) = [ 5. ]
 
   x = [[ 1.,  2.],
[ 3.,  4.],
[ 5.,  6.]]
 
-  // takes elements with specified indices along axis 0
+  // In this case we will get rows 0 and 1, then 1 and 2. Along axis 0
   take(x, [[0,1],[1,2]]) = [[[ 1.,  2.],
  [ 3.,  4.]],
 

-- 
To stop receiving notification emails like this one, please contact
j...@apache.org.

[GitHub] piiswrong closed pull request #9717: Doc improvement

[GitBox]

piiswrong closed pull request #9717: Doc improvement
URL: https://github.com/apache/incubator-mxnet/pull/9717
 
 
   

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 (from a fork), the diff is supplied
below (as it won't show otherwise due to GitHub magic):

diff --git a/src/operator/tensor/indexing_op.cc 
b/src/operator/tensor/indexing_op.cc
index b90a95dcf7..cce4537ae3 100644
--- a/src/operator/tensor/indexing_op.cc
+++ b/src/operator/tensor/indexing_op.cc
@@ -352,12 +352,16 @@ will have shape ``(i0, i1, d1, d2)``, computed by::
- `mode`- Only `clip` mode is supported for now.
 
 Examples::
+  x = [4.  5.  6.]
+
+  // Trivial case, take the second element along the first axis.
+  take(x, [1]) = [ 5. ]
 
   x = [[ 1.,  2.],
[ 3.,  4.],
[ 5.,  6.]]
 
-  // takes elements with specified indices along axis 0
+  // In this case we will get rows 0 and 1, then 1 and 2. Along axis 0
   take(x, [[0,1],[1,2]]) = [[[ 1.,  2.],
  [ 3.,  4.]],
 


 


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[GitHub] piiswrong commented on issue #9727: Refactored storage handling

[GitBox]

piiswrong commented on issue #9727: Refactored storage handling
URL: https://github.com/apache/incubator-mxnet/pull/9727#issuecomment-363870638
 
 
   What's the benefit of using shared_ptr for storage? Chunks are already in 
shared_ptr and never copied anyway.
   
   How much performance impact does refactoring to shared_ptr and std::function 
have?
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[GitHub] reminisce commented on issue #9723: Specify lint-versions and fix docker build issues due to nvidia-384

[GitBox]

reminisce commented on issue #9723: Specify lint-versions and fix docker build 
issues due to nvidia-384
URL: https://github.com/apache/incubator-mxnet/pull/9723#issuecomment-363871035
 
 
   Can this be merged?


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[GitHub] piiswrong commented on issue #9727: Refactored storage handling

[GitBox]

piiswrong commented on issue #9727: Refactored storage handling
URL: https://github.com/apache/incubator-mxnet/pull/9727#issuecomment-363870638
 
 
   What's the benefit of using shared_ptr for storage? Chunks are already in 
shared_ptr and never copied anyway.
   
   How much performance is lost by refactoring to shared_ptr and std::function?
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[GitHub] yzhliu closed issue #8549: What's the equivalent of tf.matmul in tensorflow

[GitBox]

yzhliu closed issue #8549: What's the equivalent of tf.matmul in tensorflow
URL: https://github.com/apache/incubator-mxnet/issues/8549
 
 
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[GitHub] yzhliu closed issue #8552: Problem in mx.io.ImageRecordIter

[GitBox]

yzhliu closed issue #8552: Problem in mx.io.ImageRecordIter
URL: https://github.com/apache/incubator-mxnet/issues/8552
 
 
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[GitHub] reminisce commented on issue #9724: undefined symbol: MXSymbolGetNumOutputs

[GitBox]

reminisce commented on issue #9724: undefined symbol: MXSymbolGetNumOutputs
URL: 
https://github.com/apache/incubator-mxnet/issues/9724#issuecomment-363865915
 
 
   Can you first check whether the file `./src/c_api/c_api_symbolic.cc` exists 
and the function `MXSymbolGetNumOutputs` is defined in that file and declared 
in `include/mxnet/c_api.h` when you built the lib?


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[GitHub] reminisce opened a new pull request #9734: Add convenience fluent method for squeeze op

[GitBox]

reminisce opened a new pull request #9734: Add convenience fluent method for 
squeeze op
URL: https://github.com/apache/incubator-mxnet/pull/9734
 
 
   ## Description ##
   To complement this https://github.com/apache/incubator-mxnet/pull/9700.
   @szha 


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[incubator-mxnet] branch v1.1.0 updated: PGP keys add liuyizhi AT apache.org (#9728)

[liuyizhi]

This is an automated email from the ASF dual-hosted git repository.

liuyizhi pushed a commit to branch v1.1.0
in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git


The following commit(s) were added to refs/heads/v1.1.0 by this push:
 new e49a1de  PGP keys add liuyizhi AT apache.org (#9728)
e49a1de is described below

commit e49a1de9167c1c6041c3e7c1635eadc3e34534fb
Author: Yizhi Liu 
AuthorDate: Wed Feb 7 10:10:20 2018 -0800

PGP keys add liuyizhi AT apache.org (#9728)
---
 KEYS | 59 +++
 1 file changed, 59 insertions(+)

diff --git a/KEYS b/KEYS
index 5e5769b..77710ad 100644
--- a/KEYS
+++ b/KEYS
@@ -423,3 +423,62 @@ 
fDiLw4372qAC4NpSLRpCIBbT33VztUOTmZgIg4zJiQGSp89dEVN8OUT/yjKQps39
 G4kKPLx5+UhNtHsaNA==
 =ZoTi
 -END PGP PUBLIC KEY BLOCK-
+
+pub   rsa4096 2018-02-07 [SC]
+  F42C1A6E634C105E8D985105CA751254E97B9FE4
+uid   [ultimate] Yizhi Liu 
+sig 3CA751254E97B9FE4 2018-02-07  Yizhi Liu 
+sub   rsa4096 2018-02-07 [E]
+sig  CA751254E97B9FE4 2018-02-07  Yizhi Liu 
+
+-BEGIN PGP PUBLIC KEY BLOCK-
+
+mQINBFp6j2ABEADO07fnGhxTkPfmRsJS65Cif6ywUVRl2ZXKi/N7DjKJdl+Ej5lG
+Oaw5cExaP0RD5iT5ZCAzfUS7UFULybEcbqgnm/RzaCrz9mx3gLa8Jx9XncagwJQU
+9GbvJxzlX8itgY9vezK1q7Ec/iwCA66suzLeY8cA68EvWwmjR/1WlE9W/gov9mSl
+Cu7QRIP9DuUHyL9ZZtYTwYTKsSaRCTv42xvkxAQ/ifinYZn31uQmW41Gqt2YFNWl
+fp1uA97dmyAKcIeOCkvpQyChspJLIcA5lQrH6RV+oyuhRoSw/ZXPwdRXS1+79arC
+e1vsMUeZPzkzSXcrpkzROOzVx1WlXR8WYcWtaXkPsgn7Icym0ngnwRbuY0JACT2F
+8MWgBlC3LQj0mrhCcr26v9ettcmeulmuY/WLIi9oDtgq2yHSbz2na+qbPRd5vDS1
+i+nD62xvejZXC7xInaoyB0f6QYpgXQKyEFO/uCUGFXCBwYAPe4XwkNozR1GmOTKP
+9ZnriJax/BIPva5JiqK5pkqOxuiGuPNdW7Bj/HvQr7F1s5LoG+Q8YSVo/KEJ2oo3
+IwU6FWZwq+KY/bVTO3fRCBD7Fgu8Eu9zw8ANIvpuq+BDo3yoUCoS83Ok+favH0K/
+jwBVFyu+/rnJc4wn7Px9/zdniaSTuxK6pAyTiUtVy6Gp73Roik5Dhu3nqwARAQAB
+tB9ZaXpoaSBMaXUgPGxpdXlpemhpQGFwYWNoZS5vcmc+iQJOBBMBCAA4FiEE9Cwa
+bmNMEF6NmFEFynUSVOl7n+QFAlp6j2ACGwMFCwkIBwIGFQoJCAsCBBYCAwECHgEC
+F4AACgkQynUSVOl7n+TMiA/+LB2vDz8ZzMRTwlnWxxhiKU8+P5QEvC7sgwg6REiq
+jEfo+Abcf/erRzMSnX0G4G6xauty4NDtieUI3X/mDKUS96yqo8Ij5NO02ltI0isG
+6edlyyjrs01yiGHNKjTDkU1f1Af+wW8/h9By6cf2x4u9VWfSUjzwkcrr1qorP0AU
+1cSXVDJNxnKKHbdsBlVC7UkUX1ZMBQq3inFIox5y1cSL34joUGRcyFtqZDoTvYMI
+ZgAiJJw1JmpQU2bte3T1/70j6za81/09ev/kN9HIfeK2Mh0IVTttvBdggmQZHhKq
+5tL70v93RUoCaRmJCvyUTaSe1o57phzOeUj8FmFhvqugnrtfYaygdvjrOZYXo5R1
+8jXiQG0lNQPuxh9Vr6dca85aP12yB6kK8/d+09PaEtirqwW32YcoNeiHtPWvEIas
+tcO+bAE6OKFWHE+3mYKr2m4hAH6CWDOa+x6p9JyciTKxEgaaXcj/q458r2S79iMe
+JknzLKw9zLPjHAm0tb45x893xnjNSSDd8DhjwwwZKCt/pZs2E0pyp08DF1a8uCId
+oQ0s4eo5Yr7tJxGpAWgd/VcrlHBmmGdqdMMUhS02BjuyVDXc+T3fbE1a5QIpHoqj
+l7lyeY+VLnOUt9Y+RyWKsDONsB3QcuMRaQQWGf7eeILMIZ+Y33qpt0/55qLbzsEY
+/265Ag0EWnqPYAEQAMLE3QGCRBZU4nGKyOIpIsWpolG8f5vnAZJwsC6g4ya3odsH
+uUknDo7Puhp7RCIxHuEtSBTf+20nFifX7GCgHAKn/mGWDk9mNWmsGpVzXcHNO0TK
+Tod6V9FE5SC3CVggK8U1PesXh0PoV2AMWq1AmzWJyivHFRefuPilu+NVRE/Mj6ZW
+bs3ApixMml/0S1Y7L5btNjG1DCZbs6i70nSuUXXXM/D0jkCYljYf8wtruzj1MN97
+NZP2nvGjyBkGw9tNxyWYirZ5jJOlzbee4rags9agxETrZ4z9S3QAFcQaKNI32Hyu
+SJELgIcx5U/uB2f19GQX/33kk26OrTAW6INUCRK6ji2y0F8IxfrHd0WXj/RFrV/o
+kQyEai5x8oC1+Rik62CEnI9EfL/WU/toHtSeFBfNrtTKa3WiXnQDfHmJBe1wfvOm
+M3QjH2ApPBwUXXblm7wBCPEjQJs+B0FIrlpJdN+KaGMMSHsz90f9QMF6GH/pgDPG
+7K1IBsP3ZqDzJi7CLnLTAf0FreLuKLix349Y4X603uNd6Fx6vK3BGWB3ZyH7D1vC
+MBBytDdb66nmQQ3QZjJBU8FCGuBwd8q32bVKbIOQTQiMUbUGe3xZozC82mB3glEU
+CO46OElD0j56GC3XGVK4utPexIX9hcQ+uSXStrwhgHd76/iFCsb1F9wR16EVABEB
+AAGJAjYEGAEIACAWIQT0LBpuY0wQXo2YUQXKdRJU6Xuf5AUCWnqPYAIbDAAKCRDK
+dRJU6Xuf5KqtEADHxHPTbl1lT/QZZ+Y+SSuDpPF4uMjUP1TPyt6LGK9O/C0raIxa
+bpCtuit9VPwcubH/krVQxqIkje1rI6kjl/+krrwnnNhjUozoQh4y0e90atgu9pho
+QGjb12vhl5P95OB/YX8ZRJ2Bt7aSTfZiUUbL0OwwgontgLFNyz9/FNp/9eSrxOco
+Mazkt5D6SrW0IBW9l5SZeNDc9yYw0CMg/5YZ5Rv++APgXHWc/WjuDMHje7hi2VFM
+12VXF+gWQZy842n5IQzRPx7Pav32iByN00qKLNUUIwgoEQwZMStC9xjooGSmqOVU
+WnMYBLiUgNTySgOhu73hZVo8VNpOseatlaIRGC2ukn8AF5TlXMKf7O9L24x6bp3B
+d7M5KUNCUDgwn0mjVjsGEcT41Rc9XtglB7aLTiKhE/LqGi1f+BQolr6nGLEQ+oVu
+b3bqratmjAE7Pw7Byzup78JPVMt8vNdjwGYg3yHW4atLS1qUQ9VNYo2l4b+DxcCv
+FxV/mAfa+07j1Z9Ep4/Pw35uanSfOo0ylGmHp/h9yh27vrF1EzwshB7DlJoo5Kfn
+IxR3jVTKye+UerEtN8yATW8CRIKO3IobUfLMDdPCLO7uzoW95cI35Y0l8JgK2NeU
+6tVZptP5mDogeAbq8PlimrXuzG9Bokct2SOO6Z51i6rSDo/ALj440EvWNw==
+=E4W5
+-END PGP PUBLIC KEY BLOCK-

-- 
To stop receiving notification emails like this one, please contact
liuyi...@apache.org.

[incubator-mxnet] branch v1.1.0 updated: revert acc changes (#9731)

[zhasheng]

This is an automated email from the ASF dual-hosted git repository.

zhasheng pushed a commit to branch v1.1.0
in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git


The following commit(s) were added to refs/heads/v1.1.0 by this push:
 new 4878fde  revert acc changes (#9731)
4878fde is described below

commit 4878fde51ba2b5c4cd0b59d7d1208319340def5a
Author: Sheng Zha 
AuthorDate: Wed Feb 7 10:07:58 2018 -0800

revert acc changes (#9731)

* Revert "avoid per-batch blocking in metric (#9636)"

This reverts commit 3fe694e7b1ed7fa6a2dcfeddeac44c14ab77b015.

* Revert "proper flatten in acc (#9619)"

This reverts commit ed823b2e187eb859d9475eb651465edf714c6c5f.

* Revert "use nd for accuracy calculation (#9583)"

This reverts commit f5f1b91ff972ad70e9131d3cd1d7408b7684.

* keep doc change
---
 python/mxnet/metric.py | 11 ---
 1 file changed, 4 insertions(+), 7 deletions(-)

diff --git a/python/mxnet/metric.py b/python/mxnet/metric.py
index f1cdae2..8bb3f6e 100644
--- a/python/mxnet/metric.py
+++ b/python/mxnet/metric.py
@@ -391,16 +391,13 @@ class Accuracy(EvalMetric):
 for label, pred_label in zip(labels, preds):
 if pred_label.shape != label.shape:
 pred_label = ndarray.argmax(pred_label, axis=self.axis)
-pred_label = pred_label.astype('int32')
-label = label.astype('int32')
+pred_label = pred_label.asnumpy().astype('int32')
+label = label.asnumpy().astype('int32')
 
 check_label_shapes(label, pred_label)
 
-if pred_label.context != label.context:
-pred_label = pred_label.as_in_context(label.context)
-
-self.sum_metric += (pred_label.flatten() == 
label.flatten()).sum().asscalar()
-self.num_inst += numpy.prod(pred_label.shape)
+self.sum_metric += (pred_label.flat == label.flat).sum()
+self.num_inst += len(pred_label.flat)
 
 
 @register

-- 
To stop receiving notification emails like this one, please contact
zhash...@apache.org.

[incubator-mxnet] branch master updated: PGP keys add liuyizhi AT apache.org (#9728)

[haibin]

This is an automated email from the ASF dual-hosted git repository.

haibin 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 5591cb7  PGP keys add liuyizhi AT apache.org (#9728)
5591cb7 is described below

commit 5591cb772c806a4d2d52cf21a773be574a66641e
Author: Yizhi Liu 
AuthorDate: Wed Feb 7 10:10:20 2018 -0800

PGP keys add liuyizhi AT apache.org (#9728)
---
 KEYS | 59 +++
 1 file changed, 59 insertions(+)

diff --git a/KEYS b/KEYS
index 5e5769b..77710ad 100644
--- a/KEYS
+++ b/KEYS
@@ -423,3 +423,62 @@ 
fDiLw4372qAC4NpSLRpCIBbT33VztUOTmZgIg4zJiQGSp89dEVN8OUT/yjKQps39
 G4kKPLx5+UhNtHsaNA==
 =ZoTi
 -END PGP PUBLIC KEY BLOCK-
+
+pub   rsa4096 2018-02-07 [SC]
+  F42C1A6E634C105E8D985105CA751254E97B9FE4
+uid   [ultimate] Yizhi Liu 
+sig 3CA751254E97B9FE4 2018-02-07  Yizhi Liu 
+sub   rsa4096 2018-02-07 [E]
+sig  CA751254E97B9FE4 2018-02-07  Yizhi Liu 
+
+-BEGIN PGP PUBLIC KEY BLOCK-
+
+mQINBFp6j2ABEADO07fnGhxTkPfmRsJS65Cif6ywUVRl2ZXKi/N7DjKJdl+Ej5lG
+Oaw5cExaP0RD5iT5ZCAzfUS7UFULybEcbqgnm/RzaCrz9mx3gLa8Jx9XncagwJQU
+9GbvJxzlX8itgY9vezK1q7Ec/iwCA66suzLeY8cA68EvWwmjR/1WlE9W/gov9mSl
+Cu7QRIP9DuUHyL9ZZtYTwYTKsSaRCTv42xvkxAQ/ifinYZn31uQmW41Gqt2YFNWl
+fp1uA97dmyAKcIeOCkvpQyChspJLIcA5lQrH6RV+oyuhRoSw/ZXPwdRXS1+79arC
+e1vsMUeZPzkzSXcrpkzROOzVx1WlXR8WYcWtaXkPsgn7Icym0ngnwRbuY0JACT2F
+8MWgBlC3LQj0mrhCcr26v9ettcmeulmuY/WLIi9oDtgq2yHSbz2na+qbPRd5vDS1
+i+nD62xvejZXC7xInaoyB0f6QYpgXQKyEFO/uCUGFXCBwYAPe4XwkNozR1GmOTKP
+9ZnriJax/BIPva5JiqK5pkqOxuiGuPNdW7Bj/HvQr7F1s5LoG+Q8YSVo/KEJ2oo3
+IwU6FWZwq+KY/bVTO3fRCBD7Fgu8Eu9zw8ANIvpuq+BDo3yoUCoS83Ok+favH0K/
+jwBVFyu+/rnJc4wn7Px9/zdniaSTuxK6pAyTiUtVy6Gp73Roik5Dhu3nqwARAQAB
+tB9ZaXpoaSBMaXUgPGxpdXlpemhpQGFwYWNoZS5vcmc+iQJOBBMBCAA4FiEE9Cwa
+bmNMEF6NmFEFynUSVOl7n+QFAlp6j2ACGwMFCwkIBwIGFQoJCAsCBBYCAwECHgEC
+F4AACgkQynUSVOl7n+TMiA/+LB2vDz8ZzMRTwlnWxxhiKU8+P5QEvC7sgwg6REiq
+jEfo+Abcf/erRzMSnX0G4G6xauty4NDtieUI3X/mDKUS96yqo8Ij5NO02ltI0isG
+6edlyyjrs01yiGHNKjTDkU1f1Af+wW8/h9By6cf2x4u9VWfSUjzwkcrr1qorP0AU
+1cSXVDJNxnKKHbdsBlVC7UkUX1ZMBQq3inFIox5y1cSL34joUGRcyFtqZDoTvYMI
+ZgAiJJw1JmpQU2bte3T1/70j6za81/09ev/kN9HIfeK2Mh0IVTttvBdggmQZHhKq
+5tL70v93RUoCaRmJCvyUTaSe1o57phzOeUj8FmFhvqugnrtfYaygdvjrOZYXo5R1
+8jXiQG0lNQPuxh9Vr6dca85aP12yB6kK8/d+09PaEtirqwW32YcoNeiHtPWvEIas
+tcO+bAE6OKFWHE+3mYKr2m4hAH6CWDOa+x6p9JyciTKxEgaaXcj/q458r2S79iMe
+JknzLKw9zLPjHAm0tb45x893xnjNSSDd8DhjwwwZKCt/pZs2E0pyp08DF1a8uCId
+oQ0s4eo5Yr7tJxGpAWgd/VcrlHBmmGdqdMMUhS02BjuyVDXc+T3fbE1a5QIpHoqj
+l7lyeY+VLnOUt9Y+RyWKsDONsB3QcuMRaQQWGf7eeILMIZ+Y33qpt0/55qLbzsEY
+/265Ag0EWnqPYAEQAMLE3QGCRBZU4nGKyOIpIsWpolG8f5vnAZJwsC6g4ya3odsH
+uUknDo7Puhp7RCIxHuEtSBTf+20nFifX7GCgHAKn/mGWDk9mNWmsGpVzXcHNO0TK
+Tod6V9FE5SC3CVggK8U1PesXh0PoV2AMWq1AmzWJyivHFRefuPilu+NVRE/Mj6ZW
+bs3ApixMml/0S1Y7L5btNjG1DCZbs6i70nSuUXXXM/D0jkCYljYf8wtruzj1MN97
+NZP2nvGjyBkGw9tNxyWYirZ5jJOlzbee4rags9agxETrZ4z9S3QAFcQaKNI32Hyu
+SJELgIcx5U/uB2f19GQX/33kk26OrTAW6INUCRK6ji2y0F8IxfrHd0WXj/RFrV/o
+kQyEai5x8oC1+Rik62CEnI9EfL/WU/toHtSeFBfNrtTKa3WiXnQDfHmJBe1wfvOm
+M3QjH2ApPBwUXXblm7wBCPEjQJs+B0FIrlpJdN+KaGMMSHsz90f9QMF6GH/pgDPG
+7K1IBsP3ZqDzJi7CLnLTAf0FreLuKLix349Y4X603uNd6Fx6vK3BGWB3ZyH7D1vC
+MBBytDdb66nmQQ3QZjJBU8FCGuBwd8q32bVKbIOQTQiMUbUGe3xZozC82mB3glEU
+CO46OElD0j56GC3XGVK4utPexIX9hcQ+uSXStrwhgHd76/iFCsb1F9wR16EVABEB
+AAGJAjYEGAEIACAWIQT0LBpuY0wQXo2YUQXKdRJU6Xuf5AUCWnqPYAIbDAAKCRDK
+dRJU6Xuf5KqtEADHxHPTbl1lT/QZZ+Y+SSuDpPF4uMjUP1TPyt6LGK9O/C0raIxa
+bpCtuit9VPwcubH/krVQxqIkje1rI6kjl/+krrwnnNhjUozoQh4y0e90atgu9pho
+QGjb12vhl5P95OB/YX8ZRJ2Bt7aSTfZiUUbL0OwwgontgLFNyz9/FNp/9eSrxOco
+Mazkt5D6SrW0IBW9l5SZeNDc9yYw0CMg/5YZ5Rv++APgXHWc/WjuDMHje7hi2VFM
+12VXF+gWQZy842n5IQzRPx7Pav32iByN00qKLNUUIwgoEQwZMStC9xjooGSmqOVU
+WnMYBLiUgNTySgOhu73hZVo8VNpOseatlaIRGC2ukn8AF5TlXMKf7O9L24x6bp3B
+d7M5KUNCUDgwn0mjVjsGEcT41Rc9XtglB7aLTiKhE/LqGi1f+BQolr6nGLEQ+oVu
+b3bqratmjAE7Pw7Byzup78JPVMt8vNdjwGYg3yHW4atLS1qUQ9VNYo2l4b+DxcCv
+FxV/mAfa+07j1Z9Ep4/Pw35uanSfOo0ylGmHp/h9yh27vrF1EzwshB7DlJoo5Kfn
+IxR3jVTKye+UerEtN8yATW8CRIKO3IobUfLMDdPCLO7uzoW95cI35Y0l8JgK2NeU
+6tVZptP5mDogeAbq8PlimrXuzG9Bokct2SOO6Z51i6rSDo/ALj440EvWNw==
+=E4W5
+-END PGP PUBLIC KEY BLOCK-

-- 
To stop receiving notification emails like this one, please contact
hai...@apache.org.

[GitHub] eric-haibin-lin closed pull request #9728: PGP keys add liuyizhi AT apache.org

[GitBox]

eric-haibin-lin closed pull request #9728: PGP keys add liuyizhi AT apache.org
URL: https://github.com/apache/incubator-mxnet/pull/9728
 
 
   

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 (from a fork), the diff is supplied
below (as it won't show otherwise due to GitHub magic):

diff --git a/KEYS b/KEYS
index 5e5769be54..77710ad5c4 100644
--- a/KEYS
+++ b/KEYS
@@ -423,3 +423,62 @@ 
fDiLw4372qAC4NpSLRpCIBbT33VztUOTmZgIg4zJiQGSp89dEVN8OUT/yjKQps39
 G4kKPLx5+UhNtHsaNA==
 =ZoTi
 -END PGP PUBLIC KEY BLOCK-
+
+pub   rsa4096 2018-02-07 [SC]
+  F42C1A6E634C105E8D985105CA751254E97B9FE4
+uid   [ultimate] Yizhi Liu 
+sig 3CA751254E97B9FE4 2018-02-07  Yizhi Liu 
+sub   rsa4096 2018-02-07 [E]
+sig  CA751254E97B9FE4 2018-02-07  Yizhi Liu 
+
+-BEGIN PGP PUBLIC KEY BLOCK-
+
+mQINBFp6j2ABEADO07fnGhxTkPfmRsJS65Cif6ywUVRl2ZXKi/N7DjKJdl+Ej5lG
+Oaw5cExaP0RD5iT5ZCAzfUS7UFULybEcbqgnm/RzaCrz9mx3gLa8Jx9XncagwJQU
+9GbvJxzlX8itgY9vezK1q7Ec/iwCA66suzLeY8cA68EvWwmjR/1WlE9W/gov9mSl
+Cu7QRIP9DuUHyL9ZZtYTwYTKsSaRCTv42xvkxAQ/ifinYZn31uQmW41Gqt2YFNWl
+fp1uA97dmyAKcIeOCkvpQyChspJLIcA5lQrH6RV+oyuhRoSw/ZXPwdRXS1+79arC
+e1vsMUeZPzkzSXcrpkzROOzVx1WlXR8WYcWtaXkPsgn7Icym0ngnwRbuY0JACT2F
+8MWgBlC3LQj0mrhCcr26v9ettcmeulmuY/WLIi9oDtgq2yHSbz2na+qbPRd5vDS1
+i+nD62xvejZXC7xInaoyB0f6QYpgXQKyEFO/uCUGFXCBwYAPe4XwkNozR1GmOTKP
+9ZnriJax/BIPva5JiqK5pkqOxuiGuPNdW7Bj/HvQr7F1s5LoG+Q8YSVo/KEJ2oo3
+IwU6FWZwq+KY/bVTO3fRCBD7Fgu8Eu9zw8ANIvpuq+BDo3yoUCoS83Ok+favH0K/
+jwBVFyu+/rnJc4wn7Px9/zdniaSTuxK6pAyTiUtVy6Gp73Roik5Dhu3nqwARAQAB
+tB9ZaXpoaSBMaXUgPGxpdXlpemhpQGFwYWNoZS5vcmc+iQJOBBMBCAA4FiEE9Cwa
+bmNMEF6NmFEFynUSVOl7n+QFAlp6j2ACGwMFCwkIBwIGFQoJCAsCBBYCAwECHgEC
+F4AACgkQynUSVOl7n+TMiA/+LB2vDz8ZzMRTwlnWxxhiKU8+P5QEvC7sgwg6REiq
+jEfo+Abcf/erRzMSnX0G4G6xauty4NDtieUI3X/mDKUS96yqo8Ij5NO02ltI0isG
+6edlyyjrs01yiGHNKjTDkU1f1Af+wW8/h9By6cf2x4u9VWfSUjzwkcrr1qorP0AU
+1cSXVDJNxnKKHbdsBlVC7UkUX1ZMBQq3inFIox5y1cSL34joUGRcyFtqZDoTvYMI
+ZgAiJJw1JmpQU2bte3T1/70j6za81/09ev/kN9HIfeK2Mh0IVTttvBdggmQZHhKq
+5tL70v93RUoCaRmJCvyUTaSe1o57phzOeUj8FmFhvqugnrtfYaygdvjrOZYXo5R1
+8jXiQG0lNQPuxh9Vr6dca85aP12yB6kK8/d+09PaEtirqwW32YcoNeiHtPWvEIas
+tcO+bAE6OKFWHE+3mYKr2m4hAH6CWDOa+x6p9JyciTKxEgaaXcj/q458r2S79iMe
+JknzLKw9zLPjHAm0tb45x893xnjNSSDd8DhjwwwZKCt/pZs2E0pyp08DF1a8uCId
+oQ0s4eo5Yr7tJxGpAWgd/VcrlHBmmGdqdMMUhS02BjuyVDXc+T3fbE1a5QIpHoqj
+l7lyeY+VLnOUt9Y+RyWKsDONsB3QcuMRaQQWGf7eeILMIZ+Y33qpt0/55qLbzsEY
+/265Ag0EWnqPYAEQAMLE3QGCRBZU4nGKyOIpIsWpolG8f5vnAZJwsC6g4ya3odsH
+uUknDo7Puhp7RCIxHuEtSBTf+20nFifX7GCgHAKn/mGWDk9mNWmsGpVzXcHNO0TK
+Tod6V9FE5SC3CVggK8U1PesXh0PoV2AMWq1AmzWJyivHFRefuPilu+NVRE/Mj6ZW
+bs3ApixMml/0S1Y7L5btNjG1DCZbs6i70nSuUXXXM/D0jkCYljYf8wtruzj1MN97
+NZP2nvGjyBkGw9tNxyWYirZ5jJOlzbee4rags9agxETrZ4z9S3QAFcQaKNI32Hyu
+SJELgIcx5U/uB2f19GQX/33kk26OrTAW6INUCRK6ji2y0F8IxfrHd0WXj/RFrV/o
+kQyEai5x8oC1+Rik62CEnI9EfL/WU/toHtSeFBfNrtTKa3WiXnQDfHmJBe1wfvOm
+M3QjH2ApPBwUXXblm7wBCPEjQJs+B0FIrlpJdN+KaGMMSHsz90f9QMF6GH/pgDPG
+7K1IBsP3ZqDzJi7CLnLTAf0FreLuKLix349Y4X603uNd6Fx6vK3BGWB3ZyH7D1vC
+MBBytDdb66nmQQ3QZjJBU8FCGuBwd8q32bVKbIOQTQiMUbUGe3xZozC82mB3glEU
+CO46OElD0j56GC3XGVK4utPexIX9hcQ+uSXStrwhgHd76/iFCsb1F9wR16EVABEB
+AAGJAjYEGAEIACAWIQT0LBpuY0wQXo2YUQXKdRJU6Xuf5AUCWnqPYAIbDAAKCRDK
+dRJU6Xuf5KqtEADHxHPTbl1lT/QZZ+Y+SSuDpPF4uMjUP1TPyt6LGK9O/C0raIxa
+bpCtuit9VPwcubH/krVQxqIkje1rI6kjl/+krrwnnNhjUozoQh4y0e90atgu9pho
+QGjb12vhl5P95OB/YX8ZRJ2Bt7aSTfZiUUbL0OwwgontgLFNyz9/FNp/9eSrxOco
+Mazkt5D6SrW0IBW9l5SZeNDc9yYw0CMg/5YZ5Rv++APgXHWc/WjuDMHje7hi2VFM
+12VXF+gWQZy842n5IQzRPx7Pav32iByN00qKLNUUIwgoEQwZMStC9xjooGSmqOVU
+WnMYBLiUgNTySgOhu73hZVo8VNpOseatlaIRGC2ukn8AF5TlXMKf7O9L24x6bp3B
+d7M5KUNCUDgwn0mjVjsGEcT41Rc9XtglB7aLTiKhE/LqGi1f+BQolr6nGLEQ+oVu
+b3bqratmjAE7Pw7Byzup78JPVMt8vNdjwGYg3yHW4atLS1qUQ9VNYo2l4b+DxcCv
+FxV/mAfa+07j1Z9Ep4/Pw35uanSfOo0ylGmHp/h9yh27vrF1EzwshB7DlJoo5Kfn
+IxR3jVTKye+UerEtN8yATW8CRIKO3IobUfLMDdPCLO7uzoW95cI35Y0l8JgK2NeU
+6tVZptP5mDogeAbq8PlimrXuzG9Bokct2SOO6Z51i6rSDo/ALj440EvWNw==
+=E4W5
+-END PGP PUBLIC KEY BLOCK-


 


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[incubator-mxnet] branch master updated: revert acc changes (#9731)

[haibin]

This is an automated email from the ASF dual-hosted git repository.

haibin 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 ad3f0d8  revert acc changes (#9731)
ad3f0d8 is described below

commit ad3f0d8faeaabf302923796d1379bec6040e8d53
Author: Sheng Zha 
AuthorDate: Wed Feb 7 10:07:58 2018 -0800

revert acc changes (#9731)

* Revert "avoid per-batch blocking in metric (#9636)"

This reverts commit 3fe694e7b1ed7fa6a2dcfeddeac44c14ab77b015.

* Revert "proper flatten in acc (#9619)"

This reverts commit ed823b2e187eb859d9475eb651465edf714c6c5f.

* Revert "use nd for accuracy calculation (#9583)"

This reverts commit f5f1b91ff972ad70e9131d3cd1d7408b7684.

* keep doc change
---
 python/mxnet/metric.py | 15 ---
 1 file changed, 4 insertions(+), 11 deletions(-)

diff --git a/python/mxnet/metric.py b/python/mxnet/metric.py
index e91fd3b..8bb3f6e 100644
--- a/python/mxnet/metric.py
+++ b/python/mxnet/metric.py
@@ -28,7 +28,6 @@ import numpy
 from .base import numeric_types, string_types
 from . import ndarray
 from . import registry
-from .context import cpu
 
 
 def check_label_shapes(labels, preds, shape=0):
@@ -389,22 +388,16 @@ class Accuracy(EvalMetric):
 """
 check_label_shapes(labels, preds)
 
-results = []
 for label, pred_label in zip(labels, preds):
 if pred_label.shape != label.shape:
 pred_label = ndarray.argmax(pred_label, axis=self.axis)
-pred_label = pred_label.astype('int32')
-label = label.astype('int32')
+pred_label = pred_label.asnumpy().astype('int32')
+label = label.asnumpy().astype('int32')
 
 check_label_shapes(label, pred_label)
 
-if pred_label.context != label.context:
-pred_label = pred_label.as_in_context(label.context)
-
-self.num_inst += pred_label.size
-results.append((pred_label.reshape((-1,)) == label.reshape((-1,)))
-   .sum().as_in_context(cpu()))
-self.sum_metric += ndarray.add_n(*results).asscalar()
+self.sum_metric += (pred_label.flat == label.flat).sum()
+self.num_inst += len(pred_label.flat)
 
 
 @register

-- 
To stop receiving notification emails like this one, please contact
hai...@apache.org.

[GitHub] eric-haibin-lin closed pull request #9731: revert acc changes

[GitBox]

eric-haibin-lin closed pull request #9731: revert acc changes
URL: https://github.com/apache/incubator-mxnet/pull/9731
 
 
   

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 (from a fork), the diff is supplied
below (as it won't show otherwise due to GitHub magic):

diff --git a/python/mxnet/metric.py b/python/mxnet/metric.py
index e91fd3b13e..8bb3f6ee0a 100644
--- a/python/mxnet/metric.py
+++ b/python/mxnet/metric.py
@@ -28,7 +28,6 @@
 from .base import numeric_types, string_types
 from . import ndarray
 from . import registry
-from .context import cpu
 
 
 def check_label_shapes(labels, preds, shape=0):
@@ -389,22 +388,16 @@ def update(self, labels, preds):
 """
 check_label_shapes(labels, preds)
 
-results = []
 for label, pred_label in zip(labels, preds):
 if pred_label.shape != label.shape:
 pred_label = ndarray.argmax(pred_label, axis=self.axis)
-pred_label = pred_label.astype('int32')
-label = label.astype('int32')
+pred_label = pred_label.asnumpy().astype('int32')
+label = label.asnumpy().astype('int32')
 
 check_label_shapes(label, pred_label)
 
-if pred_label.context != label.context:
-pred_label = pred_label.as_in_context(label.context)
-
-self.num_inst += pred_label.size
-results.append((pred_label.reshape((-1,)) == label.reshape((-1,)))
-   .sum().as_in_context(cpu()))
-self.sum_metric += ndarray.add_n(*results).asscalar()
+self.sum_metric += (pred_label.flat == label.flat).sum()
+self.num_inst += len(pred_label.flat)
 
 
 @register


 


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[GitHub] eric-haibin-lin commented on issue #9728: PGP keys add liuyizhi AT apache.org

[GitBox]

eric-haibin-lin commented on issue #9728: PGP keys add liuyizhi AT apache.org
URL: https://github.com/apache/incubator-mxnet/pull/9728#issuecomment-363857948
 
 
   cc @mbaijal 


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[GitHub] thinksanky commented on issue #9548: updated release version to 1.1.0 on the mainpage and re-arranged news?

[GitBox]

thinksanky commented on issue #9548: updated release version to 1.1.0 on the 
mainpage and re-arranged news?
URL: https://github.com/apache/incubator-mxnet/pull/9548#issuecomment-363852146
 
 
   Rephrased title


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[GitHub] vijayzutshi commented on issue #9602: MXNET not working in R

[GitBox]

vijayzutshi commented on issue #9602: MXNET not working in R
URL: 
https://github.com/apache/incubator-mxnet/issues/9602#issuecomment-363849800
 
 
   Hi,
   
   
   So I installed CPU version and it works ok.
   
   
   Thanks
   
   Vijay
   
   
   
   From: jeremiedb 
   Sent: January 29, 2018 12:38 AM
   To: apache/incubator-mxnet
   Cc: Vijay Zutshi; Author
   Subject: Re: [apache/incubator-mxnet] MXNET not working in R (#9602)
   
   
   I don't have access to a Windows GPU, but could you confirm if the CPU 
version can be installed and works properly? The pre-compiled Windows haven't 
been refreshed for a while, in the meantime, maybe you could consider running 
through docker? https://hub.docker.com/r/mxnet/r-lang/tags/
   Sorry for the partial answer!
   
   ?
   You are receiving this because you authored the thread.
   Reply to this email directly, view it on 
GitHub,
 or mute the 
thread.
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[GitHub] larroy commented on issue #9652: Fallout from simplifying/fixing im2rec.py commandline boolean options.

[GitBox]

larroy commented on issue #9652: Fallout from simplifying/fixing im2rec.py 
commandline boolean options.
URL: https://github.com/apache/incubator-mxnet/pull/9652#issuecomment-363836870
 
 
   @szha 


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[GitHub] larroy commented on a change in pull request #9717: Doc improvement

[GitBox]

larroy commented on a change in pull request #9717: Doc improvement
URL: https://github.com/apache/incubator-mxnet/pull/9717#discussion_r166682684
 
 

 ##
 File path: tests/python/unittest/test_ndarray.py
 ##
 @@ -590,9 +590,8 @@ def gt_topk(dat, axis, ret_typ, k, is_ascend):
 gt = gt_topk(a_npy, axis=None, ret_typ="indices", k=5*5*5*5, 
is_ascend=False)
 assert_almost_equal(nd_ret_argsort, gt)
 
-# test topk with a big shape
-a = mx.nd.arange(0, 54686454, step=1, repeat=1)
-assert_almost_equal(a.topk(k=54686454).asnumpy(), a.asnumpy()[::-1])
+a = mx.nd.arange(0, 1024, step=1, repeat=1)
+assert_almost_equal(a.topk(k=1024).asnumpy(), a.asnumpy()[::-1])
 
 Review comment:
   yes, my bad


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

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

[GitBox]

reminisce commented on a change in pull request #9552: [REQUEST FOR REVIEW | DO 
NOT MERGE] Model Quantization with Calibration
URL: https://github.com/apache/incubator-mxnet/pull/9552#discussion_r166679292
 
 

 ##
 File path: tests/ci_build/Dockerfile.build_cuda
 ##
 @@ -1,4 +1,4 @@
-FROM nvidia/cuda:8.0-cudnn7-devel
 
 Review comment:
   Yeah, it was originally cudnn5 and I changed it in hope of running cudnn7, 
but wasn't successful. So I changed it back and Marco helped me setup another 
docker file with cudnn7.


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[GitHub] daleydeng opened a new issue #9733: BUG in MultiBoxTargetForward when there is single box label

[GitBox]

daleydeng opened a new issue #9733: BUG in MultiBoxTargetForward when there is 
single box label
URL: https://github.com/apache/incubator-mxnet/issues/9733
 
 
   There was a constraint on num_label in MultiBoxTargetForward function at 
multibox_target.cu:359 which will cause error when input label has only one 
box, is it safe to remove it?
   
   PS: cpu version multibox_target.cc has no thus constraints.
   
   template
   inline void MultiBoxTargetForward(const Tensor _target,
  const Tensor _mask,
  const Tensor _target,
  const Tensor ,
  const Tensor ,
  const Tensor _preds,
  const Tensor _space,
  const float overlap_threshold,
  const float background_label,
  const float negative_mining_ratio,
  const float negative_mining_thresh,
  const int minimum_negative_samples,
  const nnvm::Tuple ) {
 const int num_batches = labels.size(0);
 const int num_labels = labels.size(1);
 const int label_width = labels.size(2);
 const int num_anchors = anchors.size(0);
 const int num_classes = cls_preds.size(1);
 CHECK_GE(num_batches, 1);
 CHECK_GT(num_labels, 2); // here
 CHECK_GE(num_anchors, 1);
 CHECK_EQ(variances.ndim(), 4);
   
   error logged as follows:
   (1, 3318, 4)  # anchor
   [[[ 1.  0.1032258   0.010.88387096  0.4133]]] #input box 
label
(1, 6, 3318) # cls_pred
   [14:31:57] 
/home/vision/miniconda3/conda-bld/mxnet_1514016712864/work/dmlc-core/include/dmlc/./logging.h:308:
 [14:31:57] src/operator/contrib/multibox_target.cu:359: Check failed: 
num_labels > 2 (1 vs. 2) 


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[GitHub] szha commented on issue #8562: Number of output from the mx.sym.SoftmaxOutput

[GitBox]

szha commented on issue #8562: Number of output from the mx.sym.SoftmaxOutput
URL: 
https://github.com/apache/incubator-mxnet/issues/8562#issuecomment-363753551
 
 
   @apache/mxnet-committers: This issue has been inactive for the past 90 days. 
It has no label and needs triage.
   
   For general "how-to" questions, our [user forum](https://discuss.mxnet.io/) 
(and [Chinese version](https://discuss.gluon.ai/)) is a good place to get help.


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[GitHub] szha commented on issue #8234: A bug in VGG_FC_ILSVRC_16_layers-symbol.json

[GitBox]

szha commented on issue #8234: A bug in VGG_FC_ILSVRC_16_layers-symbol.json
URL: 
https://github.com/apache/incubator-mxnet/issues/8234#issuecomment-363753558
 
 
   @apache/mxnet-committers: This issue has been inactive for the past 90 days. 
It has no label and needs triage.
   
   For general "how-to" questions, our [user forum](https://discuss.mxnet.io/) 
(and [Chinese version](https://discuss.gluon.ai/)) is a good place to get help.


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[GitHub] ZiyueHuang commented on issue #9732: Factorization Machine can't run

[GitBox]

ZiyueHuang commented on issue #9732: Factorization Machine can't run
URL: 
https://github.com/apache/incubator-mxnet/issues/9732#issuecomment-363740311
 
 
   You can try a higher version of mxnet, such as 1.0.0. For installation, you 
can refer to https://mxnet.incubator.apache.org/install/index.html.


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[GitHub] ZiyueHuang commented on a change in pull request #9625: sparse regression operators

[GitBox]

ZiyueHuang commented on a change in pull request #9625: sparse regression 
operators
URL: https://github.com/apache/incubator-mxnet/pull/9625#discussion_r166565547
 
 

 ##
 File path: src/operator/regression_output.cc
 ##
 @@ -90,7 +90,7 @@ The parameter `grad_scale` can be used to change this scale 
to `grad_scale/m`.
 )code" ADD_FILELINE);
 
 MXNET_OPERATOR_REGISTER_REGRESSION_BWD(_backward_linear_reg_out, 
mshadow_op::minus, true);
-
+/*
 
 Review comment:
   Of course. I am just removing some instantiated templates and try to see 
what causes this error. Seems that I should divide three regression operators 
in different files (not sure yet).


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[GitHub] ningyuwhut opened a new issue #9732: Factorization Machine can't run

[GitBox]

ningyuwhut opened a new issue #9732: Factorization Machine can't run
URL: https://github.com/apache/incubator-mxnet/issues/9732
 
 
   I run train.py in the example/sparse/factorization_machine dir.and I met the 
error:
   
   Traceback (most recent call last):
 File "train.py", line 81, in 
   train_data = mx.io.LibSVMIter(data_libsvm=args.data_train, 
data_shape=(num_features,),
   AttributeError: 'module' object has no attribute 'LibSVMIter'
   
   my mxnet version is 0.110, is this a version problem? and how can I solve it?
   
   Thanks!


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[GitHub] KellenSunderland commented on issue #9723: Specify lint-versions and fix docker build issues due to nvidia-384

[GitBox]

KellenSunderland commented on issue #9723: Specify lint-versions and fix docker 
build issues due to nvidia-384
URL: https://github.com/apache/incubator-mxnet/pull/9723#issuecomment-363697919
 
 
   "Get CUDA stubs in build_cuda from cuda-8-0 instead of nvidia-384."  The 
stubs actually come from the base image.  We're installing full CUDA w/ and w/o 
stubs here.  In the future we can just use stubs alone, and not worry about 
installing any package, but this will require us to revisit how we're testing 
the cpp packaging.


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[GitHub] chinakook commented on a change in pull request #9492: fix print_summary bug and add groups of convolution

[GitBox]

chinakook commented on a change in pull request #9492: fix print_summary bug 
and add groups of convolution
URL: https://github.com/apache/incubator-mxnet/pull/9492#discussion_r166542757
 
 

 ##
 File path: python/mxnet/visualization.py
 ##
 @@ -134,17 +134,23 @@ def print_layer_summary(node, out_shape):
 pre_filter = pre_filter + int(shape[0])
 cur_param = 0
 if op == 'Convolution':
-if ("no_bias" in node["attrs"]) and int(node["attrs"]["no_bias"]):
-cur_param = pre_filter * int(node["attrs"]["num_filter"])
+if ("no_bias" in node["attrs"]) and node["attrs"]["no_bias"] == 
'True':
+num_group = int(node["attrs"]["num_group"]) if \
+   ("num_group" in node["attrs"]) else 1
+cur_param = (pre_filter * int(node["attrs"]["num_filter"])) \
+   // num_group
 for k in _str2tuple(node["attrs"]["kernel"]):
 cur_param *= int(k)
 else:
-cur_param = pre_filter * int(node["attrs"]["num_filter"])
+num_group = int(node["attrs"]["num_group"]) if \
+   ("num_group" in node["attrs"]) else 1
+cur_param = (pre_filter * int(node["attrs"]["num_filter"])) \
+   // num_group
 for k in _str2tuple(node["attrs"]["kernel"]):
 cur_param *= int(k)
 cur_param += int(node["attrs"]["num_filter"])
 elif op == 'FullyConnected':
-if ("no_bias" in node["attrs"]) and int(node["attrs"]["no_bias"]):
+if ("no_bias" in node["attrs"]) and node["attrs"]["no_bias"] == 
'True':
 
 Review comment:
   I see. Then use 
   ```python 
   node["attrs"]["no_bias"] == 'True' 
   ``` 
   back?


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

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

[GitBox]

KellenSunderland commented on a change in pull request #9552: [REQUEST FOR 
REVIEW | DO NOT MERGE] Model Quantization with Calibration
URL: https://github.com/apache/incubator-mxnet/pull/9552#discussion_r166541617
 
 

 ##
 File path: tests/ci_build/Dockerfile.build_cuda8_cudnn7
 ##
 @@ -0,0 +1,26 @@
+FROM nvidia/cuda:8.0-cudnn7-devel
 
 Review comment:
   This Dockerfile looks sufficient to me to test cudnn7.


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

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

[GitBox]

KellenSunderland commented on a change in pull request #9552: [REQUEST FOR 
REVIEW | DO NOT MERGE] Model Quantization with Calibration
URL: https://github.com/apache/incubator-mxnet/pull/9552#discussion_r166541524
 
 

 ##
 File path: tests/ci_build/Dockerfile.build_cuda
 ##
 @@ -1,4 +1,4 @@
-FROM nvidia/cuda:8.0-cudnn7-devel
 
 Review comment:
   Maybe a typo?  Looks like you created a separate Dockerfile for cudnn7 
right?  


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services