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

Sean Owen commented on MAHOUT-838:
----------------------------------

Lance I tried just this, but I still get compile errors. Your posted 
ConfusionMatrix.java and MatrixWritable.java are identical to HEAD, but are 
missing methods the new code requires.

Where is cmdump.props used? is it needed?

It feels like the matrix and confusion matrix dumper have similar roles. I'd 
imagine they both go in examples, or both in integration? Integration feels a 
bit righter.

Finally I'd have a run at scrubbing style -- there are some unused imports, 
unthrown exceptions, constants that aren't static or final, etc.

Once you sort out your env, make up another omnibus patch here with all 
relevant changes.
                
> Make the confusion matrix writable to a file when testing classifiers
> ---------------------------------------------------------------------
>
>                 Key: MAHOUT-838
>                 URL: https://issues.apache.org/jira/browse/MAHOUT-838
>             Project: Mahout
>          Issue Type: Improvement
>            Reporter: Grant Ingersoll
>            Priority: Minor
>         Attachments: ConfusionMatrix.java, MAHOUT-838.patch, 
> MAHOUT-838_mini.patch, MatrixWritable.java, bayes-cm-10.html
>
>
> If you have a lot of labels for a classifier, the confusion matrix is hard to 
> fit in terminal window.  Would be nice if we could write it out to a file.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to