[GitHub] szha commented on a change in pull request #7304: gluon bce loss

2017-08-11 Thread git
szha commented on a change in pull request #7304: gluon bce loss
URL: https://github.com/apache/incubator-mxnet/pull/7304#discussion_r132810196
 
 

 ##
 File path: python/mxnet/gluon/loss.py
 ##
 @@ -148,13 +150,48 @@ def __init__(self, weight=None, batch_axis=0, **kwargs):
 super(L1Loss, self).__init__(weight, batch_axis, **kwargs)
 
 def hybrid_forward(self, F, output, label, sample_weight=None):
-if F is ndarray:
-loss = ndarray.abs(output - label.reshape(output.shape))
+label = _reshape_label_as_output(F, output, label)
+loss = F.abs(output - label)
+loss = _apply_weighting(F, loss, self._weight, sample_weight)
+return F.mean(loss, axis=self._batch_axis, exclude=True)
+
+
+class BinaryCrossEntropyLoss(Loss):
+r"""The cross-entropy loss for binary classification.
+
+BCE loss is useful when training logistic regression.
+
+.. math::
+loss(o, t) = - 1/n \sum_i (t[i] * log(o[i]) + (1 - t[i]) * log(1 - 
o[i]))
+
+
+Parameters
+--
+from_sigmoid : bool, default is `False`
+Whether the input is from the output of sigmoid. Set this to false 
will make
+the loss calculate sigmoid and then BCE, which is more numerically 
stable through
+log-sum-exp trick.
+weight : float or None
+Global scalar weight for loss.
+sample_weight : Symbol or None
+Per sample weighting. Must be broadcastable to
+the same shape as loss. For example, if loss has
+shape (64, 10) and you want to weight each sample
+in the batch, `sample_weight` should have shape (64, 1).
+batch_axis : int, default 0
+The axis that represents mini-batch.
+"""
+def __init__(self, from_sigmoid=False, weight=None, batch_axis=0, 
**kwargs):
+super(BinaryCrossEntropyLoss, self).__init__(weight, batch_axis, 
**kwargs)
+self._from_sigmoid = from_sigmoid
+
+def hybrid_forward(self, F, output, label, sample_weight=None):
+label = _reshape_label_as_output(F, output, label)
+if not self._from_sigmoid:
+max_val = F.maximum(-output, 0)
+loss = output - output*label + max_val + 
F.log(F.exp(-max_val)+F.exp(-output-max_val))
 
 Review comment:
   added
 

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] asmushetzel commented on issue #7147: cuda support for linalg-functions, restructuring of linalg interfaces

2017-08-11 Thread git
asmushetzel commented on issue #7147: cuda support for linalg-functions, 
restructuring of linalg interfaces
URL: https://github.com/apache/incubator-mxnet/pull/7147#issuecomment-321958146
 
 
   Should all be o.k. now but Jenkins seems to be non-responsive. Have to wait 
until it is up again. 
 

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 #7430: [RCNN] remove self-implemented speedometer

2017-08-11 Thread git
ZiyueHuang commented on issue #7430: [RCNN] remove self-implemented speedometer
URL: https://github.com/apache/incubator-mxnet/pull/7430#issuecomment-321954780
 
 
   ping @precedenceguo @piiswrong 
 

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] pracheer commented on issue #7337: MXNet -> Apple CoreML converter (Review-only. Don't merge)

2017-08-11 Thread git
pracheer commented on issue #7337: MXNet -> Apple CoreML converter 
(Review-only. Don't merge)
URL: https://github.com/apache/incubator-mxnet/pull/7337#issuecomment-321948577
 
 
   Previous pull request: https://github.com/apache/incubator-mxnet/pull/7337
 

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] pracheer commented on issue #7337: MXNet -> Apple CoreML converter (Review-only. Don't merge)

2017-08-11 Thread git
pracheer commented on issue #7337: MXNet -> Apple CoreML converter 
(Review-only. Don't merge)
URL: https://github.com/apache/incubator-mxnet/pull/7337#issuecomment-321948577
 
 
   Previous pull request: https://github.com/apache/incubator-mxnet/pull/7337
 

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] nswamy closed pull request #7438: MXNet -> Apple CoreML converter

2017-08-11 Thread git
nswamy closed pull request #7438: MXNet -> Apple CoreML converter
URL: https://github.com/apache/incubator-mxnet/pull/7438
 
 
   
 

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] pracheer commented on issue #7337: MXNet -> Apple CoreML converter (Review-only. Don't merge)

2017-08-11 Thread git
pracheer commented on issue #7337: MXNet -> Apple CoreML converter 
(Review-only. Don't merge)
URL: https://github.com/apache/incubator-mxnet/pull/7337#issuecomment-321947013
 
 
   Superseded by https://github.com/apache/incubator-mxnet/pull/7438
 

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] pracheer opened a new pull request #7438: MXNet -> Apple CoreML converter

2017-08-11 Thread git
pracheer opened a new pull request #7438: MXNet -> Apple CoreML converter
URL: https://github.com/apache/incubator-mxnet/pull/7438
 
 
   This tool helps convert MXNet models into [Apple 
CoreML](https://developer.apple.com/documentation/coreml) format which can then 
be run on Apple devices.
   
   This has been tested against MXNet 0.10.0 as well as against the current 
master branch.
   
   Example of a sample command to convert squeeze-net model.?
   `python mxnet_coreml_converter.py --model-prefix='squeezenet_v1.1' --epoch=0 
--input-shape='{"data":"3,227,227"}' --mode=classifier 
--pre-processing-arguments='{"image_input_names":"data"}' --class-labels 
classLabels.txt --output-file="squeezenetv11.mlmodel"`
   
   This tools is currently able to convert Inception/Resnet/VGG/Squeezenet/NiN 
models. Support for more vision models as well as nlp models will be added 
later.
   
   Unit tests have been added which test 1/ each layer individually 2/ some of 
the standard models from mxnet zoo against random data as well as on a subset 
of imagenet data. In all the tests we compare the accuracy of MXNet vs that of 
CoreML using the converted model.
   
   @mli @piiswrong @madjam @arank @jiajiechen @srikris
 

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] pracheer closed pull request #7337: MXNet -> Apple CoreML converter (Review-only. Don't merge)

2017-08-11 Thread git
pracheer closed pull request #7337: MXNet -> Apple CoreML converter 
(Review-only. Don't merge)
URL: https://github.com/apache/incubator-mxnet/pull/7337
 
 
   
 

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 a change in pull request #7393: add depthwise convolution's gpu version optimization

2017-08-11 Thread git
piiswrong commented on a change in pull request #7393: add depthwise 
convolution's gpu version optimization
URL: https://github.com/apache/incubator-mxnet/pull/7393#discussion_r132803681
 
 

 ##
 File path: src/operator/convolution-inl.h
 ##
 @@ -97,6 +98,8 @@ struct ConvolutionParam : public 
dmlc::Parameter {
 .set_default(dmlc::optional())
 .describe("Set layout for input, output and weight. Empty for\n"
   "default layout: NCW for 1d, NCHW for 2d and NCDHW for 3d.");
+DMLC_DECLARE_FIELD(depthwise_conv_off).set_default(false)
+.describe("whether to turn off depthwise convolution for this layer");
 
 Review comment:
   Let's remove this if there are no important reasons. Convolution has too 
many switches.
 

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 closed pull request #7374: fix consistency of cpu/gpu in stn

2017-08-11 Thread git
piiswrong closed pull request #7374: fix consistency of cpu/gpu in stn
URL: https://github.com/apache/incubator-mxnet/pull/7374
 
 
   
 

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] haehn opened a new issue #7436: NDArrayIter fails on large data (bug in tensor_blob.h)

2017-08-11 Thread git
haehn opened a new issue #7436: NDArrayIter fails on large data (bug in 
tensor_blob.h)
URL: https://github.com/apache/incubator-mxnet/issues/7436
 
 
   ```
   print X_train.shape, Y_train.shape, X_train.nbytes
   print X_val.shape, Y_val.shape, X_val.nbytes
   print X_test.shape, Y_test.shape, X_test.nbytes
   (212700, 6, 119, 119) (212700,) 36144536400
   (70900, 6, 119, 119) (70900,) 12048178800
   (70900, 6, 119, 119) (70900,) 12048178800
   
   #
   #
   #
   t0 = time.time()
   batch_size = 100
   train_iter = mx.io.NDArrayIter(data=X_train, label=Y_train, 
batch_size=batch_size)
   val_iter = mx.io.NDArrayIter(data=X_val, label=Y_val, batch_size=batch_size)
   test_iter = mx.io.NDArrayIter(data=X_test, label=Y_test, 
batch_size=batch_size)
   print 'iterators configured', time.time()-t0, 'seconds'
   
   ---
   TypeError Traceback (most recent call last)
in ()
 4 t0 = time.time()
 5 batch_size = 100
   > 6 train_iter = mx.io.NDArrayIter(data=X_train, label=Y_train, 
batch_size=batch_size)
 7 val_iter = mx.io.NDArrayIter(data=X_val, label=Y_val, 
batch_size=batch_size)
 8 test_iter = mx.io.NDArrayIter(data=X_test, label=Y_test, 
batch_size=batch_size)
   
   /home/dhaehn/D1/lib/python2.7/site-packages/mxnet/io.pyc in __init__(self, 
data, label, batch_size, shuffle, last_batch_handle, data_name, label_name)
   577 super(NDArrayIter, self).__init__(batch_size)
   578 
   --> 579 self.data = _init_data(data, allow_empty=False, 
default_name=data_name)
   580 self.label = _init_data(label, allow_empty=True, 
default_name=label_name)
   581 
   
   /home/dhaehn/D1/lib/python2.7/site-packages/mxnet/io.pyc in _init_data(data, 
allow_empty, default_name)
   485 except:
   486 raise TypeError(("Invalid type '%s' for %s, "  % 
(type(v), k)) + \
   --> 487 "should be NDArray or numpy.ndarray")
   488 
   489 return list(data.items())
   
   TypeError: Invalid type '' for data, should be NDArray 
or numpy.ndarray
   ```
   
   but the error actually comes from **tensor_blob.h**
   
   ```
   a = mx.nd.array(X_train[0:5]) # no problem
   a = mx.nd.array(X_train[0:6]) # fails
   MXNetError: [16:37:29] include/mxnet/././tensor_blob.h:247: Check failed: 
this->shape_.Size() == shape.Size() (509796 vs. 802992704) 
TBlob.get_with_shape: new and old shape do not match total elements
   
   Stack trace returned 10 entries:
   [bt] (0) 
/home/dhaehn/D1/lib/python2.7/site-packages/mxnet/libmxnet.so(+0x18b0dc) 
[0x7f655a9510dc]
   [bt] (1) 
/home/dhaehn/D1/lib/python2.7/site-packages/mxnet/libmxnet.so(+0x27d680) 
[0x7f655aa43680]
   [bt] (2) 
/home/dhaehn/D1/lib/python2.7/site-packages/mxnet/libmxnet.so(+0x27db65) 
[0x7f655aa43b65]
   [bt] (3) 
/home/dhaehn/D1/lib/python2.7/site-packages/mxnet/libmxnet.so(+0xc8047d) 
[0x7f655b44647d]
   [bt] (4) 
/home/dhaehn/D1/lib/python2.7/site-packages/mxnet/libmxnet.so(+0xc5d29b) 
[0x7f655b42329b]
   [bt] (5) 
/home/dhaehn/D1/lib/python2.7/site-packages/mxnet/libmxnet.so(MXNDArraySyncCopyFromCPU+0xa)
 [0x7f655b2f0a1a]
   [bt] (6) /lib64/libffi.so.6(ffi_call_unix64+0x4c) [0x7f66065e3dcc]
   [bt] (7) /lib64/libffi.so.6(ffi_call+0x1f5) [0x7f66065e36f5]
   [bt] (8) 
/home/dhaehn/D1/lib64/python2.7/lib-dynload/_ctypes.so(_ctypes_callproc+0x30b) 
[0x7f66067f6c8b]
   [bt] (9) /home/dhaehn/D1/lib64/python2.7/lib-dynload/_ctypes.so(+0xaa85) 
[0x7f66067f0a85]
   
   # but..
   a = mx.nd.array(X_train[5:8]) # no problem
   ```
   
   This is related to #6195
   
   
 

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] haehn commented on issue #3030: Does mxnet support .npy format file?

2017-08-11 Thread git
haehn commented on issue #3030: Does mxnet support .npy format file? 
URL: 
https://github.com/apache/incubator-mxnet/issues/3030#issuecomment-321934681
 
 
   the same error is thrown if the array is large.. the underlying exception is 
different tho:
   
   ```
   print X_train.shape, Y_train.shape, X_train.nbytes
   print X_val.shape, Y_val.shape, X_val.nbytes
   print X_test.shape, Y_test.shape, X_test.nbytes
   (212700, 6, 119, 119) (212700,) 36144536400
   (70900, 6, 119, 119) (70900,) 12048178800
   (70900, 6, 119, 119) (70900,) 12048178800
   
   #
   #
   #
   t0 = time.time()
   batch_size = 100
   train_iter = mx.io.NDArrayIter(data=X_train, label=Y_train, 
batch_size=batch_size)
   val_iter = mx.io.NDArrayIter(data=X_val, label=Y_val, batch_size=batch_size)
   test_iter = mx.io.NDArrayIter(data=X_test, label=Y_test, 
batch_size=batch_size)
   print 'iterators configured', time.time()-t0, 'seconds'
   
   ---
   TypeError Traceback (most recent call last)
in ()
 4 t0 = time.time()
 5 batch_size = 100
   > 6 train_iter = mx.io.NDArrayIter(data=X_train, label=Y_train, 
batch_size=batch_size)
 7 val_iter = mx.io.NDArrayIter(data=X_val, label=Y_val, 
batch_size=batch_size)
 8 test_iter = mx.io.NDArrayIter(data=X_test, label=Y_test, 
batch_size=batch_size)
   
   /home/dhaehn/D1/lib/python2.7/site-packages/mxnet/io.pyc in __init__(self, 
data, label, batch_size, shuffle, last_batch_handle, data_name, label_name)
   577 super(NDArrayIter, self).__init__(batch_size)
   578 
   --> 579 self.data = _init_data(data, allow_empty=False, 
default_name=data_name)
   580 self.label = _init_data(label, allow_empty=True, 
default_name=label_name)
   581 
   
   /home/dhaehn/D1/lib/python2.7/site-packages/mxnet/io.pyc in _init_data(data, 
allow_empty, default_name)
   485 except:
   486 raise TypeError(("Invalid type '%s' for %s, "  % 
(type(v), k)) + \
   --> 487 "should be NDArray or numpy.ndarray")
   488 
   489 return list(data.items())
   
   TypeError: Invalid type '' for data, should be NDArray 
or numpy.ndarray
   ```
   
   but the real problem is
   
   ```
   a = mx.nd.array(X_train[0:5]) # no problem
   a = mx.nd.array(X_train[0:6]) # fails
   MXNetError: [16:37:29] include/mxnet/././tensor_blob.h:247: Check failed: 
this->shape_.Size() == shape.Size() (509796 vs. 802992704) 
TBlob.get_with_shape: new and old shape do not match total elements
   
   Stack trace returned 10 entries:
   [bt] (0) 
/home/dhaehn/D1/lib/python2.7/site-packages/mxnet/libmxnet.so(+0x18b0dc) 
[0x7f655a9510dc]
   [bt] (1) 
/home/dhaehn/D1/lib/python2.7/site-packages/mxnet/libmxnet.so(+0x27d680) 
[0x7f655aa43680]
   [bt] (2) 
/home/dhaehn/D1/lib/python2.7/site-packages/mxnet/libmxnet.so(+0x27db65) 
[0x7f655aa43b65]
   [bt] (3) 
/home/dhaehn/D1/lib/python2.7/site-packages/mxnet/libmxnet.so(+0xc8047d) 
[0x7f655b44647d]
   [bt] (4) 
/home/dhaehn/D1/lib/python2.7/site-packages/mxnet/libmxnet.so(+0xc5d29b) 
[0x7f655b42329b]
   [bt] (5) 
/home/dhaehn/D1/lib/python2.7/site-packages/mxnet/libmxnet.so(MXNDArraySyncCopyFromCPU+0xa)
 [0x7f655b2f0a1a]
   [bt] (6) /lib64/libffi.so.6(ffi_call_unix64+0x4c) [0x7f66065e3dcc]
   [bt] (7) /lib64/libffi.so.6(ffi_call+0x1f5) [0x7f66065e36f5]
   [bt] (8) 
/home/dhaehn/D1/lib64/python2.7/lib-dynload/_ctypes.so(_ctypes_callproc+0x30b) 
[0x7f66067f6c8b]
   [bt] (9) /home/dhaehn/D1/lib64/python2.7/lib-dynload/_ctypes.so(+0xaa85) 
[0x7f66067f0a85]
   
   # but..
   a = mx.nd.array(X_train[5:8]) # no problem
   ```
 

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] haehn commented on issue #4696: run errors

2017-08-11 Thread git
haehn commented on issue #4696: run errors
URL: 
https://github.com/apache/incubator-mxnet/issues/4696#issuecomment-321934353
 
 
   pip install mxnet-cu80 worked for me
 

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] asmushetzel commented on issue #7147: cuda support for linalg-functions, restructuring of linalg interfaces

2017-08-11 Thread git
asmushetzel commented on issue #7147: cuda support for linalg-functions, 
restructuring of linalg interfaces
URL: https://github.com/apache/incubator-mxnet/pull/7147#issuecomment-321932374
 
 
   It's in the final rounds of testing. Goal is to have it integrated next week.
   It would be awesome if you can put Volta/Tensorcore in there. Another topic 
that I discussed with Marek Kolodziej is whether NVidia people can also 
investigate ways how to do more efficient batch-implementations of potrf/trmm. 
One important use case will be having a batch of relatively small matrices and 
there the current implementation won't be that great (and cuBlas/cuSolver miss 
native batch processing for these operations). 
 

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] tqchen commented on issue #7434: fix a formula typo in doc

2017-08-11 Thread git
tqchen commented on issue #7434: fix a formula typo in doc
URL: https://github.com/apache/incubator-mxnet/pull/7434#issuecomment-321927323
 
 
   @piiswrong It is indeed should be channel?
 

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] nswamy commented on a change in pull request #7435: Website fix

2017-08-11 Thread git
nswamy commented on a change in pull request #7435: Website fix
URL: https://github.com/apache/incubator-mxnet/pull/7435#discussion_r132793400
 
 

 ##
 File path: docs/build_version_doc/AddVersion.py
 ##
 @@ -59,6 +59,9 @@
 continue
 with open(os.path.join(path, name), 'r') as html_file:
 content = bs(html_file, 'html.parser')
+if os.path.join(path, name) == args.file_path + 'index.html':
+content.find(id='example-link')['href'] = \
+'https://github.com/dmlc/mxnet/tree/%s/example' % 
(args.current_version)
 
 Review comment:
   https://github.com/apache/incubator-mxnet/tree/%s/example' % 
(args.current_version)
 

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] haehn commented on issue #6195: mx.io.NDArrayIter cannot work with large memmap npy file.

2017-08-11 Thread git
haehn commented on issue #6195: mx.io.NDArrayIter cannot work with large memmap 
npy file.
URL: 
https://github.com/apache/incubator-mxnet/issues/6195#issuecomment-321930283
 
 
   This is an issue for us. Is there any fix planned?
 

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] kevinthesun opened a new pull request #7435: Website fix

2017-08-11 Thread git
kevinthesun opened a new pull request #7435: Website fix
URL: https://github.com/apache/incubator-mxnet/pull/7435
 
 
   1. Remove dmlc copyright.
   2. Add link to CoreML converter
   3. Modify some logics in building versioning website.
 

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] zjjxsjh commented on issue #7434: fix a formula typo in doc

2017-08-11 Thread git
zjjxsjh commented on issue #7434: fix a formula typo in doc
URL: https://github.com/apache/incubator-mxnet/pull/7434#issuecomment-321926015
 
 
   num_channel is in the input dimension
   
   num_filter is in the output dimension
   
   the num_channel dimension should be summed up

 

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] zjjxsjh commented on issue #7434: fix a formula typo in doc

2017-08-11 Thread git
zjjxsjh commented on issue #7434: fix a formula typo in doc
URL: https://github.com/apache/incubator-mxnet/pull/7434#issuecomment-321926015
 
 
   num_channel is in the input dimension
   
   num_filter is in the output dimension
   
   the num_channel dimension should be summed up

 

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] kevinthesun opened a new pull request #5: Chnage to 0.11-RC

2017-08-11 Thread git
kevinthesun opened a new pull request #5: Chnage to 0.11-RC
URL: https://github.com/apache/incubator-mxnet-site/pull/5
 
 
   Only merge this after 0.11-RC released.
 

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 closed pull request #7434: fix a formula typo in doc

2017-08-11 Thread git
piiswrong closed pull request #7434: fix a formula typo in doc
URL: https://github.com/apache/incubator-mxnet/pull/7434
 
 
   
 

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 #7434: fix a formula typo in doc

2017-08-11 Thread git
piiswrong commented on issue #7434: fix a formula typo in doc
URL: https://github.com/apache/incubator-mxnet/pull/7434#issuecomment-321923552
 
 
   It is num filter
 

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] lxn2 commented on issue #7367: Code freeze - halt code merges

2017-08-11 Thread git
lxn2 commented on issue #7367: Code freeze - halt code merges
URL: 
https://github.com/apache/incubator-mxnet/issues/7367#issuecomment-321922254
 
 
   Unless it's a very small change or bug fix, please stop merging PRs today.
 

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 lxn2-jenkinsfile deleted (was 551fa25)

2017-08-11 Thread lxn2
This is an automated email from the ASF dual-hosted git repository.

