Skip to content
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

feat(accuracy): add --verbose to print table of per-chart results #41

Merged
merged 1 commit into from
Jun 11, 2024

Conversation

mikix
Copy link
Contributor

@mikix mikix commented Jun 11, 2024

accuracy --verbose will now print a table of chart id / label / classification for the annotator, like this:

    ╭──────────┬──────────┬────────────────╮
    │ Chart ID │ Label    │ Classification │
    ├──────────┼──────────┼────────────────┤
    │ 1        │ Cough    │ TP             │
    │ 1        │ Fatigue  │ TP             │
    ├──────────┼──────────┼────────────────┤
    │ 4        │ Cough    │ FP             │
    │ 4        │ Fatigue  │ TP             │
    ╰──────────┴──────────┴────────────────╯

This is in addition to normal stat output. If --save is passed, the verbose bit is still only printed to the console.

Fixes: #40

Checklist

  • Consider if documentation (like in docs/) needs to be updated
  • Consider if tests should be added

@mikix mikix force-pushed the mikix/verbose branch 2 times, most recently from e852a56 to 9ca793f Compare June 11, 2024 20:00
@mikix mikix changed the title accuracy --verbose WIP feat(accuracy): add --verbose to print table of per-chart results Jun 11, 2024
accuracy --verbose will now print a table of chart id / label /
classification for the annotator, like this:

╭──────────┬──────────┬────────────────╮
│ Chart ID │ Label    │ Classification │
├──────────┼──────────┼────────────────┤
│ 1        │ Cough    │ TP             │
│ 1        │ Fatigue  │ TP             │
├──────────┼──────────┼────────────────┤
│ 4        │ Cough    │ FP             │
│ 4        │ Fatigue  │ TP             │
╰──────────┴──────────┴────────────────╯

This is in addition to normal stat output. If --save is passed,
the verbose bit is still only printed to the console.
@mikix mikix marked this pull request as ready for review June 11, 2024 20:04
truth: str,
annotator: str,
save: bool = False,
verbose: bool = False,

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: is this the only way we're going to want to make this more verbose? I suppose you can always change it later

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ooh i like this trick

@mikix mikix merged commit e31ceaf into main Jun 11, 2024
2 checks passed
@mikix mikix deleted the mikix/verbose branch June 11, 2024 20:37
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.

verbose output option
2 participants