[
https://issues.apache.org/jira/browse/MATH-584?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Randall Scarberry updated MATH-584:
-----------------------------------
Attachment: kmeans_plus_plus.patch
Noticed that one of the KMeanPlusPlusClusterer tests got caught in an infinite
loop with my patched version, because of extremely small distances.
Apply this patch instead -- it ensures an exit from the while loop.
> 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: 2.2.1
>
> 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
