Re: Issue 1694 in sympy: solve has many issues with fractions

2010-08-06 Thread sympy
Comment #105 on issue 1694 by asmeurer: solve has many issues with fractions http://code.google.com/p/sympy/issues/detail?id=1694 That's pretty easy to fix. You just need to import Integer from sympy.core.numbers in Basic.count_ops(). Funny how that has never showed up before. -- You

Re: Issue 1694 in sympy: solve has many issues with fractions

2010-08-06 Thread sympy
Updates: Cc: asmeurer Comment #106 on issue 1694 by smichr: solve has many issues with fractions http://code.google.com/p/sympy/issues/detail?id=1694 It's already fixed in commit 1923 and changes to make count_ops return a count is implemented. If someone could review the mods made

Re: Issue 1694 in sympy: solve has many issues with fractions

2010-08-06 Thread sympy
Updates: Labels: -NeedsReview Comment #107 on issue 1694 by asmeurer: solve has many issues with fractions http://code.google.com/p/sympy/issues/detail?id=1694 So the earlier commit has already passed review? Just looking at the 1923b commit, it looks fine, except you have and Add,

Re: Issue 2009 in sympy: Document why solve((x-y,y),x) is None

2010-08-06 Thread sympy
Comment #10 on issue 2009 by asmeurer: Document why solve((x-y,y),x) is None http://code.google.com/p/sympy/issues/detail?id=2009 See also issue 1694, which had some discussion about having solve() return sets instead of lists (around comments 79-84). Would this SolutionSet be like RootOf?

Re: Issue 1694 in sympy: solve has many issues with fractions

2010-08-06 Thread sympy
Comment #108 on issue 1694 by smichr: solve has many issues with fractions http://code.google.com/p/sympy/issues/detail?id=1694 Vinzent said it looked find (see 1923). The Basic.count_ops could never be called with an iterable since it's a method of basic. The tuple trap is avoided by Basic

Re: Issue 1694 in sympy: solve has many issues with fractions

2010-08-06 Thread sympy
Comment #109 on issue 1694 by smichr: solve has many issues with fractions http://code.google.com/p/sympy/issues/detail?id=1694 The cherry picking went fine. I am running tests on a branch named 1923. -- You received this message because you are subscribed to the Google Groups sympy-issues

Re: Issue 1694 in sympy: solve has many issues with fractions

2010-08-06 Thread sympy
Comment #110 on issue 1694 by smichr: solve has many issues with fractions http://code.google.com/p/sympy/issues/detail?id=1694 All tests passed (64 bit). -- You received this message because you are subscribed to the Google Groups sympy-issues group. To post to this group, send email to

Re: Issue 1923 in sympy: count_ops doesn't return a count (by default)

2010-08-06 Thread sympy
Updates: Labels: -NeedsBetterPatch Comment #6 on issue 1923 by smichr: count_ops doesn't return a count (by default) http://code.google.com/p/sympy/issues/detail?id=1923 This is in final review, branch 1923 if you have any final comments. -- You received this message because you are

Re: Issue 1923 in sympy: count_ops doesn't return a count (by default)

2010-08-06 Thread sympy
Issue 1923: count_ops doesn't return a count (by default) http://code.google.com/p/sympy/issues/detail?id=1923 This issue is now blocking issue 1694. See http://code.google.com/p/sympy/issues/detail?id=1694 -- You received this message because you are listed in the owner or CC fields of this

Re: Issue 2008 in sympy: Inconsistencies for Pow**n and exp***n

2010-08-06 Thread sympy
Comment #2 on issue 2008 by asmeurer: Inconsistencies for Pow**n and exp***n http://code.google.com/p/sympy/issues/detail?id=2008 Actually, exp does it unequivocally: In [6]: print exp(x)**y exp(x*y) Is that really valid always? And you could argue that [1] makes things easier, because you

Re: Issue 1694 in sympy: solve has many issues with fractions

2010-08-06 Thread sympy
Comment #112 on issue 1694 by asmeurer: solve has many issues with fractions http://code.google.com/p/sympy/issues/detail?id=1694 Then other errors occur (TypeError: object of type 'Integral' has no len()). Yeah, I noticed after I posted that comment that it was trying to do len on a

Re: Issue 1266 in sympy: update SympyDevelopment

2010-08-06 Thread sympy
Updates: Labels: Documentation Comment #4 on issue 1266 by asmeurer: update SympyDevelopment http://code.google.com/p/sympy/issues/detail?id=1266 I just updated that page the other day to use git, because it confused someone. I basically just replaced the hg commands with their

Re: Issue 1949 in sympy: Doctests for polynomials module

2010-08-06 Thread sympy
Comment #7 on issue 1949 by asmeurer: Doctests for polynomials module http://code.google.com/p/sympy/issues/detail?id=1949 Just a warning. You can't use Header == in any docstring that gets imported by Sphinx, because it causes the build to fail (do cd doc; make html). This is