-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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] Fixes outlier detection results exploration color legend display. #129058
[ML] Fixes outlier detection results exploration color legend display. #129058
Conversation
Pinging @elastic/ml-ui (:ml) |
...analytics/pages/analytics_exploration/components/outlier_exploration/outlier_exploration.tsx
Outdated
Show resolved
Hide resolved
...on/data_frame_analytics/pages/analytics_exploration/components/outlier_exploration/common.ts
Show resolved
Hide resolved
💚 Build SucceededMetrics [docs]Async chunks
Unknown metric groupsESLint disabled line counts
Total ESLint disabled count
History
To update your PR or re-run it, just comment with: cc @walterra |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested latest changes and LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
…color legend display. (#129058) Fixes the display of the color legend for the outlier results table. The check to display the legend was based on just checking if the first row on display had feature influence information. However, depending on the jobs results, not every row might be populated with feature influence information. This PR fixes it by displaying the color legend in any case when feature influence calculation is enabled and calculating the necessary featureCount value by looking at all available rows instead of just the first row. Unit tests have been added for the getFeatureCount() function. (cherry picked from commit 2c9d607)
💚 All backports created successfully
Note: Successful backport PRs will be merged automatically after passing CI. Questions ?Please refer to the Backport tool documentation |
…color legend display. (#129058) (#129222) Fixes the display of the color legend for the outlier results table. The check to display the legend was based on just checking if the first row on display had feature influence information. However, depending on the jobs results, not every row might be populated with feature influence information. This PR fixes it by displaying the color legend in any case when feature influence calculation is enabled and calculating the necessary featureCount value by looking at all available rows instead of just the first row. Unit tests have been added for the getFeatureCount() function. (cherry picked from commit 2c9d607) Co-authored-by: Walter Rafelsberger <walter@elastic.co>
Summary
Fixes #127119.
Fixes the display of the color legend for the outlier results table. The check to display the legend was based on just checking if the first row on display had feature influence information. However, depending on the jobs results, not every row might be populated with feature influence information. This PR fixes it by displaying the color legend in any case when feature influence calculation is enabled and calculating the necessary
featureCount
value by looking at all available rows instead of just the first row. Unit tests have been added for thegetFeatureCount()
function.Checklist