bugGNU Octave - Bugs: bug #45142, image package: strel disk does not...

 
 

bug #45142: image package: strel disk does not work any more

Submitter:  Hartmut <hardy>
Submitted:  Tue 19 May 2015 02:20:27 PM UTC
   
 
Category:  Octave Package Severity:  3 - Normal
Priority:  5 - Normal Item Group:  Regression
Status:  Invalid / Not an Octave Bug Assigned to:  None
Originator Name:  Open/Closed:  * Closed
Release:  * 4.0.0 Operating System:  * Any
Fixed Release:  None Planned Release:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Tue 02 Jun 2015 04:09:51 PM UTC, comment #4: 

Retagging release from 4.0.0-rc4 to 4.0.0.

John W. Eaton <jwe>
Group administrator
Tue 19 May 2015 04:05:36 PM UTC, comment #3: 

Yes. We haven't implemented the N option so we defaulted to N = 0 . Once that option gets implemented, we'd have to change the default which is a backwards incompatible change. I think it's better to do it now, forcing people to actually specify N.

Carnë Draug <carandraug>
Group Member
Tue 19 May 2015 03:47:20 PM UTC, comment #2: 

In my documentation I read


 -- Function File: strel ("disk", RADIUS)
 -- Function File: strel ("disk", RADIUS, N)
     Create disk shaped flat structuring element.  RADIUS must be a
     positive integer.

     The optional N argument *must* have a value of zero but the default
     value is 4.  This is to prevent future backwards incompatibilty
     since MATLAB default is also 4 but at the moment only 0 has been
     implemented.


And I am trying to set the FIRST parameter RADIUS to 4 (which is a positive integer) and leave the SECOND parameter N empty. This is valid syntax according to the above quote. But strel only works if I also explicitly set the second parameter N to something (and the only implemented choice is 0). So this does still work:


disk = strel('disk', 4, 0);


...

!! And now I have just realised that this totally what the doc says!! So just close this bug report please.

This new behavior makes my current scripts not work anymore :( But seems that I need to wait until N=4 (which is the default Matlab, and image-2.4.0 values) will be implemented at some time in the future.

Hartmut <hardy>
Tue 19 May 2015 02:54:51 PM UTC, comment #1: 


According to strel documentation, strel ('disk',N)
is implemented only for N = 0.
but


>> strel ('disk',0)
error: strel: RADIUS must be a positive integer
error: called from
    strel at line 256 column 9


Avinoam Kalma <avinoam>
Group Member
Tue 19 May 2015 02:20:27 PM UTC, original submission:  

Here is a little script to show what I mean:


clear
close all
pkg load image

disk = strel('disk',4);
-verbatim-

This perfectly valid syntax only produces an error message with the current release of image-2.4.0 (under Octave 4.0.0-rc4). I am shure that this worked with image-2.2.2 (under Octave 3.8.2).

Here is the error message:


error: strel: N for disk shape not yet implemented, use N of 0


Hartmut <hardy>

 

(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 avinoam (Posted a comment)
  • -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
    2015-06-02 jwe Release4.0.0-rc4 4.0.0
    2015-05-19 carandraug StatusNone Invalid / Not an Octave Bug
        Open/ClosedOpen Closed

    Back to the top

    Powered by Savane 3.13-caa5.
    Corresponding source code