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

Possible collaborate based on notast blog #178

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

notast
Copy link

@notast notast commented May 16, 2019

Hi Jack

I'm Benjamin and I'm the user behind notast. Instead of manually mapping the 3_digit identifier range for each chapter as I have done so on the blog, I have used I used icd9_chapters and a few tidyverse scripts. (lines 141- 158).

It is possible to extract the major, sub_chapter and chapter from the dataset using icd::chapters_to_map. However, there are two disadvantages.
[1] This technique may be longer than what I have done. In the blog, I simply merge the ICD-9 data with my dataset to extract all of the above.
[2] It does not provide me with the 3_digit identifier range for each chapter. I'm keen to use the 3_digit identifier range instead of the full chapter name because it looks neater as a legend for the visualizations.

Rmarkdown for blogpost found on https://notast.netlify.com/post/what-s-that-disease-called-overview-of-icd-package/. I have used  I used `icd9_chapters` and a few `tidyverse` scripts instead of the manually mapping chapter names
@jackwasey
Copy link
Owner

Thanks so much for this. Much appreciated. I have a lot on my plate right now - not forgotten.

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.

2 participants