pharmaverse / pharmaverse/tern
Refactor risk diff related functions (in particular `tabulate_rsp_subgroups`)
Open
Nobody has claimed this yet.
enhancement
sme
- Dominant language
- R
- Stars
- 106
- Forks
- 34
- Avg merge
- 8d 9h
- Merged PRs (30d)
- 6
Description
This refactor mainly aims at having a parameter to split the risk diff estimate and CI on different columns instead of a single one. On the side, it may be valuable to refactor afun_risk to respect modern {tern} function structure.
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 reading the risk-difference-related functions, especially tabulate_rsp_subgroups and afun_risk, to understand their current responsibilities and existing parameter structure. Done should include a parameter that places the risk-difference estimate and confidence interval in separate columns, with afun_risk refactored toward the modern {tern} function structure.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- r
- Domain
- data
- Issue type
- Refactor
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 30/100