[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)
Sat, 11 Feb 2017 19:08:22 -0500 (EST)
Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Firefox/45.0
Follow-up Comment #3, patch #9252 (project octave):
Ok. I just wanted to make sure that the missing houghlines and houghpeaks
should be dependent on hough rather than the other way around.
After changing the line breaks from dos to unix
I made this changes (see inline comments)
The only important change I made is about handling of non binary input. Can
you check Matlab behaviour? It's very rare for Matlab to actually check, it
usually just converts it to logical.
If that's ok with you, I will push that to Octave Forge.
Reply to this item at:
Message sent via/by Savannah