-
Notifications
You must be signed in to change notification settings - Fork 97
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Link to NOT_APPLICABLE docs in Shiny app UI. #550
Comments
Hi @MelaniePhilofsky ! Could you please clarify which error message you're referring to? The NOT_APPLICABLE status is designed for the scenario you're describing - these are checks that can be disregarded because a column or table is not populated. Docs have more info: https://ohdsi.github.io/DataQualityDashboard/articles/CheckStatusDefinitions.html#not-applicable |
Thank you for the quick reply! The documentation is very helpful. I’ll close this issue since the documentation fully explains the STATUS = ‘NOT APPLICABLE”.Is there a way we can link each DQD check result with the documentation explaining the error?Cheers,MelanieOn May 25, 2024, at 9:36 AM, Katy Sadowski ***@***.***> wrote:
Hi @MelaniePhilofsky ! Could you please clarify which error message you're referring to?
The NOT_APPLICABLE status is designed for the scenario you're describing - these are checks that can be disregarded because a column or table is not populated. Docs have more info: https://ohdsi.github.io/DataQualityDashboard/articles/CheckStatusDefinitions.html#not-applicable
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: ***@***.***>
|
That's a good idea, Melanie. I'll keep this issue open and pull it into work for the upcoming release which is focused on documentation. |
While reviewing our most recent, DQD v2.6, results. I came across some new checks with wording that might be misinterpreted by users reviewing the checks.
The checks have DQD STATUS = "NOT APPLICABLE".
The error message states the data are "missing". This is a little nit picky, but missing Implies the data should be there. However, these fields are not required. And we have purposely not populated the field (i.e. Payer_Plan_Period.sponsor_concept_id) since we do not have these data. I would suggest "missing" be replace by "not present".
The text was updated successfully, but these errors were encountered: