[Octave-patch-tracker] [patch #9252] image package: hough.m (Matlab compatible wrapper around hough_line.cc)

[Hartmut]

Follow-up Comment #2, patch #9252 (project octave):

* hough_line.cc (Octave):

** I have no objections to deprecating hough_line.cc once we have the (mostly)
Matlab compatible hough.m present. The new hough.m can do everything that
previously did hough_line.cc, plus hough.m has some additional "convinience
options" like the Matlab version has.

** The old hough_line.cc does need to stay inside the image package, though.
Probably as a private function. Because the new hough.m is mostly a wrapper
script around hough_line.cc. The real computational work is still done by
hough_line.cc

* houghpeaks.m and houghlines.m (Matlab):

** Those Matlab functions are "convenience" scripts, that do useful
post-processing of the results of hough.m. In the Octave image package we
currently do not have any compatible pendants that I know of. (So those would
probably stay missing functions in Octave.)

** An Octave version of houghpeaks.m might be doable by writing a wrapper
script around our present immaximas.m script.

** An Octave version of houghlines.m would be a fully new implementation of
this task in Octave. It shouldn't be too big a job, but I don't know of any
alreay existing Octave script that already aims at the same task.

* houghtf.m  (Octave): 
** Our (Octave-only) current function houghtf.m is very short and simple.
Calling it with one (or without any) option redirects the call to
hough_line.cc. And calling it with the other options redirects the call to
hough_circle.m. If we deprecate hough_line.cc then we might also deprecate
houghtf.m

* hough_circle.m (Octave) and imfindcircles.m (Matlab):
** Our hough_circle.m script does the circular Hough transform. This is
computationally independent of the linear Hough transform in hough_line.cc and
hough.m. So hough_circle.m would need to stay. In order to also substitute it
with a Matlab compatible function, one would need to implement imfindcircles.m
in Octave (it also does the circular Hough transform).



    _______________________________________________________

Reply to this item at:

  <http://savannah.gnu.org/patch/?9252>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.gnu.org/