Peer Review - impute_sdtm
barnett11 opened this issue · 1 comments
barnett11 commented
Background Information
Linked to #99
@reesnj FYI
In description for impute_sdtm:
Text should read with the addition of under "Value"
Should we explicitly state varout will be POSIXct datetime format?
We should probably add the imputation rules into the Data Handling Rules vignette, along with the handling missing dates section?
impute_sdtm processing:
It works well for the standard Roche conventions of partial dates, but I'm wondering if we need to cover other potential ISO date conventions as below from SDTMIG?
Definition of Done
No response
reesnj commented
Hello @barnett11, please confirm whether this issue can now be closed after merging the updates into devel or whether further updates may be required.