Updates:
Labels: EasyToFix
Comment #7 on issue 2453 by [email protected]: abs(sympy.I * sympy.pi)
evaluates incorrectly
http://code.google.com/p/sympy/issues/detail?id=2453
I biseced this to
commit ebe8a7e5670c871d057f781a62d13394b5fdd96d
Author: Tom Bachmann <[email protected]>
Date: Thu Jun 30 16:45:25 2011 +0100
Fix {Mul,Add}.is_irrational.
The function used to return True if any factor was irrational.
This is nonsense: neither I*pi, nor sqrt(8)/sqrt(2) are irrational.
[One might argue about I*pi, but irrational implies real under the
current assumptions system.]
We now return True iff there is exactly one irrational factor and
all other factors are rational.
Similar fix for add.
TODO this should be fixed in assocOp.is_template... see issue 2563
which was long before that XFAIL test was added. So the decorator just
needs to be removed.
--
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.
