#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
-~----------~----~----~----~------~----~------~--~---

Reply via email to