Re: Issue 2432 in sympy: Tests without assert

2011-05-27 Thread sympy
Comment #1 on issue 2432 by smi...@gmail.com: Tests without assert http://code.google.com/p/sympy/issues/detail?id=2432 I'll put a pull for the ones I found. Could you run the tests? https://github.com/sympy/sympy/pull/370 -- You received this message because you are subscribed to the Google

Issue 2433 in sympy: Polys module function poly() needs gens argument

2011-05-27 Thread sympy
Status: Accepted Owner: ness...@googlemail.com CC: matt...@gmail.com Labels: Type-Defect Priority-Medium Polynomial New issue 2433 by ness...@googlemail.com: Polys module function poly() needs gens argument http://code.google.com/p/sympy/issues/detail?id=2433 poly(1) and related calls fail.

Issue 2434 in sympy: Polynomials are sorted incorrectly w.r.t. the grevlex term order

2011-05-27 Thread sympy
Status: Accepted Owner: j.yehde...@gmail.com Labels: Type-Defect Priority-Medium New issue 2434 by j.yehde...@gmail.com: Polynomials are sorted incorrectly w.r.t. the grevlex term order http://code.google.com/p/sympy/issues/detail?id=2434 Sorting polynomials wrt the grevlex term order gives

Re: Issue 2434 in sympy: Polynomials are sorted incorrectly w.r.t. the grevlex term order

2011-05-27 Thread sympy
Comment #1 on issue 2434 by j.yehde...@gmail.com: Polynomials are sorted incorrectly w.r.t. the grevlex term order http://code.google.com/p/sympy/issues/detail?id=2434 I believe I know why it doesn't work: Polynomials (at least sd polynomials) are sorted via def sdp_sort(f, O): Sort

Re: Issue 2434 in sympy: Polynomials are sorted incorrectly w.r.t. the grevlex term order

2011-05-27 Thread sympy
Comment #2 on issue 2434 by j.yehde...@gmail.com: Polynomials are sorted incorrectly w.r.t. the grevlex term order http://code.google.com/p/sympy/issues/detail?id=2434 I believe I know why it doesn't work: Polynomials (at least sd polynomials) are sorted via def sdp_sort(f, O): Sort

Re: Issue 2433 in sympy: Polys module function poly() needs gens argument

2011-05-27 Thread sympy
Comment #1 on issue 2433 by ness...@googlemail.com: Polys module function poly() needs gens argument http://code.google.com/p/sympy/issues/detail?id=2433 Also, it is slightly unfortunate then when generators *are* passed, and the coefficients contain irrationals like 2**(S(1)/3), then

Re: Issue 2415 in sympy: Poly(x, x) * I != I * Poly(x, x)

2011-05-27 Thread sympy
Updates: Status: Started Comment #4 on issue 2415 by ronan.l...@gmail.com: Poly(x, x) * I != I * Poly(x, x) http://code.google.com/p/sympy/issues/detail?id=2415 This is an important issue. And PR #359 [https://github.com/sympy/sympy/pull/359] fixes it. The only problem is that

Issue 2435 in sympy: Cholesky Decomposition methods to be added in Matrix

2011-05-27 Thread sympy
Status: Accepted Owner: sherjilo...@gmail.com Labels: Type-Defect Priority-High Milestone-Release0.7.0 New issue 2435 by sherjilo...@gmail.com: Cholesky Decomposition methods to be added in Matrix http://code.google.com/p/sympy/issues/detail?id=2435 Cholesky decomposition is theoretically

Re: Issue 2371 in sympy: Completely untested modules in sympy.polys

2011-05-27 Thread sympy
Updates: Labels: NeedsReview Comment #5 on issue 2371 by ronan.l...@gmail.com: Completely untested modules in sympy.polys http://code.google.com/p/sympy/issues/detail?id=2371 This is public code, since it's accessible via public names, and the Repr option is actually imported but

Issue 2436 in sympy: eval(str(Derivative()))

2011-05-27 Thread sympy
Status: Accepted Owner: renato.c...@gmail.com Labels: Type-Defect Priority-Medium EasyToFix Printing New issue 2436 by renato.c...@gmail.com: eval(str(Derivative())) http://code.google.com/p/sympy/issues/detail?id=2436 str(Derivative(f(x), x)) 'D(f(x), x)' eval(_) NameError: name 'D' is not

Re: Issue 2218 in sympy: Derivative issues

2011-05-27 Thread sympy
Updates: Status: Fixed Comment #3 on issue 2218 by renato.c...@gmail.com: Derivative issues http://code.google.com/p/sympy/issues/detail?id=2218 Fixed in c20062e7 and 2361dd86. -- You received this message because you are subscribed to the Google Groups sympy-issues group. To post to

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

