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

Add functions for PIT and LOO-PIT #338

Open
TeemuSailynoja opened this issue Jan 19, 2024 · 6 comments
Open

Add functions for PIT and LOO-PIT #338

TeemuSailynoja opened this issue Jan 19, 2024 · 6 comments

Comments

@TeemuSailynoja
Copy link

loo_pit is currently in rstantools but needs implementation for discrete variables, and should probably be moved to posterior.
pit isn't currently available, but is used, for example, by bayesplot, where I have currently implemented the computation.

My questions are:
1 . PIT would require observations in addition to the draws object, which to my understanding isn't the case for other functions. Is this a problem?
2. Should there be a new subtitle on the functions page for PIT and LOO-PIT to fall under?
3. Should they be in the same function as regular PIT is just LOO-PIT with even weights?

@avehtari
Copy link
Collaborator

  1. I think it fits here. It's still generic so that there is no dependency on where the draws come from, and if pit function would support draws objects it would be natural here. Yesterday @paul-buerkner also agreed initially
  2. Makes sense
  3. Makes sense

@paul-buerkner
Copy link
Collaborator

I think that PIT in posterior does make sense. Will we want to use it for other stuff than plotting though? If yes, than posterior is I think good. If we only ever want to use it for bayesplot, then perhaps also bayesplot could be a reasonable target(?)

@paul-buerkner
Copy link
Collaborator

Just to clarify, are we talking about the methods or the generic (or both)?

@avehtari
Copy link
Collaborator

  • PIT values could be summarised with scalar diagnostic value, but the preferred way is graphics, but we may want to use other plots than what is available in bayesplot (which is what I was doing)
    1. methods, 2. it would be nice if the method function could be called with any posterior draws object type

@paul-buerkner
Copy link
Collaborator

paul-buerkner commented Jan 19, 2024 via email

@jgabry
Copy link
Member

jgabry commented Jan 22, 2024

Sounds good to me too

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants