[GSoC-PortA] chart.* as generics

[Ross Bennett]

Brian,

I have just a couple quick questions regarding chart.Weights and
chart.RiskReward as generic functions and cleaning up the documentation as
you did with the versioned functions.

My plan is to use an alias for the chart.Weights.* functions for the
different classes and then have a generic chart.Weights function with a
call to UseMethod("chart.Weights"). The first argument is RP for
chart.Weights.RP, DE for chart.Weights.DE, and so on. Should the arguments
of the chart.Weights.* functions be the same?

Should we have the @param and other tags only for one function? Would that
go on the generic chart.Weights function?

My approach would be the same for the chart.Scatter.* functions.

Thanks,
Ross


On Sun, Aug 18, 2013 at 1:21 PM, Brian G. Peterson <brian at braverock.com>wrote:

> Sounds good.  - Brian
>
>
> On 08/18/2013 03:06 PM, Ross Bennett wrote:
>
>> Brian,
>>
>> Thanks for clarifying, I was worried that might be the case. I could add
>> a new function named chart.RiskReward and make that our generic function
>> and use an alias for the existing chart.Scatter.* functions. It could
>> look something like this:
>>
>> chart.RiskReward.optimize.**portfolio.ROI <- chart.Scatter.ROI
>> chart.RiskReward.optimize.**portfolio.random <- chart.Scatter.RP
>>
>> Would that be ok and worthwhile to do?
>>
>> Ross
>>
>>
>>
>> On Sun, Aug 18, 2013 at 12:05 PM, Brian G. Peterson <brian at braverock.com
>> <mailto:brian at braverock.com>> wrote:
>>
>>     On 08/18/2013 01:35 PM, Ross Bennett wrote:
>>
>>
>>         The PerformanceAnalytics packages already has a chart.Scatter
>>         function.
>>         Will we have any naming or masking issues if I make chart.Scatter
>> a
>>         function for the different optimize.portfolio.objects?
>>
>>
>>     It's easy to make chart.Weights a generic, but it will be much
>>     harder with chart.Scatter.*
>>
>>     Because PerformanceAnalytics is on CRAN and much more widely used,
>>
>>     we'd need to match the argument list, and create a
>>     chart.Scatter.default in PerfA.  I don't see any way to match the
>>     argument list though.
>>
>>     --
>>     Brian
>>
> ______________________________**_________________
> GSoC-PortA mailing list
> GSoC-PortA at lists.r-forge.r-**project.org<GSoC-PortA at lists.r-forge.r-project.org>
> http://lists.r-forge.r-**project.org/cgi-bin/mailman/**listinfo/gsoc-porta<http://lists.r-forge.r-project.org/cgi-bin/mailman/listinfo/gsoc-porta>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.r-forge.r-project.org/pipermail/gsoc-porta/attachments/20130819/d15b8044/attachment.html>