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

Gaps in days are not accounted for #1

Open
r4j4h opened this issue Jan 13, 2015 · 0 comments
Open

Gaps in days are not accounted for #1

r4j4h opened this issue Jan 13, 2015 · 0 comments

Comments

@r4j4h
Copy link
Owner

r4j4h commented Jan 13, 2015

Right now, if we give in data for days 13 and 15, then the moving average and percent change account these as neighboring nodes.

Really though, day 14 is missing. Behavior in this edge case should be configurable.

Potential paths:

  • Ignore the missing data point and assume it stays the same
  • Bucket the missing data points with 0
  • Throw an error

Bucketing the missing data points with 0 can be done before handing the group in, but would nice to have built-in as it is easy to run in to.

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

No branches or pull requests

1 participant