Updates:
        Status: Started

Comment #7 on issue 2315 by matt...@gmail.com: integrate(1/(x**2 + n**2, x) failure, where n is an integer
http://code.google.com/p/sympy/issues/detail?id=2315

Yes, to_field() will fix this problem (you can also use auto=True in .div()). However, I would change this to:

    if f.degree() < g.degree():
        f, g = -g, f

    f = f.to_field()
    g = g.to_field()

    p, q = f.div(g)

because if q != 0, then in .gcdex() conversion will have to happen once again (.gcdex() always switches to a field, because otherwise the computation doesn't make sense).

Also look into 271-272. This is where Polys are constructed without domain tracking. But anyway, changing log_to_atan() is fine, because e.g. if someone would like to use this implementation for teaching integration algorithms, then log_to_atan() should be a standalone function.

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

Reply via email to