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
ENH: Betareg rebased #4238
ENH: Betareg rebased #4238
Conversation
This pull request introduces 3 alerts - view on lgtm.com new alerts:
Comment posted by lgtm.com |
still failures, looks like it fails when scipy <= 0.16 and passes with newer scipy. (bfgs changes again?) |
|
||
results_class = getattr(self, 'results_class', | ||
GenericLikelihoodModelResults) | ||
genericmlefit = results_class(self, mlefit) |
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.
Is this now generically implemented already in master?
see #2122
count_model uses result_class
(which is missing an "s")
this is a rebased version of my branch https://github.com/josef-pkt/statsmodels/tree/betareg_2030_3 of PR #2030
I just did the rebase, I don't know what the status is
The unit tests still have some commented out yield statements without corresponding assert
When I use
assert_allclose(links.logit()(rslt.params[-2:]), expected_methylation_precision['Estimate'], 1e-3)
instead of the yield, then I get a large test failure
So there are still problems with convergence when
link_precision=links.identity()
.(Which I don't think makes a lot of sense anyway, given the positivity constraint on variance/precision)
or there are problems in the unit test themselves.