2011-05-27 Thread sympy
Comment #13 on issue 2388 by ronan.l...@gmail.com: The value of str(expr) depends on a global setting http://code.google.com/p/sympy/issues/detail?id=2388 It doesn't seem to actually be a bug in Python - I searched through the tracker. From the discussions linked, it seems to be considered

Re: Issue 2421 in sympy: python(expr) issues

2011-05-27 Thread sympy
Comment #2 on issue 2421 by smi...@gmail.com: python(expr) issues http://code.google.com/p/sympy/issues/detail?id=2421 yes, for sqrt see issue 2359 There is also an issue for Piecewise: the python and str are the same. Piecewise((1, [0, 1))) print python(_) e = Piecewise((1, [0,

Re: Issue 2434 in sympy: Polynomials are sorted incorrectly w.r.t. the grevlex term order

2011-05-27 Thread sympy
Updates: Labels: NeedsReview j.yehdegho Comment #3 on issue 2434 by asmeurer: Polynomials are sorted incorrectly w.r.t. the grevlex term order http://code.google.com/p/sympy/issues/detail?id=2434 https://github.com/sympy/sympy/pull/371 -- You received this message because you are

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

2011-05-27 Thread sympy
Comment #20 on issue 2308 by smi...@gmail.com: Should we rename primitive()? (and Expr.primitive() should call primitive()) http://code.google.com/p/sympy/issues/detail?id=2308 Why shouldn't `(2*x/3 + 4*y/9).primitive()` give `(2/3, x + 2*y/3)`? I don't know much about the algebra fields,

Re: Issue 2433 in sympy: Polys module function poly() needs gens argument

2011-05-27 Thread sympy
Comment #2 on issue 2433 by asmeurer: Polys module function poly() needs gens argument http://code.google.com/p/sympy/issues/detail?id=2433 I think the gens kwarg should do it, i.e., In [7]: Poly(x*y, gens=[x]) Out[7]: Poly(y*x, x, domain='ZZ[y]') But poly() should have a *gens anyway.

Re: Issue 2415 in sympy: Poly(x, x) * I != I * Poly(x, x)

2011-05-27 Thread sympy
Comment #5 on issue 2415 by asmeurer: Poly(x, x) * I != I * Poly(x, x) http://code.google.com/p/sympy/issues/detail?id=2415 I say just XFAIL the match test. The only way to fix that is to implement matching in Poly. -- You received this message because you are subscribed to the Google

Re: Issue 2435 in sympy: Cholesky Decomposition methods to be added in Matrix

2011-05-27 Thread sympy
Updates: Labels: -Milestone-Release0.7.0 Milestone-Release0.7.1 NeedsReview sherjilozair Comment #1 on issue 2435 by asmeurer: Cholesky Decomposition methods to be added in Matrix http://code.google.com/p/sympy/issues/detail?id=2435 What pull request is this? -- You received this

Re: Issue 2436 in sympy: eval(str(Derivative()))

2011-05-27 Thread sympy
Comment #1 on issue 2436 by asmeurer: eval(str(Derivative())) http://code.google.com/p/sympy/issues/detail?id=2436 By the way, if we do my idea from 1620, that would fix it too. -- You received this message because you are subscribed to the Google Groups sympy-issues group. To post to this

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

2011-05-27 Thread sympy
Comment #14 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 Well, the Python guys don't consider it to be a bug per se, but I definitely do. If we make __repr__ == srepr, then [10] below would give the

Re: Issue 2421 in sympy: python(expr) issues

2011-05-27 Thread sympy
Comment #3 on issue 2421 by smi...@gmail.com: python(expr) issues http://code.google.com/p/sympy/issues/detail?id=2421 The problem in the last comment is that you couldn't run that as python code and have it work. Also, there is a strange error when trying to S to srepr form of piecewise:

Re: Issue 2421 in sympy: python(expr) issues

2011-05-27 Thread sympy
Comment #4 on issue 2421 by asmeurer: python(expr) issues http://code.google.com/p/sympy/issues/detail?id=2421 It's probably because it's printing as Function('ExprCondPair') (see the return from srepr()). -- You received this message because you are subscribed to the Google Groups

Issue 2437 in sympy: poly.subs doesn't work well with mpmath findroot

2011-05-27 Thread sympy
Status: New Owner: Labels: Type-Defect Priority-Medium New issue 2437 by memm...@gmail.com: poly.subs doesn't work well with mpmath findroot http://code.google.com/p/sympy/issues/detail?id=2437 Defining a symbolic polynomial and using this within mpmath's findroot function doesn't

Re: Issue 2421 in sympy: python(expr) issues

2011-05-27 Thread sympy
Comment #5 on issue 2421 by smi...@gmail.com: python(expr) issues http://code.google.com/p/sympy/issues/detail?id=2421 That's what I though, but cos() prints the same way. -- You received this message because you are subscribed to the Google Groups sympy-issues group. To post to this group,

Re: Issue 2435 in sympy: Cholesky Decomposition methods to be added in Matrix

2011-05-27 Thread sympy
Comment #2 on issue 2435 by sherjilo...@gmail.com: Cholesky Decomposition methods to be added in Matrix http://code.google.com/p/sympy/issues/detail?id=2435 Aaron, Can't we make it to this release ? -- You received this message because you are subscribed to the Google Groups sympy-issues

Re: Issue 2435 in sympy: Cholesky Decomposition methods to be added in Matrix

2011-05-27 Thread sympy
Comment #3 on issue 2435 by sherjilo...@gmail.com: Cholesky Decomposition methods to be added in Matrix http://code.google.com/p/sympy/issues/detail?id=2435 https://github.com/sympy/sympy/pull/184 -- You received this message because you are subscribed to the Google Groups sympy-issues

Re: Issue 2435 in sympy: Cholesky Decomposition methods to be added in Matrix

2011-05-27 Thread sympy
Comment #4 on issue 2435 by asmeurer: Cholesky Decomposition methods to be added in Matrix http://code.google.com/p/sympy/issues/detail?id=2435 We can, if it gets in soon enough, but the Milestone-Release0.7.0 tag is for issues that block the release, which does not describe this issue.

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

2011-05-27 Thread sympy
Comment #15 on issue 2388 by Vinzent.Steinberg: The value of str(expr) depends on a global setting http://code.google.com/p/sympy/issues/detail?id=2388 I remember the discussion. Basically they said that str(list(...)) will always use repr() on the content of the list. This is why we

Issue 2438 in sympy: repr() does not eval() to the same value

2011-05-27 Thread sympy
Status: New Owner: Labels: Type-Defect Priority-Medium New issue 2438 by five...@gmail.com: repr() does not eval() to the same value http://code.google.com/p/sympy/issues/detail?id=2438 a = Symbol('a') b = a/(2*a) print (b, eval(repr(b))) # (1/2, 0) from __future__ import division print

Re: Issue 2307 in sympy: Duplicate methods: as_coeff_mul and as_coeff_Mul

2011-05-27 Thread sympy
Comment #21 on issue 2307 by matt...@gmail.com: Duplicate methods: as_coeff_mul and as_coeff_Mul http://code.google.com/p/sympy/issues/detail?id=2307 btw. One more place where as_coeff_Capital_letter is useful is new combsimp(), but as we don't have as_coeff_Add and future of

Re: Issue 2433 in sympy: Polys module function poly() needs gens argument

2011-05-27 Thread sympy
Comment #3 on issue 2433 by matt...@gmail.com: Polys module function poly() needs gens argument http://code.google.com/p/sympy/issues/detail?id=2433 To construct polynomials you should really use Poly. poly() is a replacement for expand() in cases where expand is too slow. After all I

Issue 2439 in sympy: collect() should allow to apply a function to coefficients

2011-05-27 Thread sympy
Status: New Owner: matt...@gmail.com Labels: Type-Defect Priority-Medium Simplify New issue 2439 by matt...@gmail.com: collect() should allow to apply a function to coefficients http://code.google.com/p/sympy/issues/detail?id=2439 I have the following expression: In [1]: var('a') Out[1]: a

Re: Issue 2433 in sympy: Polys module function poly() needs gens argument

2011-05-27 Thread sympy
Updates: Status: Started Owner: matt...@gmail.com Cc: -matt...@gmail.com Labels: Milestone-Release0.7.0 NeedsReview Comment #4 on issue 2433 by matt...@gmail.com: Polys module function poly() needs gens argument

Re: Issue 2433 in sympy: Polys module function poly() needs gens argument

2011-05-27 Thread sympy
Comment #5 on issue 2433 by asmeurer: Polys module function poly() needs gens argument http://code.google.com/p/sympy/issues/detail?id=2433 Why does Poly not just do what poly() does? I was telling him to use poly() because his code was too slow with Poly(). -- You received this message

Re: Issue 2421 in sympy: python(expr) issues

2011-05-27 Thread sympy
Comment #6 on issue 2421 by asmeurer: python(expr) issues http://code.google.com/p/sympy/issues/detail?id=2421 Well, that's probably wrong too. cos != Function('cos'). -- You received this message because you are subscribed to the Google Groups sympy-issues group. To post to this group, send

Re: Issue 2438 in sympy: repr() does not eval() to the same value

2011-05-27 Thread sympy
Comment #1 on issue 2438 by asmeurer: repr() does not eval() to the same value http://code.google.com/p/sympy/issues/detail?id=2438 This is discussed over at issue 2388. We make __repr__ return the same thing as __str__ because str(list) calls repr() on the elements. We have the

Re: Issue 2439 in sympy: collect() should allow to apply a function to coefficients

2011-05-27 Thread sympy
Updates: Status: Accepted Comment #1 on issue 2439 by asmeurer: collect() should allow to apply a function to coefficients http://code.google.com/p/sympy/issues/detail?id=2439 Sounds like a good idea to me. -- You received this message because you are subscribed to the Google Groups

Re: Issue 2307 in sympy: Duplicate methods: as_coeff_mul and as_coeff_Mul

2011-05-27 Thread sympy
Comment #22 on issue 2307 by asmeurer: Duplicate methods: as_coeff_mul and as_coeff_Mul http://code.google.com/p/sympy/issues/detail?id=2307 I thing we should either keep both or remove as_coeff_lowercase and just require a call to .args in the second argument of as_coeff_Uppercase

Re: Issue 2338 in sympy: Use predicates and propositions instead of Assume objects

2011-05-27 Thread sympy
Updates: Labels: -NeedsReview Comment #12 on issue 2338 by ronan.l...@gmail.com: Use predicates and propositions instead of Assume objects http://code.google.com/p/sympy/issues/detail?id=2338 The pull request is in, but 'ask(x, Q.even)' still needs to be replaced with

Re: Issue 2411 in sympy: Revert _op_priority deletion

2011-05-27 Thread sympy
Updates: Labels: Ronan.Lamy asmeurer Comment #5 on issue 2411 by asmeurer: Revert _op_priority deletion http://code.google.com/p/sympy/issues/detail?id=2411 It seems that this will take longer to fix than we can afford before this release, so I have reverted the _op_priority removal.

Re: Issue 2433 in sympy: Polys module function poly() needs gens argument

2011-05-27 Thread sympy
Comment #6 on issue 2433 by matt...@gmail.com: Polys module function poly() needs gens argument http://code.google.com/p/sympy/issues/detail?id=2433 Because Poly is a constructor and it already does a lot. It can construct a polynomial from a Poly instance, an expression, a list, tuple or

Re: Issue 2433 in sympy: Polys module function poly() needs gens argument

2011-05-27 Thread sympy
Comment #7 on issue 2433 by asmeurer: Polys module function poly() needs gens argument http://code.google.com/p/sympy/issues/detail?id=2433 But that doesn't make sense. I don't see why Poly shouldn't be as smart as poly() regarding this. You have to use poly() unless you want things to

Re: Issue 2433 in sympy: Polys module function poly() needs gens argument

2011-05-27 Thread sympy
Updates: Labels: -Milestone-Release0.7.0 Milestone-Release0.7.1 Comment #8 on issue 2433 by asmeurer: Polys module function poly() needs gens argument http://code.google.com/p/sympy/issues/detail?id=2433 (No comment was entered for this change.) -- You received this message because

Re: Issue 879 in sympy: f(x).series(x, 0, 3) doesn't work

2011-05-27 Thread sympy
Updates: Blockedon: 1620 Comment #4 on issue 879 by renato.c...@gmail.com: f(x).series(x, 0, 3) doesn't work http://code.google.com/p/sympy/issues/detail?id=879 (No comment was entered for this change.) -- You received this message because you are subscribed to the Google Groups

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

2011-05-27 Thread sympy
Issue 1620: Allow derivatives of unknown functions evaluated at a point http://code.google.com/p/sympy/issues/detail?id=1620 This issue is now blocking issue 879. See http://code.google.com/p/sympy/issues/detail?id=879 -- You received this message because you are listed in the owner or CC

Re: Issue 2307 in sympy: Duplicate methods: as_coeff_mul and as_coeff_Mul

2011-05-27 Thread sympy
Comment #23 on issue 2307 by smi...@gmail.com: Duplicate methods: as_coeff_mul and as_coeff_Mul http://code.google.com/p/sympy/issues/detail?id=2307 It's a headache either way to do what the other is doing: to miimic as_coeff_mul() c, m = expr.as_coeff_Mul() margs = Mul.make_args(m)

Re: Issue 2307 in sympy: as_coeff_mul()[0] != as_coeff_Mul()[0] when the coeff is Number (not Rational)

2011-05-27 Thread sympy
Updates: Summary: as_coeff_mul()[0] != as_coeff_Mul()[0] when the coeff is Number (not Rational) Labels: -Priority-High -Milestone-Release0.7.0 -NeedsReview -smichr Priority-Medium Milestone-Release0.7.1 Comment #24 on issue 2307 by asmeurer: as_coeff_mul()[0] != as_coeff_Mul()[0] when