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

Gilles commented on MATH-917:
-----------------------------

* "Clusterer" should be an "abstract" class (if we are to be consistent with 
the changes that have been done for a few years).
* It seems that accessing each component of a "Clusterable" separately is not 
efficient, the more so that usually _all_ components are needed. I don't see 
the advantage over simply requiring to return a "double[]"; any practical use 
case for the proposed API?
* I would not include "IntegerPoint" and "DoublePoint" in the "main" 
repository. They should go in the "test" part (if unit tests need them).

                
> More distance measurements are needed in o.a.c.m.stat.clustering.
> -----------------------------------------------------------------
>
>                 Key: MATH-917
>                 URL: https://issues.apache.org/jira/browse/MATH-917
>             Project: Commons Math
>          Issue Type: Improvement
>            Reporter: Reid Hochstedler
>             Fix For: 4.0
>
>         Attachments: clustering_v2.zip, clustering.zip
>
>
> Currently only Euclidean distance is used for distance measurement, it would 
> be easy to quickly add Manhattan and Chebyshev distance among others.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to