-
Notifications
You must be signed in to change notification settings - Fork 1
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
Added all the necessary modifications to the Epidemiology Project #2
base: master
Are you sure you want to change the base?
Conversation
bmi_based_strategy.py was not added in the correct directory
… generation file, adding 3 more vaccination strategy to the already existing strategies
…eenness and eigenvector strategy all of which inlcude an asymptoatic compartment
…ination strategies employed
Remove the folder |
Why have you made changes to |
It was
It had some errors when I clicked on it...... That's why I changed it |
Oh okay I will do it, so apart from this everything else is fine right? |
src/model/tests/test_model.py
Outdated
metadata=metadata, | ||
df=df | ||
) | ||
<<<<<<< HEAD |
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.
Fix all merge conflicts including this one
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.
Done
# Simulate new infected people | ||
def get_new_infected(self, infected_contact): | ||
new_infected = [ | ||
person for person in infected_contact if random.random() <= self.beta] | ||
person for person in infected_contact & self.susceptible if random.random() <= self.beta] |
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.
What exactly are you checking here? Are you sure this will work? I don't think this will tell you if a person is present in both the sets.
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.
What exactly are you checking here? Are you sure this will work? I don't think this will tell you if a person is present in both the sets.
Yeah, it's an intersection operation right ?, When i checked it long back, it was working fine
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.
Oh right.
No description provided.