Shared exclusion implementation for all existing quality measures #440
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Shared exclusion tables
…alth#429) * add new clinical concepts * add new observation models * add new observation models * add observation model to final list * update models yml * increment package version and update dbt docs * remove diabetes in pregnancy codes * fix observation staging and casting issues
Nqf2372 shared exclusion
…deepsonshrestha/tuva into shared_exclusion_implementation
…tation Shared exclusion implementation
|
Hey @deepsonshrestha, I ran this branch on one of our customer datasets, and I'm getting some differences in counts that we need to look at. Check Slack for screenshots of the comparisons with main. Can you look at your changes again? The two measures that seem to be affected are NQF0034 and NQF2372. |
|
Workflow has finished with the following statuses:
|
sarah-tuva
approved these changes
Apr 30, 2024
|
We are good to go with this PR. With the latest version of the branch, I was able to validate against real data and confirmed that the differences in exclusion reasons were mostly related to changes in descriptions. The CI tests passed as well. Merging this one in! |
Sunnyinho
reviewed
May 1, 2024
| , exclusion_flag | ||
| , '{{ var('tuva_last_run')}}' as tuva_last_run | ||
| from add_data_types |
tuvaforrest
pushed a commit
that referenced
this pull request
May 7, 2024
* feat: changed exclusions * feat: shared exclusions for hospice and snp * feat: shared exclusions for frailty * feat: added common exclusion models and materialized as tables * docs: updated schema names * feat: updated the project version * refactor: changed the schema names and removed older excluion logic * docs: updated the tuva version * feat: updated the seed files * feat: deno models * feat: created denominator model and updated perf period * feat: updated both deno conditions * docs: updated the docs and specified the schema * osteo_numerator_procedure_only * schema definition for nqf0053 numerator in yml * changed codes from fracture diagnoses to procedures * pharmacy_claims-numerator initial * fix: refactored the source tables and updated the lookback period logic * feat: numerator from procedure and pharmacy_claims * minor: aesthetic changes * Add observation suspecting logic for HCC 48 - Morbid Obesity (#429) * add new clinical concepts * add new observation models * add new observation models * add observation model to final list * update models yml * increment package version and update dbt docs * remove diabetes in pregnancy codes * fix observation staging and casting issues * minor: denominator encounter filters * feat: updated encounter type * feat: numerator untested * feat: numerator with recorded date from deno * feat: updated emergency encounter type * feat: created custom exclusions for NQF0053 * docs: updated the docs and specified the schema * revert: reverted ED in encounter type * feat: numerator and denominator * feat: added lookback period for december * docs: 0053 numerator column description added in yml * minor: lookback fieldname change * feat: 0053 long model and docs * docs: staging medication table column update yml * refactor: combined two same ctes into one * minor: numerator * feat: created exclusion model * feat: updated long table * feat: exclusions_utsav * docs: exclusion yaml * feat: exclusion with shared exclusion name updated * feat: added final exclusion table * feat: added nqf0053 in summary tables * docs: removed unused columns * fix: removed unused columns in NQf0059 diabetes * docs: updated the tuva project version to 0.7.17 * minor: #15 pull request aesthetic changes * docs: updated seed with rx code ndc * feat: nqf0053 PR changes * feat: added age and exclusion type for measure specific exclusion * feat: implemeted shared exclusion and refactored the code * docs: removed unused columns from schema.yml * feat: updated the exclusions table * docs: updated the columns * removed older files * feat: denominator PR changes Co-authored-by: Sarah Morgan <sarah@tuvahealth.com> * prune: removed unused frailty exclusion * docs: tuva version update * feat: exclusion PR changes * fix: shared palliative concept added * fix: Palliative care diagnosis added in nqf2372 exclusion * docs: updated fields of 2372_exclusions, pruned unused model description * feat: implemented shared exclusion for NQF2372 * fix: performance period issue * fix: nqf2372 exclusion's fields edits in long model * feat: updated palliative concept name according to value concept seed * feat: implemented shared exclusion logic * prune: removed unused exclusion tables and updated the docs * BUGFIX: Correct union logic for shared exclusions in Quality Measures (#437) fix union logic * Add HCC suspecting logic for CKD (#432) * adding staging model for lab result * adding intermediate models lab suspects * add lab suspects to final model * update package version and dbt docs * update data type for lab results * add data_source to join for egfr lab suspect logic * update package version and dbt docs * NQF0053 Osteoporosis Management in women who had a fracture (#436) * feat: changed exclusions * feat: shared exclusions for hospice and snp * feat: shared exclusions for frailty * feat: added common exclusion models and materialized as tables * docs: updated schema names * feat: updated the project version * refactor: changed the schema names and removed older excluion logic * docs: updated the tuva version * feat: updated the seed files * feat: deno models * feat: created denominator model and updated perf period * feat: updated both deno conditions * docs: updated the docs and specified the schema * osteo_numerator_procedure_only * schema definition for nqf0053 numerator in yml * changed codes from fracture diagnoses to procedures * pharmacy_claims-numerator initial * fix: refactored the source tables and updated the lookback period logic * feat: numerator from procedure and pharmacy_claims * minor: aesthetic changes * Add observation suspecting logic for HCC 48 - Morbid Obesity (#429) * add new clinical concepts * add new observation models * add new observation models * add observation model to final list * update models yml * increment package version and update dbt docs * remove diabetes in pregnancy codes * fix observation staging and casting issues * minor: denominator encounter filters * feat: updated encounter type * feat: numerator untested * feat: numerator with recorded date from deno * feat: updated emergency encounter type * feat: created custom exclusions for NQF0053 * docs: updated the docs and specified the schema * revert: reverted ED in encounter type * feat: numerator and denominator * feat: added lookback period for december * docs: 0053 numerator column description added in yml * minor: lookback fieldname change * feat: 0053 long model and docs * docs: staging medication table column update yml * refactor: combined two same ctes into one * minor: numerator * feat: created exclusion model * feat: updated long table * feat: exclusions_utsav * docs: exclusion yaml * feat: exclusion with shared exclusion name updated * feat: added final exclusion table * feat: added nqf0053 in summary tables * docs: removed unused columns * fix: removed unused columns in NQf0059 diabetes * docs: updated the tuva project version to 0.7.17 * minor: #15 pull request aesthetic changes * docs: updated seed with rx code ndc * feat: nqf0053 PR changes * feat: denominator PR changes Co-authored-by: Sarah Morgan <sarah@tuvahealth.com> * docs: tuva version update * feat: exclusion PR changes --------- Co-authored-by: Deepson Shrestha <shresthadeepson317@gmail.com> Co-authored-by: Deepson Shrestha <53638102+deepsonshrestha@users.noreply.github.com> Co-authored-by: Sarah Morgan <sarah@tuvahealth.com> * Patch 0.7.17 (#439) * docs: revert package version * fix: reorder columns * docs: update dbt docs * docs: update dbt docs * fix: remove trailing blank rows * BUGFIX: Correct union logic for shared exclusions in Quality Measures (#437) fix union logic * Add HCC suspecting logic for CKD (#432) * adding staging model for lab result * adding intermediate models lab suspects * add lab suspects to final model * update package version and dbt docs * update data type for lab results * add data_source to join for egfr lab suspect logic * update package version and dbt docs * NQF0053 Osteoporosis Management in women who had a fracture (#436) * feat: changed exclusions * feat: shared exclusions for hospice and snp * feat: shared exclusions for frailty * feat: added common exclusion models and materialized as tables * docs: updated schema names * feat: updated the project version * refactor: changed the schema names and removed older excluion logic * docs: updated the tuva version * feat: updated the seed files * feat: deno models * feat: created denominator model and updated perf period * feat: updated both deno conditions * docs: updated the docs and specified the schema * osteo_numerator_procedure_only * schema definition for nqf0053 numerator in yml * changed codes from fracture diagnoses to procedures * pharmacy_claims-numerator initial * fix: refactored the source tables and updated the lookback period logic * feat: numerator from procedure and pharmacy_claims * minor: aesthetic changes * Add observation suspecting logic for HCC 48 - Morbid Obesity (#429) * add new clinical concepts * add new observation models * add new observation models * add observation model to final list * update models yml * increment package version and update dbt docs * remove diabetes in pregnancy codes * fix observation staging and casting issues * minor: denominator encounter filters * feat: updated encounter type * feat: numerator untested * feat: numerator with recorded date from deno * feat: updated emergency encounter type * feat: created custom exclusions for NQF0053 * docs: updated the docs and specified the schema * revert: reverted ED in encounter type * feat: numerator and denominator * feat: added lookback period for december * docs: 0053 numerator column description added in yml * minor: lookback fieldname change * feat: 0053 long model and docs * docs: staging medication table column update yml * refactor: combined two same ctes into one * minor: numerator * feat: created exclusion model * feat: updated long table * feat: exclusions_utsav * docs: exclusion yaml * feat: exclusion with shared exclusion name updated * feat: added final exclusion table * feat: added nqf0053 in summary tables * docs: removed unused columns * fix: removed unused columns in NQf0059 diabetes * docs: updated the tuva project version to 0.7.17 * minor: #15 pull request aesthetic changes * docs: updated seed with rx code ndc * feat: nqf0053 PR changes * feat: denominator PR changes Co-authored-by: Sarah Morgan <sarah@tuvahealth.com> * docs: tuva version update * feat: exclusion PR changes --------- Co-authored-by: Deepson Shrestha <shresthadeepson317@gmail.com> Co-authored-by: Deepson Shrestha <53638102+deepsonshrestha@users.noreply.github.com> Co-authored-by: Sarah Morgan <sarah@tuvahealth.com> * Patch 0.7.17 (#439) * docs: revert package version * fix: reorder columns * docs: update dbt docs * docs: update dbt docs * fix: remove trailing blank rows * feat: NQF2372 exclusion filtered * fix: syntax error * refactor: shared hospice palliative reorder * feat: added relevant concept names * refactor: changed to lowercase * docs: updated docs for NQF0034 * docs: 0034 yaml update * docs: updated the docs * docs: updated numerator for osteoporosis * docs: updated yaml for 0053 * Update dbt_project.yml * refactor: use existing cte for mastectomy * fix: made changes to age irrelvant fields * fix: shared exclusion acute inpatient filter * refactor: mastectomy exclusion segregated --------- Co-authored-by: utsavpaudel <utsavpaudel111@gmail.com> Co-authored-by: Sarah Morgan <sarah@tuvahealth.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Describe your changes
This PR aims to implement the shared exclusion logic for all the existing quality measures.
How has this been tested?
This was tested by querying against the production
summary_countstable and same numbers were generated.Reviewer focus
Palliative concept names are inconsistent across different measures. However only the concept names incorporated in the production have been used.
Checklist before requesting a review
(Optional) Gif of how this PR makes you feel
Loom link