[Octave-bug-tracker] [bug #63162] Inaccurate "Events" location in ode45, ode23(s)

[John W. Eaton]

Follow-up Comment #9, bug #63162 (project octave):

It would be good to have separate patches for each problem you're trying to
fix.  If that's not possible, or would mean a lot of extra work at this point,
then we can work with one larger patch.  But in the future, it would be best
to try to limit the scope of changes and not lump too many fixes in one
change.

The change you proposed in comment #6 is OK, but I think you could just write


fcn = @(t, y) feval (fcn, t, y, varargin{2:end});


If you do need to have "funarguments" defined some other reason, then you need
to use


fcn = @(t, y) feval (fcn, t, y, funarguments{:})


to pass the extra arguments individually to the original user-supplied
function instead of collected in a single cell array argument.

Also, I assume "feval" is needed because the original function could be passed
as a function handle, inline function, or character array that names the
function so that the simple function call syntax "fcn (t, y, ...)" won't work.


    _______________________________________________________

Reply to this item at:

  <https://savannah.gnu.org/bugs/?63162>

_______________________________________________
Message sent via Savannah
https://savannah.gnu.org/