-
Notifications
You must be signed in to change notification settings - Fork 65
[ML] Improve time series decomposition in the presence of change points #198
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
Merged
tveasey
merged 29 commits into
elastic:master
from
tveasey:enhancement/improve-timeseries-decomposition-with-change-points
Oct 3, 2018
Merged
Changes from all commits
Commits
Show all changes
29 commits
Select commit
Hold shift + click to select a range
77f16a2
Work on segmentation of expanding window
tveasey ced0e1e
Bug fixes
tveasey 5670ffa
Merge branch 'master' into enhancement/improve-timeseries-decompositi…
tveasey 6fdf20f
Finish up segmentation
tveasey c2bd6fe
Wire in piecewise constant scaling segmentation into the periodicity …
tveasey 84ee3e8
Wire piecewise linear trend in to periodicity hypothesis testing
tveasey 05e5d2d
Bug fixes
tveasey b0c85c5
Update test threshold
tveasey 1c47455
Speed up segmentation using linear models
tveasey e96eae3
Penalise trend segmentation in hypothesis selection. Fix remaining un…
tveasey 12a95a9
A couple of bug fixes and knock improvements given new test behaviour
tveasey 29a7573
Merge branch 'master' into enhancement/improve-timeseries-decompositi…
tveasey 294f1ad
Revert experiment
tveasey 350b9db
Merge branch 'master' into enhancement/improve-timeseries-decompositi…
tveasey b6cb46d
Compiler warnings and incorrect argument type
tveasey b9b2b86
Suppress verbose test logging
tveasey 5f6ce4c
Numerical hardening
tveasey 83786e6
Some bug fixes
tveasey 14ff203
Merge branch 'master' into enhancement/improve-timeseries-decompositi…
tveasey 5a04b22
Unused include
tveasey 78c7cd3
Merge branch 'master' into enhancement/improve-timeseries-decompositi…
tveasey cc2fb5e
Merge branch 'master' into enhancement/improve-timeseries-decompositi…
tveasey 8499551
Fix merge
tveasey 3bbd244
More descriptive test stats member names as per review comment
tveasey a8b78dd
Correct comment
tveasey f979a64
Remove unused variable
tveasey f6aa320
Docs
tveasey e13828b
Merge branch 'master' into enhancement/improve-timeseries-decompositi…
tveasey 1ca85fd
Some tidy ups and fix unit tests after merge
tveasey File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not sure that the comments here quite match the type of the member variable
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Agreed. This used to be a bool but we actually need the number of segments in the calling code so this changed. I'll update.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See this commit.