patchGNU Octave - Patches: patch #9412, image package: new function...

 
 

patch #9412: image package: new function houghlines.m

Submitter:  Hartmut <hardy>
Submitted:  Fri 21 Jul 2017 08:30:29 PM UTC
   
 
Category:  Forge : new function Priority:  5 - Normal
Status:  Done Privacy:  Public
Assigned to:  None Open/Closed:  Closed
* Mandatory Fields

Add a New Comment Rich Markup
   

Jump to the original submission

Mon 05 Nov 2018 10:14:38 PM UTC, comment #6: 
Avinoam Kalma <avinoam>
Group Member
Sun 04 Nov 2018 09:13:09 AM UTC, comment #5: 

demo houghlines depends on houghpeaks, so I would like to push them together. Any comments on houghpeaks?

Avinoam Kalma <avinoam>
Group Member
Sun 28 Oct 2018 08:48:31 PM UTC, comment #4: 

OK, I will review the new function and push it.

Avinoam Kalma <avinoam>
Group Member
Wed 24 Oct 2018 05:31:13 PM UTC, comment #3: 

Yes, in my opinion this is ready to push. (The latest version, V2.) But please also have a closer look yourself, to properly review it before pushing.

Hartmut <hardy>
Mon 15 Oct 2018 08:04:48 PM UTC, comment #2: 

I would like to push this function.
Is it ready?

Avinoam Kalma <avinoam>
Group Member
Wed 02 Aug 2017 06:23:37 PM UTC, comment #1: 

new version of houghlines.m with some coding style issues fixed:

https://bitbucket.org/hgimpel/octave-image/commits/acf5efbcd852fa786da5f011651d50bcfea85e65

also attached as file (V2).

(file #41394)

Hartmut <hardy>
Fri 21 Jul 2017 08:30:29 PM UTC, original submission:  

I have implemented the missing function "houghlines.m" for the image package.

This function extracts all line segments from a BW image, when given the results of a Hough transform (from the functions hough.m and houghpeaks.m).

The algorithm is a straigtforward implementation and might not be runtime optimal, yet. It uses the information from the Matlab help page and from the book (Gonzalez,Woods,Eddins) that Matlab mentions on its help page of the connected function houghpeaks.

You can find the new file on my bitbucket repository:

https://bitbucket.org/hgimpel/octave-image/commits/9425df685b4a80beda314a1d29487718664c1395

The file is on the default branch there, with the bookmark "new-houghlines" attached.

I will also add the function file (houghlines_V1.m) to this report.

I would be happy to hear comments or feedback on this.

Hartmut <hardy>

 

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

Attach Files:
   
   
Comment:
   

Attached Files
file #41394:  houghlines_V2.m added by hardy (13KiB - text/x-objcsrc)
file #41282:  houghlines_V1.m added by hardy (12KiB - text/x-objcsrc)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by avinoam (Posted a comment)
  • -email is unavailable- added by hardy (Submitted the item)
  • -email is unavailable- added by hardy (adding people who care about the image package)
  • -email is unavailable- added by hardy (adding people who care about the image package)
  •  

    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-11-05 avinoam StatusNone Done
        Open/ClosedOpen Closed
    2017-08-02 hardy Attached File- Added houghlines_V2.m, #41394
    2017-07-21 hardy Attached File- Added houghlines_V1.m, #41282
        Carbon-Copy- Added avinoam
        Carbon-Copy- Added carandraug

    Back to the top

    Powered by Savane 3.13-cf05.
    Corresponding source code