Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

support missing cbox in missingSummary #403

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

couthcommander
Copy link
Contributor

issue #349

Count checkbox as missing when it is displaying under branching logic and no items are selected.

Consider the scenario with a checkbox group defined by five choices. If none are selected, is that five missing values or one? What is the desired output?

> missingSummary(rcon)
  record_id redcap_repeat_instrument redcap_repeat_instance n_missing                                                                     missing
1         1                     <NA>                   <NA>         6 notes, earlybed___1, earlybed___2, earlybed___3, earlybed___4, earlybed___5

> missingSummary(rcon)
  record_id redcap_repeat_instrument redcap_repeat_instance n_missing         missing
1         1                     <NA>                   <NA>         6 notes, earlybed

> missingSummary(rcon)
  record_id redcap_repeat_instrument redcap_repeat_instance n_missing         missing
1         1                     <NA>                   <NA>         2 notes, earlybed

@spgarbet
Copy link
Member

Hm. I would say one.

@spgarbet
Copy link
Member

@mcfibb Thoughts?

@mcfibb
Copy link

mcfibb commented Jul 15, 2024

Hey! Thanks for asking for my input. I would say count as "1 Missing Value". The question is the unit, not the options. Make sense?

@spgarbet spgarbet marked this pull request as ready for review July 31, 2024 16:59
@spgarbet spgarbet marked this pull request as draft July 31, 2024 16:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants