Skip to content

[DOCS] Update machine learning screenshots#44555

Merged
lcawl merged 6 commits intoelastic:masterfrom
lcawl:ml-updates
Sep 25, 2019
Merged

[DOCS] Update machine learning screenshots#44555
lcawl merged 6 commits intoelastic:masterfrom
lcawl:ml-updates

Conversation

@lcawl
Copy link
Contributor

@lcawl lcawl commented Aug 30, 2019

Summary

This PR updates screenshots in Kibana Guide (https://www.elastic.co/guide/en/kibana/7.4/xpack-ml.html) to reflect changes in the machine learning UI. It also adds a brief overview of the data frame analytics features in Kibana.

Preview: http://kibana_44555.docs-preview.app.elstc.co/guide/en/kibana/master/xpack-ml.html

Checklist

Use strikethroughs to remove checklist items you don't feel are applicable to this PR.

- [ ] This was checked for cross-browser compatibility, including a check against IE11
- [ ] Any text added follows EUI's writing guidelines, uses sentence case text and includes i18n support
- [ ] Documentation was added for features that require explanation or tutorials
- [ ] Unit or functional tests were updated or added to match the most common scenarios
- [ ] This was checked for keyboard-only and screenreader accessibility

For maintainers

[skip ci]

@elasticmachine
Copy link
Contributor

Pinging @elastic/ml-ui

@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-docs

@lcawl lcawl changed the title [DOCS] [DOCS] Update machine learning screenshots Aug 30, 2019
@lcawl lcawl added release_note:skip Skip the PR/issue when compiling release notes v7.4.1 labels Sep 23, 2019
@lcawl lcawl removed the WIP Work in progress label Sep 23, 2019
@lcawl lcawl marked this pull request as ready for review September 23, 2019 23:31
Copy link
Contributor

@szabosteve szabosteve left a comment

Choose a reason for hiding this comment

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

Only one smaller comment, totally nitpicking.
Otherwise, LG2M.

{anomaly-detect-cap} runs in and scales with {es}, and includes an
intuitive UI on the {kib} *Machine Learning* page for creating {anomaly-jobs}
and understanding results.
cyber attacks, or business issues becomes impractical.
Copy link
Contributor

@szabosteve szabosteve Sep 24, 2019

Choose a reason for hiding this comment

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

Nitpicking!
The intro in this form is a bit weird for me because it only states that the human effort to inspect dashboards for spotting cyber attacks becomes impractical. After this sentence, I would expect a positive statement that suggests a solution for making the effort practical or offers an alternative, but the next sentence is about licensing.

What if we would add something like:

As datasets increase in size and complexity, the human effort required to
inspect dashboards or maintain rules for spotting infrastructure problems,
cyber attacks, or business issues becomes impractical. {es} machine
learning features – such as anomaly detection or outlier detection – make
it easier to notice suspicious activities with minimal human interference.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks, that's a great suggestion!

Copy link
Contributor

@peteharverson peteharverson left a comment

Choose a reason for hiding this comment

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

The Data Visualizer screenshot should be retaken as the red charts were a bug - they should be blue and this was fixed for 7.4.0 in #44499.

Also, this was not edited in this PR, but could some more user-friendly text be used in place of the URL form of the first link here. Such as Machine learning in the Elastic stack?

image


If you have a license that includes the {ml-features}, you can create
{oldetection} {dfanalytics-jobs} and view their results on the
*Machine Learning* > *Analytics* page in {kib}. For example:
Copy link
Contributor

Choose a reason for hiding this comment

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

This style is inconsistent with the rest of the page - I'd remove the *Machine Learning* > part

@lcawl
Copy link
Contributor Author

lcawl commented Sep 24, 2019

The Data Visualizer screenshot should be retaken as the red charts were a bug - they should be blue and this was fixed for 7.4.0 in #44499.

Good catch, thanks @peteharverson

@lcawl
Copy link
Contributor Author

lcawl commented Sep 25, 2019

@elasticmachine, run elasticsearch-ci/docs

Copy link
Contributor

@peteharverson peteharverson left a comment

Choose a reason for hiding this comment

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

Latest edits and screenshots LGTM

@lcawl lcawl merged commit 5777a02 into elastic:master Sep 25, 2019
@lcawl lcawl deleted the ml-updates branch September 25, 2019 15:08
lcawl added a commit to lcawl/kibana that referenced this pull request Sep 25, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:ml release_note:skip Skip the PR/issue when compiling release notes Team:Docs v7.4.1 v7.5.0 v8.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants