#5537: [with patch, mostly positive review] bug in __cmp__ in
permgroup_element.pyx
--------------------------+-------------------------------------------------
Reporter: jhpalmieri | Owner: robertwb
Type: defect | Status: new
Priority: major | Milestone: sage-3.4.2
Component: group_theory | Keywords:
--------------------------+-------------------------------------------------
Comment(by mvngu):
Replying to [comment:5 robertwb]:
{{{
> {{{
> sage: a = SymmetricGroup(20).random_element(); b =
SymmetricGroup(10).random_element()
> sage: time v = [a == b for _ in xrange(2000)]
> CPU times: user 0.00 s, sys: 0.00 s, total: 0.00 s
> Wall time: 0.00 s
> sage: timeit("a==b")
> 625 loops, best of 3: 240 ns per loop
> }}}
>
> vs. the old code
>
> {{{
> sage: a = SymmetricGroup(20).random_element(); b =
SymmetricGroup(10).random_element()
> sage: timeit("a==b")
> 625 loops, best of 3: 3.19 µs per loop
> }}}
}}}
Hi Robert. Is it possible for you to give some system info for those
particular timing statistics? I think it's good to include both the
statistics and some accompanying system/architecture info in a release
tour.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/5537#comment:12>
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
-~----------~----~----~----~------~----~------~--~---
