-
Notifications
You must be signed in to change notification settings - Fork 80
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
Weird Way to update the Infection Status of SI model #227
Comments
Hi i went through the past issues and discovered that this beta vs num_infected_neighbors have been discussed before But I kind of agree with #96 (comment) If we adopt the 1 - (1-beta)**num_infected_neighbors, then what is the definition of beta? |
It is not weird at all. This is the probability of getting infected considering multiple independent contacts with infected individuals (each contact with prob beta of passing the infection). |
That's likely a typo. Let's say that a linear scaling is not correct because it doesn't guarantee that the final probably value lies in [0,1]. Moreover, it doesn't model the independence of multiple infection sources. |
i see. thank you very much for the clarification! |
in
ndlib/models/epidemics/SEISModel.py
, in the commit f80b144#diff-81ae80e5ab1d17420f19b47e51b93b8999cee1ad70db53e2aff5309ad3cd8899the beta * num_infected_neighbors was changed into a 1 - (1-beta)**num_infected_neighbors.
I am wondering why was there such a change in the formula? The probability of beta should be linearly scaled with the number of infected neighbors right? Thanks for the hard work!
The text was updated successfully, but these errors were encountered: