Comment #7 on issue 2014 by [email protected]: 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 in the diff.
For the others. Honestly sometimes I just tried if the tests still worked so I am not sure how to write good tests. I just used the chance because of comment #3.
To the part with hasattr and raise TypeError and similar. I agree, it's unpythonic. I just kept it and didn't change. I would either remove those or alternativly add better messages. Mostly the exceptions don't even show the problematic argument which would be better if the for statement itself raises the error. And when the errors are raised deep in the call stack, as a user it is often hard to understand the root cause. So to make it short, if there is an agreement to remove those kinds of TypeErrors or change them in something better I grep through the sources and change it everywhere.
-- You received this message because you are subscribed to the Google Groups "sympy-issues" group. To post to this group, send email to [email protected]. To unsubscribe from this group, send email to [email protected]. For more options, visit this group at http://groups.google.com/group/sympy-issues?hl=en.
