-
Notifications
You must be signed in to change notification settings - Fork 416
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
Fix auc calculation and add tests #197
Conversation
Codecov Report
@@ Coverage Diff @@
## master #197 +/- ##
==========================================
+ Coverage 96.63% 96.65% +0.02%
==========================================
Files 180 180
Lines 5760 5738 -22
==========================================
- Hits 5566 5546 -20
+ Misses 194 192 -2
Flags with carried forward coverage won't be shown. Click here to find out more. Continue to review full report at Codecov.
|
Stable sort that we are trying to do doesn't work: https://colab.research.google.com/gist/maximsch2/4341c3b01482f8b38ea0f0ed70affd33/stable-sort-in-pytorch.ipynb |
@maximsch2 I see your point that no matter what we do, we will never get stable sorting on cpu until torch v1.9 is released. Could you please add a note to the affected metrics, that users can expect slightly different results on cpu vs gpu. |
Co-authored-by: Nicki Skafte <[email protected]>
Hello @maximsch2! Thanks for updating this PR. There are currently no PEP 8 issues detected in this Pull Request. Cheers! 🍻 Comment last updated at 2021-04-23 14:17:17 UTC |
Before submitting
What does this PR do?
Fixes #195
Closes #196
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 🙃