-
Notifications
You must be signed in to change notification settings - Fork 9
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
Patient Pathways within JuliaHealth - Part 1 #124
base: dev
Are you sure you want to change the base?
Conversation
Once the build has completed, you can preview your PR at this URL: https://juliahealth.org/juliahealth.github.io-previews/previews/PR124/JuliaHealth-Blogs/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @Jay-sanjay -- sorry it took so long to review this! Between GSoC reviews, sickness, and school, all things got a bit busy! Had a chance to sit down and do a nice review of this.
First of all, this is AWESOME!!! Really great work on this and like the story so far.
Second, I did leave a bunch of comments but don't be discouraged by that! I am happy to clarify and support in any way.
Third, you brought up an exciting idea which is, I think we could actually make this blog post be computable (i.e. running the code so we can automate updating the results). What do you think?
Again, thanks for writing this -- it's exciting to see things come together!
@@ -20,6 +20,7 @@ | |||
<li><a href="https://github.com/JuliaHealth">GitHub</a></li> | |||
<li><a href="/related-organizations">Related Organizations</a></li> | |||
<li><a href="/meeting-notes">Meeting Notes</a></li> | |||
<li><a href="/JuliaHealth-Blogs">JuliaHealth Blogs</a></li> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we rename the file to just "blogs".md? Also, this page should be more of an aggregator that has the following per blog post:
- Blog post title
- Blog post description
- Authors
- Date
JuliaHealth-Blogs.md
Outdated
- Health analytics built specifically for working with OMOP CDM databases | ||
|
||
|
||
* [`OMOPCDMCohortCreator`](https://github.com/JuliaHealth/OMOPCDMCohortCreator.jl) - Create cohorts from databases utilizing the OMOP CDM |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Strangely, this doesn't render correctly in the preview...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Exactly, the last two commits to this PR is for that thing only, but still not fixed
JuliaHealth-Blogs.md
Outdated
``` | ||
|
||
|
||
7. Now this are some of the required functions that probabily would be directly be useable via `occ` after the new version of [`OMOPCDMCohortCreator`](https://github.com/JuliaHealth/OMOPCDMCohortCreator.jl) is released. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hasn't this functionality been merged already? Can't we remove this section? Or did I miss something?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes we can remove it now, as the new version for occ is here
JuliaHealth-Blogs.md
Outdated
|
||
``` | ||
|
||
#So now that we have the dataset to work with as well as all the functions to work with also ready, we from here can start work form PATHWAYS-STUDY. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What are you trying to say here? Don't follow entirely -- is this where you are introducing what we are doing?
JuliaHealth-Blogs.md
Outdated
combined_df = sort!(combined_df, :drug_exposure_start_date) | ||
``` | ||
|
||
* An important thing to notice here is that some start and end dates seems to be preety weird like below: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hmm... This is an interesting question you have brought up here... Franklin can actually execute and render cells from Franklin markdown documents (like this one). I am leaning towards us actually being able to make this whole document computable (see what I mean here: https://franklinjl.org/code/#evaluating_code)
That way, the tables render properly and you don't have to worry about copying and pasting results all the time. What do you think Jay?
This blog provides a detailed walkthrough of the process for exploring patient pathways within JuliaHealth using various Julia packages and functions. It's a comprehensive guide to working with the OMOP CDM database and analyzing specific drug pathways followed by the patients within the database