bugGNU Octave - Bugs: bug #53963, [octave forge] (lssa) Unit test in...

 
 

bug #53963: [octave forge] (lssa) Unit test in fastlscomplex is failing

Submitter:  Rafael Laboissière <rlaboiss>
Submitted:  Wed 23 May 2018 12:55:58 PM UTC
   
 
Category:  Octave Package Severity:  3 - Normal
Priority:  5 - Normal Item Group:  Inaccurate Result
Status:  Wont Fix Assigned to:  lostbard
Originator Name:  Open/Closed:  * Closed
Release:  * other Operating System:  * GNU/Linux
Fixed Release:  None Planned Release:  None
* Mandatory Fields

Post a Comment

Add a New Comment Rich Markup
   

Discussion

Mon 23 Jul 2018 05:13:39 PM UTC, comment #4: 

Pushed https://sourcefor ... 8242db75661004da/, marked as a xfail for bug #48905, so will close this bug report and leave the one associated with the failure open.

John Donoghue <lostbard>
Group Member
Mon 23 Jul 2018 04:51:43 PM UTC, comment #3: 

Having done that, bug #48905 would suggest that potentially something is wrong with the algorithm and it should actally be failing.

I think that original values should probally go back and the test marked as a xfail for now.

John Donoghue <lostbard>
Group Member
Mon 23 Jul 2018 04:16:43 PM UTC, comment #2: 

updated https://sourcefor ... b9180322277ec525/

test passes ok on fedora 28 (x86_64) linux and Windows 10 using Octave 4.4.0.

John Donoghue <lostbard>
Group Member
Wed 23 May 2018 01:12:26 PM UTC, comment #1: 

Sorry, I messed with the Summary of this bug report.  I meant: "[octave forge] (lssa) Unit test in fastlscomplex is failing".

Please, change it.

Rafael Laboissière <rlaboiss>
Wed 23 May 2018 12:55:58 PM UTC, original submission:  

The following unit test of fastlscomplex is failing on my system:


 maxfreq = 4 / ( 2 * pi );
 t = [0:0.008:8];
 x = ( 2 .* sin (maxfreq .* t) +
       3 .* sin ( (3 / 4) * maxfreq .* t)-
       0.5 .* sin ((1/4) * maxfreq .* t) -
       0.2 .* cos (maxfreq .* t) +
       cos ((1/4) * maxfreq .* t));
 assert (fastlscomplex (t, x, maxfreq, 2, 2),
       [(-0.400924546169395 - 2.371555305867469i), ...
        (1.218065147708429 - 2.256125004156890i), ...
        (1.935428592212907 - 1.539488163739336i), ...
        (2.136692292751917 - 0.980532175174563i)], 5e-10);
!!!!! test failed
ASSERT errors for:  assert (fastlscomplex (t, x, maxfreq, 2, 2),[(-0.400924546169395 - 2.371555305867469i), (1.218065147708429 - 2.256125004156890i), (1.935428592212907 - 1.539488163739336i), (2.136692292751917 - 0.980532175174563i)],5e-10)

  Location  |  Observed  |  Expected  |  Reason
    (1)      -0.40092-2.3716i -0.40092-2.3716i   Abs err 2.3279e-08 exceeds tol 5e-10 by 2e-08
    (2)      1.2181-2.2561i 1.2181-2.2561i   Abs err 6.4488e-10 exceeds tol 5e-10 by 1e-10
    (3)      0.17512-2.3014i 1.9354-1.5395i   Abs err 1.9181 exceeds tol 5e-10 by 2
    (4)      1.0019-2.0248i 2.1367-0.98053i   Abs err 1.5422 exceeds tol 5e-10 by 2


I am using a Debian amd64 system with Octave 4.4.0 and g++ 7.3.0.  Here are the values that I obtain from the code above:


  -0.400924541120278 - 2.371555283143016i
   1.218065147789378 - 2.256125003517112i
   0.175123244470191 - 2.301447978287065i
   1.001866643837121 - 2.024841512887229i


Rafael Laboissière <rlaboiss>

 

Attached Files

This item currently has no attached files.

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

Attach Files:
   
   
Comment:
   

 

Dependencies

This item does not depend on any other items.

No items depend on this one.

 

Mail Notification Carbon-Copy List

Carbon-Copy List
  • -email is unavailable- added by lostbard (Posted a comment)
  • -email is unavailable- added by pantxo (Updated the item)
  • -email is unavailable- added by rlaboiss (Submitted the item)
  •  

    Votes

    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.

     

    History

    Follow 5 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2018-07-23 lostbard StatusReady For Test Wont Fix
        Open/ClosedOpen Closed
    2018-07-23 lostbard StatusNone Ready For Test
        Assigned toNone lostbard
    2018-05-23 pantxo Summary[octave forge] (lssa) Test [octave forge] (lssa) Unit test in fastlscomplex is failing

    Back to the top

    Powered by Savane 3.16-ed84.
    Corresponding source code