-
Notifications
You must be signed in to change notification settings - Fork 5
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
Toy spatial #225
Toy spatial #225
Conversation
…y-spatial Merge new spatial changes into toy population simulations
Codecov Report
@@ Coverage Diff @@
## main #225 +/- ##
===========================================
Coverage 100.00% 100.00%
===========================================
Files 105 47 -58
Lines 3380 1122 -2258
===========================================
- Hits 3380 1122 -2258 |
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.
Great work, really nice plots! One tiny comment, but I'm happy to pre-approve this on the basis that you add that
" location_x.append(x)\n", | ||
" location_y.append(y)\n", | ||
"\n", | ||
"new_susceptible = susceptible[:-(total_population - sum(susceptible))]+[x + 1 for x in susceptible[-(total_population - sum(susceptible)):]]\n", |
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.
I don't find this line very intuitive - it might be fine in a codebase (although I am not sure), but in an exemplar notebook it should definitely be commented to explain what you are doing here!
Summary
Example notebook to demonstrate the generation of approximately uniform populations and show the comparison of epidemics in different density examples.
Further work
Similar notebook to be constructed to demonstrate the application of interventions