[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Octave-patch-tracker] [patch #9000] Polygon functions for geometry/mapp
From: |
Philip Nienhuis |
Subject: |
[Octave-patch-tracker] [patch #9000] Polygon functions for geometry/mapping package |
Date: |
Sat, 1 Oct 2016 10:10:15 +0000 (UTC) |
User-agent: |
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:43.0) Gecko/20100101 Firefox/43.0 SeaMonkey/2.40 |
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/
- [Octave-patch-tracker] [patch #9000] Polygon functions for geometry/mapping package,
Philip Nienhuis <=