#7380: Replace Graph.vertices() by Graph.vertex_iterator when possible
--------------------------+-------------------------------------------------
 Reporter:  ncohen        |         Owner:  rlm                           
     Type:  defect        |        Status:  closed                        
 Priority:  major         |     Milestone:  sage-duplicate/invalid/wontfix
Component:  graph theory  |    Resolution:  wontfix                       
 Keywords:                |        Author:                                
 Upstream:  N/A           |      Reviewer:                                
   Merged:                |   Work_issues:                                
--------------------------+-------------------------------------------------

Comment(by ylchapuy):

 Sorry for commenting a closed ticket, but...

 If I look at the docstring for `vertices` I see (in the default case):
 {{{
 return sorted(list(self.vertex_iterator()))
 }}}

 A new list is created, doubling the memory needed, and this list is sorted
 which might take time too.

 I still think it might make sense to replace `vertices` with
 `vertex_iterator` when possible.

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