insightsengineering / insightsengineering/teal.modules.clinical

migrate from `useNA` to `na.rm` arguments in teal module arguments

Open
#46 0 comments 0 reactions 0 assignees View on GitHub
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

Open the contributing 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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.