#1956: implement multivariate truncated power series arithmetic
-------------------------------------------+--------------------------------
Reporter: was | Owner: pernici
Type: enhancement | Status: needs_review
Priority: major | Milestone: sage-4.7
Component: commutative algebra | Keywords: multivariate
power series
Work_issues: | Upstream: N/A
Reviewer: Martin Albrecht, Simon King | Author: Niles Johnson
Merged: | Dependencies:
-------------------------------------------+--------------------------------
Comment(by malb):
Hi, Simon good catch about the {{{__base_ring}}} attribute.
@Niles: as you realised in your code (in {{{__init__}}} of the ring),
having double underscore attributes makes it hard to inherit because
double underscore names are prefixed by the class name. If you want
subclasses to use these attributes then you'll need to switch to a single
underscore. The single underscore expresses that the attribute is meant to
be private, i.e., not treated casually.
With double underscores there is no such thing as consistency, the point
of them is to break consistency to '''prevent''' people from using them,
if they are not in exactly the right class of the hierarchy. The design is
broken if you have to set {{{__poly_ring}}} yourself twice to make it work
for the various classes in the inheritance hierarchy.
I agree that other rings do it wrong, too. But there's no need to repeat
this wrong approach in a new class?
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/1956#comment:121>
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.
