Github user thvasilo commented on the pull request:
https://github.com/apache/flink/pull/861#issuecomment-115574532
Hello Sachin,
although we will for now postpone adding the histogram function to
DataSetUtil, we should still have some external functionality for this PR.
We discussed this a bit with Till and the best place to put the histogram
function now is in the package.scala of the ml package.
You will see that there already some pimp-my-class style functions there,
like RichLabeledDataSet that has the writeAsLibSVM function.
So you can do the same and have an implicit RichDoubleDataSet class that
defines the histogram function for DataSet[Double]. Once we have that we can
merge this.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
