[Rcpp-devel] RcppMLPACK ?

[Qiang Kou]

Thanks for the opinions!

On Wed, Jun 11, 2014 at 7:38 AM, Dirk Eddelbuettel <edd at debian.org> wrote:

> And smart how you just added the little bit from Boost
> we don't have in BH (program_options).
>

As far as I know all parts in BH package are headers-only, and
"program_options" is required by MLPACK to handle command line options. As
I see, it will not be used, since we don't need those options when calling
it in R.


> Should the cpp files in inst/include be in src/, or maybe src/mlpack
> instead?


To be frank, I don't know which place is better. I just follow the
structures of RcppArmadillo.


> It may be less than ideal that the kmeans example hides the base function.
> Maybe make it mlKmeans, or keep it unexported, or ... ?
>

Of course, I will change names of functions, at least not the same with R
built-in functions.

Best,

KK
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.r-forge.r-project.org/pipermail/rcpp-devel/attachments/20140611/3e160ff5/attachment.html>