Issue 1828 in sympy: Visual factorint()

2010-02-11 Thread sympy
Status: Accepted Owner: asmeurer Labels: Type-Defect Priority-Medium NeedsReview New issue 1828 by asmeurer: Visual factorint() http://code.google.com/p/sympy/issues/detail?id=1828 Add visual option to factorint() This lets you get a visual factorization of an integer, which is much easier to r

Re: Issue 1828 in sympy: Visual factorint()

2010-02-13 Thread sympy
Comment #1 on issue 1828 by smichr: Visual factorint() http://code.google.com/p/sympy/issues/detail?id=1828 +1 This seems like a good option to have and although I tried to think of other cases where one would want this, none come to mind. So perhaps it's unique to factorint. I would recomme

Re: Issue 1828 in sympy: Visual factorint()

2010-02-13 Thread sympy
Comment #2 on issue 1828 by asmeurer: Visual factorint() http://code.google.com/p/sympy/issues/detail?id=1828 OK, I repushed with the suggestion you made. The way I had it before, I wanted it to be clear that they both returned the same thing, but perhaps your ordering is better. I can't th

Re: Issue 1828 in sympy: Visual factorint()

2010-02-15 Thread sympy
Updates: Status: Fixed Labels: -NeedsReview Comment #3 on issue 1828 by Vinzent.Steinberg: Visual factorint() http://code.google.com/p/sympy/issues/detail?id=1828 Thank you, it is in. -- You received this message because you are listed in the owner or CC fields of this issue, or

Re: Issue 1828 in sympy: Visual factorint()

2010-02-20 Thread sympy
Updates: Status: Started Cc: Vinzent.Steinberg Labels: -Priority-Medium Priority-Critical NeedsReview Comment #4 on issue 1828 by asmeurer: Visual factorint() http://code.google.com/p/sympy/issues/detail?id=1828 I forgot to test this in Python2.5/2.4, which apparently doe

Re: Issue 1828 in sympy: Visual factorint()

2010-02-26 Thread sympy
Updates: Status: Fixed Labels: -NeedsReview PassedReview Comment #5 on issue 1828 by asmeurer: Visual factorint() http://code.google.com/p/sympy/issues/detail?id=1828 Chris reviewed on a comment at github, and I pushed it in. -- You received this message because you are listed i