Replace loo::psis with posterior::pareto_khat (when available)
Closed this issue · 6 comments
Currently loo::psis
is used to calculate the pareto-k diagnostics, but there is work to have a pareto_khat
diagnostic function in posterior. Using this instead would remove the dependency on loo
and allow for loo
to suggest iwmm
(as discussed here).
I can make a branch implementing this change and create a PR once the corresponding functions are merged into posterior
I made a draft of this change in the branch https://github.com/topipa/iwmm/tree/use-posterior-k. Seems to work locally with the current posterior package branch https://github.com/n-kall/posterior/tree/pareto_k (Pull request here) .
Nice! There's also https://github.com/n-kall/iwmm/tree/posterior-pareto-k in case there is some detail helpful there
The PR for posterior implementing pareto-k diagnostics is now merged! So this can go ahead, but there might have been some changes since this draft
That's great! When I made the draft branch, I did not realize you had already done similar changes in your fork. So my changes are mostly duplicate of yours 🙈 I can take a look at both and pick relevant changes from both of them.
No worries, I forgot to mention my progress earlier.
I can take a look at both and pick relevant changes from both of them.
This sounds perfect! Let me know if you have any questions on the pareto-k functionality in posterior or my attempt in the fork