#13781: Doctests in rings/polynomial/polynomial_modn_dense_ntl.pyx test the
wrong
implementation
-----------------------+----------------------------------------------------
Reporter: mjo | Owner: AlexGhitza
Type: defect | Status: new
Priority: major | Milestone: sage-5.6
Component: algebra | Keywords:
Work issues: | Report Upstream: N/A
Reviewers: | Authors:
Merged in: | Dependencies:
Stopgaps: |
-----------------------+----------------------------------------------------
A common doctest in `polynomial_modn_dense_ntl.pyx` goes something like
this:
{{{
sage: R.<x> = Integers(100)[]
sage: (x + 5) * 3
3*x + 15
}}}
At some point, I guess, this used to construct a
Polynomial_dense_modn_ntl_zz. But now it looks like the default
implementation uses flint:
{{{
sage: R.<x> = Integers(100)[]
sage: type((x + 5) * 3)
<type 'sage.rings.polynomial.polynomial_zmod_flint.Polynomial_zmod_flint'>
}}}
So a large number of the functions in this file are actually untested,
because they're testing the flint implementation instead of the ntl one.
It looks like this is the workaround:
{{{
sage: R.<x> = PolynomialRing(Integers(100), implementation='ntl')
sage: type((x + 5) * 3)
<type
'sage.rings.polynomial.polynomial_modn_dense_ntl.Polynomial_dense_modn_ntl_zz'>
}}}
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/13781>
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.
