Skip to content
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

Overview and Checkresults counts are not matching when missing specific checkNames #565

Closed
Bharathi1991 opened this issue Sep 10, 2024 · 5 comments

Comments

@Bharathi1991
Copy link

Bharathi1991 commented Sep 10, 2024

I am using R 4.3.1 and DQD 2.6.1. I am noticing a difference between checkResults and Overview object in the results produced by executeDQChecks method in DQD Dashboard when certain checkNames are omitted from the list. These checkNames are defined here in calculateNotApplicableStatus.R. In the absence of any one of these 3 checkNames, the code does not update the default value for passed column which is zero (initialized here and updated within calculateNotApplicableStatus.R here. So, if we miss to add those 3 checkNames, this results in this mismatch. Screenshots showing this behavior can be seen below,

resultsViewer

Showing 0 for passed column
image (1)

Overview

results$overview object showing a non-zero countPassed value
image3

DQD Dashboard

Results from inaccurate checkResults object
image (2)
Relevant OHDSI Forum post - https://forums.ohdsi.org/t/dqd-results-mismatch-between-overview-and-checkresults-when-missing-specific-checknames/22337

Can anyone help me understand why we require these 3 speciifc checks to update the column. Just curious to know if these 3 checks are of some significance among the rest. If not, can we please update the code so that the overview and checkResults objects are always in alignment with each other.

@katy-sadowski
Copy link
Collaborator

Thanks @Bharathi1991 for moving your report over here! One thing to confirm - when you do include those 3 checks, you do not experience this issue with the pass/fail counts in the summary table? Also, are you seeing any issue with check statuses in the full list of check results, or does this just impact the summary table?

@Bharathi1991
Copy link
Author

Hi @katy-sadowski , Can you please clarify if you referring to the table that gets created in results schema dqdashboard_results by summary table?

The checkResults object and the table that gets created (dqdashboard_results) are in alignment with each other. So they are both equally impacted.

And when we include all these 3 checks in checkNames, then the isPassed column in checkResults is accurate and it matches with the counts in results$Overview object. When missing any one of those 3 checks, the checkResults and dqdashboard_results have inaccurate check status values.

Hope I understood your question correctly. Can you please let me know if my answer is unclear.

@katy-sadowski
Copy link
Collaborator

Sorry I wasn't clear and thanks for the additional info! In my comment I was referring to the Shiny app UI, and "summary table" = the last screenshot in your post, and "full list of check results" = this:

Screenshot 2024-09-12 at 7 39 58 PM

So you did answer my first question. In my second question, I'd like to know if all checks are listed correctly with the correct statuses in the list of checks in the Shiny App.

@katy-sadowski
Copy link
Collaborator

@Bharathi1991 any update here?

@katy-sadowski
Copy link
Collaborator

closing for now - @Bharathi1991 please re-open if you're still experiencing issues.

@katy-sadowski katy-sadowski closed this as not planned Won't fix, can't repro, duplicate, stale Dec 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants