-
Notifications
You must be signed in to change notification settings - Fork 2
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
forecast ILI #23
forecast ILI #23
Conversation
…arams without needing focustools dependency
@stephenturner fyi i just ported over the glm functions code from fluforce-init also moved the example code from fluforce-init into scratch (saw you mention something similar in a commit on this PR so wanted you to have a heads up): https://github.com/signaturescience/fiphde/blob/main/scratch/worfklow.R all seems to be working by using glm functions now in |
Nice, thanks for that. I'm running through the old code in fluforce-init and it's taking forever. I'm going to merge in main on this branch but it looks like we have a merge conflict I'll work out |
Merge branch 'main' into 7-forcast-ili # Conflicts: # R/fiphde.R
…coded ili with forecasted values
prep for keyed forecasts
State-level forecasting
…vpnagraj we'll have to chat on this one
issue with the |
oh and note column renaming (year=>epiyear, week=>epiweek ) in 8d85d50 |
still some rough edges to polish with docs / code cleanup. but overall this looks good enough to merge to main to me. i did make one change to add a "param_space" named list arg to |
I think this works after a small change in e11a52c. One thing to note, you must specify all 6 params even if you only want to restrict one. But works, and rcmdchecks OK |
First cut at function to forecast ILI (fixes #7 when closed).
Example usage below. Object of interest in the returned list is
ili_bound
, which gives you a regular (non-tsibble) tibble with historical recorded data andhorizon
number of forecasted weeks into the future, bound in a tibble.See help for
?fiphde::forecast_ili
.Currently only tested with
location="US"
. Possible we could use location as a key and retrieve forecasted ILI for all states. Keeping this in draft until I get something tested there.