octave-patch-tracker
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Octave-patch-tracker] [patch #9000] Polygon functions for geometry/mapp


From: Juan Pablo Carbajal
Subject: [Octave-patch-tracker] [patch #9000] Polygon functions for geometry/mapping package
Date: Wed, 1 Feb 2017 01:45:44 +0000 (UTC)
User-agent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/56.0.2924.76 Safari/537.36

Follow-up Comment #33, patch #9000 (project octave):

Hi Philip,

The redundancy is there because some function in geometry already use isP*CCW,
I made isP*CW just for you :D.

I guess you will have to provide a wrapper in mapping for the naming 'ispolycw
=@(p) isPolygonCW(p)'. I am not sure how much overhead is that (should be an
m-file of course). We should ask jwe, maybe there is a way of creating
aliases. Could you ask in the mailing list?

Regarding comment #31:

- Camel case is inherited from MatGEom, I also do not like it. Remember that I
will make a function to clip polygons with an option to decide which engine to
use...just as I did with the orientation. The frontend will be called
'clipPolygon'. It will accept "geometry" and "clipper" as before. You wil have
to write a wrapper for the naming and argument convention in mapping.

- I do not plan to add the boost version in the next release. I am very
disappointed with that development. Specially because of the huge
dependencies. Maybe later on when I have time to distill it.

Cheers


    _______________________________________________________

Reply to this item at:

  <http://savannah.gnu.org/patch/?9000>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.gnu.org/




reply via email to

[Prev in Thread] Current Thread [Next in Thread]