Thanks for the opinions! On Wed, Jun 11, 2014 at 7:38 AM, Dirk Eddelbuettel <e...@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
_______________________________________________ Rcpp-devel mailing list Rcpp-devel@lists.r-forge.r-project.org https://lists.r-forge.r-project.org/cgi-bin/mailman/listinfo/rcpp-devel