#10187: Update ecl and maxima
----------------------------+-----------------------------------------------
   Reporter:  vbraun        |       Owner:  tbd                                 
    
       Type:  defect        |      Status:  needs_review                        
    
   Priority:  major         |   Milestone:  sage-4.6.1                          
    
  Component:  packages      |    Keywords:                                      
    
     Author:  Volker Braun  |    Upstream:  Workaround found; Bug reported 
upstream.
   Reviewer:                |      Merged:                                      
    
Work_issues:                |  
----------------------------+-----------------------------------------------

Comment(by aginiewicz):

 I looked trough doctest patch and compared it with mine from #8731, I have
 few comments.

 I think that doctest fix for sage/calculus/functional.py should be
 modified - it was documentation of the use of assume, but now - when
 maxima can calculate this integral without assumption, isn't that same as
 applying assumption to result? And the integral is checked in other
 doctest now. Maybe there is better example that still needs the assume to
 get result?

 Also the doctest in sage/misc/functional.py was here to demonstrate
 numerical approximation of integral that cannot be evaluated iirc, and now
 when it can be evaluated, I think it should be changed to something that
 cannot be - like Jason did in patch in #8731.

 Finally, there is typo in doctest to sage/symbolic/expression.pyx - it
 should say that this doctest is here to check that #7334 not #7344 is
 fixed (#7344 point to libjpeg issue so it cannot be it) - and it should
 stay in here unmodified. Also - the log thing - it's not exactly
 regression, it's feature. This is because of Maxima ticket 947808 -
 
http://sourceforge.net/tracker/?func=detail&aid=947808&group_id=4933&atid=104933
 - they now try to keep the expression as factored as they can without
 using factor. This results in observed behaviour. This doctest I think
 cannot be changed as it demonstrated that ticket is solved, but we should
 apply "x = x.simplify_rational()" after "x = x.simplify_log('one')" in
 definition of simplify_full - that way full simplify would result in same
 results as before I think, and #7334 would be still fixed. Now, this
 doctest shows nothing.

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