-
-
Notifications
You must be signed in to change notification settings - Fork 34
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 R2 #201
Comments
@sims1253 do you need something else than what is in the PRs mentioned above, or can we close this issue? |
We do have rmse, but we're still missing R2 (which is available in brms and rstanarm, but not as plain loo package version) |
At least the brms version does not use psis, which is why I wrote the version from my pr. |
brms |
True. Sorry, I'm currently on vacation and only have my phone and only found the Bayes r2 function. |
It would be nice if the loo objects weren't hardcoded to use elpd. For a study I wanted to include rmse and R2 and had to build fake loo objects in order to use the loo_compare functionality. Code is here.
The text was updated successfully, but these errors were encountered: