#10750: Fix solve so that additional args are properly handled
-------------------------+--------------------------------------------------
Reporter: kcrisman | Owner: burcin
Type: defect | Status: needs_review
Priority: major | Milestone:
Component: symbolics | Keywords: solve, maxima, Maxima
Work_issues: | Upstream: N/A
Reviewer: | Author: John Palmieri
Merged: | Dependencies:
-------------------------+--------------------------------------------------
Comment(by kcrisman):
> I'm marking as this as "needs review", but it may just be a first draft.
I certainly haven't tested it very thoroughly.
I think you changed the previous test, which is fine. But we should catch
the `solution_dict` issue as well (that is to say we should test that that
error no longer occurs).
To test this, one should especially try to make things which previously
would have gone into `Expression.solve` break. For instance, we would
want to make sure multiplicities still could work as well as possible. I
think your code will keep that, but it's what would want to be checked. I
do not think I will have time to do that right now, sadly.
Someone would also want to just check whether this is enough to close
#10444 as a dup, which jhpalmieri is suggesting and I suspect is true.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/10750#comment:10>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica,
and MATLAB
--
You received this message because you are subscribed to the Google Groups
"sage-trac" 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/sage-trac?hl=en.
