You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@mahout.apache.org by "Ted Dunning (JIRA)" <ji...@apache.org> on 2013/04/22 21:59:16 UTC

[jira] [Commented] (MAHOUT-1196) LogisticModelParameters uses csv.getTargetCategories() even if csv is not used.

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

Ted Dunning commented on MAHOUT-1196:
-------------------------------------

Do you have a suggested patch?
                
> LogisticModelParameters uses csv.getTargetCategories() even if csv is not used.
> -------------------------------------------------------------------------------
>
>                 Key: MAHOUT-1196
>                 URL: https://issues.apache.org/jira/browse/MAHOUT-1196
>             Project: Mahout
>          Issue Type: Bug
>          Components: Classification
>    Affects Versions: 0.8
>         Environment: All
>            Reporter: Vineet Krishnan
>            Priority: Trivial
>              Labels: CSV, Classifier, LogisticModelParameters
>   Original Estimate: 1h
>  Remaining Estimate: 1h
>
> saveTo(OutputStream out) tries to get csv.getTargetCategories() even when it has already been set. In a case when CsvRecordFactory is not used, this gives a NullPointerException when saveTo() is called.
> IMHO a simple null check for targetCategories is sufficient.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira