-
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] Adds popover help for multiclass confusion matrix #101732
Conversation
Pinging @elastic/ml-ui (:ml) |
...nalytics_exploration/components/classification_exploration/confusion_matrix_help_popover.tsx
Outdated
Show resolved
Hide resolved
One other thing we mention in https://www.elastic.co/guide/en/machine-learning/current/flightdata-classification.html#flightdata-classification-evaluate that might be worth considering adding is this feature: "You can also use the Training and Testing filter options to refine the contents of the matrix. Thus you can see how well the model performs on previously unseen data." I believe the method for filtering the matrix changed in one of the releases, so it took me a bit to figure out how to do it again the last time I tried. |
…s/analytics_exploration/components/classification_exploration/confusion_matrix_help_popover.tsx Co-authored-by: Lisa Cawley <lcawley@elastic.co>
Yes, there is a selector in the top right corner of the page that can switch between Training/Testing. However, that toggle changes the context for the entire page, not only for the confusion matrix. For this reason, I wouldn't highlight it in the popover help of the confusion matrix. I added, however, a sentence about the fact that the user can see the exact number of occurrences if they select a cell in the matrix. |
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.
The popover text content looks good, but just added a small suggestion for the layout of the title section.
...alytics/pages/analytics_exploration/components/classification_exploration/evaluate_panel.tsx
Show resolved
Hide resolved
💚 Build SucceededMetrics [docs]Module Count
Async chunks
History
To update your PR or re-run it, just comment with: |
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
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.
Yay, congrats on your first awesome popover!
Summary
This PR adds help for the "Normalized confusion matrix for the entire dataset" section of the classification exploration page, using an EUI popover.
Checklist
Delete any items that are not applicable to this PR.