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

[ML] Anomaly swim lane embeddable navigation and filter actions #71082

Merged
merged 21 commits into from
Jul 14, 2020

Conversation

darnautov
Copy link
Contributor

@darnautov darnautov commented Jul 8, 2020

Summary

Related meta issue #66553.

Introduces the following swim lane embeddable actions:

  • Open swim lane in the Anomaly Explorer preserving pagination. Supports both executions from the embeddable panel context menu and cell selection trigger
    Jul-09-2020 15-13-07
    Jul-09-2020 15-14-43
  • Apply the selected time range to the dashboard. In case a single cell has been clicked it uses the interval to populate the time range.
    Jul-09-2020 14-58-55
  • Add filters with selected influencer values. Affect all the other visualizations on the dashboard.
    Jul-09-2020 15-10-13

ML URL Generator (#69265)

ML URL generator has been introduced to produce URLs to the Anomaly Explorer page.

Checklist

@darnautov darnautov force-pushed the ML-66553-swim-lane-drilldown branch from 703d1d5 to ec75f9f Compare July 9, 2020 12:47
@darnautov darnautov self-assigned this Jul 9, 2020
@darnautov darnautov mentioned this pull request Jul 9, 2020
10 tasks
@elastic elastic deleted a comment from kibanamachine Jul 9, 2020
@darnautov darnautov added :ml Feature:Embedding Embedding content via iFrame Feature:UIActions UI actions. These are client side only, not related to the server side actions.. release_note:enhancement v7.9.0 v8.0.0 and removed Feature:Embedding Embedding content via iFrame labels Jul 9, 2020
@darnautov darnautov marked this pull request as ready for review July 9, 2020 13:23
@darnautov darnautov requested a review from a team as a code owner July 9, 2020 13:23
@elasticmachine
Copy link
Contributor

Pinging @elastic/ml-ui (:ml)

@darnautov darnautov added the Feature:Anomaly Detection ML anomaly detection label Jul 9, 2020
parent?: IContainer
) {
super(
initialInput,
{
jobIds: initialInput.jobIds,
swimlaneType: initialInput.swimlaneType,
defaultTitle: initialInput.title,
Copy link
Contributor

Choose a reason for hiding this comment

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

I don't seem to be able to edit the embeddable panel title, when I add a new embeddable, or edit an existing one. Do you see the same?

image

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'm able to edit the title, but there was an issue with a swim lane initializer. Fixed in 5c39b47

Copy link
Contributor

Choose a reason for hiding this comment

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

I can edit it now, but only by keyboard navigation into the text box. But this affects all the visualizations on my dashboard; not just the swim lane embeddable.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

You might need to run yarn kbn clean and re-bootstrap. Changing the title works for me for all panels on the dashboard.

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

Code LGTM ⚡

@qn895
Copy link
Member

qn895 commented Jul 14, 2020

Code LGTM.

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.

Tested latest edits and LGTM!

@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Build metrics

@kbn/optimizer bundle module count

id value diff baseline
ml 389 +2 387

History

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

@darnautov darnautov merged commit 50f8db7 into elastic:master Jul 14, 2020
@darnautov darnautov deleted the ML-66553-swim-lane-drilldown branch July 14, 2020 14:17
darnautov added a commit to darnautov/kibana that referenced this pull request Jul 14, 2020
…tic#71082)

* [ML] dragSelect as part of ExplorerSwimlane component

* [ML] use wrapper ref

* [ML] rename callback

* [ML] WIP open in anomaly explorer

* [ML] MlUrlGenerator unit tests

* [ML] WIP actions

* [ML] restore pagination

* [ML] fix fromPage on initial load

* [ML] fix cell selection, filter and time range actions

* [ML] update url generator params

* [ML] prevent label text selection on drag select

* [ML] fix types and unit tests

* [ML] fix embeddable init

* [ML] fix swim lane unit tests

* [ML] change action label, use filter action only for single cell click

* [ML] fix time range bounds

* [ML] fix TS issues

* [ML] fix pagination persistence

* [ML] use viewByFrom the embeddable input
gmmorris added a commit to gmmorris/kibana that referenced this pull request Jul 14, 2020
* master: (21 commits)
  [Maps] 7.9 design improvements (elastic#71563)
  [ML] Changing all calls to ML endpoints to use internal user (elastic#70487)
  [eventLog] prevent log writing when initialization fails (elastic#71339)
  [Observability] landing page always being displayed (elastic#71494)
  [IM] Address data stream copy feedback (elastic#71615)
  [Logs UI] Anomalies page dataset filtering (elastic#71110)
  [data.search.aggs] Remove `use_field_mapping` from top hits agg (elastic#71168)
  [ML] Anomaly swim lane embeddable navigation and filter actions (elastic#71082)
  Fixes typo in siem_cloudtrail job description (elastic#71569)
  Require granted API Keys to have a name (elastic#71623)
  Update  getUsageForCollection (elastic#71609)
  Only fetch saved elements once (elastic#71310)
  [SecuritySolution][Resolver] Adding siem index and guarding process ancestry (elastic#71570)
  [APM] Additional data telemetry changes (elastic#71112)
  [Visualize] Fix export table for table export links (elastic#71249)
  [Search] Server side search API (elastic#70446)
  use inclusive language (elastic#71607)
  [Security Solution] Hide timeline footer when Resolver is open (elastic#71516)
  [Index template wizard] Remove shadow and use border for components panels (elastic#71606)
  [ML] Kibana API endpoint for histogram chart data (elastic#70976)
  ...
darnautov added a commit that referenced this pull request Jul 14, 2020
…) (#71648)

* [ML] dragSelect as part of ExplorerSwimlane component

* [ML] use wrapper ref

* [ML] rename callback

* [ML] WIP open in anomaly explorer

* [ML] MlUrlGenerator unit tests

* [ML] WIP actions

* [ML] restore pagination

* [ML] fix fromPage on initial load

* [ML] fix cell selection, filter and time range actions

* [ML] update url generator params

* [ML] prevent label text selection on drag select

* [ML] fix types and unit tests

* [ML] fix embeddable init

* [ML] fix swim lane unit tests

* [ML] change action label, use filter action only for single cell click

* [ML] fix time range bounds

* [ML] fix TS issues

* [ML] fix pagination persistence

* [ML] use viewByFrom the embeddable input
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature:Anomaly Detection ML anomaly detection Feature:UIActions UI actions. These are client side only, not related to the server side actions.. :ml release_note:enhancement v7.9.0 v8.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants