Hi Daniel, On Thu, Nov 25, 2010 at 3:48 AM, Daniel Fetchinson <[email protected]> wrote: > Hi folks, > > This is my first message as I've just tried using sympy on the > recommendation of Robert Kern on python-list. Congratulation on this > great python module!
Thanks! > > One issue I came up against immediately is that I have a matrix with > fractions.Fraction entries and when trying to feed this matrix into > sympy it gets automatically converted to floats. After snooping around > in the code I found that the reason is that the function sympify > ignores fractions.Fraction instances and does the conversion to floats > when in fact sympy is able to represent rationals. So why not test for > fractions.Fraction instances and in case it matches convert it to > sympy-rationals? Indeed, this should be done. Would you mind sending us a patch? Just modify this file: https://github.com/sympy/sympy/blob/master/sympy/core/sympify.py to take fractions into account. Just ask for more details if you run into any problems. And write some tests for that too, in the tests subdirectory. Ondrej -- You received this message because you are subscribed to the Google Groups "sympy" 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?hl=en.
