Status: Valid
Owner: ----
Labels: Type-Defect Priority-Medium

New issue 3688 by smi...@gmail.com: rational=True should be used with as_coeff_Mul to exclude oo
http://code.google.com/p/sympy/issues/detail?id=3688

Just putting this here as a TODO that can be closed once it is known that this situation no longer exists in SymPy.

The change to oo not being rational requires that any call to as_coeff_Mul, that is followed by treating the extracted coefficient as a Rational, be changed to use the rational=True flag in order to watch out for infinites that may be in the expression


    c, m = e.as_coeff_Mul()  # use rational=True!
    ...
    if c.q == foo ...  <--- if this follows, it will fail with oo

Some sort of proximity search or boobytrapping(*) might be able to highlight areas where this change needs to be made.

(*) modify as_coeff_Mul to always return oo and look for the exceptions that might occur (as opposed to test failures which, of course, will be many).

--
You received this message because this project is configured to send all issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings

--
You received this message because you are subscribed to the Google Groups 
"sympy-issues" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to sympy-issues+unsubscr...@googlegroups.com.
To post to this group, send email to sympy-issues@googlegroups.com.
Visit this group at http://groups.google.com/group/sympy-issues?hl=en.
For more options, visit https://groups.google.com/groups/opt_out.


Reply via email to