Comment #9 on issue 2305 by gdrummo...@gmail.com: Integer(3).sqrt() == 1 (Rename Integer.sqrt to Integer.isqrt)
http://code.google.com/p/sympy/issues/detail?id=2305

These methods have been added by Mateusz as part of polys. Removing them will break polys functionality.

Some commits I found:
707b715bd2d25444bcc68a2eef8be4c59349e596
661a8dfc47515c273c41301d203cba900e9e585a
afc470666a327b682788f58f2a1a035c8add637b

Some of this code is not fully covered by tests (Mateusz's comment on pull request: https://github.com/sympy/sympy/pull/263#issuecomment-1061047). So I'm hesitant to make further changes because I don't know the effect these changes will have on polys code.

However, as far as I can see, if I make the requested changes, then the following files will need to be updated:
polys/domains/sympyintegerring.py
polys/domains/groundtypes.py


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

Reply via email to