You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@commons.apache.org by "Luc Maisonobe (JIRA)" <ji...@apache.org> on 2011/06/05 18:29:47 UTC

[jira] [Resolved] (MATH-584) KMeansPlusPlusClusterer incorrectly selects initial cluster centers and is unnecessarily slow

     [ https://issues.apache.org/jira/browse/MATH-584?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Luc Maisonobe resolved MATH-584.
--------------------------------

       Resolution: Fixed
    Fix Version/s:     (was: 2.2.1)
                   3.0

Fixed in subversion repository as of r1132448.

Thanks for the report and for the patch.

> KMeansPlusPlusClusterer incorrectly selects initial cluster centers and is unnecessarily slow
> ---------------------------------------------------------------------------------------------
>
>                 Key: MATH-584
>                 URL: https://issues.apache.org/jira/browse/MATH-584
>             Project: Commons Math
>          Issue Type: Bug
>    Affects Versions: 2.2
>         Environment: All environments
>            Reporter: Randall Scarberry
>              Labels: cluster, clustering
>             Fix For: 3.0
>
>         Attachments: kmeans_plus_plus.patch, kmeans_plus_plus.patch
>
>
> The chooseInitialClusters() method declares sum as an int, when it should be double.  It also is quite slow because it contains a lot of unnecessary computation.  I'll attached a patch which corrects the problems.
> I found the problems while comparing an optimized implementation of KMeans++ I've been working on with the one in commons math. 

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira