loo icon indicating copy to clipboard operation
loo copied to clipboard

loo_predictive_metric and loo_crps could accept psis objects to reduce computation

Open jgabry opened this issue 2 years ago • 2 comments

If the user already has a psis object then loo_predictive_metric and loo_[s]crps could use that object instead of rerunning psis internally (similar to what E_loo already does, except in this case it would be optional). If no psis object is provided then they could compute it as they do now. @LeeviLindgren @avehtari what do you think? (This doesn't need to be done right now, it could be for the next release after this one.)

jgabry avatar Mar 22 '23 23:03 jgabry

I prefer the behavior in different functions to be consistent, so if E_loo does that then it would be good to have in other functions, too

avehtari avatar Mar 23 '23 08:03 avehtari

Agreed, makes sense

LeeviLindgren avatar Mar 24 '23 06:03 LeeviLindgren