#17659: make symbolic series subclass of Expression
-------------------------+-------------------------------------------------
       Reporter:  rws    |        Owner:
           Type:         |       Status:  positive_review
  enhancement            |    Milestone:  sage-6.10
       Priority:  major  |   Resolution:
      Component:         |    Merged in:
  symbolics              |    Reviewers:  Vincent Delecroix
       Keywords:         |  Work issues:
        Authors:  Ralf   |       Commit:
  Stephan                |  d5cc4241d638adb1329adbc737fa29591fc204b6
Report Upstream:  N/A    |     Stopgaps:
         Branch:         |
  u/rws/17659-1          |
   Dependencies:         |
-------------------------+-------------------------------------------------
Changes (by vdelecroix):

 * status:  needs_review => positive_review


Old description:

> Making `Expression.series` create `SymbolicSeries` (a subclass of
> `Expression`) and moving the series code into a separate file
> `series.pyx`
> * reflects the fact that expressions and series don't commute
> * allows upcoming fixes that don't clutter `Expression` methods with `if
> ex.is_a_series():`
> * makes for better documentation via having a `Symbolic Series` ref man
> page
>
> In refactoring language this is "replace conditional with polymorphism".
>
> #16203, #17400, and 17402 depend on this.

New description:

 Making `Expression.series` create `SymbolicSeries` (a subclass of
 `Expression`) and moving the series code into a separate file `series.pyx`
 * reflects the fact that expressions and series don't commute
 * allows upcoming fixes that don't clutter `Expression` methods with `if
 ex.is_a_series():`
 * makes for better documentation via having a `Symbolic Series` ref man
 page

 In refactoring language this is "replace conditional with polymorphism".

 #16203, #17400, and #17402 depend on this.

--

Comment:

 Good for me.

--
Ticket URL: <http://trac.sagemath.org/ticket/17659#comment:36>
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