lxn2 pushed a change to branch lxn2-jenkinsfile
in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git.


 was 551fa25  Add new variables to Jenkinsfile (#7220)

The revisions that were on this branch are still contained in
other references; therefore, this change does not discard any commits
from the repository.

-- 
To stop receiving notification emails like this one, please contact
['"comm...@mxnet.apache.org" '].


[incubator-mxnet] branch master updated: Add more license files (#7429)

2017-08-11 Thread lxn2
This is an automated email from the ASF dual-hosted git repository.

lxn2 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 a5373e6   Add more license files (#7429)
a5373e6 is described below

commit a5373e6af0c31988ada048a9ebbc84a937a7baf2
Author: lxn2 
AuthorDate: Fri Aug 11 14:12:47 2017 -0700

 Add more license files (#7429)

* Add more licenses

* Revert "Add more licenses"

This reverts commit 8395a84d21a0cebaf909ec277b7b5d6feffa1412.

* Add license files
---
 .../jni/org/dmlc/mxnet/MxnetException.java | 19 +++
 amalgamation/jni/org/dmlc/mxnet/Predictor.java | 21 -
 cpp-package/example/alexnet.cpp| 20 +++-
 cpp-package/example/charRNN.cpp| 20 +++-
 .../example/feature_extract/feature_extract.cpp| 20 +++-
 .../feature_extract/prepare_data_with_opencv.cpp   | 20 +++-
 cpp-package/example/googlenet.cpp  | 20 +++-
 cpp-package/example/inception_bn.cpp   | 20 +++-
 cpp-package/example/lenet.cpp  | 20 +++-
 cpp-package/example/lenet_with_mxdataiter.cpp  | 20 +++-
 cpp-package/example/mlp.cpp| 20 +++-
 cpp-package/example/mlp_cpu.cpp| 20 +++-
 cpp-package/example/mlp_gpu.cpp| 20 +++-
 cpp-package/example/resnet.cpp | 20 +++-
 cpp-package/example/test_score.cpp | 20 +++-
 cpp-package/include/mxnet-cpp/executor.hpp | 20 +++-
 cpp-package/include/mxnet-cpp/io.hpp   | 20 +++-
 cpp-package/include/mxnet-cpp/kvstore.hpp  | 20 +++-
 cpp-package/include/mxnet-cpp/monitor.hpp  | 20 +++-
 cpp-package/include/mxnet-cpp/ndarray.hpp  | 20 +++-
 cpp-package/include/mxnet-cpp/operator.hpp | 20 +++-
 cpp-package/include/mxnet-cpp/optimizer.hpp| 20 +++-
 cpp-package/include/mxnet-cpp/symbol.hpp   | 20 +++-
 .../tree_lstm/lib/CollapseUnaryTransformer.java| 19 +++
 example/gluon/tree_lstm/lib/ConstituencyParse.java | 21 -
 example/gluon/tree_lstm/lib/DependencyParse.java   | 19 +++
 example/rcnn/rcnn/cython/gpu_nms.hpp   | 19 +++
 example/rcnn/rcnn/pycocotools/maskApi.c| 19 +++
 example/ssd/tools/caffe_converter/make_win32.bat   | 17 +
 perl-package/AI-MXNet/examples/calculator.pl   | 18 ++
 perl-package/AI-MXNet/examples/char_lstm.pl| 18 ++
 .../AI-MXNet/examples/cudnn_lstm_bucketing.pl  | 20 +++-
 perl-package/AI-MXNet/examples/lstm_bucketing.pl   | 18 ++
 perl-package/AI-MXNet/examples/mnist.pl| 22 --
 perl-package/AI-MXNet/examples/plot_network.pl | 18 ++
 setup-utils/install-mxnet-windows-python.bat   | 17 +
 tools/caffe_converter/make_win32.bat   | 17 +
 tools/license_header.py|  9 ++---
 38 files changed, 702 insertions(+), 29 deletions(-)

diff --git a/amalgamation/jni/org/dmlc/mxnet/MxnetException.java 
b/amalgamation/jni/org/dmlc/mxnet/MxnetException.java
index c342cfa..08d80d6 100644
--- a/amalgamation/jni/org/dmlc/mxnet/MxnetException.java
+++ b/amalgamation/jni/org/dmlc/mxnet/MxnetException.java
@@ -1,3 +1,22 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements.  See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership.  The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License.  You may obtain a copy of the License at
+ *
+ *   http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied.  See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
+
 package org.dmlc.mxnet;
 
 public class MxnetException extends Exception {
diff --git a/amalgamation/jni/org/dmlc/mxnet/Predictor.java 
b/amalgamation/jni/org/dmlc/mxnet/Predictor.java
index a91312a..53152dc 100644
--- 

[GitHub] lxn2 closed pull request #7429: Add more license files

2017-08-11 Thread git
lxn2 closed pull request #7429:  Add more license files
URL: https://github.com/apache/incubator-mxnet/pull/7429
 
 
   
 

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] lxn2 commented on issue #7429: Add more license files

2017-08-11 Thread git
lxn2 commented on issue #7429:  Add more license files
URL: https://github.com/apache/incubator-mxnet/pull/7429#issuecomment-321919977
 
 
   PR builds passed, got approval, will merge 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] srochel commented on a change in pull request #7432: Prepare for v0.11.0 release

2017-08-11 Thread git
srochel commented on a change in pull request #7432: Prepare for v0.11.0 release
URL: https://github.com/apache/incubator-mxnet/pull/7432#discussion_r132782073
 
 

 ##
 File path: NEWS.md
 ##
 @@ -1,5 +1,35 @@
 MXNet Change Log
 
+## 0.11.0
 
 Review comment:
   should we include the Keras and CoreML section as well from 
https://cwiki.apache.org/confluence/display/MXNET/v0.11.0+Release+Notes ?
 

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 opened a new pull request #7433: fix autograd memory cost

2017-08-11 Thread git
piiswrong opened a new pull request #7433: fix autograd memory cost
URL: https://github.com/apache/incubator-mxnet/pull/7433
 
 
   
 

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 closed pull request #7403: Add autograd function

2017-08-11 Thread git
piiswrong closed pull request #7403: Add autograd function
URL: https://github.com/apache/incubator-mxnet/pull/7403
 
 
   
 

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] kottmann commented on issue #7123: Add installation instruction for Fedora (#7107)

2017-08-11 Thread git
kottmann commented on issue #7123: Add installation instruction for Fedora 
(#7107)
URL: https://github.com/apache/incubator-mxnet/pull/7123#issuecomment-321909870
 
 
   @sandeep-krishnamurthy I think it would also add value without CI, because 
it will make it easier for people to figure it out them self and if it is 
broken they could also send a PR to fix it again.
   
   About point 4, today you still need to add this include 
/usr/include/openblas manually, maybe we can somehow fix that in the build and 
people would get away transforming the Ubuntu steps to Fedora. 
   Do you think it would be worth fixing that?
 

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 autograd function (#7403)

2017-08-11 Thread jxie
This is an automated email from the ASF dual-hosted git repository.

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


The following commit(s) were added to refs/heads/master by this push:
 new 8ae2970  Add autograd function (#7403)
8ae2970 is described below

commit 8ae2970083aa5bcdf37c3d85fcfead39c1802f40
Author: Eric Junyuan Xie 
AuthorDate: Fri Aug 11 13:26:48 2017 -0700

Add autograd function (#7403)

* fix optimizer

* add imread

* add autograd Function

* add function

* fix

* fix

* fix

* fix

* fix

* fix
---
 docs/api/python/autograd.md  |   1 +
 include/mxnet/c_api.h|  35 +-
 include/mxnet/ndarray.h  |   8 ++
 include/mxnet/op_attr_types.h|   1 +
 python/mxnet/autograd.py | 167 +-
 python/mxnet/base.py |  10 ++
 python/mxnet/gluon/data/dataset.py   |  26 +---
 python/mxnet/gluon/data/vision.py| 109 -
 python/mxnet/image/detection.py  |  12 +-
 python/mxnet/image/image.py  |  89 ++
 python/mxnet/operator.py |  11 +-
 src/c_api/c_api_function.cc  | 199 +++
 src/c_api/c_api_ndarray.cc   |   8 ++
 src/executor/attach_op_execs_pass.cc |   2 +-
 src/io/image_io.cc   | 160 ++---
 src/ndarray/autograd.cc  |  18 +--
 src/operator/custom/custom.cc|  12 +-
 tests/python/unittest/test_autograd.py   |  47 
 tests/python/unittest/test_gluon_data.py |  31 +++--
 19 files changed, 809 insertions(+), 137 deletions(-)

diff --git a/docs/api/python/autograd.md b/docs/api/python/autograd.md
index 444e01f..de81884 100644
--- a/docs/api/python/autograd.md
+++ b/docs/api/python/autograd.md
@@ -86,6 +86,7 @@ Detailed tutorials are available in Part 1 of
 set_recording
 is_recording
 mark_variables
+Function
 ```
 
 ## API Reference
diff --git a/include/mxnet/c_api.h b/include/mxnet/c_api.h
index 7a45099..2289354 100644
--- a/include/mxnet/c_api.h
+++ b/include/mxnet/c_api.h
@@ -145,6 +145,7 @@ enum CustomOpPropCallbacks {
   kCustomOpPropInferType
 };
 
+
 typedef int (*CustomOpFBFunc)(int /*size*/, void** /*ptrs*/, int* /*tags*/,
   const int* /*reqs*/, const int /*is_train*/,
   void* /*state*/);
@@ -164,6 +165,17 @@ typedef int (*CustomOpPropCreator)(const char* 
/*op_type*/, const int /*num_kwar
const char** /*keys*/, const char** 
/*values*/,
struct MXCallbackList* /*ret*/);
 
+
+enum CustomFunctionCallbacks {
+  kCustomFunctionBackward,
+  kCustomFunctionDelete
+};
+
+typedef int (*CustomFunctionBwdFunc)(int /*num_ograds*/, int /*num_igrads*/, 
void** /*ptrs*/,
+ const int* /*reqs*/, const int 
/*is_train*/,
+ void* /*state*/);
+typedef int (*CustomFunctionDelFunc)(void* /*state*/);
+
 /*!
  * \brief return str message of the last error
  *  all function in this file will return 0 when success
@@ -639,6 +651,12 @@ MXNET_DLL int MXAutogradBackwardEx(mx_uint num_output,
NDArrayHandle* ograd_handles,
int retain_graph,
int is_train);
+/*
+ * \brief get the graph constructed by autograd.
+ * \param handle ndarray handle
+ * \param out output symbol handle
+ */
+MXNET_DLL int MXAutogradGetSymbol(NDArrayHandle handle, SymbolHandle *out);
 /*!
  * \brief create cached operator
  */
@@ -1686,8 +1704,23 @@ MXNET_DLL int MXRtcPush(RtcHandle handle, mx_uint 
num_input, mx_uint num_output,
  * \brief Delete a MXRtc object
 */
 MXNET_DLL int MXRtcFree(RtcHandle handle);
-
+/*
+ * \brief register custom operators from frontend.
+ * \param op_type name of custom op
+ * \param creator
+ */
 MXNET_DLL int MXCustomOpRegister(const char* op_type, CustomOpPropCreator 
creator);
+/*
+ * \brief record custom function for backward later.
+ * \param num_inputs number of input NDArrays.
+ * \param inputs handle to input NDArrays.
+ * \param num_outputs number of output NDArrays.
+ * \param outputs handle to output NDArrays.
+ * \param callbacks callbacks for backward function.
+ */
+MXNET_DLL int MXCustomFunctionRecord(int num_inputs, NDArrayHandle *inputs,
+ int num_outputs, NDArrayHandle *outputs,
+ MXCallbackList *callbacks);
 
 #ifdef __cplusplus
 }
diff --git a/include/mxnet/ndarray.h b/include/mxnet/ndarray.h
index d40b549..d7dff40 100644
--- a/include/mxnet/ndarray.h
+++ b/include/mxnet/ndarray.h
@@ -329,6 +329,14 @@ class NDArray {
 

[GitHub] nswamy commented on a change in pull request #7429: Add more license files

2017-08-11 Thread git
nswamy commented on a change in pull request #7429:  Add more license files
URL: https://github.com/apache/incubator-mxnet/pull/7429#discussion_r132773202
 
 

 ##
 File path: cpp-package/example/mlp.cpp
 ##
 @@ -1,5 +1,23 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements.  See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership.  The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License.  You may obtain a copy of the License at
+ *
+ *   http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied.  See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
+
 /*!
- * Copyright (c) 2015 by Contributors
 
 Review comment:
   same
 

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] nswamy commented on a change in pull request #7429: Add more license files

2017-08-11 Thread git
nswamy commented on a change in pull request #7429:  Add more license files
URL: https://github.com/apache/incubator-mxnet/pull/7429#discussion_r132773100
 
 

 ##
 File path: cpp-package/example/lenet.cpp
 ##
 @@ -1,5 +1,23 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements.  See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership.  The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License.  You may obtain a copy of the License at
+ *
+ *   http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied.  See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
+
 /*!
 
 Review comment:
   remove empty comment lines line 20, 21
 

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] nswamy commented on a change in pull request #7429: Add more license files

2017-08-11 Thread git
nswamy commented on a change in pull request #7429:  Add more license files
URL: https://github.com/apache/incubator-mxnet/pull/7429#discussion_r132773178
 
 

 ##
 File path: cpp-package/example/lenet_with_mxdataiter.cpp
 ##
 @@ -1,5 +1,23 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements.  See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership.  The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License.  You may obtain a copy of the License at
+ *
+ *   http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied.  See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
+
 /*!
 
 Review comment:
   same as above
 

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] nswamy commented on a change in pull request #7432: Prepare for v0.11.0 release

2017-08-11 Thread git
nswamy commented on a change in pull request #7432: Prepare for v0.11.0 release
URL: https://github.com/apache/incubator-mxnet/pull/7432#discussion_r132770265
 
 

 ##
 File path: NEWS.md
 ##
 @@ -1,5 +1,35 @@
 MXNet Change Log
 
 
 Review comment:
   News and Readme update should be pushed after the release has been made.
 

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] nswamy commented on a change in pull request #7432: Prepare for v0.11.0 release

2017-08-11 Thread git
nswamy commented on a change in pull request #7432: Prepare for v0.11.0 release
URL: https://github.com/apache/incubator-mxnet/pull/7432#discussion_r132770514
 
 

 ##
 File path: README.md
 ##
 @@ -22,6 +22,7 @@ deep learning systems, and interesting insights of DL 
systems for hackers.
 
 What's New
 --
+* [Version 0.11.0 Release](https://github.com/dmlc/mxnet/releases/tag/v0.11.0) 
- MXNet 0.11.0 Release.
 
 Review comment:
   same as above. should be 0.11.0-rc0 
 

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] nswamy commented on a change in pull request #7432: Prepare for v0.11.0 release

2017-08-11 Thread git
nswamy commented on a change in pull request #7432: Prepare for v0.11.0 release
URL: https://github.com/apache/incubator-mxnet/pull/7432#discussion_r132770381
 
 

 ##
 File path: NEWS.md
 ##
 @@ -1,5 +1,35 @@
 MXNet Change Log
 
+## 0.11.0
 
 Review comment:
   I don't think we can call it 0.11.0 before the vote succeeds. It should be 
0.11.0-rc0
 

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] DickJC123 commented on issue #7147: cuda support for linalg-functions, restructuring of linalg interfaces

2017-08-11 Thread git
DickJC123 commented on issue #7147: cuda support for linalg-functions, 
restructuring of linalg interfaces
URL: https://github.com/apache/incubator-mxnet/pull/7147#issuecomment-321898602
 
 
   What is the status on readying this PR for acceptance?  We're considering 
adapting this code to support NVIDIA's new Volta/TensorCore technology.
 

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] sandeep-krishnamurthy commented on a change in pull request #7432: Prepare for v0.11.0 release

2017-08-11 Thread git
sandeep-krishnamurthy commented on a change in pull request #7432: Prepare for 
v0.11.0 release
URL: https://github.com/apache/incubator-mxnet/pull/7432#discussion_r132750942
 
 

 ##
 File path: NEWS.md
 ##
 @@ -1,5 +1,35 @@
 MXNet Change Log
 
+## 0.11.0
+- Major Features
 
 Review comment:
   Headers probably ? - "##"
 

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] sandeep-krishnamurthy commented on issue #7123: Add installation instruction for Fedora (#7107)

2017-08-11 Thread git
sandeep-krishnamurthy commented on issue #7123: Add installation instruction 
for Fedora (#7107)
URL: https://github.com/apache/incubator-mxnet/pull/7123#issuecomment-321881050
 
 
   @kottmann - Thank you for the contribution. However, I see below issues to 
accept this contribution.
   
   1) We do not have CI setup for Fedora distribution.
   2) Due to very few customer/developer base, I do not see it as the higher 
priority task that will be planned soon. 
   3) Without tests, I see a greater risk in having the installation guide on 
the docs site. As it can lead to broken installation guide and we will never 
notice.
   4) Following the Ubuntu installation guide, I am hopeful that users will be 
able to find equivalent installation command for Fedora.
 

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 a change in pull request #7390: adding ranking metrics (precision/recall) at position K.

2017-08-11 Thread git
eric-haibin-lin commented on a change in pull request #7390: adding ranking 
metrics (precision/recall) at position K. 
URL: https://github.com/apache/incubator-mxnet/pull/7390#discussion_r132745728
 
 

 ##
 File path: python/mxnet/ranking_metrics.py
 ##
 @@ -0,0 +1,46 @@
+"""
+A function that implements ranking metrics
+(precision, recall, coverage and converted coverage)
+for a given position K
+"""
+
+def metrics_at_k(Y, Yhat, K):
+"""
+Parameters
+--
+Y  : dictionary with key = sample index and value = list of positive 
indices of features
+Yhat: dict with key = sample index
+  and value = ORDERED list of indices of features, according to 
some score
+K  : position at which to compute score
+
+Returns
+---
+pre: precision at K
+rec: recall at K
+convcoverage: converted coverage at K
+coverage   : coverage at K
+
+Examples
+
+>>>Ytrue = {1: [1,2,3,4]}
+>>>Yhat = {1:[1,2,3,4,5,6,7,8,9]}
+>>>k = 2
+>>>print(metrics_at_k(Ytrue,Yhat,k))
 
 Review comment:
   Could you add the result of the print 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] eric-haibin-lin commented on a change in pull request #7390: adding ranking metrics (precision/recall) at position K.

2017-08-11 Thread git
eric-haibin-lin commented on a change in pull request #7390: adding ranking 
metrics (precision/recall) at position K. 
URL: https://github.com/apache/incubator-mxnet/pull/7390#discussion_r132746129
 
 

 ##
 File path: python/mxnet/ranking_metrics.py
 ##
 @@ -0,0 +1,46 @@
+"""
+A function that implements ranking metrics
+(precision, recall, coverage and converted coverage)
+for a given position K
+"""
+
+def metrics_at_k(Y, Yhat, K):
+"""
+Parameters
+--
+Y  : dictionary with key = sample index and value = list of positive 
indices of features
+Yhat: dict with key = sample index
+  and value = ORDERED list of indices of features, according to 
some score
 
 Review comment:
   Does the model always produce ordered result? is that why you're adding the 
`ordered` restriction 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] eric-haibin-lin commented on a change in pull request #7390: adding ranking metrics (precision/recall) at position K.

2017-08-11 Thread git
eric-haibin-lin commented on a change in pull request #7390: adding ranking 
metrics (precision/recall) at position K. 
URL: https://github.com/apache/incubator-mxnet/pull/7390#discussion_r132746805
 
 

 ##
 File path: python/mxnet/ranking_metrics.py
 ##
 @@ -0,0 +1,46 @@
+"""
 
 Review comment:
   I don't think it's a good idea adding this as a standalone file under 
`python/mxnet`. Is it more reasonable to put integrate this metric in 
`python/mxnet/metric.py` file? 
 

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] mli commented on issue #7429: Add more license files

2017-08-11 Thread git
mli commented on issue #7429:  Add more license files
URL: https://github.com/apache/incubator-mxnet/pull/7429#issuecomment-321873782
 
 
   LGTM, 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


[incubator-mxnet] branch ApacheRat2 updated (8395a84 -> 8b55872)

2017-08-11 Thread lxn2
This is an automated email from the ASF dual-hosted git repository.

lxn2 pushed a change to branch ApacheRat2
in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git.


from 8395a84  Add more licenses
 new 6a765e4  Revert "Add more licenses"
 new 8b55872  Add license files

The 2 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.


Summary of changes:
 docs/_static/cn.svg| 17 --
 docs/_static/jquery-1.11.1.js  | 19 ---
 docs/_static/js/auto_module_index.js   | 21 +---
 docs/_static/js/clipboard.min.js   | 21 +---
 docs/_static/js/copycode.js| 19 ---
 docs/_static/js/navbar.js  | 25 ++-
 docs/_static/js/options.js | 19 ---
 docs/_static/js/search.js  | 23 ++
 docs/_static/js/sidebar.js | 37 ++
 docs/_static/mxnet-theme/footer.html   | 17 --
 docs/_static/mxnet-theme/index.html| 19 +--
 docs/_static/mxnet-theme/layout.html   | 17 --
 docs/_static/mxnet-theme/navbar.html   | 17 --
 docs/_static/mxnet.css | 23 ++
 docs/_static/searchtools_custom.js | 37 ++
 docs/_static/selectlang.js | 25 ++-
 docs/_static/us.svg| 17 --
 scala-package/assembly/linux-x86_64-cpu/pom.xml| 17 --
 .../src/main/assembly/assembly.xml | 17 --
 scala-package/assembly/linux-x86_64-gpu/pom.xml| 17 --
 .../src/main/assembly/assembly.xml | 17 --
 .../osx-x86_64-cpu/main/assembly/assembly.xml  | 17 --
 scala-package/assembly/osx-x86_64-cpu/pom.xml  | 17 --
 .../osx-x86_64-cpu/src/main/assembly/assembly.xml  | 17 --
 scala-package/assembly/pom.xml | 17 --
 scala-package/core/pom.xml | 17 --
 scala-package/examples/pom.xml | 17 --
 scala-package/init-native/linux-x86_64/pom.xml | 17 --
 scala-package/init-native/osx-x86_64/pom.xml   | 17 --
 scala-package/init-native/pom.xml  | 17 --
 scala-package/init/pom.xml | 17 --
 scala-package/macros/pom.xml   | 17 --
 scala-package/native/linux-x86_64-cpu/pom.xml  | 17 --
 scala-package/native/linux-x86_64-gpu/pom.xml  | 17 --
 scala-package/native/osx-x86_64-cpu/pom.xml| 17 --
 scala-package/native/pom.xml   | 17 --
 scala-package/pom.xml  | 17 --
 scala-package/spark/pom.xml| 17 --
 tools/license_header.py|  9 ++
 39 files changed, 33 insertions(+), 706 deletions(-)

-- 
To stop receiving notification emails like this one, please contact
['"comm...@mxnet.apache.org" '].


[incubator-mxnet] 01/02: Revert "Add more licenses"

2017-08-11 Thread lxn2
This is an automated email from the ASF dual-hosted git repository.

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

commit 6a765e4c641fb44241121c05426422ebaf32adb8
Author: Nguyen 
AuthorDate: Fri Aug 11 10:26:57 2017 -0700

Revert "Add more licenses"

This reverts commit 8395a84d21a0cebaf909ec277b7b5d6feffa1412.
---
 .../jni/org/dmlc/mxnet/MxnetException.java | 19 ---
 amalgamation/jni/org/dmlc/mxnet/Predictor.java | 21 +---
 cpp-package/example/alexnet.cpp| 20 +---
 cpp-package/example/charRNN.cpp| 20 +---
 .../example/feature_extract/feature_extract.cpp| 20 +---
 .../feature_extract/prepare_data_with_opencv.cpp   | 20 +---
 cpp-package/example/googlenet.cpp  | 20 +---
 cpp-package/example/inception_bn.cpp   | 20 +---
 cpp-package/example/lenet.cpp  | 20 +---
 cpp-package/example/lenet_with_mxdataiter.cpp  | 20 +---
 cpp-package/example/mlp.cpp| 20 +---
 cpp-package/example/mlp_cpu.cpp| 20 +---
 cpp-package/example/mlp_gpu.cpp| 20 +---
 cpp-package/example/resnet.cpp | 20 +---
 cpp-package/example/test_score.cpp | 20 +---
 cpp-package/include/mxnet-cpp/executor.hpp | 20 +---
 cpp-package/include/mxnet-cpp/io.hpp   | 20 +---
 cpp-package/include/mxnet-cpp/kvstore.hpp  | 20 +---
 cpp-package/include/mxnet-cpp/monitor.hpp  | 20 +---
 cpp-package/include/mxnet-cpp/ndarray.hpp  | 20 +---
 cpp-package/include/mxnet-cpp/operator.hpp | 20 +---
 cpp-package/include/mxnet-cpp/optimizer.hpp| 20 +---
 cpp-package/include/mxnet-cpp/symbol.hpp   | 20 +---
 docs/_static/cn.svg| 17 --
 docs/_static/jquery-1.11.1.js  | 19 ---
 docs/_static/js/auto_module_index.js   | 21 +---
 docs/_static/js/clipboard.min.js   | 21 +---
 docs/_static/js/copycode.js| 19 ---
 docs/_static/js/navbar.js  | 25 ++-
 docs/_static/js/options.js | 19 ---
 docs/_static/js/search.js  | 23 ++
 docs/_static/js/sidebar.js | 37 ++
 docs/_static/mxnet-theme/footer.html   | 17 --
 docs/_static/mxnet-theme/index.html| 19 +--
 docs/_static/mxnet-theme/layout.html   | 17 --
 docs/_static/mxnet-theme/navbar.html   | 17 --
 docs/_static/mxnet.css | 23 ++
 docs/_static/searchtools_custom.js | 37 ++
 docs/_static/selectlang.js | 25 ++-
 docs/_static/us.svg| 17 --
 .../tree_lstm/lib/CollapseUnaryTransformer.java| 19 ---
 example/gluon/tree_lstm/lib/ConstituencyParse.java | 21 +---
 example/gluon/tree_lstm/lib/DependencyParse.java   | 19 ---
 example/rcnn/rcnn/cython/gpu_nms.hpp   | 19 ---
 example/rcnn/rcnn/pycocotools/maskApi.c| 19 ---
 example/ssd/tools/caffe_converter/make_win32.bat   | 17 --
 perl-package/AI-MXNet/examples/calculator.pl   | 18 ---
 perl-package/AI-MXNet/examples/char_lstm.pl| 18 ---
 .../AI-MXNet/examples/cudnn_lstm_bucketing.pl  | 20 +---
 perl-package/AI-MXNet/examples/lstm_bucketing.pl   | 18 ---
 perl-package/AI-MXNet/examples/mnist.pl| 22 ++---
 perl-package/AI-MXNet/examples/plot_network.pl | 18 ---
 scala-package/assembly/linux-x86_64-cpu/pom.xml| 17 --
 .../src/main/assembly/assembly.xml | 17 --
 scala-package/assembly/linux-x86_64-gpu/pom.xml| 17 --
 .../src/main/assembly/assembly.xml | 17 --
 .../osx-x86_64-cpu/main/assembly/assembly.xml  | 17 --
 scala-package/assembly/osx-x86_64-cpu/pom.xml  | 17 --
 .../osx-x86_64-cpu/src/main/assembly/assembly.xml  | 17 --
 scala-package/assembly/pom.xml | 17 --
 scala-package/core/pom.xml | 17 --
 scala-package/examples/pom.xml | 17 --
 scala-package/init-native/linux-x86_64/pom.xml | 17 --
 scala-package/init-native/osx-x86_64/pom.xml   | 17 --
 scala-package/init-native/pom.xml  | 17 --
 scala-package/init/pom.xml 

[incubator-mxnet] 02/02: Add license files

2017-08-11 Thread lxn2
This is an automated email from the ASF dual-hosted git repository.

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

commit 8b55872b728ae37648456b8cb38f66e88fdc4b2b
Author: Nguyen 
AuthorDate: Fri Aug 11 10:32:26 2017 -0700

Add license files
---
 .../jni/org/dmlc/mxnet/MxnetException.java | 19 +++
 amalgamation/jni/org/dmlc/mxnet/Predictor.java | 21 -
 cpp-package/example/alexnet.cpp| 20 +++-
 cpp-package/example/charRNN.cpp| 20 +++-
 .../example/feature_extract/feature_extract.cpp| 20 +++-
 .../feature_extract/prepare_data_with_opencv.cpp   | 20 +++-
 cpp-package/example/googlenet.cpp  | 20 +++-
 cpp-package/example/inception_bn.cpp   | 20 +++-
 cpp-package/example/lenet.cpp  | 20 +++-
 cpp-package/example/lenet_with_mxdataiter.cpp  | 20 +++-
 cpp-package/example/mlp.cpp| 20 +++-
 cpp-package/example/mlp_cpu.cpp| 20 +++-
 cpp-package/example/mlp_gpu.cpp| 20 +++-
 cpp-package/example/resnet.cpp | 20 +++-
 cpp-package/example/test_score.cpp | 20 +++-
 cpp-package/include/mxnet-cpp/executor.hpp | 20 +++-
 cpp-package/include/mxnet-cpp/io.hpp   | 20 +++-
 cpp-package/include/mxnet-cpp/kvstore.hpp  | 20 +++-
 cpp-package/include/mxnet-cpp/monitor.hpp  | 20 +++-
 cpp-package/include/mxnet-cpp/ndarray.hpp  | 20 +++-
 cpp-package/include/mxnet-cpp/operator.hpp | 20 +++-
 cpp-package/include/mxnet-cpp/optimizer.hpp| 20 +++-
 cpp-package/include/mxnet-cpp/symbol.hpp   | 20 +++-
 .../tree_lstm/lib/CollapseUnaryTransformer.java| 19 +++
 example/gluon/tree_lstm/lib/ConstituencyParse.java | 21 -
 example/gluon/tree_lstm/lib/DependencyParse.java   | 19 +++
 example/rcnn/rcnn/cython/gpu_nms.hpp   | 19 +++
 example/rcnn/rcnn/pycocotools/maskApi.c| 19 +++
 example/ssd/tools/caffe_converter/make_win32.bat   | 17 +
 perl-package/AI-MXNet/examples/calculator.pl   | 18 ++
 perl-package/AI-MXNet/examples/char_lstm.pl| 18 ++
 .../AI-MXNet/examples/cudnn_lstm_bucketing.pl  | 20 +++-
 perl-package/AI-MXNet/examples/lstm_bucketing.pl   | 18 ++
 perl-package/AI-MXNet/examples/mnist.pl| 22 --
 perl-package/AI-MXNet/examples/plot_network.pl | 18 ++
 setup-utils/install-mxnet-windows-python.bat   | 17 +
 tools/caffe_converter/make_win32.bat   | 17 +
 tools/license_header.py|  9 ++---
 38 files changed, 702 insertions(+), 29 deletions(-)

diff --git a/amalgamation/jni/org/dmlc/mxnet/MxnetException.java 
b/amalgamation/jni/org/dmlc/mxnet/MxnetException.java
index c342cfa..08d80d6 100644
--- a/amalgamation/jni/org/dmlc/mxnet/MxnetException.java
+++ b/amalgamation/jni/org/dmlc/mxnet/MxnetException.java
@@ -1,3 +1,22 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements.  See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership.  The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License.  You may obtain a copy of the License at
+ *
+ *   http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied.  See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
+
 package org.dmlc.mxnet;
 
 public class MxnetException extends Exception {
diff --git a/amalgamation/jni/org/dmlc/mxnet/Predictor.java 
b/amalgamation/jni/org/dmlc/mxnet/Predictor.java
index a91312a..53152dc 100644
--- a/amalgamation/jni/org/dmlc/mxnet/Predictor.java
+++ b/amalgamation/jni/org/dmlc/mxnet/Predictor.java
@@ -1,3 +1,22 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements.  See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright 

[GitHub] lxn2 opened a new pull request #7432: Prepare for v0.11.0 release

2017-08-11 Thread git
lxn2 opened a new pull request #7432: Prepare for v0.11.0 release
URL: https://github.com/apache/incubator-mxnet/pull/7432
 
 
   This PR prepares for v0.11.0 release:
   
   - Bumps up version number
   - Adds updates to README & NEWS
 

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 #7429: Add more license files

2017-08-11 Thread git
piiswrong commented on issue #7429:  Add more license files
URL: https://github.com/apache/incubator-mxnet/pull/7429#issuecomment-321866643
 
 
   scala test fails
 

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] ptrendx commented on issue #7424: train_mnist.py failed: TypeError: __init__() got an unexpected keyword argument 'multi_precision'

2017-08-11 Thread git
ptrendx commented on issue #7424: train_mnist.py failed: TypeError: __init__() 
got an unexpected keyword argument 'multi_precision'
URL: 
https://github.com/apache/incubator-mxnet/issues/7424#issuecomment-321865944
 
 
   Strange, I can't reproduce that... Are you sure you compiled and "pip 
install -e ." the cloned version?
 

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] ptrendx commented on issue #7384: Parameter 'dtype' in the function 'ImageRecordIter' doesn't work

2017-08-11 Thread git
ptrendx commented on issue #7384: Parameter 'dtype' in the function 
'ImageRecordIter' doesn't work
URL: 
https://github.com/apache/incubator-mxnet/issues/7384#issuecomment-321864377
 
 
   Did it work in the previous version of the ImageRecordIter (you can test 
that by changing ImageRecordIter to ImageRecordIter_v1)? Judging by the code it 
should not work (it should either give you an error or result in a silent 
corruption), so the dtype option is just wrong and should not be there. For 
example to get UInt8 data type you need to use ImageRecordUInt8Iter.
   The best way of getting fp16 data is to cast it to fp16 after it is produced 
in fp32 (I assume you are using GPU - this gives additional bonus of double 
buffering of data copies to GPU, so should generally be faster than just using 
ImageRecordIter and outputting fp16 format). Example of doing that is in 
example/image-classification/symbols/alexnet.py (resnet, resnet-v1, 
inception-v3 and v4 also have that, but alexnet is the simplest).
 

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] haehn commented on issue #791: Databases with numpy

2017-08-11 Thread git
haehn commented on issue #791: Databases with numpy
URL: https://github.com/apache/incubator-mxnet/issues/791#issuecomment-321859424
 
 
   @yl2612 @yangshao did you solve this? if yes, could you please share code?
 

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] Tao-Sun opened a new issue #7431: AttributeError: 'module' object has no attribute 'stack'

2017-08-11 Thread git
Tao-Sun opened a new issue #7431: AttributeError: 'module' object has no 
attribute 'stack'
URL: https://github.com/apache/incubator-mxnet/issues/7431
 
 
   For bugs or installation issues, please provide the following information.
   The more information you provide, the more likely people will be able to 
help you.
   
   ## Environment info
   Operating System: OSX and Ubuntu
   
   Compiler: Not sure. Installed according to 
http://mxnet.io/get_started/install.html.
   
   Package used (Python/R/Scala/Julia): Python
   
   MXNet version: Not sure. Installed according to 
http://mxnet.io/get_started/install.html.
   
   Or if installed from source:
   
   MXNet commit hash (`git rev-parse HEAD`):
   
   If you are using python package, please provide
   
   Python version and distribution:
   
   If you are using R package, please provide
   
   R `sessionInfo()`:
   
   ## Error Message:
   Please paste the full error message, including stack trace.
   
   ## Minimum reproducible example
   if you are using your own code, please provide a short script that 
reproduces the error.
   
   ## Steps to reproduce
   or if you are running standard examples, please provide the commands you 
have run that lead to the error.
   
   1. import mxnet as mx
   2.mx.nd.stack
   
   Comment:
   I installed both on my Apple Mbp (pip) and Ubuntu (14.04, docker) following 
the steps in Not sure http://mxnet.io/get_started/install.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] kottmann commented on issue #7123: Add installation instruction for Fedora (#7107)

2017-08-11 Thread git
kottmann commented on issue #7123: Add installation instruction for Fedora 
(#7107)
URL: https://github.com/apache/incubator-mxnet/pull/7123#issuecomment-321847051
 
 
   @madjam I tested this with Fedora 26, and let me test it as well with Fedora 
25. Older versions are not supported anymore.
 

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 opened a new pull request #7430: remove self-implemented speedometer

2017-08-11 Thread git
ZiyueHuang opened a new pull request #7430: remove self-implemented speedometer
URL: https://github.com/apache/incubator-mxnet/pull/7430
 
 
   Because mx.callback.Speedometer has parameter `auto_rest` now, its behavior 
is the same with the self-implemented Speedometer when `auto_rest=false`.
 

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] madjam commented on issue #7123: Add installation instruction for Fedora (#7107)

2017-08-11 Thread git
madjam commented on issue #7123: Add installation instruction for Fedora (#7107)
URL: https://github.com/apache/incubator-mxnet/pull/7123#issuecomment-321846027
 
 
   what version of Fedora is this tested on? We have CI tests for validating 
installation instructions. Not sure if Fedora is popular enough to warrant 
being tested in CI.
   @sandeep-krishnamurthy what do you think?
 

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] lxn2 opened a new pull request #7429: Add more license files

2017-08-11 Thread git
lxn2 opened a new pull request #7429:  Add more license files
URL: https://github.com/apache/incubator-mxnet/pull/7429
 
 
   Added more extension handlers to license_header.py. Made it verbose so we 
keep track of what we're whitelisting/skipping.
   
   Added license headers to files with newly added extension headers.
 

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] 01/01: Add more licenses

2017-08-11 Thread lxn2
This is an automated email from the ASF dual-hosted git repository.

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

commit 8395a84d21a0cebaf909ec277b7b5d6feffa1412
Author: Nguyen 
AuthorDate: Fri Aug 11 08:05:51 2017 -0700

Add more licenses
---
 .../jni/org/dmlc/mxnet/MxnetException.java | 19 +++
 amalgamation/jni/org/dmlc/mxnet/Predictor.java | 21 +++-
 cpp-package/example/alexnet.cpp| 20 +++-
 cpp-package/example/charRNN.cpp| 20 +++-
 .../example/feature_extract/feature_extract.cpp| 20 +++-
 .../feature_extract/prepare_data_with_opencv.cpp   | 20 +++-
 cpp-package/example/googlenet.cpp  | 20 +++-
 cpp-package/example/inception_bn.cpp   | 20 +++-
 cpp-package/example/lenet.cpp  | 20 +++-
 cpp-package/example/lenet_with_mxdataiter.cpp  | 20 +++-
 cpp-package/example/mlp.cpp| 20 +++-
 cpp-package/example/mlp_cpu.cpp| 20 +++-
 cpp-package/example/mlp_gpu.cpp| 20 +++-
 cpp-package/example/resnet.cpp | 20 +++-
 cpp-package/example/test_score.cpp | 20 +++-
 cpp-package/include/mxnet-cpp/executor.hpp | 20 +++-
 cpp-package/include/mxnet-cpp/io.hpp   | 20 +++-
 cpp-package/include/mxnet-cpp/kvstore.hpp  | 20 +++-
 cpp-package/include/mxnet-cpp/monitor.hpp  | 20 +++-
 cpp-package/include/mxnet-cpp/ndarray.hpp  | 20 +++-
 cpp-package/include/mxnet-cpp/operator.hpp | 20 +++-
 cpp-package/include/mxnet-cpp/optimizer.hpp| 20 +++-
 cpp-package/include/mxnet-cpp/symbol.hpp   | 20 +++-
 docs/_static/cn.svg| 17 ++
 docs/_static/jquery-1.11.1.js  | 19 +++
 docs/_static/js/auto_module_index.js   | 21 +++-
 docs/_static/js/clipboard.min.js   | 21 +++-
 docs/_static/js/copycode.js| 19 +++
 docs/_static/js/navbar.js  | 25 +--
 docs/_static/js/options.js | 19 +++
 docs/_static/js/search.js  | 23 --
 docs/_static/js/sidebar.js | 37 --
 docs/_static/mxnet-theme/footer.html   | 17 ++
 docs/_static/mxnet-theme/index.html| 19 ++-
 docs/_static/mxnet-theme/layout.html   | 17 ++
 docs/_static/mxnet-theme/navbar.html   | 17 ++
 docs/_static/mxnet.css | 23 --
 docs/_static/searchtools_custom.js | 37 --
 docs/_static/selectlang.js | 25 +--
 docs/_static/us.svg| 17 ++
 .../tree_lstm/lib/CollapseUnaryTransformer.java| 19 +++
 example/gluon/tree_lstm/lib/ConstituencyParse.java | 21 +++-
 example/gluon/tree_lstm/lib/DependencyParse.java   | 19 +++
 example/rcnn/rcnn/cython/gpu_nms.hpp   | 19 +++
 example/rcnn/rcnn/pycocotools/maskApi.c| 19 +++
 example/ssd/tools/caffe_converter/make_win32.bat   | 17 ++
 perl-package/AI-MXNet/examples/calculator.pl   | 18 +++
 perl-package/AI-MXNet/examples/char_lstm.pl| 18 +++
 .../AI-MXNet/examples/cudnn_lstm_bucketing.pl  | 20 +++-
 perl-package/AI-MXNet/examples/lstm_bucketing.pl   | 18 +++
 perl-package/AI-MXNet/examples/mnist.pl| 22 +++--
 perl-package/AI-MXNet/examples/plot_network.pl | 18 +++
 scala-package/assembly/linux-x86_64-cpu/pom.xml| 17 ++
 .../src/main/assembly/assembly.xml | 17 ++
 scala-package/assembly/linux-x86_64-gpu/pom.xml| 17 ++
 .../src/main/assembly/assembly.xml | 17 ++
 .../osx-x86_64-cpu/main/assembly/assembly.xml  | 17 ++
 scala-package/assembly/osx-x86_64-cpu/pom.xml  | 17 ++
 .../osx-x86_64-cpu/src/main/assembly/assembly.xml  | 17 ++
 scala-package/assembly/pom.xml | 17 ++
 scala-package/core/pom.xml | 17 ++
 scala-package/examples/pom.xml | 17 ++
 scala-package/init-native/linux-x86_64/pom.xml | 17 ++
 scala-package/init-native/osx-x86_64/pom.xml   | 17 ++
 scala-package/init-native/pom.xml  | 17 ++
 scala-package/init/pom.xml | 17 ++
 scala-package/macros/pom.xml   | 17 

[incubator-mxnet] branch ApacheRat2 created (now 8395a84)

2017-08-11 Thread lxn2
This is an automated email from the ASF dual-hosted git repository.

lxn2 pushed a change to branch ApacheRat2
in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git.


  at 8395a84  Add more licenses

This branch includes the following new commits:

 new 8395a84  Add more licenses

The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.


-- 
To stop receiving notification emails like this one, please contact
['"comm...@mxnet.apache.org" '].


[GitHub] punisher-n commented on issue #6023: pip install error: No matching distribution found for mxnet-cu80

2017-08-11 Thread git
punisher-n commented on issue #6023: pip install error: No matching 
distribution found for mxnet-cu80
URL: 
https://github.com/apache/incubator-mxnet/issues/6023#issuecomment-321838490
 
 
   I am having the same issues with my machine
   --
   
 Could not find a version that satisfies the requirement setup.py (from 
versions: )
   No matching distribution found for setup.py
   
   please help me on this..
 

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 ApacheRat2 deleted (was 186266c)

2017-08-11 Thread lxn2
This is an automated email from the ASF dual-hosted git repository.

lxn2 pushed a change to branch ApacheRat2
in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git.


 was 186266c  Add more license files

This change permanently discards the following revisions:

 discard 186266c  Add more license files

-- 
To stop receiving notification emails like this one, please contact
['"comm...@mxnet.apache.org" '].


[GitHub] lxn2 closed pull request #7418: Enable Apache Rat + add more license headers

2017-08-11 Thread git
lxn2 closed pull request #7418: Enable Apache Rat + add more license headers
URL: https://github.com/apache/incubator-mxnet/pull/7418
 
 
   
 

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] kimtang commented on issue #7428: Linking error while compiling for R-package in windows

2017-08-11 Thread git
kimtang commented on issue #7428: Linking error while compiling for R-package 
in windows
URL: 
https://github.com/apache/incubator-mxnet/issues/7428#issuecomment-321817681
 
 
   I was able to compile the latest mxnet in windows with im2rec. I think this 
is the last missing steps to get im2rec to R ( windows). Any help are really 
appreciated.
   
   Thanks,
   
   Kim
   
 

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] kimtang opened a new issue #7428: Linking error while compiling for R-package in windows

2017-08-11 Thread git
kimtang opened a new issue #7428: Linking error while compiling for R-package 
in windows
URL: https://github.com/apache/incubator-mxnet/issues/7428
 
 
   For bugs or installation issues, please provide the following information.
   The more information you provide, the more likely people will be able to 
help you.
   
   ## Environment info
   Operating System: Windows 10
   
   Compiler: mingw_64
   
   Package used (Python/R/Scala/Julia): R
   
   MXNet version: See below
   
   Or if installed from source:
   
   MXNet commit hash (`git rev-parse HEAD`): 
89e3ee3ea7c223db8c65ddd8c94c6e787d7c52df
   
   If you are using python package, please provide
   
   Python version and distribution:
   
   If you are using R package, please provide
   
   R `sessionInfo()`:
   R version 3.4.0 (2017-04-21)
   Platform: x86_64-w64-mingw32/x64 (64-bit)
   Running under: Windows 10 x64 (build 14393)
   
   Matrix products: default
   
   locale:
   [1] LC_COLLATE=English_United Kingdom.1252
   [2] LC_CTYPE=English_United Kingdom.1252
   [3] LC_MONETARY=English_United Kingdom.1252
   [4] LC_NUMERIC=C
   [5] LC_TIME=English_United Kingdom.1252
   
   attached base packages:
   [1] stats graphics  grDevices utils datasets  methods   base
   
   loaded via a namespace (and not attached):
   [1] compiler_3.4.0
   ## Error Message:
   Please paste the full error message, including stack trace.
   > R CMD INSTALL --no-multiarch R-package
   * installing to library 'C:/Users/kuent/OneDrive/Dokumente/R/win-library/3.4'
   * installing *source* package 'mxnet' ...
   ** libs
   c:/Rtools/mingw_64/bin/g++ -shared -s -static-libgcc -o mxnet.dll tmp.def 
executor.o export.o im2rec.o io.o kvstore.o mxnet.o ndarray.o symbol.o 
-LC:/PROGRA~1/R/R-34~1.0/bin/x64 -lRlapack -LC:/PROGRA~1/R/R-34~1.0/bin/x64 
-lRblas -L../inst/libs/x64/ -llibmxnet -Ld:/Compiler/gcc-4.9.3/local330/lib/x64 
-Ld:/Compiler/gcc-4.9.3/local330/lib -LC:/PROGRA~1/R/R-34~1.0/bin/x64 -lR
   im2rec.o:im2rec.cc:(.text+0x209b): undefined reference to 
`dmlc::InputSplit::Create(char const*, unsigned int, unsigned int, char const*)'
   im2rec.o:im2rec.cc:(.text+0x24ce): undefined reference to 
`dmlc::Stream::Create(char const*, char const*, bool)'
   im2rec.o:im2rec.cc:(.text+0x2cd8): undefined reference to 
`dmlc::Stream::Create(char const*, char const*, bool)'
   im2rec.o:im2rec.cc:(.text+0x2eb1): undefined reference to 
`cv::imdecode(cv::_InputArray const&, int)'
   im2rec.o:im2rec.cc:(.text+0x3048): undefined reference to 
`cv::Mat::Mat(cv::Mat const&, cv::Range const&, cv::Range const&)'
   im2rec.o:im2rec.cc:(.text+0x3140): undefined reference to 
`cv::fastFree(void*)'
   im2rec.o:im2rec.cc:(.text+0x3208): undefined reference to 
`cv::fastFree(void*)'
   im2rec.o:im2rec.cc:(.text+0x3310): undefined reference to 
`cv::resize(cv::_InputArray const&, cv::_OutputArray const&, cv::Size_, 
double, double, int)'
   im2rec.o:im2rec.cc:(.text+0x339e): undefined reference to 
`cv::imencode(cv::String const&, cv::_InputArray const&, std::vector >&, std::vector 
const&)'
   im2rec.o:im2rec.cc:(.text+0x33a8): undefined reference to 
`cv::String::deallocate()'
   im2rec.o:im2rec.cc:(.text+0x34d8): undefined reference to 
`cv::fastFree(void*)'
   im2rec.o:im2rec.cc:(.text+0x356d): undefined reference to 
`cv::fastFree(void*)'
   im2rec.o:im2rec.cc:(.text+0x35ab): undefined reference to 
`dmlc::RecordIOWriter::WriteRecord(void const*, unsigned long long)'
   im2rec.o:im2rec.cc:(.text+0x3c3c): undefined reference to 
`cv::String::allocate(unsigned long long)'
   im2rec.o:im2rec.cc:(.text+0x3c65): undefined reference to 
`cv::Mat::copySize(cv::Mat const&)'
   im2rec.o:im2rec.cc:(.text+0x3ce0): undefined reference to 
`cv::Mat::Mat(cv::Mat const&, cv::Range const&, cv::Range const&)'
   im2rec.o:im2rec.cc:(.text+0x3dd1): undefined reference to 
`cv::fastFree(void*)'
   im2rec.o:im2rec.cc:(.text+0x3e9d): undefined reference to 
`cv::fastFree(void*)'
   im2rec.o:im2rec.cc:(.text+0x3eaa): undefined reference to 
`cv::Mat::deallocate()'
   im2rec.o:im2rec.cc:(.text+0x3eb7): undefined reference to 
`cv::Mat::deallocate()'
   im2rec.o:im2rec.cc:(.text+0x3fa5): undefined reference to 
`cv::Mat::copyTo(cv::_OutputArray const&) const'
   im2rec.o:im2rec.cc:(.text+0x40a0): undefined reference to 
`cv::fastFree(void*)'
   im2rec.o:im2rec.cc:(.text+0x416c): undefined reference to 
`cv::fastFree(void*)'
   im2rec.o:im2rec.cc:(.text+0x4253): undefined reference to 
`cv::Mat::copyTo(cv::_OutputArray const&) const'
   im2rec.o:im2rec.cc:(.text+0x4350): undefined reference to 
`cv::fastFree(void*)'
   im2rec.o:im2rec.cc:(.text+0x441c): undefined reference to 
`cv::fastFree(void*)'
   im2rec.o:im2rec.cc:(.text+0x44de): undefined reference to 
`cv::Mat::deallocate()'
   im2rec.o:im2rec.cc:(.text+0x44eb): undefined reference to 
`cv::Mat::deallocate()'
   im2rec.o:im2rec.cc:(.text+0x48c9): undefined reference to 
`cv::Mat::deallocate()'
   

[GitHub] eldercrow commented on issue #7281: Gradient function not returning enough gradient

2017-08-11 Thread git
eldercrow commented on issue #7281: Gradient function not returning enough 
gradient
URL: 
https://github.com/apache/incubator-mxnet/issues/7281#issuecomment-321803587
 
 
   Yes, I am using a custom layer, and I cannot simply put the code since it's 
too messy. I will try to make a failure case.
 

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] pras1406 closed issue #7397: OSError: /home_dir/mxnet/mxnet/python/mxnet/../../lib/libmxnet.so: undefined symbol: cblas_ddot

2017-08-11 Thread git
pras1406 closed issue #7397: OSError: 
/home_dir/mxnet/mxnet/python/mxnet/../../lib/libmxnet.so: undefined symbol: 
cblas_ddot
URL: https://github.com/apache/incubator-mxnet/issues/7397
 
 
   
 

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] pras1406 commented on issue #7397: OSError: /home_dir/mxnet/mxnet/python/mxnet/../../lib/libmxnet.so: undefined symbol: cblas_ddot

2017-08-11 Thread git
pras1406 commented on issue #7397: OSError: 
/home_dir/mxnet/mxnet/python/mxnet/../../lib/libmxnet.so: undefined symbol: 
cblas_ddot
URL: 
https://github.com/apache/incubator-mxnet/issues/7397#issuecomment-321800168
 
 
   @szha yes specifying 
   make -j8 ADD_CFLAGS="-I/home_dir/OpenBLAS_build/include" 
ADD_LDFLAGS="-L/home_dir/OpenBLAS_build/lib -lopenblas" and then exporting 
LD_LIBRARY_PATH=/OpenBLAS_build/lib:$LD_LIBRARY_PATH
   solved the issue.
   
   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] crazy-cat commented on issue #7393: add depthwise convolution's gpu version optimization

2017-08-11 Thread git
crazy-cat commented on issue #7393: add depthwise convolution's gpu version 
optimization
URL: https://github.com/apache/incubator-mxnet/pull/7393#issuecomment-321776682
 
 
   @piiswrong All checks have passed.
 

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] AihahaFox opened a new issue #7427: how to set dataiter with multi data?

2017-08-11 Thread git
AihahaFox opened a new issue #7427: how to set dataiter with multi data?
URL: https://github.com/apache/incubator-mxnet/issues/7427
 
 
   hey guys, I tried to rewrite Dataiter to provide multi data, for func 
provide_data(), I set it be [('data1', shape1), ('data2', shape2)]. and bind 
this iter with module. But there is problem in _parse_data_desc function. The 
data_name must be ['data'], cannot change to any other name. so how can I solve 
this problem
   
 

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] miraclewkf commented on issue #3475: Is there any easy way to monitor internal output of each layers?

2017-08-11 Thread git
miraclewkf commented on issue #3475: Is there any easy way to monitor internal 
output of each layers?
URL: 
https://github.com/apache/incubator-mxnet/issues/3475#issuecomment-321752847
 
 
   Is it the right way to create monitor: monitor = Monitor(...), it doesn't 
work for me.
 

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] ysh329 commented on a change in pull request #7363: Add tensorboard configure into ./common/fit.py and ./train_mnist.py

2017-08-11 Thread git
ysh329 commented on a change in pull request #7363: Add tensorboard configure 
into ./common/fit.py and ./train_mnist.py
URL: https://github.com/apache/incubator-mxnet/pull/7363#discussion_r132632691
 
 

 ##
 File path: example/image-classification/common/fit.py
 ##
 @@ -168,10 +168,16 @@ def fit(args, network, data_loader, **kwargs):
 
 # callbacks that run after each batch
 batch_end_callbacks = [mx.callback.Speedometer(args.batch_size, 
args.disp_batches)]
+eval_end_callbacks = []
 if 'batch_end_callback' in kwargs:
 cbs = kwargs['batch_end_callback']
 batch_end_callbacks += cbs if isinstance(cbs, list) else [cbs]
 
+# tensorboard logs
+if "eval_end_callback" in kwargs:
+   cbs = kwargs['eval_end_callback']
+eval_end_callbacks += cbs if isinstance(cbs, list) else [cbs]
 
 Review comment:
   modified as direct way
 

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] ysh329 commented on a change in pull request #7363: Add tensorboard configure into ./common/fit.py and ./train_mnist.py

2017-08-11 Thread git
ysh329 commented on a change in pull request #7363: Add tensorboard configure 
into ./common/fit.py and ./train_mnist.py
URL: https://github.com/apache/incubator-mxnet/pull/7363#discussion_r132612116
 
 

 ##
 File path: example/image-classification/common/fit.py
 ##
 @@ -168,10 +168,16 @@ def fit(args, network, data_loader, **kwargs):
 
 # callbacks that run after each batch
 batch_end_callbacks = [mx.callback.Speedometer(args.batch_size, 
args.disp_batches)]
+eval_end_callbacks = []
 if 'batch_end_callback' in kwargs:
 cbs = kwargs['batch_end_callback']
 batch_end_callbacks += cbs if isinstance(cbs, list) else [cbs]
 
 Review comment:
   @zihaolucky 
   I assigned `eval_end_callbacks` with same value as `batch_end_ballbacks`, 
but log about validation set still print for each epoch, not 
`args.disp_batches`.
   ```Python
   batch_end_callbacks = [mx.callback.Speedometer(args.batch_size, 
args.disp_batches)]
   eval_end_callbacks = [mx.callback.Speedometer(args.batch_size, 
args.disp_batches)]
   ```
 

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 #7425: Tensorcore fullyconnected support

2017-08-11 Thread git
piiswrong commented on issue #7425: Tensorcore fullyconnected support
URL: https://github.com/apache/incubator-mxnet/pull/7425#issuecomment-321739426
 
 
   This is overly complicated. Can we integrate it into 
https://github.com/apache/incubator-mxnet/pull/7147/files#diff-a1979d5d2d7e1986eab94cc82bcccd78R101
 and use use this in the original fc operator?
 

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