[jira] Commented: (MAHOUT-138) Convert main() methods to use Commons CLI for argument processing

2009-09-14 Thread Isabel Drost (JIRA)

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

Isabel Drost commented on MAHOUT-138:
-

Hmm - the patch seems to be out of sync with trunk. From looking at it, it also 
seems it contains two changes - the CLI support and adding a 
RandomSeedGenerator?

> Convert main() methods to use Commons CLI for argument processing
> -
>
> Key: MAHOUT-138
> URL: https://issues.apache.org/jira/browse/MAHOUT-138
> Project: Mahout
>  Issue Type: Improvement
>Reporter: Grant Ingersoll
>Assignee: Grant Ingersoll
>Priority: Minor
> Fix For: 0.2
>
> Attachments: MAHOUT-138.patch
>
>
> Commons CLI is in the classpath and makes it much easier to handle command 
> line args and they are more self-documenting when done right.  We should 
> convert our main methods to use CLI

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



[jira] Commented: (MAHOUT-138) Convert main() methods to use Commons CLI for argument processing

2009-09-14 Thread Isabel Drost (JIRA)

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

Isabel Drost commented on MAHOUT-138:
-

>From a first glimpse at the code, it looks like there are quite a few other 
>classes that need switching as well (grepped through the code base, so no 
>guarantee that there are no false positives):

   * 
./core/src/main/java/org/apache/mahout/clustering/fuzzykmeans/FuzzyKMeansJob.java
   * 
./core/src/main/java/org/apache/mahout/clustering/dirichlet/DirichletJob.java
   * 
./core/src/main/java/org/apache/mahout/clustering/dirichlet/DirichletDriver.java
   * 
./core/src/main/java/org/apache/mahout/clustering/meanshift/MeanShiftCanopyJob.java
   * 
./core/src/main/java/org/apache/mahout/clustering/meanshift/MeanShiftCanopyDriver.java
   * 
./core/src/main/java/org/apache/mahout/classifier/bayes/mapreduce/bayes/BayesDriver.java
   * 
./core/src/main/java/org/apache/mahout/classifier/bayes/mapreduce/bayes/BayesThetaNormalizerDriver.java
   * 
./core/src/main/java/org/apache/mahout/classifier/bayes/mapreduce/cbayes/CBayesNormalizedWeightDriver.java
   * 
./core/src/main/java/org/apache/mahout/classifier/bayes/mapreduce/cbayes/CBayesDriver.java
   * 
./core/src/main/java/org/apache/mahout/classifier/bayes/mapreduce/cbayes/CBayesThetaDriver.java
   * 
./core/src/main/java/org/apache/mahout/classifier/bayes/mapreduce/cbayes/CBayesThetaNormalizerDriver.java
   * 
./core/src/main/java/org/apache/mahout/classifier/bayes/mapreduce/common/BayesWeightSummerDriver.java
   * 
./core/src/main/java/org/apache/mahout/classifier/bayes/mapreduce/common/BayesFeatureDriver.java
   * 
./core/src/main/java/org/apache/mahout/classifier/bayes/mapreduce/common/BayesTfIdfDriver.java
   * ./core/src/main/java/org/apache/mahout/cf/taste/hadoop/RecommenderJob.java
   * 
./core/src/main/java/org/apache/mahout/cf/taste/hadoop/SlopeOneDiffsToAveragesJob.java
   * 
./core/src/main/java/org/apache/mahout/cf/taste/hadoop/SlopeOnePrefsToDiffsJob.java
   * 
./examples/src/main/java/org/apache/mahout/clustering/syntheticcontrol/dirichlet/Job.java
   * 
./examples/src/main/java/org/apache/mahout/clustering/syntheticcontrol/canopy/Job.java
   * 
./examples/src/main/java/org/apache/mahout/clustering/syntheticcontrol/canopy/InputDriver.java
   * 
./examples/src/main/java/org/apache/mahout/clustering/syntheticcontrol/kmeans/Job.java
   * 
./examples/src/main/java/org/apache/mahout/clustering/syntheticcontrol/meanshift/Job.java
   * 
./examples/src/main/java/org/apache/mahout/clustering/syntheticcontrol/meanshift/InputDriver.java
   * 
./examples/src/main/java/org/apache/mahout/clustering/syntheticcontrol/meanshift/OutputDriver.java
   * 
