|
From: | Carnë Draug |
Subject: | [Octave-patch-tracker] [patch #9252] image package: hough.m (Matlab compatible wrapper around hough_line.cc) |
Date: | Sat, 11 Feb 2017 19:08:22 -0500 (EST) |
User-agent: | 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 https://bitbucket.org/carandraug/octave-image/commits/6d07c845470fd438e03cedb7b56d487b9cad4463 I made this changes (see inline comments) https://bitbucket.org/carandraug/octave-image/commits/bae473473e18dbcee6fb570bb85cec983ca9d45b 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: <http://savannah.gnu.org/patch/?9252> _______________________________________________ Message sent via/by Savannah http://savannah.gnu.org/
[Prev in Thread] | Current Thread | [Next in Thread] |