bugGNU Octave - Bugs: bug #37200, conv2 doesn't check for input...

 
 

bug #37200: conv2 doesn't check for input number of dimensions

Submitter:  CarnĂ« Draug <carandraug>
Submitted:  Mon 27 Aug 2012 01:44:41 AM UTC
   
 
Category:  Libraries Severity:  3 - Normal
Priority:  5 - Normal Item Group:  None
Status:  Fixed Assigned to:  None
Originator Name:  Open/Closed:  * Closed
Release:  * 3.6.2 Operating System:  * Any
Fixed Release:  None Planned Release:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Mon 27 Aug 2012 05:20:00 AM UTC, comment #1: 

I added a check that the inputs to conv2 are not multi-dimensional matrices (http://hg.savannah.gnu.org/hgweb/octave/rev/2cdace1ba287).  If you do need multi-dimensional convolution see convn.

Rik <rik5>
Group administrator
Mon 27 Aug 2012 01:44:41 AM UTC, original submission:  

Hi

I just noticed that conv2 does not accept the number of dimensions. Obviously on the following example I'm giving crap to the function so I should expect an error but would be nice to get a more meaningful error. Specially since images are loaded as MxNx3 by default:


octave:13> a = uint8 (randi (255, 10, 10, 3));
octave:14> conv2 (a, ones(3))
error: invalid conversion of uint8 matrix to Matrix


The error even suggests that there's a problem on the the type but making it a double just returns "invalid conversion of NDArray to Matrix". Same goes for the seconed argument.

Carnë Draug <carandraug>
Group Member

 

(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 carandraug (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 2 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2012-08-27 rik5 StatusNone Fixed
        Open/ClosedOpen Closed

    Back to the top

    Powered by Savane 3.13-caa5.
    Corresponding source code