Hum, actually canopies should be passed to runJob in place of the null.
That main method is not used in any of the unit tests but it would fail
if run. 

Good eyes,
Jeff

-----Original Message-----
From: Grant Ingersoll (JIRA) [mailto:[EMAIL PROTECTED] 
Sent: Tuesday, February 19, 2008 6:33 PM
To: [email protected]
Subject: [jira] Commented: (MAHOUT-3) Build initial canopy clustering
prototype


    [
https://issues.apache.org/jira/browse/MAHOUT-3?page=com.atlassian.jira.p
lugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12570546#a
ction_12570546 ] 

Grant Ingersoll commented on MAHOUT-3:
--------------------------------------

Jeff,

I noticed in ClusterDriver in the main() that the canopies argument
(args[1]) is ignored.  That seems a bit strange.  Is this just a relic
from an older way of doing it?

public static void main(String[] args) {
    String points = args[0];
//HERE
    String canopies = args[1];
    String output = args[2];
    String measureClassName = args[3];
    float t1 = new Float(args[4]);
    float t2 = new Float(args[5]);
    String jarLocation = "apache-mahout-0.1-dev.jar";
    if (args.length > 6){
      jarLocation = args[6];
    }
    runJob(points, null, output, measureClassName, t1, t2, jarLocation);
  }

I can just decrement them, if that is cool.

> Build initial canopy clustering prototype
> -----------------------------------------
>
>                 Key: MAHOUT-3
>                 URL: https://issues.apache.org/jira/browse/MAHOUT-3
>             Project: Mahout
>          Issue Type: New Feature
>          Components: Clustering
>            Reporter: Jeff Eastman
>            Assignee: Grant Ingersoll
>            Priority: Minor
>             Fix For: 0.1
>
>         Attachments: MAHOUT-3.diff, MAHOUT-3a.diff, MAHOUT-3b.diff,
MAHOUT-3c.diff, MAHOUT-3d.diff, MAHOUT-3e.diff, MAHOUT-3f.diff,
MAHOUT-3g.patch
>
>
> I'd like to reserve some namespace, specifically
org.apache.mahout.clustering.canopy to use for an initial prototype of
canopy clustering. I'm going to start with a little unit test to get the
basic algorithm sorted out, then M/R it.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to