You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
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
The issue is atomic
The issue description is documented
The issue title describes the problem succinctly
Developers are assigned to the issue
Labels are assigned to the issue
The text was updated successfully, but these errors were encountered:
Feature Request Description
We currently have four functions that do the bulk of redcap processing work.
clean_redcap()
hasdistill_repeat_table()
anddistill_nonrepeat_table()
.clean_redcap_long()
hasdistill_repeat_table_long()
anddistill_nonrepeat_table_long()
. There's a good amount of duplicated code between these helper functions.Proposed Solution
Combine
distill_repeat_table()
anddistill_nonrepeat_table()
into a singledistill_table()
. Combinedistill_repeat_table_long()
anddistill_nonrepeat_table_long()
into a singledistill_table_long()
. This will reduce duplication without going so far as to mix our longitudinal and non-longitudinal processing together.Checklist
The text was updated successfully, but these errors were encountered: