[Patches] [ python-Patches-529408 ] fix random.gammavariate bug #527139

noreply@sourceforge.net noreply@sourceforge.net
Sun, 17 Mar 2002 12:46:45 -0800


Patches item #529408, was opened at 2002-03-13 23:15
You can respond by visiting: 
http://sourceforge.net/tracker/?func=detail&atid=305470&aid=529408&group_id=5470

Category: Library (Lib)
Group: Python 2.3
Status: Open
Resolution: None
Priority: 5
Submitted By: John Machin (sjmachin)
Assigned to: Nobody/Anonymous (nobody)
Summary: fix random.gammavariate bug #527139

Initial Comment:
random.gammavariate() doesn't work for gamma < 0.5

See detailed comment on bug # 527139

----------------------------------------------------------------------

>Comment By: John Machin (sjmachin)
Date: 2002-03-18 07:46

Message:
Logged In: YES 
user_id=480138

OK; I understand the problems with the patch. Not sure 
about the way forward -- shall I prepare a patch that just 
fixes gammavariate() and leaves stdgamma() there (with 
warning in the comments: deprecated? will be removed in 
2.x?)? Do you want it real soon now (for 2.2.1)?



----------------------------------------------------------------------

Comment By: Tim Peters (tim_one)
Date: 2002-03-18 06:42

Message:
Logged In: YES 
user_id=31435

Michael, this definitely doesn't belong in 2.2.1 as-is, 
because it removes a currently-exported name (buggy or not, 
sensible or not, somebody may be using random.stdgamma now 
and be happy with it).

John, if you're going to remove stdgamma, you need also to 
remove its (string) name from the module's __all__ list 
(right before the _verify() function).

----------------------------------------------------------------------

Comment By: Tim Peters (tim_one)
Date: 2002-03-17 04:38

Message:
Logged In: YES 
user_id=31435

Possibly, depending on whether it belongs in 2.3 -- I'm 
spread too thin to review it now.

----------------------------------------------------------------------

Comment By: Michael Hudson (mwh)
Date: 2002-03-17 03:53

Message:
Logged In: YES 
user_id=6656

Tim, do you think this should go into 2.2.1?

----------------------------------------------------------------------

You can respond by visiting: 
http://sourceforge.net/tracker/?func=detail&atid=305470&aid=529408&group_id=5470