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

Patient Pathways within JuliaHealth - Part 1 #124

Open
wants to merge 5 commits into
base: dev
Choose a base branch
from
Open

Patient Pathways within JuliaHealth - Part 1 #124

wants to merge 5 commits into from

Conversation

Jay-sanjay
Copy link
Member

@Jay-sanjay Jay-sanjay commented Feb 1, 2024

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

Copy link

github-actions bot commented Feb 1, 2024

Once the build has completed, you can preview your PR at this URL: https://juliahealth.org/juliahealth.github.io-previews/previews/PR124/JuliaHealth-Blogs/

Copy link
Member

@TheCedarPrince TheCedarPrince left a 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>
Copy link
Member

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:

  1. Blog post title
  2. Blog post description
  3. Authors
  4. Date

JuliaHealth-Blogs.md Outdated Show resolved Hide resolved
- 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
Copy link
Member

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...

Copy link
Member Author

@Jay-sanjay Jay-sanjay Feb 22, 2024

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 Show resolved Hide resolved
JuliaHealth-Blogs.md Outdated Show resolved Hide resolved
```


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.
Copy link
Member

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?

Copy link
Member Author

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


```

#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.
Copy link
Member

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 Show resolved Hide resolved
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:
Copy link
Member

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?

JuliaHealth-Blogs.md Outdated Show resolved Hide resolved
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.

None yet

2 participants