Skip to content
This repository was archived by the owner on Jun 3, 2025. It is now read-only.

[transformers] do not ignore any labels in conll eval pipeline#903

Merged
bfineran merged 1 commit intomainfrom
transformers-eval-conll-patch
Feb 6, 2023
Merged

[transformers] do not ignore any labels in conll eval pipeline#903
bfineran merged 1 commit intomainfrom
transformers-eval-conll-patch

Conversation

@bfineran
Copy link
Copy Markdown
Contributor

@bfineran bfineran commented Feb 6, 2023

fixes a bug where outputs ignored by the default ignore_labels value would cause a size mismatch with the expected labels length during metrics computation

test_plan:
verified manually. research team to QA

@bfineran bfineran self-assigned this Feb 6, 2023
Copy link
Copy Markdown
Contributor

@dbogunowicz dbogunowicz left a comment

Choose a reason for hiding this comment

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

So by default zeros were being omitted and this caused the issue?

@bfineran bfineran merged commit 9e5202e into main Feb 6, 2023
@bfineran bfineran deleted the transformers-eval-conll-patch branch February 6, 2023 17:08
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants