#6479: [with patch, needs work] desolve for 2nd order ODE with initial condition
gives wrong answer
-------------------------+--------------------------------------------------
   Reporter:  gmhossain  |       Owner:  burcin    
       Type:  defect     |      Status:  needs_work
   Priority:  major      |   Milestone:  sage-4.2  
  Component:  calculus   |    Keywords:            
Work_issues:             |      Author:            
   Reviewer:             |      Merged:            
-------------------------+--------------------------------------------------
Changes (by wdj):

  * status:  needs_review => needs_work


Comment:

 The improvements are *fantastic*!

 However, some of the docstrings do not follow proper format.
 For example, in your desolve_rk4 function, you do not indent
 the Sage code in the EXAMPLES section correctly. Also, if a
 function can produce different types of output (eg, a plot or
 a list of points, depending on the optional parameters), both
 should be illustrated in the examples. I don't know if this
 improper formatting screws up the sage -test script or not.
 There is also some improper indentation in other sections,
 such as OUTPUT, for functions such as desolve_rk4, for example.

 I hope you can please fix these.

 A request: in your new functions desolve_rk4 and desolve_system_rk4
 there is an option called endpoint, with default value 10. I would
 prefer an option called endpoints with a default value of [0,10]
 (or something), so that a range can be plotted other than from
 ics[0] to endpoint. If it is too much hassle, fine (you can just add
 plots together to get that anyway...).

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

Reply via email to