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

[8.17] [8.x] Backport 119578 + 114289 (#119638) #119644

Merged

Conversation

jan-elastic
Copy link
Contributor

Backports the following commits to 8.17:

* Refactor change point detection (elastic#114289)

* Move change detection code to separate class

* Uniformize ChangeDetector and SkipeAndDipDetector

* Separate ChangeDetectorTests and ChangePointAggregatorTests.

* Public entrypoint for change point detection

* Move p-value computation to ChangeDetector

* Move main entrypoint to a separate file

* Fix change point detection for uncertain non-stationary distributions. (elastic#119578)

* Fix change point detection for uncertain non-stationary distributions.

* Replace -1 by ChangeType.NO_CHANGE_POINT
@jan-elastic jan-elastic added :ml Machine learning auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport Team:ML Meta label for the ML team labels Jan 7, 2025
@elasticsearchmachine elasticsearchmachine merged commit 9d2eed1 into elastic:8.17 Jan 7, 2025
15 checks passed
@jan-elastic jan-elastic deleted the backport/8.17/pr-119638 branch January 7, 2025 15:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport :ml Machine learning Team:ML Meta label for the ML team v8.17.1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants