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(info): add --labels flag to print label statistics #38

Merged
merged 1 commit into from
Jun 10, 2024

Conversation

mikix
Copy link
Contributor

@mikix mikix commented Jun 10, 2024

With chart-review info --labels, you'll get a table summarizing how often each label was used by each annotator, plus a unioned view of how often each label was used by any annotator.

I Also took out the little list of labels printed with just bare info since it felt redundant and let me simplify that view a bit. Not committed to this stance, but it felt cleaner to me.

I've reviewed with Andy about some of the more "what would a researcher want to see?" choices here - like the "union" Any-annotator section and using the final grouped labels instead of the intermediate labels.

image

Checklist

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

ill: [insomnia, chickenpox, ebola]
animal: [dog, cat, fox]
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Just changing to a less charged example.

Choose a reason for hiding this comment

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

Copy link
Contributor Author

Choose a reason for hiding this comment

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

OK but cooties is on that list and that's definitely real, so...

ill: [insomnia, chickenpox, ebola]
animal: [dog, cat, fox]

Choose a reason for hiding this comment

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

@mikix mikix force-pushed the mikix/info-labels branch from c45e1ca to 39252a3 Compare June 10, 2024 17:45
With `chart-review info --labels`, you'll get a table summarizing how
often each label was used by each annotator, plus a unioned view of
how often each label was used by _any_ annotator.

I Also took out the little list of labels printed with just bare `info`
since it felt redundant and let me simplify that view a bit. Not
committed to this stance, but it felt cleaner to me.
@mikix mikix force-pushed the mikix/info-labels branch from 39252a3 to 00c2803 Compare June 10, 2024 17:46
@mikix
Copy link
Contributor Author

mikix commented Jun 10, 2024

Just pushed a quick update to sort the list of annotator names, FYI.

Will land this and then release 1.2.0 I think

@mikix mikix merged commit f69caea into main Jun 10, 2024
2 checks passed
@mikix mikix deleted the mikix/info-labels branch June 10, 2024 17:50
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.

2 participants