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

ENH: Add confusion_matrix_statistics function in contingency_tables #9118

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

pasrakan
Copy link

@pasrakan pasrakan commented Jan 8, 2024

  • tests added / passed.
  • code/documentation is well formatted.
  • properly formatted commit message. See
    NumPy's guide.
  • This commit generates classification performance metrics using square tables. The metrics include Accuracy, No Information Rate, Kappa, Sensitivity, Specificity, Positive Predictive Value, Negative Predictive Value, Balanced Accuracy, and F1. The code draws inspiration from Caret package in R

  • The overall metrics are structured in a dictionary.

  • The class-specific metrics output is structured in rows representing different metrics and columns representing different classes (class 1, class 2, class 3, class 4).

  • The test values are verified with confusionMatrix function in Caret package in R.

@pep8speaks
Copy link

pep8speaks commented Jan 8, 2024

Hello @pasrakan! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

There are currently no PEP 8 issues detected in this Pull Request. Cheers! 🍻

Comment last updated at 2024-02-23 22:42:29 UTC

Xiang Gu added 3 commits January 7, 2024 22:16
fix test_confusion_matrix_statistics_4_categories()
@pasrakan
Copy link
Author

Hi @bashtage ! Could you please review my Pull Request? It has been open for more than one month, and I would appreciate your feedback. Thank you!

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.

None yet

2 participants