stan-dev / bayesplot

bayesplot R package for plotting Bayesian models
https://mc-stan.org/bayesplot
GNU General Public License v3.0
433 stars 83 forks source link

copy/move `rhat()` and `neff_ratio()` to rstan(arm)? #119

Closed strengejacke closed 10 months ago

strengejacke commented 7 years ago

I was looking for functions to extract rhat and the ratio for number of effective samples, and found those in the brms-package for brmsfit-objects, but not in rstan or rstanarm for stanreg-objects (where I would expect those functions to be).

Instead, I later realized that these are in the bayesplot-pkg. Wouldn't it make more sense to have the functions (also) in rstan/rstanarm as well?

tjmahr commented 7 years ago

@jgabry would the generics go in rstantools and the S3 methods go into rstan and rstanarm? I'm not sure about the stanverse organization in this regard.

jgabry commented 6 years ago

Circling back through old issues. Yes I think having these as rstantools generics makes the most sense. I’ll open an issue for it over there.

Sent with GitHawk

jgabry commented 6 years ago

https://github.com/stan-dev/rstantools/issues/31

Sent with GitHawk

avehtari commented 10 months ago

I think these are now in posterior. Maybe this issue can be closed?

strengejacke commented 10 months ago

Yes, I think so, too.