#8603: Partial sums are off for Fourier series of piecewise functions
------------------------+---------------------------------------------------
   Reporter:  novoselt  |          Owner:  burcin      
       Type:  defect    |         Status:  needs_review
   Priority:  minor     |      Milestone:  sage-4.7.1  
  Component:  calculus  |       Keywords:  sd31        
Work_issues:            |       Upstream:  N/A         
   Reviewer:            |         Author:  David Joyner
     Merged:            |   Dependencies:              
------------------------+---------------------------------------------------

Comment(by burcin):

 Replying to [comment:5 kcrisman]:
 > Thanks, David; I don't have time to review this now, but appreciate it.
 >
 > Andrey and I were discussing this at Sage Days 31, and thought that
 maybe changing the behavior instead to match Taylor series would be good,
 but if this was in fact what you had intended all along, then this
 solution is better.

 The `.series()` method of symbolic expressions cut off in a pythonic way,
 like David's change here. If `.taylor()` does something different we
 should change it.

 This is a trivial change. I'd be happy to give a positive review if it
 passes all tests but the patch bot doesn't seem to be working for some
 reason.

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

Reply via email to