Skip to content
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

SBC histogram #146

Open
dpsimpson opened this issue Apr 25, 2018 · 6 comments
Open

SBC histogram #146

dpsimpson opened this issue Apr 25, 2018 · 6 comments

Comments

@dpsimpson
Copy link
Collaborator

There is a code for making the histogram for the SBC paper in the sbc_histogram branch.

@jgabry
Copy link
Member

jgabry commented May 9, 2019

@avehtari Can this be closed given #178 and #179? Aren't the SBC plots a subset of the new diagnostic plots?

@avehtari
Copy link
Contributor

avehtari commented May 9, 2019

That issue and that pull request so far discuss functions which would take chains as an argument, and ranks are computed by combining the chains. In SBC the prior draw and posterior draws in one iteration are forming one rank, and sbc_histogram would probably take these ranks as argument (and the extra thing in the plotting is adding the interval butterfly behind).

Both mcmc_rank_histogram and sbc_histogram could use the same code for the plotting part, but it's not clear whether mcmc_rank_histogram should have that interval butterfly, ecdf difference with envelope is better for intervals. There would be more code to share in ecdf difference with envelope plot, but again the input arguments are different.

@jgabry
Copy link
Member

jgabry commented May 9, 2019

Ok, that makes sense, thanks.

@jpritikin
Copy link

If you don't mind, I can try merging your code from the SBC paper with my attempt?

@jgabry jgabry removed their assignment May 16, 2019
@jgabry
Copy link
Member

jgabry commented May 16, 2019

If the code on Dan's branch that he refers to above is useful go for it. Not sure if that's what you mean. If there's other code in with the paper that's useful I'm sure that's fine too.

@jgabry
Copy link
Member

jgabry commented May 16, 2019

You can definitely take whatever we already have in the code for the simple sbc plot in rstan and put it into this new bayesplot function with your additions. Up to you what you want to take and what you want to rewrite.

jpritikin added a commit to jpritikin/bayesplot that referenced this issue May 17, 2019
jpritikin added a commit to jpritikin/bayesplot that referenced this issue May 17, 2019
jpritikin added a commit to jpritikin/bayesplot that referenced this issue May 17, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants