Partner services #248
Partner services #248
Conversation
* feat: create new setting for atlanta with latinos * upgrade python version in github actions
|
Codecov ReportAttention: Patch coverage is
❗ Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files@@ Coverage Diff @@
## main #248 +/- ##
==========================================
- Coverage 97.79% 90.05% -7.75%
==========================================
Files 33 33
Lines 2681 2855 +174
==========================================
- Hits 2622 2571 -51
- Misses 59 284 +225
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚨 Try these New Features:
|
| self.progress_to_aids(model) | ||
|
|
There was a problem hiding this comment.
Why was this moved to after progress_to_aids?
| model.run_random.random() < test_prob | ||
| and not self.dx # has not yet been diagnosed | ||
| ): | ||
| if self.active and model.time >= model.params.hiv.start_time: |
There was a problem hiding this comment.
hiv.start_time should be determined above. Nothing should happen with HIV before the start time.
| self.tested_negative = True | ||
| self.tested_negative_time = model.time | ||
|
|
||
| if self.tested_negative and self.tested_negative_time < model.time: |
There was a problem hiding this comment.
What does this do? Comment!
| @@ -201,11 +219,22 @@ def enroll_cap(self, model: "model.TITAN", haart_params: ObjMap): | |||
| # HAART agents based on % of diagnosed agents | |||
| num_dx_agents = self.agent.hiv.dx_counts[race][sex_type] # type: ignore[attr-defined] | |||
| num_haart_agents = self.counts[race][sex_type] | |||
|
|
|||
| # take value from dictionary for cap | |||
| if num_haart_agents < (haart_params.cap * num_dx_agents): | |||
There was a problem hiding this comment.
Why was this deleted?
| if self.ever and self.agent.location.params.haart.use_reinit: | ||
| if model.run_random.random() < haart_params.reinit.prob: | ||
| self.initiate(model.run_random, haart_params, "prob") | ||
|
|
||
| # if agent has been diagnosed through PS, use PS treatment (haart) probability | ||
| elif self.agent.partner_tracing.ps_dx: |
There was a problem hiding this comment.
Just for safety, I would also check that partner tracing is enabled
| stats["step_3"] += self.info_stat | ||
| stats["step_4"] += self.contact_stat | ||
|
|
||
| if self.ps_participant: |
There was a problem hiding this comment.
for the rest of these, combine if statements where applicable.
| stats["step_4"] += self.contact_stat | ||
|
|
||
| if self.ps_participant: | ||
| if self.ps_participant_time == time: |
There was a problem hiding this comment.
What's the difference between participate_time and ps_participant_time?
| ): | ||
|
|
||
| # if target model is racial, use appropriate proabability | ||
| if "Racial" in params.prep.target_model and not self.agent.partner_tracing.tested_negative: |
There was a problem hiding this comment.
Why can an agent who tested negative not go on PrEP?
| self.enroll(model.run_random, model.time) | ||
| else: |
There was a problem hiding this comment.
Where did this else clause go?
| type: int | ||
| default: 1 | ||
|
|
||
| white: |
There was a problem hiding this comment.
this should be defined more similarly to the demographics, as a sub-dict with races as keys. As it is now, it's hardcoded to these specific races. See: https://github.com/pph-collective/TITAN/blob/main/titan/params/demographics.yml
No description provided.