#7159: Graph.merge_vertices, and a bug in edge_boundary
----------------------------+-----------------------------------------------
Reporter: ncohen | Owner: rlm
Type: defect | Status: needs_work
Priority: major | Milestone: sage-4.2.1
Component: graph theory | Keywords:
Work_issues: | Author:
Reviewer: | Merged:
----------------------------+-----------------------------------------------
Changes (by AJonsson):
* status: positive_review => needs_work
Comment:
I read the trac guidelines more closely and there is a last tiny issue
before this patch can be said to be perfect:
"Bug Fixes Must Be Doctested: The patch that fixes an issue must also
contain a doctest specifically to test the problem."
So all that is missing is a test that displays the expected behavior of
edge_boundary(), and that would fail without your patch. For example
something like this:
{{{
sage: G=graphs.DiamondGraph()
sage: G.edge_boundary([0,1])
[(0, 2, None), (1, 2, None), (1, 3, None)]
}}}
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/7159#comment:7>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica,
and MATLAB
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups
"sage-trac" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to
[email protected]
For more options, visit this group at
http://groups.google.com/group/sage-trac?hl=en
-~----------~----~----~----~------~----~------~--~---