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

**HACK** Dump population at given intervals. #545

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from
Draft

Conversation

jamesturner246
Copy link
Contributor

DO NOT MERGE

Hack to enable whole population dumping (baseline and intervention) of risk factors at selected times.

Control this by setting an environment variable before running healthgps:

export HGPS_DUMP_TIMES=0,10,25

# healthgps command here

Try it out locally if you can to check you see how it works, and if there's anything else you need dumped.

Resolves #535

@jamesturner246 jamesturner246 requested a review from jzhu20 October 4, 2024 13:32
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

clang-tidy made some suggestions

src/HealthGPS/analysis_module.cpp Outdated Show resolved Hide resolved
src/HealthGPS/analysis_module.cpp Outdated Show resolved Hide resolved
src/HealthGPS/analysis_module.cpp Outdated Show resolved Hide resolved
src/HealthGPS/analysis_module.cpp Outdated Show resolved Hide resolved
src/HealthGPS/analysis_module.cpp Outdated Show resolved Hide resolved
jamesturner246 and others added 2 commits October 4, 2024 14:43
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Copy link

codecov bot commented Oct 4, 2024

Codecov Report

Attention: Patch coverage is 0% with 42 lines in your changes missing coverage. Please review.

Project coverage is 48.89%. Comparing base (f52b01e) to head (3a5d430).

Files with missing lines Patch % Lines
src/HealthGPS/analysis_module.cpp 0.00% 42 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #545      +/-   ##
==========================================
- Coverage   49.00%   48.89%   -0.11%     
==========================================
  Files         164      164              
  Lines        7816     7833      +17     
  Branches     1063     1068       +5     
==========================================
  Hits         3830     3830              
- Misses       3977     3994      +17     
  Partials        9        9              

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

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.

[ENHANCEMENT] Produce individual data
1 participant