[
https://issues.apache.org/jira/browse/FLINK-2030?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14728685#comment-14728685
]
ASF GitHub Bot commented on FLINK-2030:
---------------------------------------
Github user chiwanpark commented on the pull request:
https://github.com/apache/flink/pull/861#issuecomment-137376194
Hi, I just reviewed the updated PR. Sorry for late.
Your implementation is nice. It seems acting like I expected. But I have to
check the following:
I'm concerned about changing `Histogram` class. It causes API breaking. I'm
not sure that this breaking is necessary. Reverting the changes of `Histogram`
would be better. Because there are many differences between
`ContinuousHistogram` and `DiscreteHistogram`, we don't need to create base
class for them.
I'm sorry about consuming time to merge this PR. It seems almost arrived
the goal. Cheer up!
> Implement an online histogram with Merging and equalization features
> --------------------------------------------------------------------
>
> Key: FLINK-2030
> URL: https://issues.apache.org/jira/browse/FLINK-2030
> Project: Flink
> Issue Type: Sub-task
> Components: Machine Learning Library
> Reporter: Sachin Goel
> Assignee: Sachin Goel
> Priority: Minor
> Labels: ML
>
> For the implementation of the decision tree in
> https://issues.apache.org/jira/browse/FLINK-1727, we need to implement an
> histogram with online updates, merging and equalization features. A reference
> implementation is provided in [1]
> [1].http://www.jmlr.org/papers/volume11/ben-haim10a/ben-haim10a.pdf
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
