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

 
 

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

Submitted by:  Hartmut <hardy>
Submitted on:  Tue 19 May 2015 02:20:27 PM UTC  
 
Category: Octave Forge PackageSeverity: 3 - Normal
Priority: 5 - NormalItem Group: Regression
Status: InvalidAssigned to: None
Originator Name: Open/Closed: Closed
Release: 4.0.0Operating System: Any

Add a New Comment(Rich Markup)
   

You are not logged in

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

 

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>
Project 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>
Project Member
Tue 19 May 2015 03:47:20 PM UTC, comment #2:

In my documentation I read

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:

...

!! 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

Avinoam Kalma <avinoam>
Project 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:

Hartmut <hardy>

 

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

Attach File(s):
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by jwe (Posted a comment)
  • -unavailable- added by avinoam (Posted a comment)
  • -unavailable- added by hardy (Submitted the item)
  •  

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

    Only project members can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 3 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Tue 02 Jun 2015 04:09:51 PM UTCjweRelease4.0.0-rc4=>4.0.0
    Tue 19 May 2015 04:05:36 PM UTCcarandraugStatusNone=>Invalid
      Open/ClosedOpen=>Closed

    Back to the top


    Powered by Savane 3.1-cleanup1