Skip to content

[DOCS] Adds multi-bucket analysis to bucket overview #158

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
merged 2 commits into from
Dec 5, 2018

Conversation

lcawl
Copy link
Contributor

@lcawl lcawl commented Nov 29, 2018

Related to elastic/elasticsearch#34233

This PR adds the concept of multi-bucket analysis to the bucket overview in https://www.elastic.co/guide/en/elastic-stack-overview/master/ml-buckets.html

@lcawl lcawl requested a review from sophiec20 November 29, 2018 23:20
@lcawl lcawl changed the title [DOCS] Adds multi-bucket analyis to bucket overview [DOCS] Adds multi-bucket analysis to bucket overview Nov 29, 2018
Copy link

@sophiec20 sophiec20 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM so far ... but I also think we should mention that you may see anomalies within the model bounds (if model plot is enabled as in the pic) ... this is because the blue shaded area is a simplified 2D representation of the single bucket model. This isn't just due to multi-bucket but it is more noticeable since. I think the blog may have worded it better.

@lcawl
Copy link
Contributor Author

lcawl commented Nov 30, 2018

Thanks, I checked out the wording of the note in https://www.elastic.co/blog/changes-to-elastic-machine-learning-anomaly-scoring-in-6-5 and added a paragraph here.

When I refresh the ML getting started tutorial (in a separate PR), I'll also add that information there (e.g. where we discuss model bounds in https://www.elastic.co/guide/en/elastic-stack-overview/master/ml-gs-job1-analyze.html)

Copy link

@sophiec20 sophiec20 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@lcawl lcawl merged commit a640978 into elastic:master Dec 5, 2018
@lcawl lcawl deleted the multi-buckets branch December 5, 2018 21:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants