#7547: improve has_multiple_edges
----------------------------+-----------------------------------------------
   Reporter:  ylchapuy      |       Owner:  rlm               
       Type:  enhancement   |      Status:  needs_review      
   Priority:  major         |   Milestone:  sage-4.3          
  Component:  graph theory  |    Keywords:                    
Work_issues:                |      Author:  Yann Laigle-Chapuy
   Upstream:  N/A           |    Reviewer:                    
     Merged:                |  
----------------------------+-----------------------------------------------
Changes (by ylchapuy):

 * cc: ncohen (added)
  * status:  new => needs_review


Old description:

> It seems to be the main bottleneck in graph plotting!
> This patch cuts down by 30% the time for `sage -t graph.py` on my
> machine... (and doctests of course still pass)

New description:

 This patch cuts down by 30% the time for sage -t graph.py on my machine...
 (and doctests of course still pass)

--

Comment:

 For the record:
 {{{
 sage: P = graphs.PetersenGraph()
 sage: D = graphs.DodecahedralGraph()
 sage: L = D.lexicographic_product(P)
 sage: L.allow_multiple_edges(True)
 sage: time L.has_multiple_edges()
 }}}

 Before: Wall time: 39.56 s

 After: Wall time: 19.32 s

 I hope I did no mistake, it's 4 a.m here...

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