-
Notifications
You must be signed in to change notification settings - Fork 67
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
[WIP] raise warning f1_score_micro equals accuracy #513
Conversation
Hello @ananyaganesh! Thanks for updating this PR. There are currently no PEP 8 issues detected in this Pull Request. Cheers! 🎉 Comment last updated at 2019-10-03 18:26:28 UTC |
update changes so that build passes
Codecov Report
@@ Coverage Diff @@
## master #513 +/- ##
=========================================
Coverage ? 54.97%
=========================================
Files ? 20
Lines ? 2843
Branches ? 0
=========================================
Hits ? 1563
Misses ? 1280
Partials ? 0
Continue to review full report at Codecov.
|
Should this still be WIP? |
Yes, this is still WIP since it's likely to change based on how we resolve #502 |
I did not realize it is WIP. I saw that the build passed, and approved. I do not find how to unapprove it. |
The coverage is down though. |
Raises warning that micro-averaged F1 score is the same as accuracy for multi-class classification.
Addresses #491