Issue 2361 in sympy: Mul.flatten needs to watch for the unevaluated Mul

2011-05-09 Thread sympy
Status: Accepted Owner: smi...@gmail.com Labels: Type-Defect Priority-Medium New issue 2361 by smi...@gmail.com: Mul.flatten needs to watch for the unevaluated Mul http://code.google.com/p/sympy/issues/detail?id=2361 1>> 9*x/(4*(1 - y)) 9*x/(4 - 4*y) 2>> d = Mul(4, 1 - y, evaluate

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

2011-05-09 Thread sympy
Comment #4 on issue 2307 by asmeurer: Duplicate methods: as_coeff_mul and as_coeff_Mul http://code.google.com/p/sympy/issues/detail?id=2307 If you want to see it, see https://github.com/asmeurer/sympy/tree/as_coeff_Mul-remove. I have done everything for the merge (I converted as_coeff_Mu

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

2011-05-09 Thread sympy
Updates: Cc: smi...@gmail.com Comment #3 on issue 2307 by asmeurer: Duplicate methods: as_coeff_mul and as_coeff_Mul http://code.google.com/p/sympy/issues/detail?id=2307 There's a difference with how the handle Real coefficients: In [1]: (4.0*x).as_coeff_mul() Out[1]: (1, (4.0, x))

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

2011-05-09 Thread sympy
Comment #2 on issue 2307 by asmeurer: Duplicate methods: as_coeff_mul and as_coeff_Mul http://code.google.com/p/sympy/issues/detail?id=2307 as_coeff_mul() is used all over the place. as_coeff_Mul() is only used in a few places. The docstring of as_coeff_Mul only says that it is there to

Re: Issue 2306 in sympy: Duplicate implementation of factorial in sympy/core/numbers.py

2011-05-09 Thread sympy
Updates: Labels: Polynomial Comment #3 on issue 2306 by asmeurer: Duplicate implementation of factorial in sympy/core/numbers.py http://code.google.com/p/sympy/issues/detail?id=2306 (No comment was entered for this change.) -- You received this message because you are subscribed to t

Re: Issue 2306 in sympy: Duplicate implementation of factorial in sympy/core/numbers.py

2011-05-09 Thread sympy
Comment #2 on issue 2306 by asmeurer: Duplicate implementation of factorial in sympy/core/numbers.py http://code.google.com/p/sympy/issues/detail?id=2306 You can try it at https://github.com/asmeurer/sympy/tree/ifactorial-remove. -- You received this message because you are subscribed to the

Re: Issue 2306 in sympy: Duplicate implementation of factorial in sympy/core/numbers.py

2011-05-09 Thread sympy
Comment #1 on issue 2306 by asmeurer: Duplicate implementation of factorial in sympy/core/numbers.py http://code.google.com/p/sympy/issues/detail?id=2306 I can't figure out how to import sympy.functions.combintorial.factorials.factorial in sympy/polys/domains/groundtypes.py without gettin

Re: Issue 1721 in sympy: Rename class 'Real'

2011-05-09 Thread sympy
Comment #9 on issue 1721 by asmeurer: Rename class 'Real' http://code.google.com/p/sympy/issues/detail?id=1721 Ronan, can you redo this now? -- You received this message because you are subscribed to the Google Groups "sympy-issues" group. To post to this group, send email to sympy-issues@goog

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

2011-05-09 Thread sympy
Comment #1 on issue 2304 by asmeurer: Remove Pure from top-level namespace http://code.google.com/p/sympy/issues/detail?id=2304 So is the issue just to remove Pure from __init__.py? -- You received this message because you are subscribed to the Google Groups "sympy-issues" group. To post to th

Re: Issue 2303 in sympy: Function to access the oeis database

2011-05-09 Thread sympy
Comment #16 on issue 2303 by asmeurer: Function to access the oeis database http://code.google.com/p/sympy/issues/detail?id=2303 I like Vinzent's idea. Have it return the string by default, and oeis(seq, open=True) will open it in the browser. -- You received this message because you are su

Re: Issue 1735 in sympy: Rename .func attribute (.args too?)

2011-05-09 Thread sympy
Comment #26 on issue 1735 by asmeurer: Rename .func attribute (.args too?) http://code.google.com/p/sympy/issues/detail?id=1735 What needs to be done here? -- You received this message because you are subscribed to the Google Groups "sympy-issues" group. To post to this group, send email to sy

Re: Issue 543 in sympy: warnings on python setup.py build

2011-05-09 Thread sympy
Comment #2 on issue 543 by asmeurer: warnings on python setup.py build http://code.google.com/p/sympy/issues/detail?id=543 This is still an issue: package init file 'sympy/assumptions/tests/__init__.py' not found (or not a regular file) package init file 'sympy/concrete/tests/__init__.py' not

Re: Issue 533 in sympy: credits: use alphabetical order + automatic statistics

2011-05-09 Thread sympy
Updates: Status: WontFix Comment #14 on issue 533 by asmeurer: credits: use alphabetical order + automatic statistics http://code.google.com/p/sympy/issues/detail?id=533 Not sure how I feel about this. I like having the authors in order in the README. And the statistics are very u

Re: Issue 2360 in sympy: Bug in geometry intersection

2011-05-09 Thread sympy
Comment #2 on issue 2360 by asmeurer: Bug in geometry intersection http://code.google.com/p/sympy/issues/detail?id=2360 See https://github.com/sympy/sympy/pull/298. -- You received this message because you are subscribed to the Google Groups "sympy-issues" group. To post to this group, send em

Re: Issue 2360 in sympy: Bug in geometry intersection

2011-05-09 Thread sympy
Issue 2360: Bug in geometry intersection http://code.google.com/p/sympy/issues/detail?id=2360 This issue is now blocking issue 2203. See http://code.google.com/p/sympy/issues/detail?id=2203 -- You received this message because you are listed in the owner or CC fields of this issue, or because yo

Re: Issue 2203 in sympy: doctest does not test IPython interactive sessions

2011-05-09 Thread sympy
Updates: Labels: -PassedReview NeedsReview asmeurer Blockedon: 2360 Comment #9 on issue 2203 by asmeurer: doctest does not test IPython interactive sessions http://code.google.com/p/sympy/issues/detail?id=2203 I have fixed the rest at https://github.com/sympy/sympy/pull/298.

Re: Issue 2217 in sympy: Patches tutorial

2011-05-09 Thread sympy
Updates: Labels: NeedsReview asmeurer Comment #26 on issue 2217 by asmeurer: Patches tutorial http://code.google.com/p/sympy/issues/detail?id=2217 I have a patch removing the patches tutorial from the docs at https://github.com/sympy/sympy/pull/298. -- You received this message becau

Re: Issue 2358 in sympy: keep_sign should be cached

2011-05-09 Thread sympy
Updates: Status: WontFix Comment #3 on issue 2358 by ronan.l...@gmail.com: keep_sign should be cached http://code.google.com/p/sympy/issues/detail?id=2358 Issue 2151 has just been fixed, so this is now irrelevant. -- You received this message because you are subscribed to the Google Gro

Re: Issue 2151 in sympy: BasicMeta.keep_sign

2011-05-09 Thread sympy
Updates: Status: Fixed Comment #20 on issue 2151 by asmeurer: BasicMeta.keep_sign http://code.google.com/p/sympy/issues/detail?id=2151 It's in. -- You received this message because you are subscribed to the Google Groups "sympy-issues" group. To post to this group, send email to sympy-

Issue 2360 in sympy: Bug in geometry intersection

2011-05-09 Thread sympy
Status: Accepted Owner: asmeurer Labels: Type-Defect Priority-Medium Geometry WrongResult New issue 2360 by asmeurer: Bug in geometry intersection http://code.google.com/p/sympy/issues/detail?id=2360 I found a bug when fixing issue 2203. There is a doctest in doc/src/modules/geometry.txt that

Re: Issue 2151 in sympy: BasicMeta.keep_sign

2011-05-09 Thread sympy
Updates: Labels: NeedsReview mattpap Ronan.Lamy Comment #19 on issue 2151 by asmeurer: BasicMeta.keep_sign http://code.google.com/p/sympy/issues/detail?id=2151 It doesn't look like this was ever linked to here. The fix is at https://github.com/sympy/sympy/pull/296 -- You received th

Re: Issue 1576 in sympy: Integral of strictly positive function is zero

2011-05-09 Thread sympy
Updates: Labels: NeedsReview asmeurer Comment #10 on issue 1576 by asmeurer: Integral of strictly positive function is zero http://code.google.com/p/sympy/issues/detail?id=1576 See https://github.com/sympy/sympy/pull/297. -- You received this message because you are subscribed to the

Re: Issue 2150 in sympy: integrate gives wrong result

2011-05-09 Thread sympy
Updates: Labels: NeedsReview asmeurer Comment #7 on issue 2150 by asmeurer: integrate gives wrong result http://code.google.com/p/sympy/issues/detail?id=2150 The fix is at https://github.com/sympy/sympy/pull/297. -- You received this message because you are subscribed to the Google Grou

Re: Issue 1892 in sympy: integrate(1/(x*(a+b*x)**3), x) fails

2011-05-09 Thread sympy
Updates: Labels: NeedsReview asmeurer Comment #9 on issue 1892 by asmeurer: integrate(1/(x*(a+b*x)**3), x) fails http://code.google.com/p/sympy/issues/detail?id=1892 My patch from issue 1793 unfortunately makes the output of this a little more complicated, but it is still correct: In

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

2011-05-09 Thread sympy
Status: Accepted Owner: asmeurer Labels: Type-Defect Priority-Medium Printing New issue 2359 by asmeurer: Should Pow(expr, S(1)/2) print as sqrt(expr) http://code.google.com/p/sympy/issues/detail?id=2359 It's annoying to have sqrt(expr) print as expr**(1/2), because have to always convert that

Re: Issue 2150 in sympy: integrate gives wrong result

2011-05-09 Thread sympy
Comment #6 on issue 2150 by asmeurer: integrate gives wrong result http://code.google.com/p/sympy/issues/detail?id=2150 With Mateusz's auto keyword to div() commit, this now returns a result. Unfortunately, it is wrong! (in master) In [1]: integrate(1/(x**2 + y**2), x) Out[1]: ⎛ⅈ⋅y

Re: Issue 1077 in sympy: ask(pi, exp(rational), etc., Q.algebraic) should return False

2011-05-09 Thread sympy
Comment #14 on issue 1077 by christian.muise: ask(pi, exp(rational), etc., Q.algebraic) should return False http://code.google.com/p/sympy/issues/detail?id=1077 Well if anyone is putting in changes to the new assumption system, it can be done as follows: If new predicate is being introduc

Re: Issue 1028 in sympy: Sum().doit() doesn't do it

2011-05-09 Thread sympy
Comment #17 on issue 1028 by asmeurer: Sum().doit() doesn't do it http://code.google.com/p/sympy/issues/detail?id=1028 You are correct in that the summation algorithms are similar to the integration algorithms (for example, the Karr algorithm has been called the counterpart to the Risch algo

Re: Issue 1077 in sympy: ask(pi, exp(rational), etc., Q.algebraic) should return False

2011-05-09 Thread sympy
Comment #13 on issue 1077 by asmeurer: ask(pi, exp(rational), etc., Q.algebraic) should return False http://code.google.com/p/sympy/issues/detail?id=1077 Yeah. Adding those predicates should make some trivial ones work, like ask(oo, Q.algebraic). And we should add Q.transcendental which

Re: Issue 2217 in sympy: Patches tutorial

2011-05-09 Thread sympy
Comment #25 on issue 2217 by asmeurer: Patches tutorial http://code.google.com/p/sympy/issues/detail?id=2217 That's a bug with the GitHub wiki. We are just using the ".. contents:: Table of Contents" reStructuredText directive. -- You received this message because you are subscribed to the

Re: Issue 2357 in sympy: bugs in PythonRationalType

2011-05-09 Thread sympy
Updates: Status: Accepted Labels: Polynomial Comment #2 on issue 2357 by asmeurer: bugs in PythonRationalType http://code.google.com/p/sympy/issues/detail?id=2357 Another problem with PythonRationalType: In [16]: S(PythonRationalType(2, 3)) Out[16]: 0.667 It should

Re: Issue 2354 in sympy: isympy -o doesn't parse bad arguments correctly

2011-05-09 Thread sympy
Comment #5 on issue 2354 by asmeurer: isympy -o doesn't parse bad arguments correctly http://code.google.com/p/sympy/issues/detail?id=2354 I gues what I meant wasy, why don't the monomials support reverse orderings? -- You received this message because you are subscribed to the Google Groups

Re: Issue 1028 in sympy: Sum().doit() doesn't do it

2011-05-09 Thread sympy
Comment #16 on issue 1028 by skr...@gmail.com: Sum().doit() doesn't do it http://code.google.com/p/sympy/issues/detail?id=1028 OK, I will use the issue tracker then. Oh, I assumed integration by parts was implemented . -- You received this message because you are subscribed to the Google Group

Re: Issue 1077 in sympy: ask(pi, exp(rational), etc., Q.algebraic) should return False

2011-05-09 Thread sympy
Comment #12 on issue 1077 by christian.muise: ask(pi, exp(rational), etc., Q.algebraic) should return False http://code.google.com/p/sympy/issues/detail?id=1077 This related to issue 2019? -- You received this message because you are subscribed to the Google Groups "sympy-issues" group. To

Re: Issue 1077 in sympy: ask(pi, exp(rational), etc., Q.algebraic) should return False

2011-05-09 Thread sympy
Comment #11 on issue 1077 by christian.muise: ask(pi, exp(rational), etc., Q.algebraic) should return False http://code.google.com/p/sympy/issues/detail?id=1077 This related to Issue 2019? - http://code.google.com/p/sympy/issues/detail?id=2019 -- You received this message because you are sub

Re: Issue 1028 in sympy: Sum().doit() doesn't do it

2011-05-09 Thread sympy
Comment #15 on issue 1028 by asmeurer: Sum().doit() doesn't do it http://code.google.com/p/sympy/issues/detail?id=1028 Feel free to edit the wiki however you like, though be aware that if you want something to never be lost, the issue tracker is the best place to put it. By the way, what l

Re: Issue 2355 in sympy: Should we .gitignore .patch and .rej files?

2011-05-09 Thread sympy
Comment #3 on issue 2355 by matt...@gmail.com: Should we .gitignore .patch and .rej files? http://code.google.com/p/sympy/issues/detail?id=2355 tags -> vim / ctags dist -> setup.py dist my -> ;) -- You received this message because you are subscribed to the Google Groups "sympy-issues" gr

Re: Issue 1077 in sympy: ask(pi, exp(rational), etc., Q.algebraic) should return False

2011-05-09 Thread sympy
Updates: Summary: ask(pi, exp(rational), etc., Q.algebraic) should return False Labels: Polynomial Assumptions Comment #10 on issue 1077 by asmeurer: ask(pi, exp(rational), etc., Q.algebraic) should return False http://code.google.com/p/sympy/issues/detail?id=1077 From the pul

Re: Issue 2355 in sympy: Should we .gitignore .patch and .rej files?

2011-05-09 Thread sympy
Comment #2 on issue 2355 by asmeurer: Should we .gitignore .patch and .rej files? http://code.google.com/p/sympy/issues/detail?id=2355 To me, anything in .gitignore should be something that you wouldn't mind deleting with ./setup.py clean or make clean (i.e., they can just be regenerated)

Re: Issue 2217 in sympy: Patches tutorial

2011-05-09 Thread sympy
Comment #24 on issue 2217 by tlora...@gmail.com: Patches tutorial http://code.google.com/p/sympy/issues/detail?id=2217 Links in the table of contents (https://github.com/sympy/sympy/wiki/Development-workflow) don't work. -- You received this message because you are subscribed to the Google G

Re: Issue 2354 in sympy: isympy -o doesn't parse bad arguments correctly

2011-05-09 Thread sympy
Updates: Status: Started Owner: matt...@gmail.com Cc: -matt...@gmail.com Labels: mattpap Comment #4 on issue 2354 by matt...@gmail.com: isympy -o doesn't parse bad arguments correctly http://code.google.com/p/sympy/issues/detail?id=2354 Yes, it should be done b

Re: Issue 2357 in sympy: bugs in PythonRationalType

2011-05-09 Thread sympy
Comment #1 on issue 2357 by matt...@gmail.com: bugs in PythonRationalType http://code.google.com/p/sympy/issues/detail?id=2357 Several bugs mentioned here are actually design decisions, e.g.: print PythonRationalType(3) 3/1 PythonRationalType is meant for internal purpose of sympy.polys, not

Re: Issue 2352 in sympy: Remove Ellipse.distance_to_center()

2011-05-09 Thread sympy
Comment #4 on issue 2352 by smi...@gmail.com: Remove Ellipse.distance_to_center() http://code.google.com/p/sympy/issues/detail?id=2352 If someone is going to do more than a 1-off calculation they should really have the formula and just do the subs, I suppose. I'm ok with removal. -- You r

Re: Issue 2352 in sympy: Remove Ellipse.distance_to_center()

2011-05-09 Thread sympy
Comment #3 on issue 2352 by ronan.l...@gmail.com: Remove Ellipse.distance_to_center() http://code.google.com/p/sympy/issues/detail?id=2352 But how can anybody understand that e.distance_to_center(1) means that, unless they know what the translation is? That operation is so specialised tha

Re: Issue 2352 in sympy: Remove Ellipse.distance_to_center()

2011-05-09 Thread sympy
Comment #2 on issue 2352 by smi...@gmail.com: Remove Ellipse.distance_to_center() http://code.google.com/p/sympy/issues/detail?id=2352 Here is a direct comparison: h[4] >>> e Ellipse(Point(1, 2), 3, 4) >>> t = Symbol('t') >>> Point.distance(e.center,e.arbitrary_point(t.name)

Re: Issue 2358 in sympy: keep_sign should be cached

2011-05-09 Thread sympy
Comment #1 on issue 2358 by smi...@gmail.com: keep_sign should be cached http://code.google.com/p/sympy/issues/detail?id=2358 see also issue 2151 -- You received this message because you are subscribed to the Google Groups "sympy-issues" group. To post to this group, send email to sympy-issues

Issue 2358 in sympy: keep_sign should be cached

2011-05-09 Thread sympy
Status: Accepted Owner: smi...@gmail.com Labels: Type-Defect Priority-Medium NeedsReview smichr New issue 2358 by smi...@gmail.com: keep_sign should be cached http://code.google.com/p/sympy/issues/detail?id=2358 see http://groups.google.com/group/sympy/t/a9b5fa5d37795473 The change discussed th

Issue 2357 in sympy: bugs in PythonRationalType

2011-05-09 Thread sympy
Status: New Owner: Labels: Type-Defect Priority-Medium New issue 2357 by mario.pe...@gmail.com: bugs in PythonRationalType http://code.google.com/p/sympy/issues/detail?id=2357 Hi, there are a few bugs in PythonRationalType from sympy import * from sympy.polys.domains.pythonrationaltype im

Re: Issue 1028 in sympy: Sum().doit() doesn't do it

2011-05-09 Thread sympy
Comment #14 on issue 1028 by skr...@gmail.com: Sum().doit() doesn't do it http://code.google.com/p/sympy/issues/detail?id=1028 I already submitted some changes. I think one alternative is the one you mention, to have some heuristics, and is the most doable in th short term. Other alternative