Conversation
|
@dsweber2 How important do you think this methodology is? Just a thought, but what if we
|
|
if the fitting is expensive, it's probably best to predict as few days as possible (I've run into this with grf). Having something that handles that filtering is probably worthwhile. That said, the most important part of this PR is definitely the filtering the output in the way |
|
Just to be clear,
Where are you imagining the hit is occurring? I agree that point 1 is a reason to avoid predicting on excess data (and one reason for implementing this function in the first place), but my current thinking, after much use, is that the fragility is a larger issue than the computation. Do you have examples where the cost to 1 is burdensome? |
Checklist
Please:
dajmcdon.
DESCRIPTIONandNEWS.md.Always increment the patch version number (the third number), unless you are
making a release PR from dev to main, in which case increment the minor
version number (the second number).
(backwards-incompatible changes to the documented interface) are noted.
Collect the changes under the next release number (e.g. if you are on
0.7.2, then write your changes under the 0.8 heading).
epiprocessversion in theDESCRIPTIONfile ifepiprocesssoonepipredictandepiprocessChange explanations for reviewer
make
get_test_dataretrieve a year by default, and make predict narrow down to the forecast_dateMagic GitHub syntax to mark associated Issue(s) as resolved when this is merged into the default branch