./examples/src/main/java/org/apache/mahout/ga/watchmaker/cd/tool/CDInfosTool.java
   * 
./examples/src/main/java/org/apache/mahout/cf/taste/example/bookcrossing/BookCrossingRecommenderEvaluatorRunner.java
   * 
./examples/src/main/java/org/apache/mahout/cf/taste/example/netflix/NetflixRecommenderEvaluatorRunner.java
   * 
./examples/src/main/java/org/apache/mahout/cf/taste/example/netflix/TransposeToByUser.java
   * 
./examples/src/main/java/org/apache/mahout/cf/taste/example/jester/JesterRecommenderEvaluatorRunner.java
   * 
./examples/src/main/java/org/apache/mahout/cf/taste/example/grouplens/GroupLensRecommenderEvaluatorRunner.java

I'd like to offer my  help with some of these.

> Convert main() methods to use Commons CLI for argument processing
> -
>
> Key: MAHOUT-138
> URL: https://issues.apache.org/jira/browse/MAHOUT-138
> Project: Mahout
>  Issue Type: Improvement
>Reporter: Grant Ingersoll
>Assignee: Grant Ingersoll
>Priority: Minor
> Fix For: 0.2
>
> Attachments: MAHOUT-138.patch
>
>
> Commons CLI is in the classpath and makes it much easier to handle command 
> line args and they are more self-documenting when done right.  We should 
> convert our main methods to use CLI

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



[jira] Commented: (MAHOUT-138) Convert main() methods to use Commons CLI for argument processing

2009-09-14 Thread Grant Ingersoll (JIRA)

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

Grant Ingersoll commented on MAHOUT-138:


I think for these, you can just start committing.  Also, good to try to have 
some consistency in argument naming across files, where possible.

> Convert main() methods to use Commons CLI for argument processing
> -
>
> Key: MAHOUT-138
> URL: https://issues.apache.org/jira/browse/MAHOUT-138
> Project: Mahout
>  Issue Type: Improvement
>Reporter: Grant Ingersoll
>Assignee: Grant Ingersoll
>Priority: Minor
> Fix For: 0.2
>
> Attachments: MAHOUT-138.patch, MAHOUT-138_fuzzyKMeansJob.patch
>
>
> Commons CLI is in the classpath and makes it much easier to handle command 
> line args and they are more self-documenting when done right.  We should 
> convert our main methods to use CLI

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



[jira] Commented: (MAHOUT-138) Convert main() methods to use Commons CLI for argument processing

2009-09-14 Thread Isabel Drost (JIRA)

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

Isabel Drost commented on MAHOUT-138:
-

Will do so and put some documentation of the command line parameters on the 
wiki while I go along.

> Convert main() methods to use Commons CLI for argument processing
> -
>
> Key: MAHOUT-138
> URL: https://issues.apache.org/jira/browse/MAHOUT-138
> Project: Mahout
>  Issue Type: Improvement
>Reporter: Grant Ingersoll
>Assignee: Grant Ingersoll
>Priority: Minor
> Fix For: 0.2
>
> Attachments: MAHOUT-138.patch, MAHOUT-138_fuzzyKMeansJob.patch
>
>
> Commons CLI is in the classpath and makes it much easier to handle command 
> line args and they are more self-documenting when done right.  We should 
> convert our main methods to use CLI

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



[jira] Commented: (MAHOUT-138) Convert main() methods to use Commons CLI for argument processing

2009-09-18 Thread Isabel Drost (JIRA)

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

Isabel Drost commented on MAHOUT-138:
-

