#13215: Skew polynomials
-------------------------------------+-------------------------------------
Reporter: caruso | Owner: tbd
Type: enhancement | Status: needs_work
Priority: major | Milestone: sage-7.3
Component: algebra | Resolution:
Keywords: skew polynomials | Merged in:
Authors: Xavier Caruso, | Reviewers: Burcin Erocal, David
Arpit Merchant, Johan Rosenkilde | Lucas, Travis Scrimshaw
Report Upstream: N/A | Work issues:
Branch: | Commit:
u/arpitdm/skew_polynomials | 271bface01f93c8dca5a1fb104da139224ae1e66
Dependencies: #13214, #13303, | Stopgaps:
#13640, #13641, #13642 |
-------------------------------------+-------------------------------------
Comment (by arpitdm):
Replying to [comment:98 tscrim]:
>
> > > - Actually, a number of functions that use `_list_c` should be moved
to the concrete class (and maybe an `@abstract_method` placeholder done in
the ABC.
> > Could you please elaborate a little what you mean by this? I'm not
sure I follow.
>
> ABC = Abstract Base Class, so the methods you have that just raise a
`NotImplementedError` (e.g., `_inplace_add`), you should have them be
blank (modulo a docstring and a doctest of a concrete implementation) with
a `@abstract_method`. These then get picked up by a failure of the
`TestSuite`.
I tried adding the @abstract_method to the `cdef void _inplace_add` method
and I got an error message saying "Cdef functions/classes cannot take
arbitrary decorators." According to
[http://docs.cython.org/en/latest/src/tutorial/pure.html their docs], I
don't think decorators are implemented yet.
--
Ticket URL: <https://trac.sagemath.org/ticket/13215#comment:100>
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 https://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.
