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

[FLINK-31030] Support more binary classification evaluation metrics #243

Closed
wants to merge 1 commit into from

Conversation

kaori-seasons
Copy link

@kaori-seasons kaori-seasons commented Jun 16, 2023

Related to FLINK-31030

Supplementary indicators: precision, recall, F-measure

Contribution Checklist

  • Make sure that the pull request corresponds to one JIRA issue. Exceptions are made for typos in JavaDoc or documentation files, which need no JIRA issue.

  • Name the pull request in the form "[FLINK-XXXX] Title of the pull request", where FLINK-XXXX should be replaced by the actual issue number.
    Typo fixes that have no associated JIRA issue should be named following this pattern: [hotfix] Title of the pull request.

  • Fill out the template below to describe the changes contributed by the pull request. That will give reviewers the context they need to do the review.

  • Each commit in the pull request has a meaningful commit message (including the JIRA id)

  • Once all items of the checklist are addressed, remove the above text and this checklist, leaving only the filled out template below.

(The sections below can be removed for hotfixes of typos)
-->

What is the purpose of the change

(For example: This pull request adds the implementation of Naive Bayes algorithm.)

Brief change log

(for example:)

  • Adds Transformer and Estimator implementation of Naive Bayes in Java and Python
  • Adds examples and documentations of Naive Bayes

Does this pull request potentially affect one of the following parts:

  • Dependencies (does it add or upgrade a dependency): (yes / no)
  • The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)

Documentation

  • Does this pull request introduce a new feature? (yes / no)
  • If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

@zhipeng93
Copy link
Contributor

@complone Thanks for the PR. It seems that there are some test failures. Can you fix them first?

@zhipeng93
Copy link
Contributor

I will close this PR for now since there is no activity during the past two months. Please feel free to reopen it later @kaori-seasons

@zhipeng93 zhipeng93 closed this Aug 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants