Also, adding a dependency on commons math doesn't necessarily give us forward motion on the dependency front.
On Fri, May 20, 2011 at 7:23 PM, Sean Owen (JIRA) <[email protected]> wrote: > > [ > https://issues.apache.org/jira/browse/MAHOUT-687?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13037229#comment-13037229] > > Sean Owen commented on MAHOUT-687: > ---------------------------------- > > OK, that is still not what the patch does though. > I think there are two good ideas in the mix here that can be committed > without controversy. First, work around setSeed() behavior by instantiating > a new RNG when called. Second, don't use a shared RNG in the sampling > Iterator. I suggest this is the substance of what to commit in this thread. > > > Random generator objects- slight refactor > > ----------------------------------------- > > > > Key: MAHOUT-687 > > URL: https://issues.apache.org/jira/browse/MAHOUT-687 > > Project: Mahout > > Issue Type: Improvement > > Reporter: Lance Norskog > > Priority: Minor > > Attachments: MAHOUT-687.patch, MAHOUT-687.patch > > > > > > Problems: > > * The uncommons RepeatableRNG classes are the basis of RandomUtils. > > ** These classes cheerfully ignore setSeed. > > * Some people in the project want to move off Uncommons anyway. > > This patch uses the org.apache.commons.math.random.RandomGenerator > classes instead of org.apache.uncommons.maths.RepeatableRNG classes. > > . > > -- > This message is automatically generated by JIRA. > For more information on JIRA, see: http://www.atlassian.com/software/jira >
