-
Notifications
You must be signed in to change notification settings - Fork 8
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
Return frequentist MMRM model object #274
Comments
We need to fit an MMRM model to the original data for all methods, right? One totally separate issue is that the MMRM to the original data does exclude "non-MAR" post-ICE data but I guess this is unavoidable. Also, it's not 100% clear to me how to do good residual diagnostics for an MMRM model but I leave that to Paul for now. |
Currently we do not fit the MMRM to the original dataset for the bmlmi or approx_bayes methods. Additionally I'm not sure it makes any sense to return it for the bayes method as its only used to initalise the stan sampler. |
Thanks for clarifying @gowerc |
@gowerc @wolbersm Just to be clear, currently |
Thanks, as long as the content of |
I'm not sure we would be able to document it other than saying its a "glmfit" object. Otherwise we have a moving target that our documentation is dependent on another teams implementation details. |
Yes, I think it's fine to just say that this contains the result of the imputation model fit returned by glmmTMB() or stan(). |
sounds good. I would also return the full object from the ancova function for the same reason to check residuals |
Just add here the relevant functions in case I forgot. There is a kind of chain relation among them
|
Hi @wolbersm ,
We have been asked a few times by Paul to add return the model object of the MMRM fit for the bootstrap methods for diagnostic / residual purposes.
Talking to @nociale our current proposal would be that we return the MMRM fit to the original dataset (only) for the condmean jackknife + bootstrap methods, for all other methods (bayesian, approx bayes, blmi) we would carry on returning nothing.
What do you think ?
The text was updated successfully, but these errors were encountered: