You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@mahout.apache.org by "Sean Owen (JIRA)" <ji...@apache.org> on 2010/05/08 09:30:47 UTC
[jira] Commented: (MAHOUT-392) Test cases for logGamma,
Distribution.normal and Distribution.beta, fix for Distribution.normal
[ 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.