-
Notifications
You must be signed in to change notification settings - Fork 10
Created a preprocessing step that limits the size of the training window #53
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
Conversation
It looks like this is failing because of cmu-delphi/epiprocess#96. Blocked for the moment. |
@dajmcdon, when using training data in I think the issue here is that when we put |
I suspect you're right. We're going to need a new |
…amespace Merge branch 'frosting' of https://github.com/cmu-delphi/epipredict into 36-step_training_window # Conflicts: # NAMESPACE
…hi/epipredict into 36-step_training_window
Addressed issue 36 by creating a preprocessing step,
step_training_window()
, that limits the size of the training window to the n most recent observations per location. See training_window.R and test-step_training_window.R for some tests.