#5223: [with patch; needs review] silly bug in flint wrapper makes it a factor
of
10 slower for division of a polynomial by an integer
------------------------------+---------------------------------------------
Reporter: was | Owner: somebody
Type: defect | Status: new
Priority: major | Milestone: sage-3.3
Component: basic arithmetic | Keywords:
------------------------------+---------------------------------------------
BEFORE:
{{{
sage: R.<x> = ZZ['x']
sage: f = 389*R.random_element(1000)
sage: timeit('f//389')
625 loops, best of 3: 228 µs per loop
}}}
AFTER:
{{{
sage: R.<x> = ZZ['x']
sage: f = 389*R.random_element(1000)
sage: timeit('f//389')
625 loops, best of 3: 48.3 µs per loop
}}}
The bug was doing the shortcut case, but then not returning and hence
doing the long case *as well*.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/5223>
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
-~----------~----~----~----~------~----~------~--~---
