Closed TheCedarPrince closed 8 months ago
@Farreeda was working through some tutorial code and noticed that the ExecuteAudit code was broken for an example she was running. Upon investigation, I realized the reason for the break is that the function is too brittle and should have a kwarg added to it that allows a user to define on what column the audit should be conducted on (code: https://github.com/JuliaHealth/OMOPCDMCohortCreator.jl/blob/77126d02e304479991fe9fde9f47f71cb3d6ebaa/src/executors.jl#LL1-L17)
ExecuteAudit
To fix this issue, a kwarg needs to be added to allow a user to define what column should be targeted for auditing and at least one test added testing this new kwarg.
Resolved by #60 -- thanks @Jay-sanjay! :smiley:
@Farreeda was working through some tutorial code and noticed that the
ExecuteAudit
code was broken for an example she was running. Upon investigation, I realized the reason for the break is that the function is too brittle and should have a kwarg added to it that allows a user to define on what column the audit should be conducted on (code: https://github.com/JuliaHealth/OMOPCDMCohortCreator.jl/blob/77126d02e304479991fe9fde9f47f71cb3d6ebaa/src/executors.jl#LL1-L17)To fix this issue, a kwarg needs to be added to allow a user to define what column should be targeted for auditing and at least one test added testing this new kwarg.