bugGNU Octave - Bugs: bug #61787, wrong output of "isreal"...

 
 

bug #61787: wrong output of "isreal" on array of uint8

Submitter:  Hartmut <hardy>
Submitted:  Mon 10 Jan 2022 05:17:23 PM UTC
   
 
Category:  Octave Function Severity:  3 - Normal
Priority:  5 - Normal Item Group:  Regression
Status:  Fixed Assigned to:  None
Originator Name:  Open/Closed:  * Closed
Release:  * 7.0.90 Operating System:  * Any
Fixed Release:  None Planned Release:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Tue 11 Jan 2022 04:20:58 PM UTC, comment #2: 

Good catch Hartmut.  I checked in jwe's patch under his name with a commit message here: http://hg.savannah.gnu.org/hgweb/octave/rev/579a21f74d6d

Marking as fixed and closing report.

Rik <rik5>
Group administrator
Mon 10 Jan 2022 06:22:04 PM UTC, comment #1: 

I think the attached change should fix it.


(file #52638)

John W. Eaton <jwe>
Group administrator
Mon 10 Jan 2022 05:17:23 PM UTC, original submission:  

Here is a regression (I think) that I observered with the current release candidate 7.0.90 under Win10, but I assume it is there under all OS'es. To demonstrate the principle, run the following code:


isreal(uint8(5):uint8(9))


  • This gives 0 under Octave 7.0.90
  • This gives 1 under Octave 6.4.0
  • This gives 1 under Matlab R2019b


I think the current result of Octave 7.0.90 is wrong. Maybe the underlying regression in Octave 7 is much wider than just the "isreal" function, I cannot judge this.

This error does not happen if you just do "isreal(5:9)" with double input arrays.

I found this, because a unit test in the image package was failing.

Hartmut <hardy>

 

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

Attach Files:
   
   
Comment:
   

Attached Files
file #52638:  range-isreal-diffs.txt added by jwe (378B - text/plain)

 

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 jwe (Updated the item)
  • -email is unavailable- added by hardy (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-01-11 rik5 StatusNone Fixed
        Open/ClosedOpen Closed
    2022-01-10 jwe Attached File- Added range-isreal-diffs.txt, #52638

    Back to the top

    Powered by Savane 3.13-758e.
    Corresponding source code