#11804: delta_qexp() fails in large characteristic
--------------------------------+-------------------------------------------
Reporter: jdemeyer | Owner: AlexGhitza
Type: defect | Status: needs_work
Priority: major | Milestone: sage-4.7.2
Component: basic arithmetic | Keywords: FLINT NTL polynomial
_mul_short _mul_trunc delta_qexp _delta_poly_modulo
Work_issues: | Upstream: N/A
Reviewer: | Author:
Merged: | Dependencies:
--------------------------------+-------------------------------------------
Changes (by robertwb):
* status: needs_review => needs_work
Comment:
I would like to see some documentation as well as examples in the new
_mul_trunc functions. Also, something like
sage: f = ...
sage: g = ...
sage: f*g
...
sage: f._mul_trunc(g, 2)
...
would be illustrative of what goes on as well as being a good test.
The function as written will crash horribly if one does
sage: f._mul_trunc(2, 2)
it should probably be declared as
cpdef Polynomial_dense_modn_ntl_zz _mul_trunc(self,
Polynomial_dense_modn_ntl_zz _right, long n)
Otherwise, the patch looks good. I am slightly concerned about performance
degradation, but fixing this bug is more important.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/11804#comment:2>
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.