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

Sean Owen commented on MAHOUT-392:
----------------------------------

Tiny comments:
- You could inline those b0, b1, etc variables for a bit more performance. Or 
at least make them constants
- Extend .math.MahoutTestCase and also don't forget the copyright block in the 
unit test

> Test cases for logGamma, Distribution.normal and Distribution.beta, fix for 
> Distribution.normal
> -----------------------------------------------------------------------------------------------
>
>                 Key: MAHOUT-392
>                 URL: https://issues.apache.org/jira/browse/MAHOUT-392
>             Project: Mahout
>          Issue Type: Bug
>            Reporter: Ted Dunning
>             Fix For: 0.4
>
>         Attachments: MAHOUT-392.patch
>
>
> I need access to the normal distribution to write test cases for random 
> matrices.  While testing that, I found it wasn't quite right so I provided an 
> alternative implementation.

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