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

[Carnë Draug]

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/