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

Sean Owen commented on MAHOUT-564:
----------------------------------

I'm cleaning house on some issues that have stalled -- this look ready-ish to 
commit.
But I get an error in the test when I apply it:

java.lang.AssertionError: Cluster 1 test 3 expected:<4> but was:<5>
        at org.junit.Assert.fail(Assert.java:91)
        at org.junit.Assert.failNotEquals(Assert.java:645)
        at org.junit.Assert.assertEquals(Assert.java:126)
        at org.junit.Assert.assertEquals(Assert.java:470)
        at 
org.apache.mahout.clustering.kmeans.TestKmeansClustering.testReferenceImplementation(TestKmeansClustering.java:150)
...

ideas?

> 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-without-star-imports.patch, 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.

Reply via email to