[
https://issues.apache.org/jira/browse/MATH-1524?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17056970#comment-17056970
]
Gilles Sadowski commented on MATH-1524:
---------------------------------------
bq. a double[] may be better
Or not... See e.g. issue MATH-1330.
I think that there is something wrong in being able to create a
{{CentroidCluster}} with an "arbitrary" center (since the {{Cluster}}
constructor does not ensure that the center is consistent with the current
contents). Moreover, the {{addPoint}} method makes it certain that consistency
can be broken (even if the center was consistent at construction).
bq. Then the "getCenterPoint" and "centroid" won't be misused.
I don't understand why the "misuse" would come from the type of the return
value being the same.
> "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)
