bugGNU Octave - Bugs: bug #62599, Minor test failure with integral...

 
 

bug #62599: Minor test failure with integral (ArrayValued)

Submitter:  Arun Giridhar <arungiridhar>
Submitted:  Wed 08 Jun 2022 06:44:51 PM UTC
   
 
Category:  Test Suite Severity:  3 - Normal
Priority:  5 - Normal Item Group:  Inaccurate Result
Status:  Fixed Assigned to:  None
Originator Name:  Open/Closed:  * Closed
Release:  * dev Operating System:  * GNU/Linux
Fixed Release:  None Planned Release:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Wed 08 Jun 2022 08:55:16 PM UTC, comment #2: 

Thanks! Passes make check for me now. Closing as fixed.

Arun Giridhar <arungiridhar>
Group Member
Wed 08 Jun 2022 07:35:05 PM UTC, comment #1: 

yes, that test was just added recently to ensure integral could now be called successfully with both parameters when it previously could not.  adding the waypoint at 0.5 cuts the cosine error, and it passes with eps on my windows system, but increasing to 2*eps is fine.  I pushed that change here:

http://hg.savannah.gnu.org/hgweb/octave/rev/b390f662a150

Nicholas Jankowski <nrjank>
Group Member
Wed 08 Jun 2022 06:44:51 PM UTC, original submission:  

Is it OK to increase the tolerance on this test to 2*eps?


>>>>> processing .../octave/scripts/general/integral.m
***** assert (integral (@(x) [sin(x), cos(x)], 0, pi, "ArrayValued", 1,
                   "Waypoints", [0.5]), [2, 0], eps);
!!!!! test failed
ASSERT errors for:  assert (integral (@(x)[sin(x), cos(x)], 0, pi, "ArrayValued", 1, "Waypoints", [0.5]),[2, 0],eps)

  Location  |  Observed  |  Expected  |  Reason
    (2)       2.7756e-16       0         Abs err 2.7756e-16 exceeds tol 2.2204e-16 by 6e-17


Arun Giridhar <arungiridhar>
Group Member

 

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

Attach Files:
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by nrjank (Posted a comment)
  • -email is unavailable- added by arungiridhar (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only group members can vote.

     

    Follow 3 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2022-06-08 arungiridhar StatusReady For Test Fixed
        Open/ClosedOpen Closed
    2022-06-08 nrjank StatusNone Ready For Test

    Back to the top

    Powered by Savane 3.13-758e.
    Corresponding source code