Status: New
Owner:
Labels: Type-Defect Priority-Medium
New issue 2014 by torstenm...@googlemail.com: sympy.solvers.solvers.solve
should also allow frozenset to hold equations
http://code.google.com/p/sympy/issues/detail?id=2014
Currently only tuple, list and set seem to be allowed.
Patc
Comment #1 on issue 2014 by jensen.oyvind: sympy.solvers.solvers.solve
should also allow frozenset to hold equations
http://code.google.com/p/sympy/issues/detail?id=2014
sympy.core.Tuple would also be very useful. Then one could do
eqs = Tuple(x + y, x - 3)
eqs.subs(x, z)
Tuple(z + y, z -
Updates:
Status: Accepted
Labels: Solvers
Comment #2 on issue 2014 by asmeurer: sympy.solvers.solvers.solve should
also allow frozenset to hold equations
http://code.google.com/p/sympy/issues/detail?id=2014
Tuple should be a different issue.
So solve() (and really, any sympy
Comment #3 on issue 2014 by mattpap: sympy.solvers.solvers.solve should
also allow frozenset to hold equations
http://code.google.com/p/sympy/issues/detail?id=2014
This is probably a leftover from the times when expressions were iterable.
It should be sufficient to use hasattr(something, '_
Comment #4 on issue 2014 by torstenm...@googlemail.com:
sympy.solvers.solvers.solve should also allow frozenset to hold equations
http://code.google.com/p/sympy/issues/detail?id=2014
Sorry for the delay. Attached is the diff file replacing most of the
remaining isinstance(x,(list, tuple)) a
Updates:
Labels: NeedsReview
Comment #5 on issue 2014 by asmeurer: sympy.solvers.solvers.solve should
also allow frozenset to hold equations
http://code.google.com/p/sympy/issues/detail?id=2014
You can fix the test_whitespace failure using the strip_whitespace utility
in the bin dir
Updates:
Labels: -NeedsReview NeedsBetterPatch
Comment #6 on issue 2014 by asmeurer: sympy.solvers.solvers.solve should
also allow frozenset to hold equations
http://code.google.com/p/sympy/issues/detail?id=2014
You should add tests. There at least needs to be a test for frozenset
Comment #7 on issue 2014 by torstenm...@googlemail.com:
sympy.solvers.solvers.solve should also allow frozenset to hold equations
http://code.google.com/p/sympy/issues/detail?id=2014
The patch already contains the additional tests for solve for lists, sets
and frozensets. See line 253-261 i
Comment #8 on issue 2014 by asmeurer: sympy.solvers.solvers.solve should
also allow frozenset to hold equations
http://code.google.com/p/sympy/issues/detail?id=2014
I see, you just put it as a doctest. I would prefer to see them as regular
tests (sympy/solvers/tests/test_solve.py), since y
Comment #9 on issue 2014 by torstenm...@googlemail.com:
sympy.solvers.solvers.solve should also allow frozenset to hold equations
http://code.google.com/p/sympy/issues/detail?id=2014
As no one disagreed, I prepare a combined patch for allowing all iterables
and removing TypeError for the ca
Comment #10 on issue 2014 by torstenm...@googlemail.com:
sympy.solvers.solvers.solve should also allow frozenset to hold equations
http://code.google.com/p/sympy/issues/detail?id=2014
ChangeLog:
- Rebased patch to current origin master
- Converted doctests to regular tesets
- Removed "raise T
Updates:
Labels: -NeedsBetterPatch NeedsReview torstenmarcoknodt
Comment #11 on issue 2014 by asmeurer: sympy.solvers.solvers.solve should
also allow frozenset to hold equations
http://code.google.com/p/sympy/issues/detail?id=2014
(No comment was entered for this change.)
--
You rece
Comment #12 on issue 2014 by smichr: sympy.solvers.solvers.solve should
also allow frozenset to hold equations
http://code.google.com/p/sympy/issues/detail?id=2014
I'm +1/2 with comments below. I have often thought that listing all
iterable types was kind of a clunky way to test for iterabi
Comment #13 on issue 2014 by smichr: sympy.solvers.solvers.solve should
also allow frozenset to hold equations
http://code.google.com/p/sympy/issues/detail?id=2014
I was favorable to the direction of the changes. Is any more going to be
done on this?
--
You received this message because y
Comment #14 on issue 2014 by smi...@gmail.com: sympy.solvers.solvers.solve
should also allow frozenset to hold equations
http://code.google.com/p/sympy/issues/detail?id=2014
A check can be done to see if any of the instances corrected in the patches
still remain and could use the ordered_it
15 matches
Mail list logo