[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: |
Sat, 28 May 2016 22:09:29 +0000 (UTC) |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/49.0.2623.75 Safari/537.36 |
Follow-up Comment #12, patch #9000 (project octave):
No plans yet for joinPolygons, lets first finish with this.
about isPolygonCCW: ispolycw collides withhte naming convention in geometry
(camel case, decide by matgeom dev, not me). Do you prefer to wirte a mapping
function that wraps it? or shall we add it to geometry and then write a
function for the sake of naming convention?
polyclip: if the code you mention will be repeated better to private/ if bugs
pop up is less places to maintain.
1. Currently it doesn't but it would be easy (although a hassle: many
functions!) to do so. I prefer to force users to pass correct input arguments.
Keep my functions cleaner and my users better trained (they understand they
programs better).
2. If it is aboutignoring the other arguments. I think only a few functions do
force Nx2, in the others Nxd would be silently ignored.
_______________________________________________________
Reply to this item at:
<http://savannah.gnu.org/patch/?9000>
_______________________________________________
Message sent via/by Savannah
http://savannah.gnu.org/
- [Octave-patch-tracker] [patch #9000] Polygon functions for geometry/mapping package, (continued)
- [Octave-patch-tracker] [patch #9000] Polygon functions for geometry/mapping package, Philip Nienhuis, 2016/05/26
- [Octave-patch-tracker] [patch #9000] Polygon functions for geometry/mapping package, Juan Pablo Carbajal, 2016/05/26
- [Octave-patch-tracker] [patch #9000] Polygon functions for geometry/mapping package, Juan Pablo Carbajal, 2016/05/26
- [Octave-patch-tracker] [patch #9000] Polygon functions for geometry/mapping package, Philip Nienhuis, 2016/05/27
- [Octave-patch-tracker] [patch #9000] Polygon functions for geometry/mapping package, Juan Pablo Carbajal, 2016/05/27
- [Octave-patch-tracker] [patch #9000] Polygon functions for geometry/mapping package, Philip Nienhuis, 2016/05/27
- [Octave-patch-tracker] [patch #9000] Polygon functions for geometry/mapping package, Juan Pablo Carbajal, 2016/05/27
- [Octave-patch-tracker] [patch #9000] Polygon functions for geometry/mapping package, Philip Nienhuis, 2016/05/28
- [Octave-patch-tracker] [patch #9000] Polygon functions for geometry/mapping package, Juan Pablo Carbajal, 2016/05/28
- [Octave-patch-tracker] [patch #9000] Polygon functions for geometry/mapping package, Philip Nienhuis, 2016/05/28
- [Octave-patch-tracker] [patch #9000] Polygon functions for geometry/mapping package,
Juan Pablo Carbajal <=
- [Octave-patch-tracker] [patch #9000] Polygon functions for geometry/mapping package, Juan Pablo Carbajal, 2016/05/29