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

Jake Mannix commented on MAHOUT-220:
------------------------------------

Ted,

  While I'm totally down with using the randomizer / hashing techniques in 
places, I don't think we should totally wed ourselves to it either - having the 
option of using the "real" vector representation should probably be implemented 
to, as people understand it better, and it's pretty standard.

bq. If you like these, we can promote them to a common area under classifier.

  They might belong in a more general place, actually.  If I'm going to use 
some of this stuff in the decompositions (although I'm not sure yet of the 
efficacy of the single hash for doing SVD), it should go somewhere in the math 
module.

> Mahout Bayes Code cleanup
> -------------------------
>
>                 Key: MAHOUT-220
>                 URL: https://issues.apache.org/jira/browse/MAHOUT-220
>             Project: Mahout
>          Issue Type: Improvement
>          Components: Classification
>    Affects Versions: 0.3
>            Reporter: Robin Anil
>            Assignee: Robin Anil
>             Fix For: 0.3
>
>         Attachments: MAHOUT-BAYES.patch, MAHOUT-BAYES.patch
>
>
> Following isabel's checkstyle, I am adding a whole slew of code cleanup with 
> the following exceptions
> 1.  Line length used is 120 instead of 80. 
> 2.  static final log is kept as is. not LOG. 

-- 
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