Mailing-List: contact issues-help@commons.apache.org; run by ezmlm
Precedence: bulk
Reply-To: issues@commons.apache.org
Date: Fri, 4 Nov 2011 17:09:51 +0000 (UTC)
From: "Phil Steitz (Commented) (JIRA)" <jira@apache.org>
To: issues@commons.apache.org
Message-ID: <33024321.365.1320426591592.JavaMail.tomcat@hel.zones.apache.org>
In-Reply-To: 
 <503227206.55673.1320326856574.JavaMail.tomcat@hel.zones.apache.org>
Subject: [jira] [Commented] (MATH-701) Seeding a default RNG
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit


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

Phil Steitz commented on MATH-701:
----------------------------------

I thought about changing the default seeding in AbstractWell and agree that would be a good idea.  I still like to supply the seed explicitly and document it in RandomDataImpl, though, so users of that class know exactly what they are getting by default.  There is a little wrinkle here, too that keeping the seeding expressed and documented in RandomDataImpl makes easier to keep track of.  If we ever implement hashcode in RandomDataImpl (or the Well generators), things could get messed up if it does not separate generator instances the way the system identity haschcode does.
                
> Seeding a default RNG
> ---------------------
>
>                 Key: MATH-701
>                 URL: https://issues.apache.org/jira/browse/MATH-701
>             Project: Commons Math
>          Issue Type: Bug
>            Reporter: Gilles
>            Assignee: Gilles
>             Fix For: 3.0
>
>
> In "RandomDataImpl":
> {code}
> private RandomGenerator getRan() {
>     if (rand == null) {
>         rand = new JDKRandomGenerator();
>         rand.setSeed(System.currentTimeMillis());
>     }
>     return rand;
> }
> {code}
> The conditional branch is used by "sample()" in "AbstractContinuousDistribution".
> When several "...Distribution" objects are instantiated in a short time interval, they are seeded with the same value.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira