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.

Sign up for GitHub

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

Jump to bottom

HIV testing for the symptomatic #182

Merged
merged 55 commits into from
Jul 8, 2024
Merged

HIV testing for the symptomatic #182

merged 55 commits into from
Jul 8, 2024

Conversation

pineapple-cat
Copy link
Collaborator

Addresses #111.

@pineapple-cat
Added symptomatic test probabilities.
ceb4d8f
@pineapple-cat
Updated relevant symptomatic testing columns.
9c5134f
@pineapple-cat
Added calculations for symptomatic test probabilities and outcomes.
05492ef
@pineapple-cat
Fixed typo.
c290e21
@pineapple-cat
Added function to mark symptomatic people for testing.
08031e5
@pineapple-cat
Added function to mark non-HIV symptomatic people for testing.
bdb02ff
@pineapple-cat
Extracted general population test marking into its own function.
8188652
@pineapple-cat
Updated main HIV testing function to make use of test marks.
4a845c8
@pineapple-cat
Added probability of testing with non-HIV symptoms.
9c59325
@pineapple-cat
Bugfixes for unit tests (transform_group issue left unsolved; awaitin…
b4a639f 
…g further discussion).
@pineapple-cat
Fixed transform_group column name issues for past time steps.
0e33c59
@pineapple-cat
Function reordering for better clarity.
e6b23ab
@pineapple-cat
Added unit test for HIV symptomatic testing.
6de1d5d
@pineapple-cat
Added unit test for non-HIV symptomatic testing.
10e098b
@pineapple-cat
Updated general HIV testing unit test.
5666ccb
@pineapple-cat
Fixed ADC risk unit test.
207550e
@pineapple-cat
Moved post-test VMMC update function to circumcision module.
6b579c4
@pineapple-cat
Updated post-VMMC HIV test marks.
7a089dc
@pineapple-cat
Updated unit tests to account for post-VMMC testing.
71208f9
@pineapple-cat
Updated call ordering of ANC HIV test marks.
d124395
@pineapple-cat
Updated unit tests to account for new ANC testing.
48a1c22
@pineapple-cat
Moved timing of ANC removal from people giving birth this time step.
7109ddf
@pineapple-cat pineapple-cat marked this pull request as ready for review May 20, 2024 13:00
@pineapple-cat pineapple-cat requested a review from mmcleod89 May 20, 2024 13:28
mmcleod89
mmcleod89 reviewed May 24, 2024
View reviewed changes
src/hivpy/hiv_testing.py Outdated Show resolved Hide resolved
mmcleod89
mmcleod89 reviewed May 24, 2024
View reviewed changes
src/hivpy/hiv_testing.py Outdated Show resolved Hide resolved
mmcleod89
mmcleod89 reviewed May 24, 2024
View reviewed changes
src/hivpy/hiv_testing.py Outdated Show resolved Hide resolved
mmcleod89
mmcleod89 reviewed Jun 13, 2024
View reviewed changes
src/hivpy/hiv_testing.py Outdated Show resolved Hide resolved
@pineapple-cat pineapple-cat requested a review from mmcleod89 June 20, 2024 17:40
mmcleod89
mmcleod89 reviewed Jun 25, 2024
View reviewed changes
src/hivpy/hiv_testing.py Outdated Show resolved Hide resolved
mmcleod89
mmcleod89 approved these changes Jul 8, 2024
View reviewed changes
Copy link
Collaborator

@mmcleod89 mmcleod89 left a comment

Choose a reason for hiding this comment

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

Hooray!

@mmcleod89 mmcleod89 merged commit b3893f2 into development Jul 8, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@andrew-phillips-1 andrew-phillips-1 andrew-phillips-1 left review comments

@mmcleod89 mmcleod89 mmcleod89 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

HIV testing for Symptomatic people (without diagnosis)
3 participants
@pineapple-cat @mmcleod89 @andrew-phillips-1