#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.

Reply via email to