Added changes to cli for FuzzyKMeans, Dirichlet and MeanShiftCanopy (see below 
for exact classes). Added documentation to the wiki (see links to command line 
client documentation at 
http://cwiki.apache.org/confluence/display/MAHOUT/ClusteringYourData ) 

./core/src/main/java/org/apache/mahout/clustering/fuzzykmeans/FuzzyKMeansJob.java
 
./core/src/main/java/org/apache/mahout/clustering/dirichlet/DirichletJob.java 
./core/src/main/java/org/apache/mahout/clustering/dirichlet/DirichletDriver.java
 
./core/src/main/java/org/apache/mahout/clustering/meanshift/MeanShiftCanopyJob.java
 
./core/src/main/java/org/apache/mahout/clustering/meanshift/MeanShiftCanopyDriver.java
 

I have added one additional helper class (DefaultOptionCreator) that provides 
methods for creating the most common options (k clusters, input, output and the 
like) to avoid copying and ensure that the same option strings are used all 
over the code.

> Convert main() methods to use Commons CLI for argument processing
> -
>
> Key: MAHOUT-138
> URL: https://issues.apache.org/jira/browse/MAHOUT-138
> Project: Mahout
>  Issue Type: Improvement
>Reporter: Grant Ingersoll
>Assignee: Grant Ingersoll
>Priority: Minor
> Fix For: 0.2
>
> Attachments: MAHOUT-138.patch, MAHOUT-138_fuzzyKMeansJob.patch
>
>
> Commons CLI is in the classpath and makes it much easier to handle command 
> line args and they are more self-documenting when done right.  We should 
> convert our main methods to use CLI

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



[jira] Commented: (MAHOUT-138) Convert main() methods to use Commons CLI for argument processing

2009-10-06 Thread Grant Ingersoll (JIRA)

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

Grant Ingersoll commented on MAHOUT-138:


I don't understand why this was moved to 0.3.  I think we are most of the way 
there, and I think it is important that all of our drivers have a consistent 
argument processing

> Convert main() methods to use Commons CLI for argument processing
> -
>
> Key: MAHOUT-138
> URL: https://issues.apache.org/jira/browse/MAHOUT-138
> Project: Mahout
>  Issue Type: Improvement
>Affects Versions: 0.2
>Reporter: Grant Ingersoll
>Assignee: Grant Ingersoll
>Priority: Minor
> Fix For: 0.3
>
> Attachments: MAHOUT-138.patch, MAHOUT-138_fuzzyKMeansJob.patch
>
>
> Commons CLI is in the classpath and makes it much easier to handle command 
> line args and they are more self-documenting when done right.  We should 
> convert our main methods to use CLI

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



[jira] Commented: (MAHOUT-138) Convert main() methods to use Commons CLI for argument processing

2009-10-06 Thread Sean Owen (JIRA)

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

Sean Owen commented on MAHOUT-138:
--

It was based on Isabel's comment. Can this be committed then? I agree with 
moving it unless it can be committed now, as it hasn't seen movement in 3 
weeks. Obviously it should be committed, and now if it's done, but if nobody's 
moving on it... what do we do?

> Convert main() methods to use Commons CLI for argument processing
> -
>
> Key: MAHOUT-138
> URL: https://issues.apache.org/jira/browse/MAHOUT-138
> Project: Mahout
>  Issue Type: Improvement
>Affects Versions: 0.2
>Reporter: Grant Ingersoll
>Assignee: Grant Ingersoll
>Priority: Minor
> Fix For: 0.3
>
> Attachments: MAHOUT-138.patch, MAHOUT-138_fuzzyKMeansJob.patch
>
>
> Commons CLI is in the classpath and makes it much easier to handle command 
> line args and they are more self-documenting when done right.  We should 
> convert our main methods to use CLI

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



[jira] Commented: (MAHOUT-138) Convert main() methods to use Commons CLI for argument processing

2009-10-06 Thread Grant Ingersoll (JIRA)

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

Grant Ingersoll commented on MAHOUT-138:


I've been just committing as I go, so I don't think we need a lot of review for 
these things.

> Convert main() methods to use Commons CLI for argument processing
> -
>
> Key: MAHOUT-138
> URL: https://issues.apache.org/jira/browse/MAHOUT-138
> Project: Mahout
>  Issue Type: Improvement
>Affects Versions: 0.2
>Reporter: Grant Ingersoll
>Assignee: Grant Ingersoll
>Priority: Minor
> Fix For: 0.3
>
> Attachments: MAHOUT-138.patch, MAHOUT-138_fuzzyKMeansJob.patch
>
>
> Commons CLI is in the classpath and makes it much easier to handle command 
> line args and they are more self-documenting when done right.  We should 
> convert our main methods to use CLI

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



[jira] Commented: (MAHOUT-138) Convert main() methods to use Commons CLI for argument processing

2009-10-06 Thread Isabel Drost (JIRA)

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

Isabel Drost commented on MAHOUT-138:
-

Sean, you can easily follow what is going on with this issue on the subversion 
commit panel:

https://issues.apache.org/jira/browse/MAHOUT-138?page=com.atlassian.jira.plugin.ext.subversion%3Asubversion-commits-tabpanel

> Convert main() methods to use Commons CLI for argument processing
> -
>
> Key: MAHOUT-138
> URL: https://issues.apache.org/jira/browse/MAHOUT-138
> Project: Mahout
>  Issue Type: Improvement
>Affects Versions: 0.2
>Reporter: Grant Ingersoll
>Assignee: Grant Ingersoll
>Priority: Minor
> Fix For: 0.3
>
> Attachments: MAHOUT-138.patch, MAHOUT-138_fuzzyKMeansJob.patch
>
>
> Commons CLI is in the classpath and makes it much easier to handle command 
> line args and they are more self-documenting when done right.  We should 
> convert our main methods to use CLI

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



[jira] Commented: (MAHOUT-138) Convert main() methods to use Commons CLI for argument processing

2009-10-07 Thread Sean Owen (JIRA)

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

Sean Owen commented on MAHOUT-138:
--

I see, there was a commit, from Isabel. Is it done then? Isabel you had 
suggested moving this to 0.3, so I suppose you're saying it's not done, but 
wonder what the delta is then.

Grant I tend to agree with quick review and commits since patches very quickly 
go stale. But my question I suppose was, if you don't want to mark this for 
0.3, who is waiting to do what for how long on this, if it is to block 0.2?

This isn't my patch at all, I'm not involved.

> Convert main() methods to use Commons CLI for argument processing
> -
>
> Key: MAHOUT-138
> URL: https://issues.apache.org/jira/browse/MAHOUT-138
> Project: Mahout
>  Issue Type: Improvement
>Affects Versions: 0.2
>Reporter: Grant Ingersoll
>Assignee: Grant Ingersoll
>Priority: Minor
> Fix For: 0.3
>
> Attachments: MAHOUT-138.patch, MAHOUT-138_fuzzyKMeansJob.patch
>
>
> Commons CLI is in the classpath and makes it much easier to handle command 
> line args and they are more self-documenting when done right.  We should 
> convert our main methods to use CLI

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



[jira] Commented: (MAHOUT-138) Convert main() methods to use Commons CLI for argument processing

2009-10-07 Thread Grant Ingersoll (JIRA)

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

Grant Ingersoll commented on MAHOUT-138:


I think we just need to go through the various main() methods and see what is 
left.

> Convert main() methods to use Commons CLI for argument processing
> -
>
> Key: MAHOUT-138
> URL: https://issues.apache.org/jira/browse/MAHOUT-138
> Project: Mahout
>  Issue Type: Improvement
>Affects Versions: 0.2
>Reporter: Grant Ingersoll
>Assignee: Grant Ingersoll
>Priority: Minor
> Fix For: 0.3
>
> Attachments: MAHOUT-138.patch, MAHOUT-138_fuzzyKMeansJob.patch
>
>
> Commons CLI is in the classpath and makes it much easier to handle command 
> line args and they are more self-documenting when done right.  We should 
> convert our main methods to use CLI

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



[jira] Commented: (MAHOUT-138) Convert main() methods to use Commons CLI for argument processing

2009-10-07 Thread Isabel Drost (JIRA)

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

Isabel Drost commented on MAHOUT-138:
-


>From the classes above, I worked through up to the classification stuff. 
>Documentation is in the wiki at: 
>http://cwiki.apache.org/confluence/display/MAHOUT/ClassifyingYourData (the 
>links with commandline in their name) and 
>http://cwiki.apache.org/confluence/display/MAHOUT/ClusteringYourData (again 
>the links with commandline in their name).

