[
https://issues.apache.org/jira/browse/FLINK-4129?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15579933#comment-15579933
]
Vasia Kalavri commented on FLINK-4129:
--------------------------------------
I think having two HITS examples could be confusing to users. Is this
implementation showcasing some feature that no other example is or could we
simply remove it in favor of the HITS driver?
> HITSAlgorithm should test for element-wise convergence
> ------------------------------------------------------
>
> Key: FLINK-4129
> URL: https://issues.apache.org/jira/browse/FLINK-4129
> Project: Flink
> Issue Type: Bug
> Components: Gelly
> Affects Versions: 1.1.0
> Reporter: Greg Hogan
> Priority: Minor
>
> {{HITSAlgorithm}} tests for convergence by summing the difference of each
> authority score minus the average score. This is simply comparing the sum of
> scores against the previous sum of scores which is not a good test for
> convergence.
> {code}
> // count the diff value of sum of authority scores
> diffSumAggregator.aggregate(previousAuthAverage -
> newAuthorityValue.getValue());
> {code}
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
