patchGNU Octave - Patches: patch #9244, image package: regionprops.m - new...

 
 

patch #9244: image package: regionprops.m - new properties ConvexHull, ConvexImage, ConvexArea, Solidity

Submitted by:  Hartmut <hardy>
Submitted on:  Mon 30 Jan 2017 11:35:20 PM UTC  
 
Category: Forge : new featurePriority: 5 - Normal
Status: NonePrivacy: Public
Assigned to: NoneOpen/Closed: Open

Add a New Comment(Rich Markup)
   

You are not logged in

Please log in, so followups can be emailed to you.

 

Fri 16 Feb 2018 06:41:39 PM UTC, comment #4:

@Marko: Yes, the V1 m-file is based on a one year old "default" branch of the image package. So it is really missing all improvements from the last 12 months. I hope it helps you anyways.

Hartmut <hardy>
Fri 16 Feb 2018 10:56:02 AM UTC, comment #3:

Thank you Hardy.

ConvexImage works as a charm.

However, there is a regresion in the orientation calculation. Orientation has been fixed in the image package 2.6.2, https://savannah.gnu.org/bugs/?49613
Is it possible regionprops you have attached doesn't contain this fix? I should be able to merge this manually...

Marko <markobe>
Thu 15 Feb 2018 03:54:20 PM UTC, comment #2:

@Marko: I will attach the file regionprops_V1.m that includes this patch (V1) already. You will need to rename it to regionprops.m and put it where your current regionprops.m sits, i.e. substitute the old function file of the image package.

Please leave a note here on the bug tracker, if those new properties work as expected for you. Thanks.

(file #43329)

Hartmut <hardy>
Thu 15 Feb 2018 02:34:03 PM UTC, comment #1:

I'm sorry for dumb question. But how do I apply patch?

Marko <markobe>
Mon 30 Jan 2017 11:35:20 PM UTC, original submission:

Attached is a PATCH (V1) that implements the missing features

  • ConvexHull
  • ConvexImage
  • ConvexArea
  • Solidity

in regionprops.m

This patch also adds

  • help strings for those new features
  • Matlab compatible tests for those new features

The tests are all passing. Only one xtest is failing, because of an imperfect workaround for bug #50188 in ConvexImage. But even those results are of same quality as the Matlab results.

Hartmut <hardy>

 

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

Attach File(s):
   
   
Comment:
   

Attached Files
file #43329:  regionprops_V1.m added by hardy (61KiB - text/x-objcsrc)

 

Depends on the following items: None found

Items that depend on this one: None found

 

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

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 4 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Thu 15 Feb 2018 03:54:20 PM UTChardyAttached File-=>Added regionprops_V1.m, #43329
    Mon 30 Jan 2017 11:35:20 PM UTChardyAttached File-=>Added regionprops_convex-properties_V1.patch, #39621
      Carbon-Copy-=>Added avinoam
      Carbon-Copy-=>Added carandraug

    Back to the top


    Powered by Savane 3.1-cleanup1