-
Notifications
You must be signed in to change notification settings - Fork 413
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
Add plot functionality to AUROC and ROC metrics #1490
Conversation
Not sure why the docs test is failing. Docs could be build successfully on my local machine. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Already looking good :)
Could you please also add test to this file:
https://github.com/Lightning-AI/metrics/blob/master/tests/unittests/utilities/test_plot.py
This is actually my first time contributing to an open source project and I am new to working with pytest. I have now included tests for BinaryAUROC and MulticlassAUROC plots. I would be glad if you could have a look on it. |
for more information, see https://pre-commit.ci
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## master #1490 +/- ##
======================================
- Coverage 88% 88% -0%
======================================
Files 216 216
Lines 11344 11396 +52
======================================
+ Hits 9969 9997 +28
- Misses 1375 1399 +24 |
for more information, see https://pre-commit.ci
What does this PR do?
Fixes part of #1406
Currently, plot functionality is added to modular metrics BinaryAUROC, MulticlassAUROC, MultilabelAUROC and BinaryROC.
Before submitting
PR review
Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.
Did you have fun?
Make sure you had fun coding 🙃