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] Meta - Classification UI #51310

Closed
22 tasks done
alvarezmelissa87 opened this issue Nov 21, 2019 · 4 comments
Closed
22 tasks done

[ML] Meta - Classification UI #51310

alvarezmelissa87 opened this issue Nov 21, 2019 · 4 comments
Assignees
Labels

Comments

@alvarezmelissa87
Copy link
Contributor

alvarezmelissa87 commented Nov 21, 2019

@elasticmachine
Copy link
Contributor

Pinging @elastic/ml-ui (:ml)

@alvarezmelissa87
Copy link
Contributor Author

Would you be up for taking a look at the checklist I've got so far? Feel free to point out anything I've missed or have gotten wrong. I've got a couple of clarification questions below, as well. Thank you!
cc @peteharverson, @sophiec20

  1. Do we want num_top_classes available in the form or just in the advanced editor?

  2. It looks like there are 2 options when evaluating classification jobs (https://www.elastic.co/guide/en/elasticsearch/reference/7.5/evaluate-dfanalytics.html#_binary_soft_classification). Is which one to use dependent on the value in num_top_classes?
    E.g. If 2 (the default) then it's binary_soft_classification, otherwise use classification? Or is the type of evaluation something the user will have to specify?

@peteharverson
Copy link
Contributor

My thoughts on #51310 (comment) are

  1. num_top_classes should not be available in the form. If only binary classification is currently done by the back-end currently, I wonder if it even needs to be specifically added in the advanced editor?

  2. The evaluation API we should be using is the classification one documented at https://www.elastic.co/guide/en/elasticsearch/reference/7.5/evaluate-dfanalytics.html#_classification_2. We need to be displaying the confusion matrix in some form in the results view, along with initially some form of tabular view where the user can compare the predicted class to the actual class.

@sophiec20
Copy link
Contributor

The num_top_classes back-end default is 2. For binary classification, there is little point in making this any more visible to the end user as we do not expect them to need to change this value.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants