Re: Issue 281 in sympy: Infinity is a subclass of Rational

2011-05-26 Thread sympy
Updates: Labels: -Priority-Medium Priority-High Comment #6 on issue 281 by asmeurer: Infinity is a subclass of Rational http://code.google.com/p/sympy/issues/detail?id=281 I just discovered this when fixing issue 2425, and I don't like it. In addition to speed issues because of

Re: Issue 281 in sympy: Infinity is a subclass of Rational

2011-05-26 Thread sympy
Comment #7 on issue 281 by asmeurer: Infinity is a subclass of Rational http://code.google.com/p/sympy/issues/detail?id=281 zoo isn't Rational: In [28]: zoo.is_Rational Out[28]: False It's an AtomicExpr. Let's reimplement oo, -oo, and nan in the same way. -- You received this message

Re: Issue 2359 in sympy: Should Pow(expr, S(1)/2) print as sqrt(expr)

2011-05-26 Thread sympy
Comment #2 on issue 2359 by smi...@gmail.com: Should Pow(expr, S(1)/2) print as sqrt(expr) http://code.google.com/p/sympy/issues/detail?id=2359 I'm guessing the sqrt is common enough to warrant a special case and I like the other suggestion of using S in the exponent. Can we give S a

Re: Issue 2359 in sympy: Should Pow(expr, S(1)/2) print as sqrt(expr)

2011-05-26 Thread sympy
Comment #3 on issue 2359 by asmeurer: Should Pow(expr, S(1)/2) print as sqrt(expr) http://code.google.com/p/sympy/issues/detail?id=2359 Actually, R(3, 4) is more characters than S(3)/4 (count them). But of course, it's more if you put it in an exponent, because of the extra parentheses.

Re: Issue 2359 in sympy: Should Pow(expr, S(1)/2) print as sqrt(expr)

2011-05-26 Thread sympy
Comment #4 on issue 2359 by matt...@gmail.com: Should Pow(expr, S(1)/2) print as sqrt(expr) http://code.google.com/p/sympy/issues/detail?id=2359 oes (x**p)**1/q always denest into x**(p/q)? No, e.g.: In [3]: print sqrt(x**3) (x**3)**(1/2) How about creating a root function (...)

Re: Issue 2359 in sympy: Should Pow(expr, S(1)/2) print as sqrt(expr)

2011-05-26 Thread sympy
Comment #5 on issue 2359 by asmeurer: Should Pow(expr, S(1)/2) print as sqrt(expr) http://code.google.com/p/sympy/issues/detail?id=2359 That was 1/2. -- 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 2425 in sympy: Pretty printing bug

2011-05-26 Thread sympy
Updates: Status: Fixed Comment #7 on issue 2425 by smi...@gmail.com: Pretty printing bug http://code.google.com/p/sympy/issues/detail?id=2425 thanks, it's in. -- You received this message because you are subscribed to the Google Groups sympy-issues group. To post to this group, send

Re: Issue 1620 in sympy: Allow derivatives of unknown functions evaluated at a point

2011-05-26 Thread sympy
Comment #32 on issue 1620 by renato.c...@gmail.com: Allow derivatives of unknown functions evaluated at a point http://code.google.com/p/sympy/issues/detail?id=1620 I re-updated this pull request (360) with a much better hash function. I also fixed Lambda.__hash__ function, it's pull

Issue 2428 in sympy: PolificationFailed needs to be imported in polysys

2011-05-26 Thread sympy
Status: Accepted Owner: smi...@gmail.com Labels: Type-Defect Priority-Medium EasyToFix New issue 2428 by smi...@gmail.com: PolificationFailed needs to be imported in polysys http://code.google.com/p/sympy/issues/detail?id=2428 I thought I filled this out already...perhaps my computer shut

Issue 2429 in sympy: add quantities support to lambdify

2011-05-26 Thread sympy
Status: New Owner: Labels: Type-Defect Priority-Medium New issue 2429 by Bartosz.Telenczuk: add quantities support to lambdify http://code.google.com/p/sympy/issues/detail?id=2429 lambdify function does not work with sympy.physics.units: In [6]: expr = 2*units.A*a In [7]: f =

Re: Issue 2424 in sympy: Cartesian Product Set

2011-05-26 Thread sympy
Updates: Labels: NeedsReview MRocklin Comment #2 on issue 2424 by asmeurer: Cartesian Product Set http://code.google.com/p/sympy/issues/detail?id=2424 (No comment was entered for this change.) -- You received this message because you are subscribed to the Google Groups sympy-issues

Re: Issue 2388 in sympy: The value of str(expr) depends on a global setting

2011-05-26 Thread sympy
Updates: Labels: NeedsReview asmeurer Comment #12 on issue 2388 by asmeurer: The value of str(expr) depends on a global setting http://code.google.com/p/sympy/issues/detail?id=2388 The patch at https://github.com/sympy/sympy/pull/350 fixes this, btw. -- You received this message

Re: Issue 2428 in sympy: PolificationFailed needs to be imported in polysys

2011-05-26 Thread sympy
Updates: Status: Duplicate Labels: Polynomial Solvers Mergedinto: 2404 Comment #1 on issue 2428 by asmeurer: PolificationFailed needs to be imported in polysys http://code.google.com/p/sympy/issues/detail?id=2428 No, you did. -- You received this message because you

Re: Issue 2404 in sympy: PolificationFailed needs to be imported in polysys

2011-05-26 Thread sympy
Comment #2 on issue 2404 by asmeurer: PolificationFailed needs to be imported in polysys http://code.google.com/p/sympy/issues/detail?id=2404 Issue 2428 has been merged into this issue. -- You received this message because you are subscribed to the Google Groups sympy-issues group. To post

Re: Issue 2404 in sympy: PolificationFailed needs to be imported in polysys

2011-05-26 Thread sympy
Updates: Labels: Solvers Polynomial Comment #3 on issue 2404 by asmeurer: PolificationFailed needs to be imported in polysys http://code.google.com/p/sympy/issues/detail?id=2404 (No comment was entered for this change.) -- You received this message because you are subscribed to the

Re: Issue 2419 in sympy: Finite Sets

2011-05-26 Thread sympy
Issue 2419: Finite Sets http://code.google.com/p/sympy/issues/detail?id=2419 This issue is now blocking issue 2430. See http://code.google.com/p/sympy/issues/detail?id=2430 -- You received this message because you are listed in the owner or CC fields of this issue, or because you starred this

Re: Issue 2424 in sympy: Cartesian Product Set

2011-05-26 Thread sympy
Issue 2424: Cartesian Product Set http://code.google.com/p/sympy/issues/detail?id=2424 This issue is now blocking issue 2430. See http://code.google.com/p/sympy/issues/detail?id=2430 -- You received this message because you are listed in the owner or CC fields of this issue, or because you

Issue 2430 in sympy: Expression Countable Set

2011-05-26 Thread sympy
Status: Accepted Owner: mrock...@gmail.com Labels: Type-Enhancement Priority-Medium New issue 2430 by mrock...@gmail.com: Expression Countable Set http://code.google.com/p/sympy/issues/detail?id=2430 I'm looking to build a basic vocabulary of sets. Currently Intervals, Unions, and the

Re: Issue 2404 in sympy: PolificationFailed needs to be imported in polysys

2011-05-26 Thread sympy
Comment #4 on issue 2404 by smi...@gmail.com: PolificationFailed needs to be imported in polysys http://code.google.com/p/sympy/issues/detail?id=2404 1 minute review at https://github.com/sympy/sympy/pull/365; please just commit it after reviewing if you can. The polysys test passes here.

Re: Issue 2430 in sympy: Expression Countable Set

2011-05-26 Thread sympy
Comment #1 on issue 2430 by christian.muise: Expression Countable Set http://code.google.com/p/sympy/issues/detail?id=2430 Why not use lambda's to make up a set's behaviour? If it's countable, then there should be a mapping from the ints to an element in the set and vice versa. A lambda can

Re: Issue 2430 in sympy: Expression Countable Set

2011-05-26 Thread sympy
Comment #2 on issue 2430 by asmeurer: Expression Countable Set http://code.google.com/p/sympy/issues/detail?id=2430 ```python``` only works in GitHub :) That could be useful. We should support both. Actually, if you implement set builder notation correctly, you should be able to just as

Re: Issue 2404 in sympy: PolificationFailed needs to be imported in polysys

2011-05-26 Thread sympy
Comment #5 on issue 2404 by smi...@gmail.com: PolificationFailed needs to be imported in polysys http://code.google.com/p/sympy/issues/detail?id=2404 Thanks, Aaron. It's in. -- You received this message because you are subscribed to the Google Groups sympy-issues group. To post to this

Re: Issue 2431 in sympy: Bug with Poly.is_irreducible

2011-05-26 Thread sympy
Comment #1 on issue 2431 by asmeurer: Bug with Poly.is_irreducible http://code.google.com/p/sympy/issues/detail?id=2431 Ah, sorry, that should be preceded by a = Poly(x, x) The point is that Poly.is_irreducible doesn't work at all. -- You received this message because you are subscribed to

Issue 2431 in sympy: Bug with Poly.is_irreducible

2011-05-26 Thread sympy
Status: Accepted Owner: asmeurer CC: matt...@gmail.com Labels: Type-Defect Priority-Medium Polynomial New issue 2431 by asmeurer: Bug with Poly.is_irreducible http://code.google.com/p/sympy/issues/detail?id=2431 In [4]: a.is_irreducible

Re: Issue 2305 in sympy: Integer.isqrt and similar shouldn't be a method

2011-05-26 Thread sympy
Updates: Cc: Vinzent.Steinberg ondrej.c...@gmail.com elliso...@gmail.com Comment #18 on issue 2305 by asmeurer: Integer.isqrt and similar shouldn't be a method http://code.google.com/p/sympy/issues/detail?id=2305 Can we get some more opinions on this? -- You received this message

Issue 2432 in sympy: Tests without assert

2011-05-26 Thread sympy
Status: Accepted Owner: asmeurer Labels: Type-Defect Priority-Medium Testing New issue 2432 by asmeurer: Tests without assert http://code.google.com/p/sympy/issues/detail?id=2432 When fixing issue 2308, I noticed that test_Add_primitive() in test_arit.py was just a bunch of equality checks

Re: Issue 2308 in sympy: Should we rename primitive()? (and Expr.primitive() should call primitive())

2011-05-26 Thread sympy
Updates: Labels: -NeedsBetterPatch NeedsReview Comment #19 on issue 2308 by asmeurer: Should we rename primitive()? (and Expr.primitive() should call primitive()) http://code.google.com/p/sympy/issues/detail?id=2308 OK, I updated the branch at https://github.com/sympy/sympy/pull/322

Re: Issue 2304 in sympy: Remove Pure from top-level namespace

2011-05-26 Thread sympy
Updates: Status: Accepted Labels: NeedsReview asmeurer Comment #3 on issue 2304 by asmeurer: Remove Pure from top-level namespace http://code.google.com/p/sympy/issues/detail?id=2304 I decided to just remove Pure completely. It was only being used as a generator for the

Re: Issue 2304 in sympy: Remove Pure from top-level namespace

2011-05-26 Thread sympy
Comment #4 on issue 2304 by asmeurer: Remove Pure from top-level namespace http://code.google.com/p/sympy/issues/detail?id=2304 It seems that Mathematic link is broken. Anyway, I was just following the link from http://mathworld.wolfram.com/AlgebraicNumberMinimalPolynomial.html, which