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/ |
TheCedarPrince
left a comment
TheCedarPrince
left a comment
There was a problem hiding this comment.
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!
| <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.
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
| - 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.
Strangely, this doesn't render correctly in the preview...
There was a problem hiding this comment.
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.
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.
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.
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.
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?
2 participants
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