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
After chatting with @DougManuel and @yulric, it appears that this duplication is due to the order in which the different flow packages were developed (i.e., cchsflow, elderflow, followed by recodeflow). Ideally, some refactoring needs to be done such that database-specific flow packages are mainly concerned with the variable sheet, details sheet, and derived variable functions, and recodeflow is taken as a dependency such that recodeflow::rec_with_table can be used with consistency.
The text was updated successfully, but these errors were encountered:
I have been browsing through the source code of flow packages and noticed that the
rec_with_table
function is defined in multiple places:After chatting with @DougManuel and @yulric, it appears that this duplication is due to the order in which the different flow packages were developed (i.e.,
cchsflow
,elderflow
, followed byrecodeflow
). Ideally, some refactoring needs to be done such that database-specific flow packages are mainly concerned with the variable sheet, details sheet, and derived variable functions, andrecodeflow
is taken as a dependency such thatrecodeflow::rec_with_table
can be used with consistency.The text was updated successfully, but these errors were encountered: