Skip to content

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
{
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.

@darnautov darnautov requested a review from peteharverson July 13, 2020 14:10
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