#11739: add_vertex w/o params should return the new vertex
----------------------------+-----------------------------------------------
   Reporter:  ddestrada     |          Owner:  jason, ncohen, rlm
       Type:  enhancement   |         Status:  needs_review      
   Priority:  minor         |      Milestone:  sage-4.8          
  Component:  graph theory  |       Keywords:  add_vertex sd35.5 
Work_issues:                |       Upstream:  N/A               
   Reviewer:                |         Author:                    
     Merged:                |   Dependencies:                    
----------------------------+-----------------------------------------------
Changes (by zimmerma):

  * keywords:  add_vertex => add_vertex sd35.5


Comment:

 I like this patch. I will run the doctests. However, shouldn't we do the
 same for {{{add_vertices}}} for consistency?
 {{{
 sage: G=Graph()
 sage: G.add_vertex()
 0
 sage: G.add_vertices([None])
 }}}

 Note the following strange thing:
 {{{
 sage: G=Graph()
 sage: G.add_vertices([None,0])
 sage: G.vertices()
 [0]
 }}}
 thus we added only one vertex.

 Paul

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/11739#comment:6>
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