Re: Issue 2014 in sympy: sympy.solvers.solvers.solve should also allow frozenset to hold equations

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

Re: Issue 2014 in sympy: sympy.solvers.solvers.solve should also allow frozenset to hold equations

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

Re: Issue 2014 in sympy: sympy.solvers.solvers.solve should also allow frozenset to hold equations

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

Re: Issue 2014 in sympy: sympy.solvers.solvers.solve should also allow frozenset to hold equations

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

Re: Issue 2014 in sympy: sympy.solvers.solvers.solve should also allow frozenset to hold equations

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

Re: Issue 2014 in sympy: sympy.solvers.solvers.solve should also allow frozenset to hold equations

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

Re: Issue 2014 in sympy: sympy.solvers.solvers.solve should also allow frozenset to hold equations

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

Re: Issue 2014 in sympy: sympy.solvers.solvers.solve should also allow frozenset to hold equations

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

Re: Issue 2014 in sympy: sympy.solvers.solvers.solve should also allow frozenset to hold equations

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))

Re: Issue 2014 in sympy: sympy.solvers.solvers.solve should also allow frozenset to hold equations

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

Re: Issue 2014 in sympy: sympy.solvers.solvers.solve should also allow frozenset to hold equations

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

Re: Issue 2014 in sympy: sympy.solvers.solvers.solve should also allow frozenset to hold equations

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,

Issue 2014 in sympy: sympy.solvers.solvers.solve should also allow frozenset to hold equations

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.

Re: Issue 2014 in sympy: sympy.solvers.solvers.solve should also allow frozenset to hold equations

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

Re: Issue 2014 in sympy: sympy.solvers.solvers.solve should also allow frozenset to hold equations

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