[GitHub] haojin2 commented on issue #10931: [MXNET-349] Histogram Operator

2018-06-01 Thread GitBox
haojin2 commented on issue #10931: [MXNET-349] Histogram Operator URL: https://github.com/apache/incubator-mxnet/pull/10931#issuecomment-394058185 @reminisce @piiswrong @anirudh2290 @rahul003 Unit tests added, this is ready, please give a review if you have time. Thanks! --

[GitHub] haojin2 commented on issue #10931: [MXNET-349] Histogram Operator

2018-06-06 Thread GitBox
haojin2 commented on issue #10931: [MXNET-349] Histogram Operator URL: https://github.com/apache/incubator-mxnet/pull/10931#issuecomment-395253134 @piiswrong Should be ready for merge, please take a look when you have time, thanks! --

[GitHub] haojin2 commented on issue #10931: [MXNET-349] Histogram Operator

2018-06-11 Thread GitBox
haojin2 commented on issue #10931: [MXNET-349] Histogram Operator URL: https://github.com/apache/incubator-mxnet/pull/10931#issuecomment-396427807 @piiswrong There's no backward pass for this function, I wonder if symbolic is necessary in this case?

[GitHub] haojin2 commented on issue #10931: [MXNET-349] Histogram Operator

2018-06-22 Thread GitBox
haojin2 commented on issue #10931: [MXNET-349] Histogram Operator URL: https://github.com/apache/incubator-mxnet/pull/10931#issuecomment-399526098 @piiswrong symbol is added for histogram. This is an automated message from the