-
-
Notifications
You must be signed in to change notification settings - Fork 69
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
Report brms refitting models #374
Comments
@rempsyc @mattansb @DominiqueMakowski I'm not sure, but I guess this is due to computing Bayes factors when retrieving the performance-table? We should try to find the bottle neck here, maybe using the profile package. @jgeller112 do you have a model you can share, so we can try to reproduce your issue? Else, maybe you find a model out of our pre-compiled models from here: https://easystats.github.io/circus/reference/index.html and just name one of those example models that behaves similarly. |
I think this is because report standardizes coefficients using the refit method (which it 100% should not do since it does not adjust the priors accordingly). |
Here you go:
|
If I understand the profiling right, one issue is And another bottle neck is Given that already the simple call to |
yes, the standardization method should be changeable to something else than "refit" (?) |
@jgeller112 Can you then try to run |
I get
|
Maybe I am missing something, but is there a way to avoid always refitting brms models using the report function? I have pretty large models and using report will either error out or run forever.
The text was updated successfully, but these errors were encountered: