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

  * author:  => Leif Leonhardy


Comment:

 Replying to [comment:27 leif]:
 > Replying to [comment:26 burcin]:
 > > The doctests for using values that are taken to be True in Leif's
 patch added a few extra seconds to the time to test
 `sage/symbolic/relation.py`.
 >
 > :-) Adding {{{# optional}}} or {{{# long}}} would have been possible,
 too.
 >
 > (Now the tests of Python {{{int}}}s vs. Sage's {{{Integer}}}s are
 omitted.)

 Since they weren't testing the functionality of `solve()`, but a
 convenience in Python, I think it's OK if they are not so extensive. I run
 doctests on these files very often, so it's rather important that even the
 long tests take a reasonable time.

 Can you review my changes BTW?

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