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

Rt crowd forecast: Population adjustment #49

Open
seabbs opened this issue May 25, 2021 · 4 comments
Open

Rt crowd forecast: Population adjustment #49

seabbs opened this issue May 25, 2021 · 4 comments
Assignees
Labels
question Further information is requested

Comments

@seabbs
Copy link
Contributor

seabbs commented May 25, 2021

In the Rt model the R estimate has population adjustment beyond the forecast horizon. This means that what you see for Rt in the forecast will actually slightly reduce as susceptibility reduces over time. There are two options. 1. Remove the control parameter so this is not the case (this will induce a divergence with the normal Rt model or 2. flag this to users.

@seabbs seabbs added the question Further information is requested label May 25, 2021
@nikosbosse
Copy link
Contributor

Does that show up in the simulation? I'd assume people tweak Rt according to what they see. If that is the case I'd argue it is not that much of an issue

@seabbs
Copy link
Contributor Author

seabbs commented May 27, 2021

No, it doesn't hence mentioning it as if it did as you say not a problem.

@nikosbosse
Copy link
Contributor

Intuitively I'd have a small preference for removing the control parameter (if we think it makes a meaningful difference). I'm not sure people will be able to understand what it does and what effect it has and it would be easiest if the app was as close as posisble to WYSIWYG.
Do you think the divergence in methodology would pose a problem? Maybe should have a quick chat about this

@seabbs
Copy link
Contributor Author

seabbs commented May 28, 2021

its the pop arg supplied to rt_obs and it needs to be updated in the Rt fit model object prior to use in the app but after use for forecasting Rt via the model.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants