-
Notifications
You must be signed in to change notification settings - Fork 79
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
Fixed plotting function for final stats #691
Conversation
MLCommons CLA bot All contributors have signed the MLCommons CLA ✍️ ✅ |
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.
Minor changes requested.
Co-authored-by: Sarthak Pati <sarthak.pati@pennmedicine.upenn.edu>
Codecov Report
@@ Coverage Diff @@
## master #691 +/- ##
=======================================
Coverage 94.62% 94.62%
=======================================
Files 116 116
Lines 8062 8062
=======================================
Hits 7629 7629
Misses 433 433
Flags with carried forward coverage won't be shown. Click here to find out more. 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
@sarthakpati Ready to review |
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.
LGTM
Fixes #334
Proposed Changes
Plotting is now worked on automatically for all columns except for per_class, per_label and global which are separated by _
Checklist
CONTRIBUTING
guide.pip install
step is needed for PR to be functional), please ensure it is reflected in all the files that control the CI, namely: python-test.yml, and all docker files [1,2,3].