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

Edits Machine Learning Overview text #64593

Closed
wants to merge 2 commits into from
Closed

Conversation

lcawl
Copy link
Contributor

@lcawl lcawl commented Apr 27, 2020

Summary

This PR edits the UI text in the Machine Learning Overview page:

image

In general, the goal is to make the text more concise.

  • Remove “For more information about machine learning in the Elastic stack please see here” It seems like an unnecessary reference to the marketing page, when they must already have the product installed.
  • Edit “If you have input… please feel free to submit…” to “If you have input…your experience, please submit..”.
  • Change “Create your first analytics job” to “Create your first data frame analytics job”?
  • Change “…and annotate it with…” description of analytics jobs so it’s clearer that this annotation is performed by the job not the user.
  • Change “Machine learning makes it easy….” to something like “Anomaly detection enables you to find unusual behavior in time series data”. Remove the unnecessary “Track one metric” sentence.

Checklist

Delete any items that are not applicable to this PR.

For maintainers

@elasticmachine
Copy link
Contributor

Pinging @elastic/ml-ui (:ml)

@kibanamachine
Copy link
Contributor

💚 Build Succeeded

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@lcawl
Copy link
Contributor Author

lcawl commented Apr 29, 2020

Closed as duplicate of #64625

@lcawl lcawl closed this Apr 29, 2020
@lcawl lcawl deleted the ml-overview branch April 29, 2020 23:27
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.

3 participants