[jira] Commented: (MAHOUT-563) CanopyEstimator - Estimate T1/T2 for CanopyClusterer

["Sean Owen (JIRA)" <ji...@apache.org>]

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

Sean Owen commented on MAHOUT-563:
----------------------------------

I think this could be fine for utils/. There are some style issues with the patch (System.out, no copyright, some unused variables, spacing, braces) but easily fixed locally. I might sprinkle some javadoc on it too.

I have a few questions:

The variable canopies is never filled out. Should it be removed? Or is this supposed to be filled out in estimate()?
There are some constant-looking local variables in estimate(). Can I make them constants?



> CanopyEstimator - Estimate T1/T2 for CanopyClusterer
> ----------------------------------------------------
>
>                 Key: MAHOUT-563
>                 URL: https://issues.apache.org/jira/browse/MAHOUT-563
>             Project: Mahout
>          Issue Type: New Feature
>          Components: Clustering
>            Reporter: Lance Norskog
>            Priority: Minor
>         Attachments: MAHOUT-563.patch
>
>
> Hunting for T1/T2 values that make an interesting Canopy set is a singularly unsatisfying task. This class estimates T1 and T2 numbers given the original set.

-- 
This message is automatically generated by JIRA.
-
For more information on JIRA, see: http://www.atlassian.com/software/jira