<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">Thanks for the opinions!<br></div><div class="gmail_quote"><br></div><div class="gmail_quote">On Wed, Jun 11, 2014 at 7:38 AM, Dirk Eddelbuettel <span dir="ltr"><<a href="mailto:edd@debian.org" target="_blank">edd@debian.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div class="">And smart how you just added the little bit from Boost<br>
</div>
we don't have in BH (program_options).<br></blockquote><div><br></div><div>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.</div>
<div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
Should the cpp files in inst/include be in src/, or maybe src/mlpack instead?</blockquote><div><br></div><div>To be frank, I don't know which place is better. I just follow the structures of RcppArmadillo.</div><div>
 </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
It may be less than ideal that the kmeans example hides the base function.<br>
Maybe make it mlKmeans, or keep it unexported, or ... ?<br></blockquote><div><br></div><div>Of course, I will change names of functions, at least not the same with R built-in functions.</div><div><br></div><div>Best,</div>
<div><br></div><div>KK</div></div>
</div></div>