bugGNU Octave - Bugs: bug #52030, cplxpair returns incorrect result

 
 

bug #52030: cplxpair returns incorrect result

Submitter:  Henk Borsje <hjborsje>
Submitted:  Fri 15 Sep 2017 08:47:39 AM UTC
   
 
Category:  Octave Function Severity:  3 - Normal
Priority:  5 - Normal Item Group:  Incorrect Result
Status:  Fixed Assigned to:  None
Originator Name:  Henk Borsje Open/Closed:  * Closed
Release:  * 4.2.1 Operating System:  * Microsoft Windows
Fixed Release:  None Planned Release:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Fri 15 Sep 2017 03:29:49 PM UTC, comment #3: 

This has been fixed on the dev branch.  See this cset


changeset:   23972:b4cf41d173dd
user:        Bill Lash
date:        Tue Aug 29 16:46:48 2017 -0400
summary:     * cplxpair.m: Improve accuracy and compatibility (bug #50124)


Since this is just an m-file, you can download the latest copy of it from the Mercurial repository and place it in your installation.  Or you can wait until the next major release of Octave (4.4.0) which will include this patch.

Rik <rik5>
Group administrator
Fri 15 Sep 2017 10:40:21 AM UTC, comment #2: 

Fixed in current dev system

Michael Godfrey <godfrey>
Group Member
Fri 15 Sep 2017 10:37:18 AM UTC, comment #1: 

It works for me
 ans = -1
Ubuntu and dev branch

Doug Stewart <dastew>
Fri 15 Sep 2017 08:47:39 AM UTC, original submission:  

z = -1.0000e+000 + 1.2246e-016i
tol = 2.2204e-10

>> cplxpair(z,tol)



MATLAB returns -1, Octave stops with the error

error: cplxpair: could not pair all complex numbers
error: called from
    cplxpair at line 114 column 9

This combination of arguments came up during the generation of a Butterworth filter.

This seems to be a different bug from the tolerance incorrect units one, that did get fixed.

Henk Borsje

Henk Borsje <hjborsje>

 

(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 rik5 (Posted a comment)
  • -email is unavailable- added by godfrey (Posted a comment)
  • -email is unavailable- added by dastew (Posted a comment)
  • -email is unavailable- added by hjborsje (Submitted the item)
  • -email is unavailable- added by hjborsje
  •  

    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
    2017-09-15 rik5 StatusNone Fixed
        Open/ClosedOpen Closed
    2017-09-15 hjborsje Carbon-Copy- Added -email is unavailable-

    Back to the top

    Powered by Savane 3.13-02a9.
    Corresponding source code