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

GitHub actions claim clinical enabled #398

Merged
merged 28 commits into from
Mar 5, 2024

Conversation

thutuva
Copy link
Member

@thutuva thutuva commented Feb 26, 2024

Describe your changes

Please include a summary of any changes.

  • added two workflows to Snowflake to only run claims models and only run clinical models
  • upgraded the dbt version to 1.6.3
  • changed source schema to input_layer
  • changed the data type of each enabled statement in every model to boolean

How has this been tested?

Please describe the tests you ran to verify your changes. Provide instructions or code to reproduce output.

  • CI testing passes

Reviewer focus

Please summarize the specific items you’d like the reviewer(s) to look into.

Checklist before requesting a review

  • I have updated the version number in dbt_project.yml file to reflect the release number of this PR
  • I have updated the docs files (by running dbt docs generate/serve and copying the necessary files into the docs folder)
  • I have commented my code as necessary
  • I have added at least one Github label to this PR
  • My code follows style guidelines
  • (Optional) I have recorded a Loom to explain this PR

(Optional) Gif of how this PR makes you feel

Loom link

https://www.loom.com/share/e1f229db8ceb42219a991d583f7bafed?sid=26eaf167-bc9a-4ac0-b4b7-69209dbb7c18

Copy link

Workflow has finished with the following statuses:

  • Snowflake: success
  • Redshift: success
  • BigQuery: success

@thutuva thutuva merged commit 784d3c1 into main Mar 5, 2024
5 checks passed
@thutuva thutuva deleted the github-actions-claim-clinical-enabled branch March 5, 2024 15:07
deepsonshrestha pushed a commit to deepsonshrestha/tuva that referenced this pull request Mar 11, 2024
* updated ci with clinical/claims vars; added dbt v1.7.3

* fixed spacing

* update job names

* changes to syntax

* troubleshooting

* troubleshooting

* troubleshooting

* troubleshooting

* Add as_bool to enabled configs

* updated github workflows

* corrected workflow syntax

* changed comman to build

* fixed syntax; added dbt v1.7.3 for testing

* updating ci to use dbt v 1.6.3

* testing chaining jobs

* chaining jobs

* fixed bq syntax

* changed query timeout; edited outside contributor workflow

* changed loading method of seeds that take over 30 sec

* added data types

* added length to data type

* edited dbt commands to optimize ci pipeline

* fixing job name in needs

* fixing needs statement

* fixed spacing

* edits to the jobs runs

---------

Co-authored-by: Eldon Jenkins <[email protected]>
sarah-tuva added a commit that referenced this pull request Mar 13, 2024
* feat: updated the value seeds

* feat: created performance period and denominator for bp

* feat: excluded hospice and palliative patients

* docs: updated column level docs and specified schema

* feat: excluded institutional and snp patients

* feat: created fraility exclusion table with older patients flag

* feat: excluded patients with dementia having age below 80

* docs: updated the docs for controlling blood pressure

* feat: created exclusion for patients with advanced illness for patients younger than 81

* docs: updated docs and configured schema for advanced illness data model

* feat: combined all exclusions into a single table

* docs: updated docs and configured schema for exclusion table

* feat: created numerator for controlling blood pressure

* docs: updated docs

* feat: created long table and updated docs

* feat: changed to using ref

* fix: updated logic while joining bp ctes

* feat: created file for excluding custom observations and procedures

* docs: updated schema and docs

* feat: added to final exclusion table

* feat: updated the numerator logic and summary tables

* feat: excluded patients manual readings

* fix: filtered patients according to dates

* docs: updated the project version

* GitHub actions claim clinical enabled (#398)

* updated ci with clinical/claims vars; added dbt v1.7.3

* fixed spacing

* update job names

* changes to syntax

* troubleshooting

* troubleshooting

* troubleshooting

* troubleshooting

* Add as_bool to enabled configs

* updated github workflows

* corrected workflow syntax

* changed comman to build

* fixed syntax; added dbt v1.7.3 for testing

* updating ci to use dbt v 1.6.3

* testing chaining jobs

* chaining jobs

* fixed bq syntax

* changed query timeout; edited outside contributor workflow

* changed loading method of seeds that take over 30 sec

* added data types

* added length to data type

* edited dbt commands to optimize ci pipeline

* fixing job name in needs

* fixing needs statement

* fixed spacing

* edits to the jobs runs

---------

Co-authored-by: Eldon Jenkins <[email protected]>

* Quality Measure Evidence Value (#409)

* quality measures updates; dbt doc updates

* quality measures updates; dbt doc updates

* Update ccsr__long_condition_category.sql

added back as_bool

* Update quality_measures__int_nqf0059_numerator.sql

removed duplicative select of lab.result

* removed duplicate select of lab result (i.e. evidence_value)

* fix: updated the logic for diastolic readings

* feat: added evidence result

* fix: addressed PR comments

* feat: updated the concept names

* fix: removed excess comma

* feat: added evidence value column in the summary long table

* docs: updated the schema.yml file for newly added evidence value across all measures

* feat: added |as bool for config tags

* fix: added new concept names

* Update models/quality_measures/quality_measures_models.yml

Co-authored-by: Sarah Morgan <[email protected]>

---------

Co-authored-by: Deepson <[email protected]>
Co-authored-by: thutuva <[email protected]>
Co-authored-by: Eldon Jenkins <[email protected]>
Co-authored-by: Sarah Morgan <[email protected]>
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

Successfully merging this pull request may close these issues.

3 participants