[jira] Commented: (MAHOUT-564) KMeansClusterer does not use distanceThreshold parameter in testConvergence(Iterable clusters, double distanceThreshold) method

["Sebastian Schelter (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/MAHOUT-564?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12970630#action_12970630 ] 

Sebastian Schelter commented on MAHOUT-564:
-------------------------------------------

Hi Frank, 

your patch looks good, but it reorders imports and introduces star imports, could you fix that? I guess you're using IntelliJ which does this automatically (I had the same problem some time ago...).



> KMeansClusterer does not use distanceThreshold parameter in testConvergence(Iterable<Cluster> clusters, double distanceThreshold) method
> ----------------------------------------------------------------------------------------------------------------------------------------
>
>                 Key: MAHOUT-564
>                 URL: https://issues.apache.org/jira/browse/MAHOUT-564
>             Project: Mahout
>          Issue Type: Bug
>          Components: Clustering
>    Affects Versions: 0.4
>            Reporter: Frank Scholten
>             Fix For: 0.5
>
>         Attachments: MAHOUT-564.patch
>
>
> While running KMeansClusterer#runKMeansIteration sequentially I noticed that the distanceThreshold parameter is not used. The convergenceDelta field is used instead. However, it's initialized at 0 and only set when creating a KMeansClusterer with a Configuration object, which is only used in a MapReduce setting.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.