Skip to content

Comments

[6.x] [ML] Fixes loading the influencers for Anomaly Explorer. (#22963)#22989

Merged
walterra merged 1 commit intoelastic:6.xfrom
walterra:backport/6.x/pr-22963
Sep 13, 2018
Merged

[6.x] [ML] Fixes loading the influencers for Anomaly Explorer. (#22963)#22989
walterra merged 1 commit intoelastic:6.xfrom
walterra:backport/6.x/pr-22963

Conversation

@walterra
Copy link
Contributor

Backports the following commits to 6.x:

- This fixes a regression introduced in elastic#22814. The influencer list wouldn't update if no cell in the swimlanes was selected.
- Renames getTopInfluencers to loadTopInfluencers to be in line with the other functions loadDataForCharts and loadAnomaliesTableData
- Changes the order of arguments for loadDataForCharts so they are the same like in loadTopInfluencers.
@walterra walterra added the backport This PR is a backport of another PR label Sep 13, 2018
@walterra walterra self-assigned this Sep 13, 2018
@walterra walterra added :ml Feature:ml-results legacy - do not use labels Sep 13, 2018
@elasticmachine
Copy link
Contributor

Pinging @elastic/ml-ui

@elasticmachine
Copy link
Contributor

💔 Build Failed

@walterra
Copy link
Contributor Author

retest

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@walterra walterra merged commit 49bfcc6 into elastic:6.x Sep 13, 2018
@walterra walterra deleted the backport/6.x/pr-22963 branch September 13, 2018 12:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR Feature:ml-results legacy - do not use :ml

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants