Skip to content
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 the social distancing measures notebook #26

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

lakhand7
Copy link

No description provided.

@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

You'll be able to see Jupyter notebook diff and discuss changes. Powered by ReviewNB.

@@ -0,0 +1,386 @@
{
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What is the thought process for initE to be zero?


Reply via ReviewNB

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have added a new commit wrt this. Have a look.

@@ -0,0 +1,386 @@
{
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do you think it'd be better to model this in two splits? One relaxed lockdown from April 20 and completely relaxed from May 2?

How do we model these two scenarios? What is Day 0 and Day 25 in real dates?


Reply via ReviewNB

Copy link
Author

@lakhand7 lakhand7 Apr 18, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Q1: Sure, I can incorporate this by relaxing the scale parameter of my graph defined at the starting of the notebook. I can increase the scale from 10 to 20 for a relaxes lock-down type scenario

Q2: The data is for April 18th at the moment. In the new commit, day 0 is April 18th, a lock-down of 15 days till May 3rd and then trend for 85 more days

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How can we rely that these params work for us? Can we use Lombardo or Wuhan for reference and get corresponding Day 0 curves and see how they compare against reality?

@@ -0,0 +1,386 @@
{
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What is the thought process behind using mean of all the simulations? Am I interpreting the code+plot correctly?


Reply via ReviewNB

Copy link
Author

@lakhand7 lakhand7 Apr 18, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The time steps for SEIRSPLUS modelling is less than 1. If we look at model.tseries, we will be able to see the exact time period for the model is predicting infections. Let's say, on day 14, there are multiple time periods for which we have S, E, D_E, D_I, I, R, F with sufficient std dev. Therefore, I take mean to get a stable value for that day.

@@ -0,0 +1,386 @@
{
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Use the absolute scale please?


Reply via ReviewNB

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I had used the default graph provided in the SEIRSPLUS library. I have removed this graph altogether in the new commit.

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