[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Octave-patch-tracker] [patch #9448] image package: new function imfindc
From: |
Carnë Draug |
Subject: |
[Octave-patch-tracker] [patch #9448] image package: new function imfindcircles.m |
Date: |
Mon, 15 Jan 2018 14:23:06 -0500 (EST) |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Firefox/52.0 |
Update of patch #9448 (project octave):
Status: None => Done
Open/Closed: Open => Closed
_______________________________________________________
Follow-up Comment #7:
> Alright. I've done some more testing of the modified version of
imfindcircles. And it seems to still behave nicely. I think this new function
is mostly ready for pushing to the repo.
Done
http://hg.code.sf.net/p/octave/image/rev/f3d64772b21a
http://hg.code.sf.net/p/octave/image/rev/fcf037d3421b
> * You could add some more "distortions" like staight lines, squares and
salt&pepper noise to the data for the demo block. This shows the advanced
capabilities of this circular Hough transform.
Ok. I added noise, a blurring filter, and a square.
> * There seems to be a bug in the new viscircles.m, at least I cannot get the
EdgeColor option to work at all. For example the demo block in imfindcircles
gives me white circles (even though the default color shoud be red as the
figure title says) and the demo block in viscircles also gives me white
circles (even though the code explicitly asks for black ones).
I can't reproduce it. I have tried with Octave 4.0, 4.2, and a ~1 month olf
4.2 with both gnuplot and qt graphics toolkit. I did found that the LineWidth
option being ignored but that seems to be an issue with the line function.
_______________________________________________________
Reply to this item at:
<http://savannah.gnu.org/patch/?9448>
_______________________________________________
Message sent via/by Savannah
http://savannah.gnu.org/