Updates:
Labels: -NeedsReview PassedReview
Comment #3 on issue 1961 by asmeurer: integration works too hard
http://code.google.com/p/sympy/issues/detail?id=1961
I still need to look at the advantages/disadvantages of pulling out
coefficients (it will probably be a good idea, but I need to make sure,
because it will change the ground domain). For now, I think your patch is
fine.
I would rather see something like the fix in the attached patch, which
correctly integrates both diff(x**x, x) and diff(x**x, x).expand() without
calling integrate() twice. But it causes integrate(x*exp(x) + x*exp(2*x),
x) to fail, for the same reason that the integral at issue 1891 fails.
So if you want to push this in now, it's fine. Tests pass. But be aware
that this code will change a lot by the end of the summer, and that better
fixes exist (such as the one in d38ccac54a4 in my integration branch for
the test you add here).
Attachments:
integration-fix.patch 3.2 KB
--
You received this message because you are subscribed to the Google Groups
"sympy-issues" 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/sympy-issues?hl=en.
