Please select a category the issue is focused on?
Function Documentation
Let us know where something needs a refresh or put your idea here!
Description
The documentation for derive_seq() currently includes an example for the APSC domain that is both technically incorrect and redundant.
The Problem:
SDTM Non-compliance: The example uses tgt_var = "APSEQ". Per SDTMIG-AP, the sequence variable for the APSC domain must be SCSEQ.
Redundancy: Once corrected to SCSEQ, the example is functionally identical to the VS / VSSEQ example already present in the documentation.
Proposed Change:
Remove the APSC code block from the @examples section in R/derive_seq.R to streamline the documentation and avoid spreading incorrect SDTM naming conventions.
Affected Page:
https://pharmaverse.github.io/sdtm.oak/reference/derive_seq.html
Please select a category the issue is focused on?
Function Documentation
Let us know where something needs a refresh or put your idea here!
Description
The documentation for
derive_seq()currently includes an example for the APSC domain that is both technically incorrect and redundant.The Problem:
SDTM Non-compliance: The example uses
tgt_var = "APSEQ". Per SDTMIG-AP, the sequence variable for theAPSCdomain must beSCSEQ.Redundancy: Once corrected to
SCSEQ, the example is functionally identical to theVS/VSSEQexample already present in the documentation.Proposed Change:
Remove the
APSCcode block from the@examplessection inR/derive_seq.Rto streamline the documentation and avoid spreading incorrect SDTM naming conventions.Affected Page:
https://pharmaverse.github.io/sdtm.oak/reference/derive_seq.html