[jira] [Commented] (MATH-917) More distance measurements are needed in o.a.c.m.stat.clustering.

classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view

[jira] [Commented] (MATH-917) More distance measurements are needed in o.a.c.m.stat.clustering.

David Costanzo (Jira)

    [ 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