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

'overrides' command in 'site_parameters' function does not override outcome rendering parameters #8

Open
htopazian opened this issue Mar 1, 2023 · 2 comments

Comments

@htopazian
Copy link
Collaborator

When creating a list of overrides, the 'site_parameters' function appears to override the default 'human_population' parameter value, but does not override the outcome rendering variables. I have not yet tested with other overrides.

site <- site::single_site(foresite::"KEN", 1)

site_par <- site_parameters(
  interventions = site$interventions,
  demography = site$demography,
  vectors = site$vectors,
  seasonality = site$seasonality,
  eir = site$eir$eir[1],
  overrides = list(human_population = 1000,
                   prevalence_rendering_min_ages = 2 * year,
                   prevalence_rendering_max_ages = 10 * year,
                   clinical_incidence_rendering_min_ages = c(0, 0) * year,
                   clinical_incidence_rendering_max_ages = c(5, 200) * year,
                   severe_incidence_rendering_min_ages = c(0, 0) * year,
                   severe_incidence_rendering_max_ages = c(5, 200) * year))

site_par$human_population # 1000
site_par$prevalence_rendering_min_ages # 365 730
site_par$prevalence_rendering_max_ages # 36499  3649
site_par$clinical_incidence_rendering_min_ages # 0 1825 5475
site_par$clinical_incidence_rendering_max_ages # 1824  5474 36499
site_par$severe_incidence_rendering_min_ages # 0 1825 5475
site_par$severe_incidence_rendering_max_ages # 1824  5474 36499
@pwinskill
Copy link
Member

The site_parameters() function has a min_ages argument that I've been using to set outputs for rates. It defaults to doing 2-10 and 1-100 for prevalence.
Let me know if you need more than that flexibility?

@htopazian
Copy link
Collaborator Author

Ah yes, I see that. I generally do one big band of the entire population for incidence, but the way you have the age bands set up, it is easy to add them in post-processing to combine age groups. Might be nice to have a max_ages argument or actually just to allow every parameter to be able to override the default malariasimulation parameters. You could keep the age bands you have set as the defaults, but still allow users to choose their own if they need (for example just ages targeted by SMC, or just under 6 months, etc).

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

No branches or pull requests

2 participants