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] Adds the class_assignment_objective parameter to classification #52763

Merged
merged 15 commits into from
Mar 12, 2020

Conversation

tveasey
Copy link
Contributor

@tveasey tveasey commented Feb 25, 2020

Adds a new parameter for classification that enables choosing whether to assign labels to maximise accuracy or to maximise the minimum class recall.

#52427.

@elasticmachine
Copy link
Collaborator

Pinging @elastic/ml-core (:ml)

@tveasey tveasey force-pushed the add-class-assignment-objective branch from 63aa35d to edfebc2 Compare February 26, 2020 16:20
Copy link
Contributor

@dimitris-athanasiou dimitris-athanasiou left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! A few things to address.

@tveasey
Copy link
Contributor Author

tveasey commented Mar 4, 2020

Thanks for the review @dimitris-athanasiou. I've addressed all your comments. Can you take another look?

@tveasey
Copy link
Contributor Author

tveasey commented Mar 5, 2020

run elasticsearch-ci/bwc
run elasticsearch-ci/default-distro

@tveasey
Copy link
Contributor Author

tveasey commented Mar 6, 2020

run elasticsearch-ci/2

Copy link
Contributor

@dimitris-athanasiou dimitris-athanasiou left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM Now just waiting for those green lights!

@tveasey tveasey merged commit 58340c2 into elastic:master Mar 12, 2020
@tveasey tveasey deleted the add-class-assignment-objective branch March 12, 2020 18:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants