#16276: fixes PolynomialElement.mod()
-------------------------------------+-------------------------------------
Reporter: Bouillaguet | Owner:
Type: defect | Status: needs_review
Priority: major | Milestone: sage-6.3
Component: basic arithmetic | Resolution:
Keywords: | Merged in:
Authors: Charles | Reviewers:
Bouillaguet | Work issues:
Report Upstream: N/A | Commit:
Branch: | 7e902de0445ec47ce835f3cb202b361793c621f5
u/Bouillaguet/ticket/16276 | Stopgaps:
Dependencies: |
-------------------------------------+-------------------------------------
Comment (by pbruin):
Is `mod = __mod__` the correct way to say that these two functions should
behave identically? If `C` is a subclass of `Polynomial` implementing
`__mod__` but not `mod`, then `C.mod` will still be equal to
`Polynomial.__mod__`, not to `C.__mod__`, if I'm not mistaken.
This is currently the case for `Polynomial_dense_modn_ntl_zz`,
`Polynomial_dense_modn_ntl_ZZ` and `Polynomial_rational_flint`. To
automatically make `mod == __mod__` for these classes, I think
`Polynomial.mod` would have to be defined as
{{{
def mod(self, other):
return self.__mod__(other)
}}}
--
Ticket URL: <http://trac.sagemath.org/ticket/16276#comment:14>
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 unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.