<div dir="ltr">Brian,<div><br></div><div style>Thanks for the note.</div><div style><br></div><div style>Regarding the S3 generic methods. My intention with the chart.Weights.EF function was that it would be a generic method for optimize.portfolio and efficient.frontier objects. Since chart.Weights is already a generic method for optimize.portfolio objects, I don't think chart.Weights can be used a generic function to chart weights along the efficient frontier. I think I should just rename chart.Weights.EF to chart.WeightsEF.</div>
<div style><br></div><div style>Regarding the other documentation issues, I'll get to work on cleaning those up.</div><div style><br></div><div style>Thanks,</div><div style>Ross</div></div><div class="gmail_extra"><br>
<br><div class="gmail_quote">On Tue, Sep 3, 2013 at 9:02 AM, Brian G. Peterson <span dir="ltr"><<a href="mailto:brian@braverock.com" target="_blank">brian@braverock.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
I just ran R CMD check on current svn, and it's kind of a mess.<br>
<br>
'R CMD build' runs fine, which is good.<br>
<br>
The vignettes take over 100 minutes to build using only one core on my 8 (physical) core desktop.  CRAN won't like that, but I think it is necessary. I think we can make a case to CRAN that we need non-toy examples for everything, and they can get over it.<br>

<br>
The rest of check is going to need some work.<br>
<br>
By far the biggest thing is missing/incorrect parameters in the documentation.  This should all be worked out by making rure the roxygen comments are up to date and that the docs are regenerated.<br>
<br>
Another big area of check wordiness is in the S3 methods (e.g. chart.Scatter, chart.Weights, extractStats) , which we had agreed should be collapsed into a single main entry as I had done with optimize.portfolio, etc.  We might need to add some @method tags to the roxygen comments.<br>

<br>
I think it's rather important to start cleaning this stuff up, we've only got a few weeks left in GSoC.<br>
<br>
Full results of 'R CMD check --no-vignettes' and the generated manual attached.<br>
<br>
Regards,<br>
<br>
Brian<span class="HOEnZb"><font color="#888888"><br>
<br>
-- <br>
Brian G. Peterson<br>
<a href="http://braverock.com/brian/" target="_blank">http://braverock.com/brian/</a><br>
Ph: <a href="tel:773-459-4973" value="+17734594973" target="_blank">773-459-4973</a><br>
IM: bgpbraverock<br>
</font></span><br>_______________________________________________<br>
GSoC-PortA mailing list<br>
<a href="mailto:GSoC-PortA@lists.r-forge.r-project.org">GSoC-PortA@lists.r-forge.r-project.org</a><br>
<a href="http://lists.r-forge.r-project.org/cgi-bin/mailman/listinfo/gsoc-porta" target="_blank">http://lists.r-forge.r-project.org/cgi-bin/mailman/listinfo/gsoc-porta</a><br>
<br></blockquote></div><br></div>