|
From: | Rik |
Subject: | Re: new ODE functions |
Date: | Fri, 16 Dec 2016 09:40:48 -0800 |
On 12/16/2016 09:00 AM,
address@hidden wrote:
Carlo, Can you modify scripts/help/__unimplemented__.m to remove ode15i, ode15s from the list of functions missing from Octave? There is also a more explanatory help message that needs updating. case {"ode113", "ode15i", "ode15s", "ode23s", "ode23t", "ode23tb"} txt = ["Octave provides lsode and ode45 for solving differential equations. ", ... "For more information try @code{help lsode}, @code{help ode45}. ", ... "Matlab-compatible ODE functions are provided by the odepkg ", ... "package. See @url{http://octave.sourceforge.net/odepkg/}."]; Other things I see are single quotes instead of double quotes in some of the code. For the input validation %!error tests I would use a message so it is clear what is being tested. This is an example from ode45.m %!error <TRANGE must be a numeric> ode45 (@fpol, {[0 25]}, [3 15 1]) instead of writing %!error ode45 (@fpol, {[0 25]}, [3 15 1]) When I run configure, I get configure: WARNING: SUNDIALS_IDA library configured with klu not found. Solvers ode15i and ode15s will be disabled. configure: WARNING: SUNDIALS_NVECSERIAL library not found. Solvers ode15i and ode15s will be disabled. For text, the convention is to use two spaces after a period to indicate a sentence break. Also, printing just one message that the solvers will be disabled is enough. --Rik |
[Prev in Thread] | Current Thread | [Next in Thread] |