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

[Philip Nienhuis]

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

Juan Pablo Carbajal wrote:
> On Sat, Sep 3, 2016 at 11:36 PM, Philip Nienhuis
>><address@hidden>  wrote:
> Hi, I will have time soon., where are the latest advances?
> I am still using sourceforge as main repo.

?? Did you reply to the patch tracker post?? that won't work properly. Better
answer thrugh the patch tracker

Wow, time ... :-)

I'm almost 95 % tru the review but ran out of time.
ATM I think it is still unwise to absorb Amr's stuff just like that, but you
are pkg maintainer not me. The issues:

- boost 1.60 vs 1.61. That itches me the most.

- boost is > 114 MB on disk; Amr did't outline how to strip boost to its
essentials. The required functions (include files) seem to be included in his
repo.

- Amr implemented some functions that were already in geometry and some I
already had implemented. I did sort out which ones are the fastest for most of
these. Amr did good work anyway here.

- We need to discuss a clear distinction between what functions go in geometry
and which ones in mapping, and related: which ones in mapping might be
wrappers for what base functions in geometry. I think most of my original post
below still stands.

I have very little time this weekend & week, starting next Friday & next
weekend I can do a lot more. Can you wait until then? In the meantime I'll
polish & sort out my work on this.


I cannot find Ar's repo URL now (no time/in a rush) but it is in the
maintainers-ML archives in a GSOC review thread.
Tonight I can look into the URL


    _______________________________________________________

Reply to this item at:

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

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