pennsignals / bayes-chime

Bayesian fit to SEIR model. An extension to Penn Medicine's CHIME tool.
MIT License
28 stars 18 forks source link

Broken reopen_cap parameter #66

Closed Mdraugelis closed 4 years ago

Mdraugelis commented 4 years ago

reopen_cap parameter isn't explicitly passed into the sir function. So the default value of 0.50 is always used.

Minor issue the reopen_cap parameter is line 660: plt.title(f"Reopening scenario, {int(reopen_speed*100)}% per day up to {int(reopen_speed*100)}% social distancing") Should be updated to line 660: plt.title(f"Reopening scenario, {int(reopen_speed*100)}% per day up to {int(reopen_cap*100)}% social distancing")

My workaround. I just changed the sir default from 0.5 to 0.3.

cranedroesch commented 4 years ago

I fixed this on Friday evening with 1a0a8201e4b344a66522bd00731c74a78596f485.