Skip to content

[ML] Add support for percentiles aggregation to Transform wizard - #60763

Merged
peteharverson merged 4 commits into
elastic:masterfrom
peteharverson:transforms-percentiles-agg
Mar 24, 2020
Merged

peteharverson merged 4 commits into
elastic:masterfrom
peteharverson:transforms-percentiles-agg

Conversation

@peteharverson

@peteharverson peteharverson commented Mar 20, 2020 •

Copy link
Copy Markdown
Contributor

Summary

Adds support for the percentiles aggregation into the Create Transform wizard.

The default percentiles (1, 5, 25, 50, 75, 95, 99) can be edited using the edit aggregation popover:

image

Checklist

Fixes #60010

@elasticmachine

Copy link
Copy Markdown
Contributor

Pinging @elastic/ml-ui (:ml)

@pheyos pheyos left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Functional test LGTM

@peteharverson
peteharverson force-pushed the transforms-percentiles-agg branch from fc34ed6 to 4fd072b Compare March 23, 2020 09:56
@peteharverson

Copy link
Copy Markdown
Contributor Author

retest

@peteharverson

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

@darnautov darnautov left a comment

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.

Latest edits LGTM

@peteharverson
peteharverson merged commit 57f9a9f into elastic:master Mar 24, 2020
@peteharverson
peteharverson deleted the transforms-percentiles-agg branch March 24, 2020 09:43
peteharverson added a commit that referenced this pull request Mar 24, 2020
) (#61044)

* [ML] Add support for percentiles aggregation to Transform wizard

* [ML] Fix type error and comments from review

* [ML] Remove unused function

Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>

Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
gmmorris added a commit to gmmorris/kibana that referenced this pull request Mar 24, 2020
* master:
  Updating our direct usage of https-proxy-agent to 5.0.0 (elastic#58296)
  allow users to unset the throttle of an alert (elastic#60964)
  [Lens] Fix bug in metric config panel (elastic#60982)
  [SearchProfiler] Minor fixes (elastic#60919)
  [ML] Renaming ML setup and start contracts (elastic#60980)
  introduce StartServicesAccessor type for `CoreSetup.getStartServices` (elastic#60748)
  [SIEM][Detection Engine] Add rule's notification alert type (elastic#60832)
  [APM] Re-revert "Collect telemetry about data/API performance" (elastic#61030)
  [NP] Graph: get rid of saved objects class wrapper (elastic#59917)
  [EPM] merge duplicate fields when creating index patterns (elastic#60957)
  [Uptime] Ml detection of duration anomalies (elastic#59785)
  [Alerting] removes unimplemented buttons from Alert Details page (elastic#60934)
  [skip-ci] Fix CODEOWNERS paths for the Pulse team (elastic#60944)
  [APM] Threshold alerts (elastic#59566)
  [ML] Add support for percentiles aggregation to Transform wizard (elastic#60763)
  Cahgen save object duplicate message (elastic#60901)
@walterra walterra mentioned this pull request Jan 20, 2022
17 of 32 tasks
patrykkopycinski pushed a commit to patrykkopycinski/kibana that referenced this pull request May 6, 2026
…stic#60763)

* [ML] Add support for percentiles aggregation to Transform wizard

* [ML] Fix type error and comments from review

* [ML] Remove unused function

Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
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.

[ML] Transforms: add support for percentiles aggregation

5 participants