#4285: [with patch, with positive review after some typos are fixed] update
desolver interface
----------------------+-----------------------------------------------------
Reporter: robertwb | Owner: burcin
Type: defect | Status: new
Priority: major | Milestone: sage-3.2
Component: calculus | Resolution:
Keywords: |
----------------------+-----------------------------------------------------
Changes (by zimmerma):
* summary: [with patch, needs review] update desolver interface => [with
patch, with positive review after some typos
are fixed] update desolver interface
Comment:
I made a review of this patch (tested against 3.1.2) and it is positive,
here are my remarks:
One issue is that in the 2nd desolve example, one would prefer y(10) to be
substituted by the initial condition y(10)=2 in the output, but this is
still
better than previously, and as Robert told me there is no point in
investing
too much in this if maxima is replaced by something else.
ics -- (optional) the initial or boundary conditions (hereafter called
x)
-> shouldn't "hereafter called x" be removed?
I see a few typos:
(in)dependant -> (in)dependent?
differeintial -> differential
if there more -> if there is more (twice)
variabe -> variable
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/4285#comment:7>
Sage <http://sagemath.org/>
Sage - Open Source Mathematical Software: Building the Car Instead of
Reinventing the Wheel
--~--~---------~--~----~------------~-------~--~----~
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
-~----------~----~----~----~------~----~------~--~---
