Re: Issue 1757 in sympy: Tools to check and correct indentation

2009-12-13 Thread sympy
Comment #15 on issue 1757 by asmeurer: Tools to check and correct indentation http://code.google.com/p/sympy/issues/detail?id=1757 I was referring to PEP 8, and it doesn't mention this. Unless there is some other one that speaks about it, other than implicitly by PEP 20 :) (maybe actually "F

Re: Issue 1757 in sympy: Tools to check and correct indentation

2009-12-13 Thread sympy
Comment #14 on issue 1757 by asmeurer: Tools to check and correct indentation http://code.google.com/p/sympy/issues/detail?id=1757 I am not sure how I feel about changing: if test: return something else: do something else to if test: return something do something else Coverag

Re: Issue 1752 in sympy: setup.py test annoyances

2009-12-13 Thread sympy
Comment #6 on issue 1752 by smichr: setup.py test annoyances http://code.google.com/p/sympy/issues/detail?id=1752 A cleaned up version is at branch 1752...I haven't had a change to look at the break issue (ctrl-C) and my system's command line is still not working for testing. -- You received

Re: Issue 1752 in sympy: setup.py test annoyances

2009-12-13 Thread sympy
Updates: Labels: -NeedsReview NeedsBetterPatch Comment #5 on issue 1752 by asmeurer: setup.py test annoyances http://code.google.com/p/sympy/issues/detail?id=1752 Keyboard interrupting from the tests works correctly now, but the branch commit log is a mess. Keyboard interrupting from t

Re: Issue 1764 in sympy: homogeneous_order(-g(x).diff(x), x, y) fails

2009-12-13 Thread sympy
Updates: Cc: smichr Labels: NeedsReview Comment #1 on issue 1764 by asmeurer: homogeneous_order(-g(x).diff(x), x, y) fails http://code.google.com/p/sympy/issues/detail?id=1764 The problem is that homogenous_order is checking if the expression is an Order or a Derivative before

Re: Issue 1762 in sympy: classify_ode doesn't match (or even mismatch) parametric ODE correctly

2009-12-13 Thread sympy
Updates: Status: Started Labels: NeedsReview Comment #3 on issue 1762 by asmeurer: classify_ode doesn't match (or even mismatch) parametric ODE correctly http://code.google.com/p/sympy/issues/detail?id=1762 The fix was simple. classify_ode was returning early if the expression

Re: Issue 1762 in sympy: classify_ode doesn't match (or even mismatch) parametric ODE correctly

2009-12-13 Thread sympy
Updates: Cc: smichr Comment #4 on issue 1762 by asmeurer: classify_ode doesn't match (or even mismatch) parametric ODE correctly http://code.google.com/p/sympy/issues/detail?id=1762 (No comment was entered for this change.) -- You received this message because you are listed in the own

Issue 1764 in sympy: homogeneous_order(-g(x).diff(x), x, y) fails

2009-12-13 Thread sympy
Status: Accepted Owner: asmeurer Labels: Type-Defect Priority-Medium New issue 1764 by asmeurer: homogeneous_order(-g(x).diff(x), x, y) fails http://code.google.com/p/sympy/issues/detail?id=1764 In [2]: homogeneous_order(g(x).diff(x), x, y) < Returns None> In [3]: homogeneous_order(-g(x).diff(x),

Re: Issue 1762 in sympy: classify_ode doesn't match (or even mismatch) parametric ODE correctly

2009-12-13 Thread sympy
Comment #2 on issue 1762 by asmeurer: classify_ode doesn't match (or even mismatch) parametric ODE correctly http://code.google.com/p/sympy/issues/detail?id=1762 I think classify_ode needs to know that f(x) is not the same function as f(x, a) (SymPy lets the same function name have a differen

Re: Issue 1762 in sympy: classify_ode doesn't match (or even mismatch) parametric ODE correctly

2009-12-13 Thread sympy
Comment #1 on issue 1762 by asmeurer: classify_ode doesn't match (or even mismatch) parametric ODE correctly http://code.google.com/p/sympy/issues/detail?id=1762 In [2]: var('a') Out[2]: a In [3]: dsolve(f(x).diff(x), f(a)) --

Re: Issue 1755 in sympy: Broken methods for trigonometric and hyperbolic functions

2009-12-13 Thread sympy
Updates: Status: Fixed Labels: -NeedsReview PassedReview Comment #4 on issue 1755 by asmeurer: Broken methods for trigonometric and hyperbolic functions http://code.google.com/p/sympy/issues/detail?id=1755 This looks good. I pushed it in. -- You received this message because y

Re: Issue 1757 in sympy: Tools to check and correct indentation

2009-12-13 Thread sympy
Updates: Cc: Vinzent.Steinberg asmeurer Labels: NeedsReview Comment #13 on issue 1757 by smichr: Tools to check and correct indentation http://code.google.com/p/sympy/issues/detail?id=1757 It would probably be a good idea to include in the tests a test for hidden conditions whic

Re: Issue 652 in sympy: Write tests for acosh, atanh, acoth

2009-12-13 Thread sympy
Updates: Summary: Write tests for acosh, atanh, acoth Comment #13 on issue 652 by asmeurer: Write tests for acosh, atanh, acoth http://code.google.com/p/sympy/issues/detail?id=652 (No comment was entered for this change.) -- You received this message because you are listed in the owner o

Re: Issue 1755 in sympy: Broken methods for trigonometric and hyperbolic functions

2009-12-13 Thread sympy
Comment #3 on issue 1755 by ronan.l...@gmail.com: Broken methods for trigonometric and hyperbolic functions http://code.google.com/p/sympy/issues/detail?id=1755 Sorry, it's in my github repository, git://github.com/rlamy/sympy.git -- You received this message because you are listed in the owne

Re: Issue 1755 in sympy: Broken methods for trigonometric and hyperbolic functions

2009-12-13 Thread sympy
Comment #2 on issue 1755 by aaron.j.gardner: Broken methods for trigonometric and hyperbolic functions http://code.google.com/p/sympy/issues/detail?id=1755 Hi, can you indicate where, specifically, branch '1755' lives and how one might acquire it? Perhaps you could simply attach the patch to

Re: Issue 1758 in sympy: fix curvilinear_coordinates example output

2009-12-13 Thread sympy
Updates: Labels: NeedsReview Comment #7 on issue 1758 by Vinzent.Steinberg: fix curvilinear_coordinates example output http://code.google.com/p/sympy/issues/detail?id=1758 I'm undecided about lamda. The rest looks fine to me. -- You received this message because you are listed in the o

Re: Issue 1763 in sympy: sympify(u"α") does not wor k

2009-12-13 Thread sympy
Updates: Summary: sympify(u"α") does not work Labels: -Priority-Medium Priority-High Milestone-Release0.7.0 Comment #1 on issue 1763 by ronan.l...@gmail.com: sympify(u"α") does not work http://code.google.com/p/sympy/issues/detail?id=1763 Actually, "α" is garbage and we shouldn'

Re: Issue 1719 in sympy: implement from sympy.abc import lower, greek

2009-12-13 Thread sympy
Comment #20 on issue 1719 by asmeurer: implement from sympy.abc import lower, greek http://code.google.com/p/sympy/issues/detail?id=1719 I like that idea. As for the tests, I guess we will just have to leave it untested. -- You received this message because you are listed in the owner or CC

Re: Issue 1719 in sympy: implement from sympy.abc import lower, greek

2009-12-13 Thread sympy
Comment #18 on issue 1719 by asmeurer: implement from sympy.abc import lower, greek http://code.google.com/p/sympy/issues/detail?id=1719 It also fails test_code_quality.py, because of the explicit imports. -- You received this message because you are listed in the owner or CC fields of this is

Re: Issue 1719 in sympy: implement from sympy.abc import lower, greek

2009-12-13 Thread sympy
Comment #19 on issue 1719 by Vinzent.Steinberg: implement from sympy.abc import lower, greek http://code.google.com/p/sympy/issues/detail?id=1719 Yeah, I fear it is not possible to disable the warning. Should we use sympy.abc.greek and sympy.abc.Greek? sympy.abc.greek.upper seems to long to

Re: Issue 1719 in sympy: implement from sympy.abc import lower, greek

2009-12-13 Thread sympy
Updates: Labels: -NeedsReview NeedsBetterPatch Comment #17 on issue 1719 by asmeurer: implement from sympy.abc import lower, greek http://code.google.com/p/sympy/issues/detail?id=1719 On Vinzent's branch, immediately when I test, I get warnings: /Users/aaronmeurer/Documents/Python/symp

Re: Issue 1719 in sympy: implement from sympy.abc import lower, greek

2009-12-13 Thread sympy
Comment #16 on issue 1719 by asmeurer: implement from sympy.abc import lower, greek http://code.google.com/p/sympy/issues/detail?id=1719 We should also have support for uppercase greek letters here. -- You received this message because you are listed in the owner or CC fields of this issue, or

Re: Issue 1758 in sympy: fix curvilinear_coordinates example output

2009-12-13 Thread sympy
Comment #6 on issue 1758 by asmeurer: fix curvilinear_coordinates example output http://code.google.com/p/sympy/issues/detail?id=1758 I pushed a fix that uses dsolve correctly to my example-fix branch. There is also a commit in there that uses lamda instead of lambda, so if you don't like th

Issue 1763 in sympy: sympify("α") does not work

2009-12-13 Thread sympy
Status: Accepted Owner: asmeurer Labels: Type-Defect Priority-Medium New issue 1763 by asmeurer: sympify("α") does not work http://code.google.com/p/sympy/issues/detail?id=1763 In [37]: S("α") -- - SympifyError

Re: Issue 1725 in sympy: Implement switch manager for expand() and possibly others

2009-12-13 Thread sympy
Comment #10 on issue 1725 by asmeurer: Implement switch manager for expand() and possibly others http://code.google.com/p/sympy/issues/detail?id=1725 By the way, is there a good reason that expand_trig and expand_complex are off by default? Oh, and I agree with Vinzent. -- You received this

Re: Issue 1725 in sympy: Implement switch manager for expand() and possibly others

2009-12-13 Thread sympy
Comment #9 on issue 1725 by Vinzent.Steinberg: Implement switch manager for expand() and possibly others http://code.google.com/p/sympy/issues/detail?id=1725 I think this is too complex and too much magic. I'd just set a reasonable default, if there is any '...=True' kwarg, I'd disable all ot

Re: Issue 1727 in sympy: fix namespace clashes

2009-12-13 Thread sympy
Comment #5 on issue 1727 by Vinzent.Steinberg: fix namespace clashes http://code.google.com/p/sympy/issues/detail?id=1727 We could also remove sum() and replace it with Sum().doit() or Sum().expand(). -- You received this message because you are listed in the owner or CC fields of this issue,

Re: Issue 1725 in sympy: Implement switch manager for expand() and possibly others

2009-12-13 Thread sympy
Updates: Cc: asmeurer Vinzent.Steinberg Comment #8 on issue 1725 by smichr: Implement switch manager for expand() and possibly others http://code.google.com/p/sympy/issues/detail?id=1725 In Issue 1724 there has been some discussion about implementing this. I can give this a go this we