[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Octave-patch-tracker] [patch #9252] image package: hough.m (Matlab comp
[Octave-patch-tracker] [patch #9252] image package: hough.m (Matlab compatible wrapper around hough_line.cc)
Tue, 7 Feb 2017 16:42:25 -0500 (EST)
Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:51.0) Gecko/20100101 Firefox/51.0
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
* 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
* 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:
Message sent via/by Savannah