Hospital Projection Update

[yu-peng]

[Wait to merge after important demos/meetings]

Updated hospitalization projection data from https://covidactnow.org/us/ny/county/new_york_county. The curve seems to be flattened compared to the data we used before. Note that a test needs to be updated too since it extends into the dates whose projections were updated.

Also, note that we are using the lowest curve of the three that assumes strict stay at home order, and continues with the total beds available number even after the projection falls below the capacity to ensure that we won't underestimate the demand.

[rcoh]

Maybe pass the model file in as a parameter with the default param set to the current model? Then we won't need to update the tests.
(You would just add new models instead of updating the old one)

[yu-peng]

Maybe pass the model file in as a parameter with the default param set to the current model? Then we won't need to update the tests.
(You would just add new models instead of updating the old one)

Yes! Trying to add a fix in ac72879. Would it work (keeping multiple versions of the projection in the repo but use different ones in testing and deployment)?

[rcoh]

FYI it sounds like we're moving to a flat model

…

On Mon, Apr 20, 2020, 8:03 PM yu-peng ***@***.***> wrote: Maybe pass the model file in as a parameter with the default param set to the current model? Then we won't need to update the tests. (You would just add new models instead of updating the old one) Yes! Trying to add a fix in ac72879 <ac72879>. Would it work (keeping multiple versions of the projection in the repo but use different ones in testing and deployment)? — You are receiving this because your review was requested. Reply to this email directly, view it on GitHub <#96 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AADYKZ4VDJRWPFVOOOS5Q3DRNTPEXANCNFSM4MMAFJSQ> .

[yu-peng]

FYI it sounds like we're moving to a flat model
…
On Mon, Apr 20, 2020, 8:03 PM yu-peng @.***> wrote: Maybe pass the model file in as a parameter with the default param set to the current model? Then we won't need to update the tests. (You would just add new models instead of updating the old one) Yes! Trying to add a fix in ac72879 <ac72879>. Would it work (keeping multiple versions of the projection in the repo but use different ones in testing and deployment)? — You are receiving this because your review was requested. Reply to this email directly, view it on GitHub <#96 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AADYKZ4VDJRWPFVOOOS5Q3DRNTPEXANCNFSM4MMAFJSQ .

Yeah....will leave the update here then.