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

Sean Owen commented on MAHOUT-696:
----------------------------------

OK -- there are still some formatting issues here but I can just fix them 
locally. You should look at how the other code is formatted; like I said, for 
instance, we use space not tab.

There is one thing that bothers me about the code here, which this patch 
copies-and-pastes (meaning, it's not a problem with this patch per se). It's 
this static field "output" which is modified by reflection to point from 
System.out to a custom object. Surely we can arrange this differently? Meh, let 
me see if I can think of a cleaner way. 

> Command line program for AdaptiveLogiscticRegression
> ----------------------------------------------------
>
>                 Key: MAHOUT-696
>                 URL: https://issues.apache.org/jira/browse/MAHOUT-696
>             Project: Mahout
>          Issue Type: Improvement
>          Components: Classification
>    Affects Versions: 0.5
>            Reporter: XiaoboGu
>            Assignee: Ted Dunning
>             Fix For: 0.6
>
>         Attachments: MAHOUT-696.patch, MAHOUT-696.patch, MAHOUT-696.patch, 
> MAHOUT-696.patch, MAHOUT-696.patch, MAHOUT-696.patch, MAHOUT-696.patch, 
> mahout-696-r1.patch, mahout-696-r2.patch, mahout-696-r3.patch, 
> mahout-696-r4.patch, mahout-696-r5.patch
>
>
> Suggested by Ted, I'll try to write a command line program for 
> AdaptiveLogicticRegression, but as I am not familir with the algorithm, I'll 
> try to write a prototype for the program from a Java developer's perspactive, 
> hope anyone else will help with the details of the algorithm.

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

        

Reply via email to