Comment #32 on issue 1778 by ronan.l...@gmail.com: Rational from string can be improved
http://code.google.com/p/sympy/issues/detail?id=1778

* I don't like the name of rsimplify: what it does appears unrelated to
simplification and there's a possible confusion with ratsimp.

* I think Rational should mimic the interface of fractions.Fraction (from std lib) as
much as possible, so Rational("3/2") should work.

* The part dealing with sympification of dicts is unrelated to Rational and should be in a separate commit, and probably reviewed in a separate issue. (BTW, I would rather
see the sympification of lists, tuples and sets removed than this added)

* I don't like that you add so much specialised code in sympify(). Perhaps this should go into a helper function called from sympify(). (Actually, it's OK like this,
this is more of a TODO for me in issue 1896.)

--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings

--
You received this message because you are subscribed to the Google Groups 
"sympy-issues" group.
To post to this group, send email to sympy-iss...@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