insightsengineering / insightsengineering/teal.modules.clinical
migrate from `useNA` to `na.rm` arguments in teal module arguments
Open
sme
- Dominant language
- R
- Stars
- 42
- Forks
- 22
- Avg merge
- 22m
- Merged PRs (30d)
- 1
Description
It is more general and not just applicable for factor tabulation... I think long term we should change the teal module argument. Then it will also be consistent with `tern`.
Provenance:
```
Creator: danielinteractive
```
Contributor guide
Research direction
Start by locating teal module arguments that use `useNA` and compare their interfaces with `tern`, as requested in the issue. Determine the complete set of modules affected and define the migration scope before changing arguments; done means the relevant teal modules consistently use `na.rm`.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- r
- Domain
- api
- Issue type
- Refactor
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 25/100