Skip to content
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.

Sign up for GitHub

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

Jump to bottom

moving_avg forecasts should not include current point #11641

Merged
merged 1 commit into from Jun 22, 2015
Merged

moving_avg forecasts should not include current point #11641

merged 1 commit into from Jun 22, 2015

Commits on Jun 22, 2015

  1. Aggregations: Moving average forecasts should not include current dat… 

    …apoint.

- Fixes tests, and removes a few special snowflake, fragile tests.
- Removes concrete implementation of predict() and moves it into
  each model so that the logic is clearer.  Because there is some
  shared checks/assertions, those remain in predict() and the main
  prediction happens in doPredict()
    @polyfractal
    polyfractal committed Jun 22, 2015
    Configuration menu
    Copy the full SHA
    5d94feb View commit details
    Browse the repository at this point in the history