-
Notifications
You must be signed in to change notification settings - Fork 4
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Alphastable fit #20
Alphastable fit #20
Conversation
Codecov Report
@@ Coverage Diff @@
## master #20 +/- ##
==========================================
+ Coverage 87.96% 89.47% +1.50%
==========================================
Files 1 1
Lines 133 152 +19
==========================================
+ Hits 117 136 +19
Misses 16 16
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It might be worth considering:
- Document the limits of parameters for which the fitting is meant to work clearly.
- Split out
AlphaStableDistribution
fromSymmetricAlphaStableDistribution
. - Make a comparison in this PR, before merging, of test cases that @azev77 mentions in Special Cases of AlphaStable #19, to see consistency.
And put those test cases from #19 in the test suite. |
Thanks for the reviews. The limits of parameters is included in the docstring. A new type |
The current
AlphaStable
fit based on Fama & Roll (1971) is limited to Symmetric Alpha Stable Distributions (as discussed in #19). This PR implements a more generalAlphaStable
fit based on McCulloch (1986) foralpha
in the range of[0.6, 2.0]
andbeta
in the range of[-1, 1]
.