bugGNU Octave - Bugs: bug #53231, Range type should implement...

 
 

bug #53231: Range type should implement division operator for scalars

Submitted by:  Rik <rik5>
Submitted on:  Tue 27 Feb 2018 01:43:27 AM UTC  
 
Category:  Interpreter Severity:  3 - Normal
Priority:  5 - Normal Item Group:  Feature Request
Status:  Confirmed Assigned to:  None
Originator Name:  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.

 

Tue 27 Feb 2018 07:34:29 PM UTC, comment #3:

As an additional data point, see bug #53199 where the extra accuracy of a Range, versus a Matrix, data type was crucial to resolving the bug.

Rik <rik5>
Project Administrator
Tue 27 Feb 2018 04:01:53 AM UTC, comment #2:

See also bug #50924. Some care needs to be used when implementing these operators. Otherwise, if accuracy and/or compatibility is important, we should probably limit the range type to be used in only a few cases (for loop ranges, for example), same as Matlab.

John W. Eaton <jwe>
Project Administrator
Tue 27 Feb 2018 02:00:18 AM UTC, comment #1:

To be clear, only a range divided by a scalar needs to be implemented, and not the reverse. In other words, this function

Rik <rik5>
Project Administrator
Tue 27 Feb 2018 01:43:27 AM UTC, original submission:

While fixing a bug in hist.m I accidentally discovered that division by a scalar is not implement for Range values.

For example:

This is rather surprising and likely to trip up users.

I was able to work around the limitation by using multiplication, but the code looks awkward.

The issue is confirmed by looking at liboctave/array/Range.h where the list of operators implemented does not include division.

This seems like a simple enough fix to introduce function prototypes for division by a scalar in Range.h and copy/modify the existing multiplication code in Range.cc to work for division.

Rik <rik5>
Project Administrator

 

(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 jwe (Posted a comment)
  • -email is unavailable- added by rik5 (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
    2018-02-27 rik5 StatusNone => Confirmed

    Back to the top


    Powered by Savane 3.3