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

Noam/mon 1644 enable f1 as a default metric #2326

Merged
merged 3 commits into from
Feb 14, 2023

Conversation

noamzbr
Copy link
Collaborator

@noamzbr noamzbr commented Feb 12, 2023

What does this implement/fix? Explain your changes.

  1. Add f1 to binary tasks
  2. Even in binary, I may want to use multiclass metrics to get all kinds of metrics that consider both classes (e.g., I may want to use f1_macro to consider also a very un-common 0 class).

@noamzbr noamzbr added the feature Feature update or code change to the package label Feb 12, 2023
@noamzbr noamzbr requested a review from a team as a code owner February 12, 2023 08:44
@noamzbr noamzbr self-assigned this Feb 12, 2023
@noamzbr noamzbr added this pull request to the merge queue Feb 13, 2023
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to no response for status checks Feb 13, 2023
@noamzbr noamzbr added this pull request to the merge queue Feb 14, 2023
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to no response for status checks Feb 14, 2023
@ItayGabbay ItayGabbay merged commit cca4c2f into main Feb 14, 2023
@delete-merged-branch delete-merged-branch bot deleted the noam/mon-1644-enable-f1-as-a-default-metric branch February 14, 2023 15:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature Feature update or code change to the package
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants