#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:  burcin    
     Type:  defect            |      Status:  assigned  
 Priority:  major             |   Milestone:  sage-3.4.1
Component:  basic arithmetic  |    Keywords:            
------------------------------+---------------------------------------------
Changes (by burcin):

  * owner:  somebody => burcin
  * status:  new => assigned
  * milestone:  sage-3.4.2 => sage-3.4.1


Comment:

 Wrong function was called for scalar division in the existing code, so we
 returned wrong results if division was not exact. Using
 `fmpz_poly_scalar_div_mpz()` fixes this problem and removes the limit on
 the size of the divisor. I didn't measure it's effects on speed.

 All tests under sage/rings/polynomial pass with
 attachment:trac_5223.take2.patch.

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

Reply via email to