Ticket #3579 (closed defect: fixed)

Opened 3 months ago

Last modified 3 months ago

[with patch, positive review] bug in RandonGNP graph constructor

Reported by: was Assigned to: rlm
Priority: major Milestone: sage-3.0.6
Component: graph theory Keywords:
Cc:

Description

17:32 < itolkov> sage: graphs.RandomGNP(n=4, p=1)
17:32 < itolkov> Traceback ... OverflowError: math range error
17:32 < itolkov> bug?
17:34 < wstein-3576> nt necessarily.
17:35 < wstein-3576> the line lp=math.log(1.0-p) shows why it doesn't work.
17:35 < wstein-3576> The docs do not ban probability 1, so yes, it is a bug.

Attachments

trac3579-random_graph_generator_bug_fix.patch (1.1 kB) - added by ekirkman on 07/10/2008 10:05:40 AM.

Change History

07/10/2008 10:05:40 AM changed by ekirkman

  • attachment trac3579-random_graph_generator_bug_fix.patch added.

07/10/2008 10:06:56 AM changed by ekirkman

  • summary changed from bug in RandonGNP graph constructor to [with patch, needs review] bug in RandonGNP graph constructor.
  • milestone changed from sage-3.0.6 to sage-3.0.5.

07/10/2008 10:09:28 AM changed by rlm

  • summary changed from [with patch, needs review] bug in RandonGNP graph constructor to [with patch, positive review] bug in RandonGNP graph constructor.

+1

07/14/2008 06:49:19 PM changed by mabshoff

  • status changed from new to closed.
  • resolution set to fixed.

Merged in Sage 3.0.6.alpha0