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

Terminology Versioning and ICD 10 CM Update #461

Merged
merged 19 commits into from
May 10, 2024
Merged

Conversation

thutuva
Copy link
Member

@thutuva thutuva commented May 10, 2024

Describe your changes

Please include a summary of any changes.

  • Created a way to version terminology.
    • Created new folders in s3 and re-pointed post hooks
  • Uploaded all seed files into s3 and removed content from dbt
  • Updated the icd-10-cm file to contain header codes. header_flag can be used to determine which codes are header codes

How has this been tested?

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

  • ran dbt seed --full-refresh and dbt build --full-refresh manually
  • dropped an existing version of terminology before reloading and manually confirmed each terminology and value set file contained data (after the contents of the seeds were removed)

Reviewer focus

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

  • check Terminology and Value set seed files to ensure they are pointing:
    • versioned_terminonolgy > 0_8_1
    • versioned_value_sets > 0_8_1
    • versioned_provider_data > 0_8_1

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/ee0f00ac1cb24165afaa05c783fd1718?sid=46f7df4e-d19e-45a8-9916-ab33aa72a263

@thutuva thutuva merged commit 91cf3bb into main May 10, 2024
5 checks passed
@thutuva thutuva deleted the terminology-versioning branch May 10, 2024 17:00
@utsavpaudel utsavpaudel mentioned this pull request May 13, 2024
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants