-
Notifications
You must be signed in to change notification settings - Fork 6
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
Add how to use pp_check to FAQ #541
Comments
I assume that this works for |
Have you checked to see if it works with linelist and weighted data? My expectation would be it doesn't
I think the first step is checking this and thinking about how it works. What happens if you pass new data? What happens if you don't enforce a type? The references to rows in https://mc-stan.org/bayesplot/reference/PPC-overview.html makes me wonder what the general support for this function for weighted data is like? |
I have reproduced this will all models and with models using straight brms. The issue is that |
pp_check()
methods
Another alternative is we add this to the docs. I am not sure where a good place for this would be right now as we currently only showcase fitting one distribution and it is choosing the distribution where I think this workflow matters most. Perhaps the other place for it could be the ebola vignette as a way of justifying the more complex stratified models? |
I had been interested to use |
As that vignette has been shortened maybe that is no longer the case. I will take another look at it this week |
Describe the bug
The
pp_check
methods frombrms
work incorrectly with theepidist_aggregate_data
.To Reproduce
Expected behavior
The figure should have a barplot with the data observed
fit$data$n
times.Session information
The text was updated successfully, but these errors were encountered: