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

Sean Owen commented on MAHOUT-743:
----------------------------------

The fixed seed is not 0. You correctly describe the behavior; is there a 
concrete problem that it solves? In general you don't want to write code with 
fixed seeds. Nevertheless I think it's fine to change the behavior, yes.

> Allow use of random seeds during unit tests
> -------------------------------------------
>
>                 Key: MAHOUT-743
>                 URL: https://issues.apache.org/jira/browse/MAHOUT-743
>             Project: Mahout
>          Issue Type: Improvement
>            Reporter: Lance Norskog
>         Attachments: MAHOUT-743.patch, RandomWrapper.patch
>
>
> The RandomUtils package has a trick for running unit tests repeatably. All 
> random generators created with RandomUtils.getRandom() are started with seed 
> 0. Unfortunately, all random generators started with 
> RandomUtils.getRandom(seed) effectively run from a seed of 0. Thus, code 
> which deliberately creates Random objects with seeds does not work correctly 
> under unit tests.
> The problem is that the logic in RandomWrapper tests for unit test mode and 
> settable seed mode in the wrong order.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to