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.

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

Add support for Graphite web holtWintersConfidenceArea() function #28

Draft
wants to merge 27 commits into
base: main
Choose a base branch
from

Conversation

carrieedwards
Copy link
Collaborator

@carrieedwards carrieedwards commented Jul 11, 2022

This PR adds support for the Graphite web holtWintersConfidenceArea() function.

The holtWintersConfidenceArea() function is defined as:

holtWintersConfidenceArea(seriesList, delta=3, bootstrapInterval='7d', seasonality='1d')
Performs a Holt-Winters forecast using the series as input data and plots the area between the upper and lower bands of the predicted forecast deviations.

@carrieedwards carrieedwards force-pushed the cedwards/add-holtWintersConfidenceArea branch from 692341e to 8f3ab69 Compare August 4, 2022 19:05
@carrieedwards carrieedwards linked an issue Aug 8, 2022 that may be closed by this pull request
npazosmendez and others added 26 commits September 7, 2022 15:37
…th not equal (#99)

Align series by step in divideBySeries if steps not equal
* Check validity of consolidation function in sortBy

* Add test for checking validity of consolidation functions

* Update validity checker function and add tests
* Parse strings as ints or floats for params of type Integer or Float

* Add parser tests

* Fix failing e2e test
* Fix xFilesFactor handling in AggregateSeries and add tests

* Add more xFilesFactor tests for removeZeroSeries
…ided (#109)

* Return ErrMissingTimeSeries if only one argument is provided

* Add test for only passing in one argument
Co-authored-by: msaf1980 <msaf1980@gmail.com>
Consolidate series by step for divideSeries
…is specified (#113)

* Prevent panic if nil series and xFilesFactor passed into removeBelowSeries

* Add tests for preventing panic
@carrieedwards carrieedwards force-pushed the cedwards/add-holtWintersConfidenceArea branch from 8f3ab69 to a0eb5f7 Compare September 14, 2022 14:31
@CLAassistant
Copy link

CLAassistant commented Sep 19, 2022

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
3 out of 4 committers have signed the CLA.

✅ npazosmendez
✅ leizor
✅ carrieedwards
❌ msaf1980
You have signed the CLA already but the status is still pending? Let us recheck it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add support for Graphite web holtWintersConfidenceArea() function
5 participants