pharmaverse / sdtmchecks

Analysis facing checks for SDTM data
https://pharmaverse.github.io/sdtmchecks/
Apache License 2.0
21 stars 8 forks source link

Identify checks where Rave specific info is used #258

Open harriscw opened 1 year ago

harriscw commented 1 year ago

e.g. grepping --SPID for form. Identify alternatives if possible to help generalize

sarabodach commented 10 months ago

The below 3 checks have --SPID as a required variable:

check_ds_ae_discon check_ae_aeacn_ds_disctx_covid check_vs_sbp_lt_dbp

sarabodach commented 10 months ago

The check check_tr_dup has --SPID as an optional variable:

This check prints out the full TRSPID if this variable exists rather than naming it RAVE with simplified information. https://github.com/pharmaverse/sdtmchecks/blob/79ab6e0fd49a7fad1755100c80097b8d528c110e/R/check_tr_dup.R#L43

This could be incorrectly looking for duplicates in mapping more than data entry if the full TRSPID is included.