#10434: add doctests from #8582 and other integration improvements from Maxima
5.22.1
-----------------------------------+----------------------------------------
   Reporter:  mvngu                |       Owner:  mvngu          
       Type:  enhancement          |      Status:  positive_review
   Priority:  major                |   Milestone:  sage-4.6.1     
  Component:  documentation        |    Keywords:                 
     Author:  Minh Van Nguyen      |    Upstream:  N/A            
   Reviewer:  Karl-Dieter Crisman  |      Merged:                 
Work_issues:                       |  
-----------------------------------+----------------------------------------
Changes (by kcrisman):

  * status:  needs_review => positive_review
  * reviewer:  => Karl-Dieter Crisman


Comment:

 Okay, correctly checks #8582 and does the right stuff from #8624 that we
 need.  I agree with Minh that the other stuff was not needed.

 Very odd about the `sqrt(x+sqrt(x))` thing.  I think that Burcin is right;
 we never enabled the abs integrate - Maxima just got better at the #8624
 integrals on its own:
 {{{

 Maxima 5.22.1 http://maxima.sourceforge.net
 using Lisp ECL 10.4.1
 Distributed under the GNU Public License. See the file COPYING.
 Dedicated to the memory of William Schelter.
 The function bug_report() provides bug reporting information.
 (%i1) integrate(abs(cos(x))*sin(x),x,%pi/2,%pi);
                                        1
 (%o1)                                  -
                                        2
 }}}

 So I respectfully request that someone with the authority reopen that
 ticket as he suggests, with new title.  Using `abs_integrate` does solve
 the square root integral in question (`sqrt(x+sqrt(x))`).

 However, I think that this one can still be merged, since there are no
 problems with these, and they were verified as correct on the relevant
 tickets, and it passes the tests, of course, and applies, of course.

 To release manager/patchbot: doc fix only, apply trac-10434_doctests.patch

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