-
-
Notifications
You must be signed in to change notification settings - Fork 19
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
estimate_dispersion() (and maybe estimate_zeroinflation()?) #136
Comments
What do you think about API here? @DominiqueMakowski |
We could add this easily, simply by calling |
I have little experience with ZI-like models to be honest, so I'll defer judgment here. If I understand, we are talking about two things:
does that make sense? |
I just saw that my comment was only addressing the 2nd point. Brenton was talking about incorporating uncertainty into intervals in his 1. point. |
Yes, sounds good. We could still use |
For models with dispersion models, it would be nice if two things were possible:
estimate_prediction()
.estimate_response()
and related functions (cf. get_predicted(): zero-inflation options insight#413) or a separateestimate_dispersion()
. Like withestimate_prediction()
, making this an alias ofestimate_response()
with default options might make sense.Relatedly, options for estimating zero-inflation predictions would be great.
The text was updated successfully, but these errors were encountered: