Weighted AUROC to omit empty classes

[BeyondTheProof]

Before submitting

• Was this discussed/approved via a Github issue? (no need for typos and docs improvements)
• Did you read the contributor guideline, Pull Request section?
• Did you make sure to update the docs?
• Did you write any new necessary tests?

What does this PR do?

Fixes #348.

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 🙃

[pep8speaks]

Hello @BeyondTheProof! Thanks for updating this PR.

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

Comment last updated at 2021-07-24 21:07:30 UTC

[codecov]

Codecov Report

Merging #376 (8af688d) into master (397a089) will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #376      +/-   ##
==========================================
+ Coverage   96.37%   96.39%   +0.01%     
==========================================
  Files         120      120              
  Lines        3756     3771      +15     
==========================================
+ Hits         3620     3635      +15     
  Misses        136      136              

Flag	Coverage Δ
Linux	76.61% <23.52%> (-0.23%)	⬇️
Windows	76.61% <23.52%> (-0.23%)	⬇️
cpu	96.34% <100.00%> (+0.01%)	⬆️
gpu	96.31% <100.00%> (+0.01%)	⬆️
macOS	96.34% <100.00%> (+0.01%)	⬆️
pytest	96.39% <100.00%> (+0.01%)	⬆️
python3.6	95.51% <100.00%> (+0.01%)	⬆️
python3.8	96.34% <100.00%> (+0.01%)	⬆️
python3.9	96.23% <100.00%> (+0.01%)	⬆️
torch1.3.1	95.51% <100.00%> (+0.01%)	⬆️
torch1.4.0	95.59% <100.00%> (+0.01%)	⬆️
torch1.9.0	96.23% <100.00%> (+0.01%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
torchmetrics/functional/classification/auroc.py	88.09% <100.00%> (+2.58%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 397a089...8af688d. Read the comment docs.

[Borda]

@BeyondTheProof mind check the failing tests?

[SkafteNicki]

Looking good, remember to also update the changelog as this influences the end user!

[BeyondTheProof]

@SkafteNicki I've addressed all your suggestions, and my tests are passing locally. However, code coverage is failing, but I don't know how to address this, and whether it is important or not in this scenario. I would appreciate this workflow approved to make sure everything is A-OK! Thanks again.

[SkafteNicki]

@BeyondTheProof do not worry about the code coverage. Only test to focus on are the once marked as "required"

[SkafteNicki]

LGTM :]