REDCapTidieR icon indicating copy to clipboard operation
REDCapTidieR copied to clipboard

Combine distill_*_table functions

Open ezraporter opened this issue 2 years ago • 0 comments

Feature Request Description

We currently have four functions that do the bulk of redcap processing work. clean_redcap() has distill_repeat_table() and distill_nonrepeat_table(). clean_redcap_long() has distill_repeat_table_long() and distill_nonrepeat_table_long(). There's a good amount of duplicated code between these helper functions.

Proposed Solution

Combine distill_repeat_table() and distill_nonrepeat_table() into a single distill_table(). Combine distill_repeat_table_long() and distill_nonrepeat_table_long() into a single distill_table_long(). This will reduce duplication without going so far as to mix our longitudinal and non-longitudinal processing together.

Checklist

  • [x] The issue is atomic
  • [x] The issue description is documented
  • [x] The issue title describes the problem succinctly
  • [ ] Developers are assigned to the issue
  • [ ] Labels are assigned to the issue

ezraporter avatar Oct 28 '22 14:10 ezraporter