[Octave-bug-tracker] [bug #46547] image package: canny detector in edge() should use LoG as first step

[Hartmut]

Follow-up Comment #5, bug #46547 (project octave):

Oops. Your are right!

Sorry for the error in my patch, this seems to be about the DATA TYPE of the
input image. In the original implementation of the gaussian filtering in
edge.m there was an "double(im)" and in my new implementation I take the input
image directly, without converting it to double before. (Funnily my two new
tests passed anyways...)

It works if you make the following change to edge.m:

-      J = imfilter (im, gauss, "replicate");
+      J = imfilter (double(im), gauss, "replicate");

Maybe you know a more clever way to convert the input image to double values
as a first step, then just do it like this :)

And maybe we should also add an additional test to check that also integer
typed input images are treated well?


    _______________________________________________________

Reply to this item at:

  <http://savannah.gnu.org/bugs/?46547>

_______________________________________________
  Nachricht gesendet von/durch Savannah
  http://savannah.gnu.org/