bugGNU Octave - Bugs: bug #56950, NaN/Inf values in ind2rgb

 
 

bug #56950: NaN/Inf values in ind2rgb

Submitted by:  Guillaume <gyom>
Submitted on:  Wed 25 Sep 2019 11:32:43 AM UTC  
 
Category:  Octave Function Severity:  3 - Normal
Priority:  5 - Normal Item Group:  Matlab Compatibility
Status:  None Assigned to:  None
Originator Name:  Guillaume Open/Closed:  Open
Release:  dev Operating System:  Any

Add a New Comment (Rich Markup)
   

You are not logged in

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

 

Mon 30 Sep 2019 09:52:19 AM UTC, comment #2: 

In my view, ind2rgb is a 'core' enough functionality that full compatibility could be obtained. I found an older bug report where I was suggesting a short fix:
https://savannah.gnu.org/bugs/?func=detailitem&item_id=41851#comment7
It fits with TMW description that "image values are clipped to the range [1, c]" and solves all the cases I came across without error or warning.

Guillaume <gyom>
Fri 27 Sep 2019 05:04:38 PM UTC, comment #1: 

Octave is generally stricter about checking its inputs than Matlab.  In this case, why would an indexed image have a NaN value?

Even if there is a use case for have a NaN value in an indexed image, it is a rare occurrence.  For most users, a NaN is an error, and for the few users for which it is not, I believe it is better for them to actively choose how they want these values to be interpreted.  This is an easy one-line piece of code as you demonstrated.  I am tempted to mark this as "Won't Fix" and have programmers choose how they want such images to be interpreted.

Rik <rik5>
Project Administrator
Wed 25 Sep 2019 11:32:43 AM UTC, original submission:  

ind2rgb does not allow NaN/Inf values in the indexed image while Matlab behaves as if they were equal to the maximal index according to the size of the colormap.

>> rgb = ind2rgb ([NaN Inf 1], jet)
rgb(:,:,1) =
   0.500000000000000   0.500000000000000                   0
rgb(:,:,2) =
     0     0     0
rgb(:,:,3) =
                   0                   0   0.562500000000000

One possible fix would be to add the following before the input check:

if isfloat (x)
  x (isnan (x) | isinf (x)) = size (map, 1);
endif

Guillaume <gyom>

 

(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 gyom (Submitted the item)
  •  

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

    Only project members can vote.

     

     

     

    Follows 1 latest change.

    Date Changed by Updated Field Previous Value => Replaced by
    2019-09-27 rik5 Item GroupUnexpected Error => Matlab Compatibility

    Back to the top


    Powered by Savane 3.5