[with patch, positive review] bug somewhere in the symbolics
This is from Hector:
I also hit this bug while doing this (taken from the "piecewise"
documentation):
sage: f1 = lambda x:1
sage: f2 = lambda x:2
sage: f = Piecewise([[(0,pi/2),f1],[(pi/2,pi),f2]])
sage: P = f.plot_fourier_series_partial_sum(15,pi,5,5) # long time
boom
...
/Users/was/s/local/lib/python2.5/sitepackages/sage/calculus/calculus.py in <lambda>(i)
3607 # We need to do this maximum to correctly handle the case where
3608 # self is something like (sin+1)
> 3609 n = max( max(map(lambda i: i.number_of_arguments(), self._operands)+[0]), len(variables) )
3610 self.__number_of_args = n
3611 return n
<type 'exceptions.AttributeError'>: 'Pi' object has no attribute 'number_of_arguments'
Change History (7)
Owner: 
changed from William Stein to Mike Hansen

Status: 
new →
assigned

Summary: 
bug somewhere in the symbolics →
[with patch, needs review] bug somewhere in the symbolics

Summary: 
[with patch, needs review] bug somewhere in the symbolics →
[with patch, positive review] bug somewhere in the symbolics

Resolution: 
→ fixed

Status: 
assigned →
closed

I also fixed this in the same way earlier today (as part of one my other patches), but my patch was just a few lines to actually fix the listed problem. The patch attached to this ticket, fixes the problem and does a HUGE amount more to vastly improve doctesting in some files, etc. I.e., this is _great_.
I have not fully reviewed the patch yet, though I've looked it over by eye and it looks very good.