[DOCS] Updates changelog for 7.0.0-alpha2#347
Merged
lcawl merged 2 commits intoelastic:masterfrom Dec 18, 2018
Merged
Conversation
droberts195
reviewed
Dec 18, 2018
docs/CHANGELOG.asciidoc
Outdated
|
|
||
| //=== Regressions | ||
| * Fixes CPoissonMeanConjugate sampling error. {ml-pull}335[#335] | ||
| * Removes out-of-phase buckets feature. {ml-pull}318[#318] (issue: {ml-issue}302[#302]) |
There was a problem hiding this comment.
We deliberately decided not to document this as the removed feature was never documented or tested and nobody should ever have used it.
Contributor
Author
There was a problem hiding this comment.
Ok! I've added "non-issue" so it doesn't get picked up.
docs/CHANGELOG.asciidoc
Outdated
| * Linear scaling change detection. {ml-pull}25[#25] | ||
| * Implements an absolute goodness-of-fit test to accept a change. {ml-pull}21[#21] | ||
| * Wires in change detection/modelling to our univariate time series model. {ml-pull}11[#11] | ||
| * First pass implementation of support functionality for change detection and modelling. {ml-pull}9[#9] |
There was a problem hiding this comment.
| Fix cause of "MERGE: Sum mode samples = 0, total samples = 4.43521.." log errors ({ml-pull}294[294]) | ||
|
|
||
| //=== Regressions | ||
| * Fixes CPoissonMeanConjugate sampling error. {ml-pull}335[#335] |
There was a problem hiding this comment.
It's true that this isn't currently in a GA version, but it will be in 6.6. So when the eventual 7.0.0 GA release notes are created it shouldn't be in there. Might it be worth adding a comment here to say it should be removed when consolidating into the GA 7.0.0 release notes?
Contributor
Author
|
Thanks @droberts195, I've cleaned up the list based on your feedback |
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
This PR removes the 6.x items from the changelog in master and adds PRs that are closed with the v7.0.0 label. It omits PRs that were labelled with release numbers that are already generally available.