bugGNU Octave - Bugs: bug #49918, No Stats available for ode15i,...

 
 

bug #49918: No Stats available for ode15i, ode15s

Submitted by:  Rik <rik5>
Submitted on:  Fri 23 Dec 2016 06:13:44 AM UTC  
 
Category: Octave FunctionSeverity: 3 - Normal
Priority: 5 - NormalItem Group: None
Status: In ProgressAssigned to: Carlo de Falco <cdf>
Originator Name: Open/Closed: Open
Release: devOperating System: Any

Add a New Comment(Rich Markup)
   

You are not logged in

Please log in, so followups can be emailed to you.

 

Fri 23 Dec 2016 05:34:03 PM UTC, comment #4:

Final issue, ode23.m and ode45.m have statistics which they print. The new ode15i.m and ode15s.m accept the Stats option, and even have code to set the internal variable options.havestats, but don't print anything.

Should a new bug report be opened about that, or should we just continue to re-purpose this one? Either we can get the statistics out of SUNDIALS in some manner, or we should just issue a warning that no statistics are available.

Rik <rik5>
Project Administrator
Fri 23 Dec 2016 05:31:21 PM UTC, comment #3:

@Carlo: I committed your cset here (http://hg.savannah.gnu.org/hgweb/octave/rev/bb452f84a299). One thing I did was change the BIST tests to use evalc to capture the output from the Stats option which was otherwise being dumped to stdout after your change.

Also, I don't think there is a reason to follow Matlab slavishly in regards to case sensitivity. When you write something like,

Matlab ignores case and sets the field name 'Stats'. I think Octave should be equally tolerant of the right-hand side argument where people might type 'ON', 'On', or 'on'. I made that additional change here (http://hg.savannah.gnu.org/hgweb/octave/rev/c9344df03da5).

Rik <rik5>
Project Administrator
Fri 23 Dec 2016 09:10:15 AM UTC, comment #2:

Actually, it seems that in Octave the 'stats' option is neglected
independent of the capitalization ...

I'm changing the title of this ticket and attaching a changeset.

(file #39298)

Carlo de Falco <cdf>
Project MemberIn charge of this item.
Fri 23 Dec 2016 08:51:23 AM UTC, comment #1:

Although it looks odd, the current behavior seems to be compatible:

Carlo de Falco <cdf>
Project MemberIn charge of this item.
Fri 23 Dec 2016 06:13:44 AM UTC, original submission:

I don't know how Matlab handles the Stats option to odeset, but Octave behaves in a case-sensitive manner.

But if you look at the code in the m-files, for example ode45.m, you find

It seems like these calls should be replaced with strcmpi. It also looks like the option NormControl requires the same change.

Rik <rik5>
Project Administrator

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach File(s):
   
   
Comment:
   

Attached Files
file #39299:  open_SCyQ6FsW.txt added by cdf (2KiB - text/plain)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by cdf (Posted a comment)
  • -unavailable- added by rik5 (Submitted the item)
  •  

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only project members can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 7 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Wed 28 Dec 2016 12:33:59 AM UTCrik5SummaryStats option is ignored by ode45=>No Stats available for ode15i, ode15s
    Fri 23 Dec 2016 05:34:03 PM UTCrik5StatusPatch Submitted=>In Progress
    Fri 23 Dec 2016 09:17:27 AM UTCcdfAttached File-=>Added open_SCyQ6FsW.txt, #39299
    Fri 23 Dec 2016 09:16:57 AM UTCcdfAttached File#39298=>Removed
    Fri 23 Dec 2016 09:10:15 AM UTCcdfAttached File-=>Added open_SCyQ6FsW.txt, #39298
      StatusNone=>Patch Submitted
      Summaryodeset and Stats option is case sensitive=>Stats option is ignored by ode45

    Back to the top


    Powered by Savane 3.1-cleanup1