Skip to content

Fix metric tests#223

Open
sananakther6642 wants to merge 4 commits into
meta-pytorch:mainfrom
sananakther6642:fix-metric-tests
Open

Fix metric tests#223
sananakther6642 wants to merge 4 commits into
meta-pytorch:mainfrom
sananakther6642:fix-metric-tests

Conversation

@sananakther6642

@sananakther6642 sananakther6642 commented Dec 14, 2025

Copy link
Copy Markdown

Summary:

  • Fix flaky MulticlassAccuracy test when a class has zero samples by explicitly handling empty classes.
  • Fix the same zero-sample-class edge case in functional multiclass_accuracy tests.
  • Fix BinaryAUROC test failures caused by float64 vs float32 dtype mismatches.
  • Skip FrechetAudioDistance VGGish tests when torchaudio prototype features are unavailable.

Test plan:

  • pytest -q torcheval/tests/metrics/classification/test_accuracy.py
  • pytest -q torcheval/tests/metrics/functional/classification/test_accuracy.py
  • pytest -q torcheval/tests/metrics/classification/test_auroc.py
  • pytest -q torcheval/tests/metrics/audio/test_fad.py
  • pytest -q torcheval/tests/metrics/functional/classification

Fixes: N/A

@meta-cla meta-cla Bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Dec 14, 2025
@sananakther6642

Copy link
Copy Markdown
Author

Hi team! 👋 Just wanted to do a gentle follow-up on this PR — it's been a few months since submission and I wanted to check if there's anything on my end I can help with, such as rebasing, addressing feedback, or making any adjustments. Happy to do whatever is needed to move this forward. Thanks for your time and for maintaining this project! 🙏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant