Status: Accepted
Owner: nicolas.pourcelot
Labels: Type-Defect Priority-Medium
New issue 2011 by nicolas.pourcelot: less self-confident simplify
http://code.google.com/p/sympy/issues/detail?id=2011
There is some case where simplify result is much more complicated than its
input value.
In inte
Updates:
Labels: EasyToFix
Comment #1 on issue 2011 by nicolas.pourcelot: less self-confident simplify
http://code.google.com/p/sympy/issues/detail?id=2011
(No comment was entered for this change.)
--
You received this message because you are subscribed to the Google Groups
"sympy-issu
Updates:
Blockedon: 1923
Comment #2 on issue 2011 by nicolas.pourcelot: less self-confident simplify
http://code.google.com/p/sympy/issues/detail?id=2011
If no one else has done it before, I'll implement it once .count_ops()
modifications will be in.
See
https://github.com/smichr/s
Updates:
Blockedon: 2237
Comment #3 on issue 2011 by nicolas@gmail.com: less self-confident
simplify
http://code.google.com/p/sympy/issues/detail?id=2011
(No comment was entered for this change.)
--
You received this message because you are subscribed to the Google Groups
"sympy
Updates:
Cc: Vinzent.Steinberg asmeurer smi...@gmail.com ronan.l...@gmail.com
ondrej.c...@gmail.com
Labels: NeedsReview
Comment #4 on issue 2011 by nicolas@gmail.com: less self-confident
simplify
http://code.google.com/p/sympy/issues/detail?id=2011
Once issue 2237 is fixed, ev
Updates:
Labels: nicolas.pourcelot
Comment #5 on issue 2011 by asmeurer: less self-confident simplify
http://code.google.com/p/sympy/issues/detail?id=2011
(No comment was entered for this change.)
--
You received this message because you are subscribed to the Google Groups
"sympy-issue
Comment #6 on issue 2011 by Vinzent.Steinberg: less self-confident simplify
http://code.google.com/p/sympy/issues/detail?id=2011
Alas, physics-related failure is not the only one on my 64 bit system:
__
Comment #7 on issue 2011 by nicolas@gmail.com: less self-confident
simplify
http://code.google.com/p/sympy/issues/detail?id=2011
"Alas, physics-related failure is not the only one"
Indeed.
I manifestly forgot to rerun all the tests after last changes...
In [6]: srepr([y**(1/b)*(1/a)**(1
Comment #8 on issue 2011 by Vinzent.Steinberg: less self-confident simplify
http://code.google.com/p/sympy/issues/detail?id=2011
So, new result is clearly simpler than previous one, or did I miss
something ?
I was assuming the tests would pass for you. If you get the same failures,
it is i
Comment #9 on issue 2011 by nicolas@gmail.com: less self-confident
simplify
http://code.google.com/p/sympy/issues/detail?id=2011
I've updated tests.
https://github.com/wxgeo/sympy/tree/2011
--
You received this message because you are subscribed to the Google Groups
"sympy-issues" group
Comment #10 on issue 2011 by nicolas@gmail.com: less self-confident
simplify
http://code.google.com/p/sympy/issues/detail?id=2011
By the way, sympy/master seems to be currently broken (one test fails in
solvers).
--
You received this message because you are subscribed to the Google Gr
Comment #11 on issue 2011 by Vinzent.Steinberg: less self-confident simplify
http://code.google.com/p/sympy/issues/detail?id=2011
Nice, now they all pass for me except the physics test.
--
You received this message because you are subscribed to the Google Groups
"sympy-issues" group.
To post t
Comment #12 on issue 2011 by Vinzent.Steinberg: less self-confident simplify
http://code.google.com/p/sympy/issues/detail?id=2011
I don't get this failure with current master.
--
You received this message because you are subscribed to the Google Groups
"sympy-issues" group.
To post to this gro
Comment #13 on issue 2011 by nicolas@gmail.com: less self-confident
simplify
http://code.google.com/p/sympy/issues/detail?id=2011
Strange.
I add this:
nicolas@nicolas-laptop:~/Programmation/sympy$ git checkout origin/master
Note: checking out 'origin/master'.
You are in 'detached HEAD
Comment #14 on issue 2011 by nicolas@gmail.com: less self-confident
simplify
http://code.google.com/p/sympy/issues/detail?id=2011
Strange.
I have this:
nicolas@nicolas-laptop:~/Programmation/sympy$ git checkout origin/master
Note: checking out 'origin/master'.
You are in 'detached HEA
Comment #15 on issue 2011 by nicolas@gmail.com: less self-confident
simplify
http://code.google.com/p/sympy/issues/detail?id=2011
Maybe this depends on architecture, since you have a 64 bit system ??
--
You received this message because you are subscribed to the Google Groups
"sympy-iss
Comment #16 on issue 2011 by Vinzent.Steinberg: less self-confident simplify
http://code.google.com/p/sympy/issues/detail?id=2011
Maybe. I also tried Python 2.6, still no failures. Can you reproduce it
with a newer version of Python like 2.6.6 or 2.7?
--
You received this message because you
Comment #17 on issue 2011 by ronan.l...@gmail.com: less self-confident
simplify
http://code.google.com/p/sympy/issues/detail?id=2011
The failure is issue 2226 - and is unrelated to this issue.
--
You received this message because you are subscribed to the Google Groups
"sympy-issues" group.
Comment #18 on issue 2011 by nicolas@gmail.com: less self-confident
simplify
http://code.google.com/p/sympy/issues/detail?id=2011
"and is unrelated to this issue"
However, as a side effect, this failure does not occur anymore in my
branch, since a new possible result was added in test_
Comment #19 on issue 2011 by nicolas@gmail.com: less self-confident
simplify
http://code.google.com/p/sympy/issues/detail?id=2011
All tests pass now in https://github.com/wxgeo/sympy/tree/2011.
--
You received this message because you are subscribed to the Google Groups
"sympy-issues"
Updates:
Labels: -NeedsReview PassedReview
Comment #20 on issue 2011 by Vinzent.Steinberg: less self-confident simplify
http://code.google.com/p/sympy/issues/detail?id=2011
Looks fine to me now, all tests pass.
--
You received this message because you are subscribed to the Google Groups
Comment #21 on issue 2011 by nicolas@gmail.com: less self-confident
simplify
http://code.google.com/p/sympy/issues/detail?id=2011
This does not pass anymore:
___ sympy/matrices/tests/test_matrices.py:te
Updates:
Labels: -PassedReview NeedsReview
Comment #22 on issue 2011 by nicolas@gmail.com: less self-confident
simplify
http://code.google.com/p/sympy/issues/detail?id=2011
(No comment was entered for this change.)
--
You received this message because you are subscribed to the Go
Comment #23 on issue 2011 by Vinzent.Steinberg: less self-confident simplify
http://code.google.com/p/sympy/issues/detail?id=2011
Hmm, I pushed in some matrix related fixes today, it might be this.
--
You received this message because you are subscribed to the Google Groups
"sympy-issues" grou
Comment #24 on issue 2011 by nicolas@gmail.com: less self-confident
simplify
http://code.google.com/p/sympy/issues/detail?id=2011
This is the code in Matrix.is_symmetric():
def symmetric(self):
if not self.is_square:
return False
m = self.clone()
m
Comment #25 on issue 2011 by asmeurer: less self-confident simplify
http://code.google.com/p/sympy/issues/detail?id=2011
You can also use git bisect to tell what commit caused a problem.
--
You received this message because you are subscribed to the Google Groups
"sympy-issues" group.
To post
Comment #26 on issue 2011 by nicolas@gmail.com: less self-confident
simplify
http://code.google.com/p/sympy/issues/detail?id=2011
Thanks for the advice, I'll use it next time.
--
You received this message because you are subscribed to the Google Groups
"sympy-issues" group.
To post to t
Comment #27 on issue 2011 by nicolas@gmail.com: less self-confident
simplify
http://code.google.com/p/sympy/issues/detail?id=2011
Pull request added: https://github.com/sympy/sympy/pull/185
--
You received this message because you are subscribed to the Google Groups
"sympy-issues" group
Comment #28 on issue 2011 by nicolas@gmail.com: less self-confident
simplify
http://code.google.com/p/sympy/issues/detail?id=2011
(Now that Matrix.is_symmetric() is improved, all tests pass again.)
--
You received this message because you are subscribed to the Google Groups
"sympy-issue
Comment #29 on issue 2011 by nicolas@gmail.com: less self-confident
simplify
http://code.google.com/p/sympy/issues/detail?id=2011
Vinzent, are you ok with Matrix.is_symmetric() change ?
May I push it (after retesting it) ?
--
You received this message because you are subscribed to the Go
Comment #30 on issue 2011 by Vinzent.Steinberg: less self-confident simplify
http://code.google.com/p/sympy/issues/detail?id=2011
See the pull request.
--
You received this message because you are subscribed to the Google Groups
"sympy-issues" group.
To post to this group, send email to sympy-
Comment #31 on issue 2011 by nicolas@gmail.com: less self-confident
simplify
http://code.google.com/p/sympy/issues/detail?id=2011
While testing, I found another failure in master with my configuration. I
opened issue 2258.
--
You received this message because you are subscribed to the
Updates:
Status: Fixed
Labels: -NeedsReview PassedReview
Comment #32 on issue 2011 by nicolas@gmail.com: less self-confident
simplify
http://code.google.com/p/sympy/issues/detail?id=2011
(No comment was entered for this change.)
--
You received this message because you ar
33 matches
Mail list logo