braverock / braverock/FactorAnalytics

fitTsfmLagLeadBeta, fitTsfmUpDn, fitTsfmMT

Open
#54 0 comments 0 reactions 2 assignees Claimed by @martinrd3D View on GitHub
enhancement question
Dominant language
R
Stars
85
Forks
63
PR merge metrics
No merged PRs in 30d

Description

Justin and I discussed that these functions should be options in the function fitTsfm.
Probably all three of these functions can go there as is, but we should check the option arguments of each, and how we pass them, as an immediate priority.

Also, fitTsfmLabLeadBeta currently implements the Dimson method without the correction of bias due to serial correlation in the market returns, and that needs to be added as an option, and furthermore the thought is to also enhance this thin trading beta capability to include the Scholes-Williams method.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.