Skip to content

minor stats fixes to printing and NER only measurement - #620

Merged
mart-r merged 2 commits into
mainfrom
stats_fixes
Sep 29, 2026
Merged

mart-r merged 2 commits into
mainfrom
stats_fixes

Conversation

@adam-sutton-1992

Copy link
Copy Markdown
Contributor

Hihi -

Just some minor changes I found while using the new stats:

  • get_stats and get_stats_calculator now print appropriate metrics for legacy and new stats. The spacing is cleaned up slightly too.
  • The NER only pipeline would assign all labels as "NER" - which was a bit ugly / confusing. I was going to set it to "0" or "1" but instead opted for "DETECTED_ENTITY" which I think is the most descriptive.

@adam-sutton-1992

Copy link
Copy Markdown
Contributor Author

How does my own ruff not spot that over indenting... honestly...

@mart-r mart-r left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Looks good

@mart-r
mart-r merged commit 8dfaa05 into main Sep 29, 2026
25 of 28 checks passed
@mart-r
mart-r deleted the stats_fixes branch September 29, 2026 12:58
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