Skip to content

[ML] Data Frame Analytics: Fix steps to be named phases.#65855

Merged
walterra merged 9 commits intoelastic:masterfrom
walterra:ml-analytics-progress-fix
May 13, 2020
Merged

[ML] Data Frame Analytics: Fix steps to be named phases.#65855
walterra merged 9 commits intoelastic:masterfrom
walterra:ml-analytics-progress-fix

Conversation

@walterra
Copy link
Copy Markdown
Contributor

@walterra walterra commented May 8, 2020

Summary

Fixes #65607.

  • Fixes wrong names and renames everything related to step to phase.
  • Renames column heading Progress per Step to just Progress.
  • Hide the progress bar percentage, instead it's available in a tooltip.
  • Changes the progress column content to Phase 1/4 [ProgressBar].

image

Checklist

Delete any items that are not applicable to this PR.

For maintainers

@walterra walterra self-assigned this May 8, 2020
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/ml-ui (:ml)

@walterra walterra added bug Fixes for quality problems that affect the customer experience release_note:skip Skip the PR/issue when compiling release notes labels May 8, 2020
@walterra walterra marked this pull request as ready for review May 8, 2020 14:54
@walterra walterra requested a review from a team as a code owner May 8, 2020 14:54
@qn895
Copy link
Copy Markdown
Member

qn895 commented May 8, 2020

Tested and code LGTM

@walterra
Copy link
Copy Markdown
Contributor Author

@elasticmachine merge upstream

Copy link
Copy Markdown
Member

@jgowdyelastic jgowdyelastic left a comment

Choose a reason for hiding this comment

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

Code LGTM

@walterra
Copy link
Copy Markdown
Contributor Author

@elasticmachine merge upstream

</EuiText>
</EuiFlexItem>
<EuiFlexItem style={{ width: '40px' }} grow={false}>
<EuiToolTip content={`Progress of phase ${currentPhase}: ${progress}%`}>
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This needs internationalizing.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Fixed in bad6c67.

Copy link
Copy Markdown
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.

One missed translation, but otherwise tested and LGTM

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 ⚡

@walterra
Copy link
Copy Markdown
Contributor Author

@elasticmachine merge upstream

@walterra
Copy link
Copy Markdown
Contributor Author

@elasticmachine merge upstream

@kibanamachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

History

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

@walterra walterra merged commit 115ede9 into elastic:master May 13, 2020
@walterra walterra deleted the ml-analytics-progress-fix branch May 13, 2020 10:16
walterra added a commit to walterra/kibana that referenced this pull request May 13, 2020
- Fixes wrong names and renames everything related to step to phase.
- Renames column heading Progress per Step to just Progress.
- Hide the progress bar percentage, instead it's available in a tooltip.
- Changes the progress column content to Phase 1/4 [ProgressBar].
walterra added a commit to walterra/kibana that referenced this pull request May 13, 2020
- Fixes wrong names and renames everything related to step to phase.
- Renames column heading Progress per Step to just Progress.
- Hide the progress bar percentage, instead it's available in a tooltip.
- Changes the progress column content to Phase 1/4 [ProgressBar].
walterra added a commit that referenced this pull request May 13, 2020
…6379)

- Fixes wrong names and renames everything related to step to phase.
- Renames column heading Progress per Step to just Progress.
- Hide the progress bar percentage, instead it's available in a tooltip.
- Changes the progress column content to Phase 1/4 [ProgressBar].
v1v added a commit to v1v/kibana that referenced this pull request May 13, 2020
* upstream/master: (223 commits)
  [Ingest] Support root level yaml variables in agent stream template (elastic#66120)
  [Snapshot Restore] Fix error when deleting snapshots behind reverse proxy (elastic#66147)
  [Lens] fix empty state for pie (elastic#66206)
  [APM] Improve e2e tests (elastic#66373)
  [ML] Data Frame Analytics: Fix steps to be named phases. (elastic#65855)
  [Discover] Encode context link filter part (elastic#63831)
  [APM] Scope APM alert creation to environment (elastic#65681)
  Move Kibana Usage collectors outside the telemetry plugin (elastic#65663)
  [ML] Data Frame Analytics: Fix confusion matrix data grid width. (elastic#65818)
  Switch to core application service (elastic#63443)
  Removes use of prefer_v2_templates (elastic#66316)
  [Maps] handle case where fit to bounds does not match any documents (elastic#66307)
  log error instead of throw (elastic#66254)
  [plugin-helpers] remove outdated postinstall task (elastic#66324)
  Spaces - migrate default space and enter space view to KP (elastic#66098)
  [APM] Change plugin id for `apm_oss` to `apmOss` (elastic#66164)
  [Maps] convert map_selectors to TS (elastic#65905)
  [uptime/usage-collector] add missing await (elastic#66079)
  [Ingest] Add additional attributes to the Datasources Saved Object (elastic#66127)
  [Endpoint]EMT-339: add new policy response schema (elastic#66264)
  ...
walterra added a commit that referenced this pull request May 13, 2020
…6378)

- Fixes wrong names and renames everything related to step to phase.
- Renames column heading Progress per Step to just Progress.
- Hide the progress bar percentage, instead it's available in a tooltip.
- Changes the progress column content to Phase 1/4 [ProgressBar].
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Fixes for quality problems that affect the customer experience Feature:Data Frame Analytics ML data frame analytics features :ml release_note:skip Skip the PR/issue when compiling release notes v7.8.0 v7.9.0 v8.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[ML] Data Frame Analytics: Improve analysis progress in job list

7 participants