Status: Accepted
Owner:
Labels: Type-Defect Priority-High Milestone-Release0.6.4
New issue 1219 by ondrej.certik: .coeff() fails for -x/8 + x*y
http://code.google.com/p/sympy/issues/detail?id=1219
In [1]: p = -x/8 + x*y
In [2]: p.coeff(x)
Out[2]: 0
In [3]: p.coeff(-x)
Out[3]: -y
In [4]: p
Updates:
Blockedon: 1220
Comment #1 on issue 1219 by ondrej.certik: .coeff() fails for -x/8 + x*y
http://code.google.com/p/sympy/issues/detail?id=1219
(No comment was entered for this change.)
--
You received this message because you are listed in the owner
or CC fields of this issue,
Updates:
Blockedon: 1221
Comment #2 on issue 1219 by ondrej.certik: .coeff() fails for -x/8 + x*y
http://code.google.com/p/sympy/issues/detail?id=1219
The biggest problem is in Wild, see the issue 1220. Another minor problem
is in
collect, see the issue 1221.
--
You received this mes
Updates:
Status: Started
Comment #3 on issue 1219 by ondrej.certik: .coeff() fails for -x/8 + x*y
http://code.google.com/p/sympy/issues/detail?id=1219
After fixing the issue 1220, it will behave like this:
In [1]: p = -x/8 + x*y
In [2]: p.coeff(x)
Out[2]: -1/8 + y
In [3]: p.coeff(-x)
Updates:
Owner: ---
Labels: -Simplify
Comment #5 on issue 1219 by fabian.seoane: .coeff() fails for -x/8 + x*y
http://code.google.com/p/sympy/issues/detail?id=1219
(No comment was entered for this change.)
--
You received this message because you are listed in the owner
or CC fi
Updates:
Owner: fabian.seoane
Labels: Simplify
Comment #4 on issue 1219 by fabian.seoane: .coeff() fails for -x/8 + x*y
http://code.google.com/p/sympy/issues/detail?id=1219
(No comment was entered for this change.)
--
You received this message because you are listed in the owner
Updates:
Owner: fabian.seoane
Comment #6 on issue 1219 by fabian.seoane: .coeff() fails for -x/8 + x*y
http://code.google.com/p/sympy/issues/detail?id=1219
(No comment was entered for this change.)
--
You received this message because you are listed in the owner
or CC fields of this iss
Updates:
Labels: NeedsReview
Comment #7 on issue 1219 by fabian.seoane: .coeff() fails for -x/8 + x*y
http://code.google.com/p/sympy/issues/detail?id=1219
(No comment was entered for this change.)
--
You received this message because you are listed in the owner
or CC fields of this issu
Updates:
Cc: ondrej.certik
Comment #8 on issue 1219 by fabian.seoane: .coeff() fails for -x/8 + x*y
http://code.google.com/p/sympy/issues/detail?id=1219
I believe this is fixed because of fixing #1221. Ondrej, can you please
verify and
close the issue?
--
You received this message bec
Updates:
Owner: ---
Comment #9 on issue 1219 by fabian.seoane: .coeff() fails for -x/8 + x*y
http://code.google.com/p/sympy/issues/detail?id=1219
(No comment was entered for this change.)
--
You received this message because you are listed in the owner
or CC fields of this issue, or bec
Comment #10 on issue 1219 by ondrej.certik: .coeff() fails for -x/8 + x*y
http://code.google.com/p/sympy/issues/detail?id=1219
No, it is not fixed yet:
In [1]: p = -x/8 + x*y
In [2]: p.coeff(x)
Out[2]: 0
In [3]: p.coeff(-x)
Out[3]: 1/8 - y
In [4]: p
Out[4]:
x
- ─ + x⋅y
8
[3] is corre
Comment #11 on issue 1219 by fabian.seoane: .coeff() fails for -x/8 + x*y
http://code.google.com/p/sympy/issues/detail?id=1219
Strange, this is what i get:
In [1]: p = -x/8 + x*y
In [2]: p.coeff(x)
Out[2]: -1/8 + y
In [3]: p.coeff(-x)
Out[3]: 1/8 - y
--
You received this message because yo
Updates:
Cc: -ondrej.certik
Comment #12 on issue 1219 by ondrej.certik: .coeff() fails for -x/8 + x*y
http://code.google.com/p/sympy/issues/detail?id=1219
It is known to work on macosx, but fails on linux.
--
You received this message because you are listed in the owner
or CC fields of
Updates:
Labels: -NeedsReview
Comment #13 on issue 1219 by fabian.seoane: .coeff() fails for -x/8 + x*y
http://code.google.com/p/sympy/issues/detail?id=1219
(No comment was entered for this change.)
--
You received this message because you are listed in the owner
or CC fields of this is
Updates:
Owner: fabian.seoane
Comment #14 on issue 1219 by fabian.seoane: .coeff() fails for -x/8 + x*y
http://code.google.com/p/sympy/issues/detail?id=1219
(No comment was entered for this change.)
--
You received this message because you are listed in the owner
or CC fields of this is
Updates:
Cc: ondrej.certik
Comment #15 on issue 1219 by fabian.seoane: .coeff() fails for -x/8 + x*y
http://code.google.com/p/sympy/issues/detail?id=1219
I think this patch fixes it. Still a couple of things:
- there must be a better way to know if an expression is multiplied by a
Comment #16 on issue 1219 by fabian.seoane: .coeff() fails for -x/8 + x*y
http://code.google.com/p/sympy/issues/detail?id=1219
sorry, "implement coeff using the helper methods of coeff" means "implement
coeff
using the helper methods of collect", obviously
--
You received this message because
Updates:
Cc: -ondrej.certik
Comment #17 on issue 1219 by ondrej.certik: .coeff() fails for -x/8 + x*y
http://code.google.com/p/sympy/issues/detail?id=1219
This patch fixes it for me, great!
Yes, I know that collect is an overkill, but it works, that's important.
But we
should definite
Updates:
Labels: NeedsReview
Comment #18 on issue 1219 by fabian.seoane: .coeff() fails for -x/8 + x*y
http://code.google.com/p/sympy/issues/detail?id=1219
I've sent the patch for review, let's get this is and open issues for
optimizations
and other stuff.
--
You received this message
Updates:
Status: Fixed
Comment #19 on issue 1219 by fabian.seoane: .coeff() fails for -x/8 + x*y
http://code.google.com/p/sympy/issues/detail?id=1219
should be fixed in
http://git.sympy.org/?p=sympy.git;a=commit;h=9a3a37e5b06dcc62991183e04b22b2f1bff3ff67
--
You received this message bec
Updates:
Status: Verified
Comment #20 on issue 1219 by ondrej.certik: .coeff() fails for -x/8 + x*y
http://code.google.com/p/sympy/issues/detail?id=1219
Before:
In [1]: p = -x/8 + x*y
In [2]: p.coeff(x)
Out[2]: 0
In [3]: p.coeff(-x)
Out[3]: 1/8 - y
now:
In [1]: p = -x/8 + x*y
In [
21 matches
Mail list logo