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

Thomas Neidhart commented on MATH-917:
--------------------------------------

Applied patch with changes + tests in r1461862:

 * changed Clusterer to an abstract class
 * changed Clusterable interface to require only one method: double[] getPoint()
 * kept DoublePoint, and it also wraps an int[]: imho this class makes sense as 
it does not require someone to implement Clusterable for simple use-cases
 * created overridden constructors for providing a DistanceMeasure
 * added tests
 * added getters for the customisable arguments of a clusterer

I guess we could continue discussing the API, but for now it is pretty fine 
imho, we can evaluate if it is well suited and adjust if needed in 4.0.

Still need to deprecate the old package.
                
> 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