#5623: [with patch, positive review] 5x speedup in all_graph_colorings
-----------------------------+----------------------------------------------
Reporter: carlohamalainen | Owner: boothby
Type: enhancement | Status: closed
Priority: minor | Milestone: sage-3.4.1
Component: graph theory | Resolution: fixed
Keywords: |
-----------------------------+----------------------------------------------
Description changed by mabshoff:
Old description:
> This patch changes all_graph_colorings to use the C++ dancing links
> implementation instead of the Cython implementation.
>
> * sage -testall is all ok
> * valgrind is ok
> * Roughly 5x speedup on some random graphs (see test.sage):
>
> Sage 3.4 timing:
> ca...@ka37:~/work/code/graphdlx$ sage test.sage
> 5 loops, best of 3: 158 ms per loop
> ca...@ka37:~/work/code/graphdlx$ sage test.sage
> 5 loops, best of 3: 158 ms per loop
> ca...@ka37:~/work/code/graphdlx$ sage test.sage
> 5 loops, best of 3: 157 ms per loop
>
> Sage 3.4 with patch timing:
>
> 5 loops, best of 3: 33.5 ms per loop
> ca...@ka37:~/work/code/graphdlx$ sage test.sage
> 5 loops, best of 3: 33.1 ms per loop
> ca...@ka37:~/work/code/graphdlx$ sage test.sage
> 5 loops, best of 3: 33.3 ms per loop
New description:
This patch changes all_graph_colorings to use the C++ dancing links
implementation instead of the Cython implementation.
* sage -testall is all ok
* valgrind is ok
* Roughly 5x speedup on some random graphs (see test.sage):
Sage 3.4 timing:
{{{
ca...@ka37:~/work/code/graphdlx$ sage test.sage
5 loops, best of 3: 158 ms per loop
ca...@ka37:~/work/code/graphdlx$ sage test.sage
5 loops, best of 3: 158 ms per loop
ca...@ka37:~/work/code/graphdlx$ sage test.sage
5 loops, best of 3: 157 ms per loop
}}}
Sage 3.4 with patch timing:
{{{
5 loops, best of 3: 33.5 ms per loop
ca...@ka37:~/work/code/graphdlx$ sage test.sage
5 loops, best of 3: 33.1 ms per loop
ca...@ka37:~/work/code/graphdlx$ sage test.sage
5 loops, best of 3: 33.3 ms per loop
}}}
--
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/5623#comment:10>
Sage <http://sagemath.org/>
Sage - Open Source Mathematical Software: Building the Car Instead of
Reinventing the Wheel
--~--~---------~--~----~------------~-------~--~----~
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
-~----------~----~----~----~------~----~------~--~---
