[jira] Commented: (MAHOUT-138) Convert main() methods to use Commons CLI for argument processing
[ 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
[ 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
[ 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
[ 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
[ 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
[ 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
[ 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
[ 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
[ 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
[ 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
[ 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
[ 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
[ 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
[ 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
[ 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
[ 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
[ 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
[ 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
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
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
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
