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

p.v sub-patent duration is immunity dependent. #318

Conversation

RJSheppard
Copy link
Member

antiparasite immunity function shapes the duration of subpatent infection/rate of recovery from subpatent infection. These values are now assigned during the variable creation and calculation of recovery rates.

In previous iterations I have kept ID in the vivax model, where it acts as antiparasite immunity. For clarity of thought, I've now added IAA and IAM antiparasite immunities, including their decay, boosting, maternal immunity and reset at death.

We must also adjust the prevalence rendering, which for p.f is based on detectability immunity, and for p.v is explicit in the human states. p_detect_lm is also inappropriate for p.v as the lm_detection is explicit.

du, and ID parameters removed for vivax.

Tests are added for du rates and for prevalence rendering.

R/render.R Outdated Show resolved Hide resolved
R/disease_progression.R Outdated Show resolved Hide resolved
@pwinskill
Copy link
Member

/benchmark

@RJSheppard RJSheppard force-pushed the vivax_competing_hazards_main_no_blood_immunity branch from 693e212 to a570996 Compare September 17, 2024 13:33
@RJSheppard RJSheppard force-pushed the vivax_competing_hazards_main_subpatent_duration_disease_progression branch 5 times, most recently from 3d918c6 to 8b1c56b Compare September 20, 2024 16:15
@RJSheppard RJSheppard force-pushed the vivax_competing_hazards_main_no_blood_immunity branch from a570996 to e733b28 Compare October 8, 2024 15:11
Base automatically changed from vivax_competing_hazards_main_no_blood_immunity to dev October 9, 2024 10:54
@RJSheppard RJSheppard force-pushed the vivax_competing_hazards_main_subpatent_duration_disease_progression branch from 8b1c56b to 3461513 Compare October 10, 2024 09:57
…tion/rate of recovery from subpatent infection in the Pv model. These values are now assigned during the variable creation and calculation of disease progression rates. Antiparasite immunity is labelled IAA and IAM for acquired and maternal immunities, where their decay, boosting, maternal immunity and reset at death are now included. The decay of the immunity occurs daily and has an impact of disease progression rate, which is now calculated and updated daily.

We must also adjust the prevalence rendering, which for p.f is based on detectability immunity, and for p.v is explicit in the human states.

`du` and ID variables and parameters are removed for vivax.

Tests are added for du rates and for prevalence rendering.

The vignette is extended to include the removal of ID and the sub-patent duration calculation.
@RJSheppard RJSheppard force-pushed the vivax_competing_hazards_main_subpatent_duration_disease_progression branch from 3461513 to e25c01b Compare October 10, 2024 11:54
@giovannic
Copy link
Member

/benchmark

Copy link

This is how benchmark results would change (along with a 95% confidence interval in relative change) if e25c01b is merged into dev:

  • 🚀large_population: 2.42m -> 2.38m [-3.19%, -0.24%]
  • ✔️small_population: 33.4s -> 33.6s [-1.55%, +2.73%]

Further explanation regarding interpretation and methodology can be found in the documentation.
Plots and raw data are available as artifacts of the workflow run.

Copy link
Member

@giovannic giovannic left a comment

Choose a reason for hiding this comment

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

Thank you!

@giovannic giovannic merged commit 8e0d250 into dev Oct 10, 2024
5 checks passed
@giovannic giovannic deleted the vivax_competing_hazards_main_subpatent_duration_disease_progression branch October 10, 2024 13:27
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.

3 participants