Currently their are only examples left to convert as well as three classes 
containing main methods from the taste code:

./core/src/main/java/org/apache/mahout/cf/taste/hadoop/RecommenderJob.java
./core/src/main/java/org/apache/mahout/cf/taste/hadoop/SlopeOneDiffsToAveragesJob.java
./core/src/main/java/org/apache/mahout/cf/taste/hadoop/SlopeOnePrefsToDiffsJob.java
./examples/src/main/java/org/apache/mahout/cf/taste/example/bookcrossing/BookCrossingRecommenderEvaluatorRunner.java
./examples/src/main/java/org/apache/mahout/cf/taste/example/netflix/NetflixRecommenderEvaluatorRunner.java
./examples/src/main/java/org/apache/mahout/cf/taste/example/netflix/TransposeToByUser.java
./examples/src/main/java/org/apache/mahout/cf/taste/example/jester/JesterRecommenderEvaluatorRunner.java
./examples/src/main/java/org/apache/mahout/cf/taste/example/grouplens/GroupLensRecommenderEvaluatorRunner.java
./examples/src/main/java/org/apache/mahout/clustering/syntheticcontrol/dirichlet/Job.java
./examples/src/main/java/org/apache/mahout/clustering/syntheticcontrol/canopy/Job.java
./examples/src/main/java/org/apache/mahout/clustering/syntheticcontrol/canopy/InputDriver.java
./examples/src/main/java/org/apache/mahout/clustering/syntheticcontrol/kmeans/Job.java
./examples/src/main/java/org/apache/mahout/clustering/syntheticcontrol/meanshift/Job.java
./examples/src/main/java/org/apache/mahout/clustering/syntheticcontrol/meanshift/InputDriver.java
./examples/src/main/java/org/apache/mahout/clustering/syntheticcontrol/meanshift/OutputDriver.java
./examples/src/main/java/org/apache/mahout/ga/watchmaker/cd/tool/CDInfosTool.java



