Updates:
Summary: as_coeff_mul()[0] != as_coeff_Mul()[0] when the coeff is Number
(not Rational)
Labels: -Priority-High -Milestone-Release0.7.0 -NeedsReview -smichr
Priority-Medium Milestone-Release0.7.1
Comment #24 on issue 2307 by asmeurer: as_coeff_mul()[0] !=
as_coeff_Mul()[0] when the coeff is Number (not Rational)
http://code.google.com/p/sympy/issues/detail?id=2307
Ah, you're right. I didn't realize that you would have to use make_args to
properly mimic as_coeff_mul(). So let's keep both. It's cleaner than
mimicking one in the other (as I my original patch did), or reimplementing
one directly in the code.
As for the Rational/Number issue, I think we should see what would need to
be done to have as_coeff_mul() pull out a Number like as_coeff_Mul() does.
There were some test failures when I changed it to do this, but I didn't
look too closely at why there were there.
--
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.
