Randall,
We had experience with that check also; the 1.4 validator seems to have addressed it to not produce the unwarranted warning anymore.
Best,
Joan
The message and description are
I think Randal may be right though in that the rule eliminates unscheduled visits based on these text comparisons.
VISIT is not null
VISIT <> Upper('UNPLAN')
VISIT <> Upper('UNPLANNED')
VISIT <> Upper('UNSCHED%')
Let me look into this. the other area for improvement is assigning this rule to SV domain as well.
I see the problem clearer now.
CODE | MESSAGE | use SVUPDES ? | Assigned to | Lookup from |
SD1017 | VISITNUM value does not match TV domain data | Y | SV | TV |
SD1018 | VISITNUM/VISIT/VISITDY values do not match TV domain data | Y | SV | TV |
SD1019 | VISITDY is populated for unplanned visit | Y | SV | |
SD1023 | VISIT/VISITNUM values do not match TV domain data | N | All subject domains | TV |
SD0065 | USUBJID/VISIT/VISITNUM values do not match SV domain data | -- | All subject domains | SV |
Because checks 0065, 1017, 1018 already cover the responsibility of check 1023. Having check 1023 not only introduces false positive as reported above, it also duplicates the error messages already reported by 0065+1017+1018.
(Just in case the logic is not clear: A. If subject domains agree with SV, and SV agrees with TV --> subject domains agree with TV. and B. If subject domains do not agree with TV --> either subject domains do not agree with SV, or SV does not agree with TV, or both)
Programmers in our company still experience this issue in V1.4.
If check SD1023 is to stay, then please modify the check so that it uses SV to identify unplanned visits, not based on the spelling of VISIT.
Because checks 0065, 1017, 1018 already cover the responsibility of check 1023. Having check 1023 not only introduce the false positive, it also duplicates the error messages already reported by 0065+1017+1018.
Just in case the logic is not clear: A) If subject domains agree with SV, and SV agrees with TV --> subject domains agree with TV. and B) If subject domains do not agree with TV --> either subject domains do not agree with SV, or SV does not agree with TV, or both
Programmers in our company still experience this issue in V1.4.
If check SD1023 is to stay, then please modify the check so that it uses SV to identify unplanned visits, not based on the spelling of VISIT.
Hi Jim,
Based on our experience SD1023 is a valid and valuable check. The issues related to violation of the SD1023 business rule are still in the Top10 list.
A combination of 0065+1017+1018 is not the same as SD1023.
We are still collecting statistics to create a more intelligent algorithm for filtering of Unplanned and Unscheduled visits to avoid false-positive messages. The usage info from the SV domain for this purpose will be tricky because there is no guarantee that those data are 100% correct. E.g., something like an "ALL VISITS" value may have description in SVUPDES.
Could you share your particular cases as examples where SD1023 produces false-positive messages?
Regards,
Sergiy
SD1023 looks at looks at the text value of VISIT to decide whether a visit is unscheduled or not. There isn’t controlled terminology for unscheduled visit descriptions, nor is there any expectation expressed in the SDTM IG for a value convention. The appropriate way to check whether a visit is unscheduled is to check to see if the VISIT in SV has SVUPDES populated. We use the convention, "UNSCHEDULED - {visit number}", e.g. "UNSCHEDULED - 20.01" for unscheduled VISIT descriptions. We are getting lots of false hits with the OpenCDISC check that must be explained in the SDRG.