Open JustinMShea opened 2 years ago
Having reviewed more of the plot.ffm examples in the vignette code Doug_versionOfSangeetha_fitFfmVignette.R, I am near to concluding that we should just do away with plot.ffm, But I will check the examples in the plot.ffm man page and see if I can get them all to work - with Kirk's help. In that case we can just leave plot.ffm in FA for now.
Ok @martinrd3D , just let me know. Any additional updates here? I'll just move it into "Under Review" on the board for now.
Kirk is working on this, and told me yesterday that he was half-way finished.
On Wed, Dec 22, 2021 at 12:53 AM Justin M. Shea @.***> wrote:
Ok, let me know. Any additional updates here? I'll just move it into "Under Review" on the board for now.
— Reply to this email directly, view it on GitHub https://github.com/braverock/FactorAnalytics/issues/70#issuecomment-999398701, or unsubscribe https://github.com/notifications/unsubscribe-auth/AP575Y2DXHUE4TXACZ7ZTY3USGGYXANCNFSM5J5FVSAA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were assigned.Message ID: @.***>
-- R. Douglas Martin Professor Emeritus in Applied Mathematics and Statistics University of Washington
@kecoli I believe your recent PR #82 was to partially fix this issue, though we still have more items to go here.
With the following example:
https://github.com/braverock/FactorAnalytics/blob/master/R/plot.ffm.R#L111-L122
option 3: Actual and Fitted asset returns
option 4: Time-series of R-squared values
Doesn't have an error, but chart isn't produced
option 8: Factor Model Return Correlation
Also some, slightly different issues with data.table methodology.