Skip to content

Add Expectations for pipeline token classification tests#46151

Open
kaixuanliu wants to merge 2 commits into
huggingface:mainfrom
kaixuanliu:xpu-expectation
Open

Add Expectations for pipeline token classification tests#46151
kaixuanliu wants to merge 2 commits into
huggingface:mainfrom
kaixuanliu:xpu-expectation

Conversation

@kaixuanliu
Copy link
Copy Markdown
Contributor

This PR Adds Expectations for pipeline token classification tests.
@ydshieh pls help review, thx!

Signed-off-by: Liu, Kaixuan <kaixuan.liu@intel.com>
Signed-off-by: Liu, Kaixuan <kaixuan.liu@intel.com>
@kaixuanliu
Copy link
Copy Markdown
Contributor Author

And BTW, this case failes on cuda as well:
tests/pipelines/test_pipelines_token_classification.py::TokenClassificationPipelineTests::test_pt_ignore_subwords_slow_tokenizer_raises, the root cause is, although we set use_fast=False in tokenizer, the tokenizer(BertTokenizer)'s use_fast attribute is still True, can we just use BertTokenizerLegacy to replace AutoTokenizer here?

@kaixuanliu
Copy link
Copy Markdown
Contributor Author

The failed CI should not be introduced by this PR.

@github-actions
Copy link
Copy Markdown
Contributor

View the CircleCI Test Summary for this PR:

https://huggingface.co/spaces/transformers-community/circle-ci-viz?pr=46151&sha=aecdc4

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.

1 participant