-
Notifications
You must be signed in to change notification settings - Fork 14
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
Feat/metapopulation model #259
Conversation
* Add rdt testing parameters * Update biting_process to consider rdt testing for mixing
- updating metapop vignette - clarifying row & col meanings
* Add rdt testing parameters * Update biting_process to consider rdt testing for mixing
* refactor transmission mixing code * fix regressions * add time caching
* Added export_mixing and import_mixing arguments to run_metapop_... * Update create_transmission_mixer to mix eir on import and foim on export * Fix regression tests
Add asymmetric transmission in mixing:
change mixing to export_mixing and import_mixing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
From my perspective this seems fine, but I don't really have the background context on what is being accomplished.
.gitignore
Outdated
@@ -12,3 +12,4 @@ doc | |||
inst/doc | |||
/doc/ | |||
/Meta/ | |||
src-i386 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure why this is needed?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@giovannic do you know why this was added?
Co-authored-by: Paul Liétar <[email protected]>
Co-authored-by: Paul Liétar <[email protected]>
Thanks @plietar, essentially we are fixing some assumptions in how the metapopulation model mixing matrix is made to allow for different effects on FOIM and EIR. We have done a lot of testing with it, but thanks for providing another set of eyes to the changes! |
No description provided.