Skip to content

St/queries to snakemake - #85

Draft
thompson318 wants to merge 40 commits into
devfrom
st/queries-to-snakemake
Draft

St/queries to snakemake#85
thompson318 wants to merge 40 commits into
devfrom
st/queries-to-snakemake

Conversation

@thompson318

@thompson318 thompson318 commented Aug 19, 2026

Copy link
Copy Markdown
Collaborator

This work in progress. Creating PR as I'll be off till monday in case anyone else wants to take some of it on.

  • There some incomplete logic to get EHR in src/electronic_health_records.py
  • Snakemake has been modified to include an ehr_lookup step.
  • Some testing has been done.
  • Runs locally, not tested on GAE yet.
  • I've added a fake_stardb option to enable snakemake integration to run without setting up a fakeuds on github actions. Longer term it would probably be better to set up a fakeuds and fake caboodle.

skeating and others added 29 commits February 2, 2026 15:06
Introduces new queries for airway and sputum/secretions data, and refactors the flow sheet query to consolidate multiple values into a wider format. A new README documents the overall data extraction goal and the current set of scripts.
@codecov

codecov Bot commented Aug 24, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 56.52%. Comparing base (4251bc1) to head (aa8f86a).

Additional details and impacted files
@@            Coverage Diff             @@
##              dev      #85      +/-   ##
==========================================
+ Coverage   50.44%   56.52%   +6.07%     
==========================================
  Files          14       15       +1     
  Lines         563      667     +104     
  Branches       47       54       +7     
==========================================
+ Hits          284      377      +93     
- Misses        263      269       +6     
- Partials       16       21       +5     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

skeating and others added 3 commits August 24, 2026 11:13
Transforms specific lab test results (e.g., CRP, WCC) from a long format into distinct columns using `MAX() FILTER`. This enables direct consumption of lab data in a wider format, simplifying downstream analysis.

Additionally, the query is updated to use parameters for `hospital_visit_id` and to include date range filtering, improving its flexibility.
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