Opened 6 years ago

Closed 6 years ago

#1249 closed defect (fixed)

[with patch, with positive review] fixes bug in graph plotting with partitions

Reported by: rlm Owned by: mhansen
Priority: major Milestone: sage-2.8.15
Component: combinatorics Keywords: graphs
Cc: Merged in:
Authors: Reviewers:
Report Upstream: Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Description


Attachments (1)

graph_plotting_partitions.patch (1.4 KB) - added by rlm 6 years ago.

Download all attachments as: .zip

Change History (4)

Changed 6 years ago by rlm

comment:1 Changed 6 years ago by cwitty

  • Summary changed from [with patch] fixes bug in graph plotting with partitions to [with patch, with positive review] fixes bug in graph plotting with partitions

Looks good to me: the example in the doctests fails before the patch and succeeds after the patch. There is one caveat: I think the call to D.show() in the first doctest chunk was supposed to be D.plot().

But IMHO good enough to apply, anyway.

comment:2 Changed 6 years ago by mabshoff

<mhansen-962> mabshoff: I think http://trac.sagemath.org/sage_trac/ticket/1249 is okay.

comment:3 Changed 6 years ago by mabshoff

  • Resolution set to fixed
  • Status changed from new to closed

Merged in 2.8.15.alpha1.

Note: See TracTickets for help on using tickets.