patchGNU Octave - Patches: patch #9607, imfindcircles

 
 

patch #9607: imfindcircles

Submitter:  Colin Foster <cfoster>
Submitted:  Sat 24 Mar 2018 09:59:48 PM UTC
   
 
Category:  Forge : new function Priority:  5 - Normal
Status:  Wont Do Privacy:  Public
Assigned to:  None Open/Closed:  Closed
* Mandatory Fields

Add a New Comment Rich Markup
   

Mon 26 Mar 2018 01:03:31 AM UTC, comment #2: 

Oh wow, that one is much nicer than mine :-) I should have looked more thoroughly before submitting it.

There probably isn't much of a need for this technique, given that the accepted implementation seems to have all of my "todo" optimizations. I'll keep my eye open for something else to try to contribute. I don't have any specific use-case, this was just something that piqued my interest.

Colin Foster <cfoster>
Sun 25 Mar 2018 03:02:47 PM UTC, comment #1: 

Thanks for you work, Colin.

But there already exists a function imfindcirlces, that I've written some months ago. See patch #9448. It is already part of the image repository and will be released with the next major release of the image package.

If you are still motivated, you could check this existing imfindcircles function and double check that it properly works in your use cases.

If you see an additional benefit of your implementation (that doesn't intend to be fully Matlab compatible), then please let us know here. Maybe the image package maintainer can then add it to the image package with a slightly different name, as an Octave-only function.

Hartmut <hardy>
Sat 24 Mar 2018 09:59:48 PM UTC, original submission:  

I've added an initial imfindcircles. It currently does the following:

Runs a canny edge detection to transform circles into rings
Creates a kernel image of the ring, one pixel wide, using the bwmorph command
Runs a matrix convolve2 function to populate the accumulator

All of these should be consistent with the Hough transform method that other packages currently use, though their implementation seems to differ slightly.

Optionally, this will plot images with detected circles.

This will return an array of circle centers, as well as the detected radii and confidence.

Colin Foster <cfoster>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

Attached Files
file #43665:  patchfile added by cfoster (6KiB - application/octet-stream)
file #43666:  example.pdf added by cfoster (36KiB - application/pdf)
file #43667:  coins.png added by cfoster (37KiB - image/png)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by mtmiller (Updated the item)
  • -email is unavailable- added by avinoam
  • -email is unavailable- added by hardy (Posted a comment)
  • -email is unavailable- added by cfoster (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only logged-in users can vote.

     

    Follow 6 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2018-03-31 mtmiller StatusNone Wont Do
        Open/ClosedOpen Closed
    2018-03-25 avinoam Carbon-Copy- Added carandraug
    2018-03-24 cfoster Attached File- Added patchfile, #43665
        Attached File- Added example.pdf, #43666
        Attached File- Added coins.png, #43667

    Back to the top

    Powered by Savane 3.13-f8d8.
    Corresponding source code