mark padgham
mark padgham
@mkuehn10 I've just put a comment in [@symbolixAU's benchmarks](https://gist.github.com/SymbolixAU/0f65a431a53da6bdaf5d60b8ff30eba9) that shows how `geodist` can fairly easily be given around a ~~3-times~~ 28% (sorry, my mistake there) speed boost, which ~~would...
This is now officially a non-trivial issue, because the [benchmark](https://gist.github.com/SymbolixAU/0f65a431a53da6bdaf5d60b8ff30eba9) clearly shows that the C code in indeed quite a bit faster than C++, and so ... ### The Question...
Now that would be an interesting idea ... but nah, I do not want to dig up my fortran skills (had to do a tiny bit of that back in...
Thanks @asardaes for the useful insights. I concur with the `Rcpp` concerns, and my really would prefer to keep this package as lightweight as possible, which would be not easy...
This is now all done via #31
Closing this for now; the additional quadtree stuff can be implemented later if desired.
All grand by me, but with one caveat: The number of exposed functions is actually pretty huge: [`geodist_init.c`](https://github.com/hypertidy/geodist/blob/master/src/geodist_init.c) lists them all: ``` C extern SEXP R_cheap(SEXP); extern SEXP R_cheap_paired(SEXP, SEXP);...
An alternative would be for me to write a C-level meta-wrapper around all of those internal functions. The latter could then be left so, and the exposed bit used to...
I'm happy to leave this issue open for a while. I'll attend to some others first, and then slowly come around to this. Thanks for the impetus!
I'm less sure about this. I've got `alphahull` code integrated with both [`spatialcluster`](https://github.com/mpadge/spatialcluster/blob/master/R/plot-fns.R#L43-L87) and [`gtfs-router`](https://github.com/ATFutures/gtfs-router/blob/master/R/isochrone.R#L226-L255), so you may feel free to pilfer from there. But redoing the code of that...