#8553: Minor bugs in solve() if dictionary result requested
----------------------------------------------------------+-----------------
   Reporter:  leif                                        |       Owner:  leif  
              
       Type:  defect                                      |      Status:  
needs_review        
   Priority:  trivial                                     |   Milestone:  
sage-4.4            
  Component:  symbolics                                   |    Keywords:  
solve, solution_dict
     Author:                                              |    Upstream:  N/A   
              
   Reviewer:  Harald Schilly, Mike Hansen, Burcin Erocal  |      Merged:        
              
Work_issues:                                              |  
----------------------------------------------------------+-----------------
Changes (by leif):

 * cc: burcin (added)
  * status:  needs_work => needs_review


Old description:

> solve() raises an index error on empty solutions if solution dictionary
> was requested; returns empty list instead of empty dictionary in other
> cases.

New description:

 {{{solve()}}} raises an {{{IndexError}}} on empty solutions if a list of
 solution ''dictionaries'' is requested.

 The description/examples may mislead w.r.t. the result type.

--

Comment:

 Sorry, unable to delete the previous patches; '''apply only the last
 one'''.

  * {{{IndexError}}} issue fixed.
  * Type of parameter {{{solution_dict}}} relaxed as suggested by Mike
 Hansen (now consistent; also in
 {{{expression.pyx}}}/{{{Expression.solve()}}}).
  * (More) Doctests added.
  * Description made more clear. Some typos fixed.

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/8553#comment:23>
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.

Reply via email to