Skip to content

[ML] Edit hyperparameter descriptions in job wizard#91072

Merged
lcawl merged 8 commits intoelastic:masterfrom
lcawl:ml-hyperparameters
Feb 12, 2021
Merged

[ML] Edit hyperparameter descriptions in job wizard#91072
lcawl merged 8 commits intoelastic:masterfrom
lcawl:ml-hyperparameters

Conversation

@lcawl
Copy link
Copy Markdown
Member

@lcawl lcawl commented Feb 11, 2021

Summary

Related to #90843 and #90738

Screenshots

Before:

image

After:
image

Copy link
Copy Markdown
Contributor

@tveasey tveasey left a comment

Choose a reason for hiding this comment

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

Couple of small suggestions but basically LGTM

lcawl and others added 4 commits February 11, 2021 10:27
…s/analytics_creation/components/advanced_step/hyper_parameters.tsx

Co-authored-by: Tom Veasey <tveasey@users.noreply.github.com>
…s/analytics_creation/components/advanced_step/hyper_parameters.tsx

Co-authored-by: Tom Veasey <tveasey@users.noreply.github.com>
…s/analytics_creation/components/advanced_step/hyper_parameters.tsx

Co-authored-by: Tom Veasey <tveasey@users.noreply.github.com>
@lcawl lcawl marked this pull request as ready for review February 11, 2021 18:45
@lcawl lcawl requested a review from a team as a code owner February 11, 2021 18:45
@lcawl lcawl added :ml release_note:skip Skip the PR/issue when compiling release notes v7.12.0 v8.0.0 labels Feb 11, 2021
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/ml-ui (:ml)

@lcawl lcawl added the Feature:Data Frame Analytics ML data frame analytics features label Feb 11, 2021
@lcawl
Copy link
Copy Markdown
Member Author

lcawl commented Feb 11, 2021

@elasticmachine merge upstream

Copy link
Copy Markdown
Contributor

@alvarezmelissa87 alvarezmelissa87 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 c64cc35 into elastic:master Feb 12, 2021
@lcawl lcawl deleted the ml-hyperparameters branch February 12, 2021 00:19
lcawl added a commit to lcawl/kibana that referenced this pull request Feb 12, 2021
Co-authored-by: Tom Veasey <tveasey@users.noreply.github.com>
lcawl added a commit that referenced this pull request Feb 12, 2021
Co-authored-by: Tom Veasey <tveasey@users.noreply.github.com>
@kibanamachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
ml 6.4MB 6.4MB -408.0B

History

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Feature:Data Frame Analytics ML data frame analytics features :ml release_note:skip Skip the PR/issue when compiling release notes v7.12.0 v8.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants