Use more functions from the posterior package
Open
A pull request for this has already been merged.
- #289 by @VisruthSK — merged
- Dominant language
- R
- Stars
- 157
- Forks
- 38
- Avg merge
- 4d 16h
- Merged PRs (30d)
- 2
Description
Release 2.7.0 imports posterior. We could use more posterior package functions. This would reduce possibility of having different implementations and thus reduce maintenance burden. Potential functions to replace
ess_rfunfft_next_good_sizeautocovariance
- part of
do_psis_ipsis_smooth_tailn_paretoenough_tail_samples
ps_khat_thresholdpareto_smooth.pareto_smooth_tailas_one_logicalshould_return_NAis_constant
ess_tailess_quantile.ess_quantile.ess.split_chains
gpdfitlxadjust_k_wipqgpd
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by reviewing the listed loo entry points, including ess_rfun, do_psis_i, pareto_smooth, ess_tail, and gpdfit, and compare them with the corresponding posterior package functions. Use merged pull request #289 as context; done means the relevant duplicate implementations have been replaced without changing behavior.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- r
- Domain
- data
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 20/100