Skip to content

Conversation

@cmmartin
Copy link
Contributor

Fixes #323

Checklist

  • My branch is up-to-date with development branch.
  • Everything works and tested on latest stable Rust.
  • Coverage and Linting have been applied

Current behaviour

  • recall
    • binary case calculates precision instead of recall due to incorrect false negative counting
    • multiclass case computes accuracy (total correct / total) instead of macro-averaged recall
  • precision
    • binary case calculates recall instead of precision due to incorrect false positive counting
    • multiclass case computes accuracy instead of macro-averaged precision

New expected behaviour

  • recall
    • binary case correctly calculates recall
    • multiclass case computes macro-averaged recall instead of accuracy
    • handles zero-division edge case
  • precision
    • binary case correctly calculates precision
    • multiclass case computes macro-averaged precision instead of accuracy
    • handles zero-division edge case

Change logs

@cmmartin cmmartin requested a review from Mec-iS as a code owner November 22, 2025 03:46
@Mec-iS
Copy link
Collaborator

Mec-iS commented Nov 22, 2025

thanks a lot. I will review this on monday

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bug: Precision and recall calculations

2 participants