> Convert main() methods to use Commons CLI for argument processing
> -
>
> Key: MAHOUT-138
> URL: https://issues.apache.org/jira/browse/MAHOUT-138
> Project: Mahout
>  Issue Type: Improvement
>Affects Versions: 0.2
>Reporter: Grant Ingersoll
>Assignee: Grant Ingersoll
>Priority: Minor
> Fix For: 0.3
>
> Attachments: MAHOUT-138.patch, MAHOUT-138_fuzzyKMeansJob.patch
>
>
> Commons CLI is in the classpath and makes it much easier to handle command 
> line args and they are more self-documenting when done right.  We should 
> convert our main methods to use CLI

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



[jira] Commented: (MAHOUT-138) Convert main() methods to use Commons CLI for argument processing

2009-10-08 Thread Sean Owen (JIRA)

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

Sean Owen commented on MAHOUT-138:
--

You are most welcome to convert 
core/src/main/java/org/apache/mahout/cf/taste/hadoop as you see fit, since 
you're familiar with this change, or would you prefer I took a shot at it?

> Convert main() methods to use Commons CLI for argument processing
> -
>
> Key: MAHOUT-138
> URL: https://issues.apache.org/jira/browse/MAHOUT-138
> Project: Mahout
>  Issue Type: Improvement
>Affects Versions: 0.2
>Reporter: Grant Ingersoll
>Assignee: Grant Ingersoll
>Priority: Minor
> Fix For: 0.3
>
> Attachments: MAHOUT-138.patch, MAHOUT-138_fuzzyKMeansJob.patch
>
>
> Commons CLI is in the classpath and makes it much easier to handle command 
> line args and they are more self-documenting when done right.  We should 
> convert our main methods to use CLI

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



[jira] Commented: (MAHOUT-138) Convert main() methods to use Commons CLI for argument processing

2009-10-08 Thread Isabel Drost (JIRA)

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

Isabel Drost commented on MAHOUT-138:
-

Sean: sure, trying to get to it as soon as I find time to do so (hopefully 
tomorrow).

> Convert main() methods to use Commons CLI for argument processing
> -
>
> Key: MAHOUT-138
> URL: https://issues.apache.org/jira/browse/MAHOUT-138
> Project: Mahout
>  Issue Type: Improvement
>Affects Versions: 0.2
>Reporter: Grant Ingersoll
>Assignee: Grant Ingersoll
>Priority: Minor
> Fix For: 0.3
>
> Attachments: MAHOUT-138.patch, MAHOUT-138_fuzzyKMeansJob.patch
>
>
> Commons CLI is in the classpath and makes it much easier to handle command 
> line args and they are more self-documenting when done right.  We should 
> convert our main methods to use CLI

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



[jira] Commented: (MAHOUT-138) Convert main() methods to use Commons CLI for argument processing

2009-10-09 Thread Isabel Drost (JIRA)

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

Isabel Drost commented on MAHOUT-138:
-

Sean - I just converted the implementation of the taste jobs in core - could 
you please have a look at the commandline option descriptions to check that 
everything is correct?

http://cwiki.apache.org/confluence/display/MAHOUT/TasteCommandLine

> Convert main() methods to use Commons CLI for argument processing
> -
>
> Key: MAHOUT-138
> URL: https://issues.apache.org/jira/browse/MAHOUT-138
> Project: Mahout
>  Issue Type: Improvement
>Affects Versions: 0.2
>Reporter: Grant Ingersoll
>Assignee: Grant Ingersoll
>Priority: Minor
> Fix For: 0.3
>
> Attachments: MAHOUT-138.patch, MAHOUT-138_fuzzyKMeansJob.patch
>
>
> Commons CLI is in the classpath and makes it much easier to handle command 
> line args and they are more self-documenting when done right.  We should 
> convert our main methods to use CLI

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



[jira] Commented: (MAHOUT-138) Convert main() methods to use Commons CLI for argument processing

2009-10-09 Thread Isabel Drost (JIRA)

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

Isabel Drost commented on MAHOUT-138:
-

Usage description for Taste examples is online in the wiki at: 

http://cwiki.apache.org/confluence/display/MAHOUT/RecommendationExamples 

Current status: 
./examples/src/main/java/org/apache/mahout/clustering/syntheticcontrol/dirichlet/Job.java
 
./examples/src/main/java/org/apache/mahout/clustering/syntheticcontrol/canopy/Job.java
 
./examples/src/main/java/org/apache/mahout/clustering/syntheticcontrol/canopy/InputDriver.java
 
./examples/src/main/java/org/apache/mahout/clustering/syntheticcontrol/kmeans/Job.java
 
./examples/src/main/java/org/apache/mahout/clustering/syntheticcontrol/meanshift/Job.java
 
./examples/src/main/java/org/apache/mahout/clustering/syntheticcontrol/meanshift/InputDriver.java
 
./examples/src/main/java/org/apache/mahout/clustering/syntheticcontrol/meanshift/OutputDriver.java
 
./examples/src/main/java/org/apache/mahout/ga/watchmaker/cd/tool/CDInfosTool.java
 

8 examples to go.

> Convert main() methods to use Commons CLI for argument processing
> -
>
> Key: MAHOUT-138
> URL: https://issues.apache.org/jira/browse/MAHOUT-138
> Project: Mahout
>  Issue Type: Improvement
>Affects Versions: 0.2
>Reporter: Grant Ingersoll
>Assignee: Grant Ingersoll
>Priority: Minor
> Fix For: 0.3
>
> Attachments: MAHOUT-138.patch, MAHOUT-138_fuzzyKMeansJob.patch
>
>
> Commons CLI is in the classpath and makes it much easier to handle command 
> line args and they are more self-documenting when done right.  We should 
> convert our main methods to use CLI

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



[jira] Commented: (MAHOUT-138) Convert main() methods to use Commons CLI for argument processing

2009-10-09 Thread Isabel Drost (JIRA)

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

Isabel Drost commented on MAHOUT-138:
-

Robin, you briefly mentioned that for the bayes classifier it does not make 
sense to start up the different phases manually. Could you please detail which 
classes should not have main-methods attached to them and which ones should 
instead be used to start a training job in this issue?

> Convert main() methods to use Commons CLI for argument processing
> -
>
> Key: MAHOUT-138
> URL: https://issues.apache.org/jira/browse/MAHOUT-138
> Project: Mahout
>  Issue Type: Improvement
>Affects Versions: 0.2
>Reporter: Grant Ingersoll
>Assignee: Grant Ingersoll
>Priority: Minor
> Fix For: 0.3
>
> Attachments: MAHOUT-138.patch, MAHOUT-138_fuzzyKMeansJob.patch
>
>
> Commons CLI is in the classpath and makes it much easier to handle command 
> line args and they are more self-documenting when done right.  We should 
> convert our main methods to use CLI

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



[jira] Commented: (MAHOUT-138) Convert main() methods to use Commons CLI for argument processing

2009-10-09 Thread Sean Owen (JIRA)

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

Sean Owen commented on MAHOUT-138:
--

Yes, looks good to me. Go ahead.

> Convert main() methods to use Commons CLI for argument processing
> -
>
> Key: MAHOUT-138
> URL: https://issues.apache.org/jira/browse/MAHOUT-138
> Project: Mahout
>  Issue Type: Improvement
>Affects Versions: 0.2
>Reporter: Grant Ingersoll
>Assignee: Grant Ingersoll
>Priority: Minor
> Fix For: 0.3
>
> Attachments: MAHOUT-138.patch, MAHOUT-138_fuzzyKMeansJob.patch
>
>
> Commons CLI is in the classpath and makes it much easier to handle command 
> line args and they are more self-documenting when done right.  We should 
> convert our main methods to use CLI

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



Re: [jira] Commented: (MAHOUT-138) Convert main() methods to use Commons CLI for argument processing

2009-10-08 Thread deneche abdelhakim
There is also a main() method in:

./examples/src/main/java/org/apache/mahout/ga/watchmaker/cd/CDGA.java

I should be able to post a patch saturday concerning CDInfosTool and CDGA.

On Thu, Oct 8, 2009 at 7:29 AM, Isabel Drost (JIRA)  wrote:
>
>    [ 
> https://issues.apache.org/jira/browse/MAHOUT-138?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12763378#action_12763378
>  ]
>
> Isabel Drost commented on MAHOUT-138:
> -
>
>
> From the classes above, I worked through up to the classification stuff. 
> Documentation is in the wiki at: 
> http://cwiki.apache.org/confluence/display/MAHOUT/ClassifyingYourData (the 
> links with commandline in their name) and 
> http://cwiki.apache.org/confluence/display/MAHOUT/ClusteringYourData (again 
> the links with commandline in their name).
>
> Currently their are only examples left to convert as well as three classes 
> containing main methods from the taste code:
>
> ./core/src/main/java/org/apache/mahout/cf/taste/hadoop/RecommenderJob.java
> ./core/src/main/java/org/apache/mahout/cf/taste/hadoop/SlopeOneDiffsToAveragesJob.java
> ./core/src/main/java/org/apache/mahout/cf/taste/hadoop/SlopeOnePrefsToDiffsJob.java
> ./examples/src/main/java/org/apache/mahout/cf/taste/example/bookcrossing/BookCrossingRecommenderEvaluatorRunner.java
> ./examples/src/main/java/org/apache/mahout/cf/taste/example/netflix/NetflixRecommenderEvaluatorRunner.java
> ./examples/src/main/java/org/apache/mahout/cf/taste/example/netflix/TransposeToByUser.java
> ./examples/src/main/java/org/apache/mahout/cf/taste/example/jester/JesterRecommenderEvaluatorRunner.java
> ./examples/src/main/java/org/apache/mahout/cf/taste/example/grouplens/GroupLensRecommenderEvaluatorRunner.java
> ./examples/src/main/java/org/apache/mahout/clustering/syntheticcontrol/dirichlet/Job.java
> ./examples/src/main/java/org/apache/mahout/clustering/syntheticcontrol/canopy/Job.java
> ./examples/src/main/java/org/apache/mahout/clustering/syntheticcontrol/canopy/InputDriver.java
> ./examples/src/main/java/org/apache/mahout/clustering/syntheticcontrol/kmeans/Job.java
> ./examples/src/main/java/org/apache/mahout/clustering/syntheticcontrol/meanshift/Job.java
> ./examples/src/main/java/org/apache/mahout/clustering/syntheticcontrol/meanshift/InputDriver.java
> ./examples/src/main/java/org/apache/mahout/clustering/syntheticcontrol/meanshift/OutputDriver.java
> ./examples/src/main/java/org/apache/mahout/ga/watchmaker/cd/tool/CDInfosTool.java
>
>
>
>> Convert main() methods to use Commons CLI for argument processing
>> -
>>
>>                 Key: MAHOUT-138
>>                 URL: https://issues.apache.org/jira/browse/MAHOUT-138
>>             Project: Mahout
>>          Issue Type: Improvement
>>    Affects Versions: 0.2
>>            Reporter: Grant Ingersoll
>>            Assignee: Grant Ingersoll
>>            Priority: Minor
>>             Fix For: 0.3
>>
>>         Attachments: MAHOUT-138.patch, MAHOUT-138_fuzzyKMeansJob.patch
>>
>>
>> Commons CLI is in the classpath and makes it much easier to handle command 
>> line args and they are more self-documenting when done right.  We should 
>> convert our main methods to use CLI
>
> --
> This message is automatically generated by JIRA.
> -
> You can reply to this email to add a comment to the issue online.
>
>


Re: [jira] Commented: (MAHOUT-138) Convert main() methods to use Commons CLI for argument processing

2009-10-08 Thread Grant Ingersoll


On Oct 8, 2009, at 4:13 AM, deneche abdelhakim wrote:


There is also a main() method in:

./examples/src/main/java/org/apache/mahout/ga/watchmaker/cd/CDGA.java

I should be able to post a patch saturday concerning CDInfosTool and  
CDGA.


Personally, I'd just commit.   The goal here is to convert to CLI.  It  
doesn't take a whole lot of review.  Only thing I would suggest is we  
be consistent about what we name arguments, so have a look at the  
other ones for precedent.


-Grant



On Thu, Oct 8, 2009 at 7:29 AM, Isabel Drost (JIRA)  
 wrote:


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


Isabel Drost commented on MAHOUT-138:
-


From the classes above, I worked through up to the classification  
stuff. Documentation is in the wiki at: http://cwiki.apache.org/confluence/display/MAHOUT/ClassifyingYourData 
 (the links with commandline in their name) and http://cwiki.apache.org/confluence/display/MAHOUT/ClusteringYourData 
 (again the links with commandline in their name).


Currently their are only examples left to convert as well as three  
classes containing main methods from the taste code:


./core/src/main/java/org/apache/mahout/cf/taste/hadoop/ 
RecommenderJob.java
./core/src/main/java/org/apache/mahout/cf/taste/hadoop/ 
SlopeOneDiffsToAveragesJob.java
./core/src/main/java/org/apache/mahout/cf/taste/hadoop/ 
SlopeOnePrefsToDiffsJob.java
./examples/src/main/java/org/apache/mahout/cf/taste/example/ 
bookcrossing/BookCrossingRecommenderEvaluatorRunner.java
./examples/src/main/java/org/apache/mahout/cf/taste/example/netflix/ 
NetflixRecommenderEvaluatorRunner.java
./examples/src/main/java/org/apache/mahout/cf/taste/example/netflix/ 
TransposeToByUser.java
./examples/src/main/java/org/apache/mahout/cf/taste/example/jester/ 
JesterRecommenderEvaluatorRunner.java
./examples/src/main/java/org/apache/mahout/cf/taste/example/ 
grouplens/GroupLensRecommenderEvaluatorRunner.java
./examples/src/main/java/org/apache/mahout/clustering/ 
syntheticcontrol/dirichlet/Job.java
./examples/src/main/java/org/apache/mahout/clustering/ 
syntheticcontrol/canopy/Job.java
./examples/src/main/java/org/apache/mahout/clustering/ 
syntheticcontrol/canopy/InputDriver.java
./examples/src/main/java/org/apache/mahout/clustering/ 
syntheticcontrol/kmeans/Job.java
./examples/src/main/java/org/apache/mahout/clustering/ 
syntheticcontrol/meanshift/Job.java
./examples/src/main/java/org/apache/mahout/clustering/ 
syntheticcontrol/meanshift/InputDriver.java
./examples/src/main/java/org/apache/mahout/clustering/ 
syntheticcontrol/meanshift/OutputDriver.java
./examples/src/main/java/org/apache/mahout/ga/watchmaker/cd/tool/ 
CDInfosTool.java





Convert main() methods to use Commons CLI for argument processing
-

Key: MAHOUT-138
URL: https://issues.apache.org/jira/browse/ 
MAHOUT-138

Project: Mahout
 Issue Type: Improvement
   Affects Versions: 0.2
   Reporter: Grant Ingersoll
   Assignee: Grant Ingersoll
   Priority: Minor
Fix For: 0.3

Attachments: MAHOUT-138.patch,  
MAHOUT-138_fuzzyKMeansJob.patch



Commons CLI is in the classpath and makes it much easier to handle  
command line args and they are more self-documenting when done  
right.  We should convert our main methods to use CLI


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




--
Grant Ingersoll
http://www.lucidimagination.com/

Search the Lucene ecosystem (Lucene/Solr/Nutch/Mahout/Tika/Droids)  
using Solr/Lucene:

http://www.lucidimagination.com/search



Re: [jira] Commented: (MAHOUT-138) Convert main() methods to use Commons CLI for argument processing

2009-10-08 Thread Isabel Drost
On Thu, 8 Oct 2009 07:52:21 -0300
Grant Ingersoll  wrote:

> Personally, I'd just commit.   The goal here is to convert to CLI.
> It doesn't take a whole lot of review.  Only thing I would suggest is
> we be consistent about what we name arguments, so have a look at the  
> other ones for precedent.

+1 You can also have a look at DefaultOptionCreator - I have created a
few methods to keep creation of the most common options (input,
output, help and the like) in one place.

Isabel