[ 
https://issues.apache.org/jira/browse/MATH-1524?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17057940#comment-17057940
 ] 

Gilles Sadowski commented on MATH-1524:
---------------------------------------

{quote}The Cluster.centroid() may only used in Clustering and Evaluator, it can 
be package private.
{quote}
Either I'm missing something, or this is backwards (from an API POV): the 
{{centroid()}} provides a property of the instance; it should thus be 
accessible by users.
{quote}getCenter can be package private, except centers should be displayed: 
[...]
{quote}
I don't get that; If the method is "private", users (like a GUI application) 
won't be able to call it...
{quote}[cluster_comparison.png|http://commons.apache.org/proper/commons-math/images/userguide/cluster_comparison.png]
{quote}
In that picture, couldn't the centers be computed using the {{centroid()}} 
method of class {{Cluster}}?

> "chooseInitialCenters" should move out from KMeansPlusPlusClusterer
> -------------------------------------------------------------------
>
>                 Key: MATH-1524
>                 URL: https://issues.apache.org/jira/browse/MATH-1524
>             Project: Commons Math
>          Issue Type: Improvement
>            Reporter: Chen Tao
>            Priority: Major
>          Time Spent: 20m
>  Remaining Estimate: 0h
>
> There are two reason for "chooseInitialCenters" should be move out from 
> "KMeansPlusPlusClusterer":
> # k-means++ clusterer is a special case of k-means clusterer, that k-means++ 
> initialize the cluster centers with k-means++ algorithm. Another case is 
> initialize the cluster centers with random points.
> # The mini batch k-means will reuse "chooseInitialCenters". 



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to