X Tutup
Skip to content

Fix typo in cell#51

Closed
syclops wants to merge 1 commit intoAllenDowney:masterfrom
syclops:master
Closed

Fix typo in cell#51
syclops wants to merge 1 commit intoAllenDowney:masterfrom
syclops:master

Conversation

@syclops
Copy link
Copy Markdown

@syclops syclops commented Oct 9, 2019

This commit addresses a typo in the code that yields unexpected results.

In the cell, results, run_simulation(...) causes the simulation to be
run correctly, but the results are never assigned back to the original
results variable, meaning that the fraction of infected individuals is
never updated over the course of the sweep. When running this code, it
then seems like the fraction of infected individuals never changes,
regardless of the number of doses (and resulting changes in beta).

This commit addresses a typo in the code that yields unexpected results.

In the cell, `results, run_simulation(...)` causes the simulation to be
run correctly, but the results are never assigned back to the original
`results` variable, meaning that the fraction of infected individuals is
never updated over the course of the sweep. When running this code, it
then seems like the fraction of infected individuals never changes,
regardless of the number of doses (and resulting changes in beta).
@AllenDowney
Copy link
Copy Markdown
Owner

Fixed with 387a6ae

Thanks!

@AllenDowney AllenDowney closed this Oct 9, 2019
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.

2 participants

X Tutup