Skip to content

Fix explicit local code resolution for tokenizers and image processors#45169

Merged
vasqu merged 2 commits intohuggingface:mainfrom
hmellor:fix-explicit-local-resolution
Apr 1, 2026
Merged

Fix explicit local code resolution for tokenizers and image processors#45169
vasqu merged 2 commits intohuggingface:mainfrom
hmellor:fix-explicit-local-resolution

Conversation

@hmellor
Copy link
Copy Markdown
Member

@hmellor hmellor commented Apr 1, 2026

In #45094 I introduced some errors to the remote code resolution when trying to detect if local code belonged to Transformers or not.

These tests were:

pytest tests/models/cohere_asr/test_modeling_cohere_asr.py::CohereAsrIntegrationTest::test_longform_english
pytest tests/models/cohere_asr/test_modeling_cohere_asr.py::CohereAsrIntegrationTest::test_non_english_with_punctuation
pytest tests/models/pp_ocrv5_mobile_rec/test_modeling_pp_ocrv5_mobile_rec.py::PPOCRV5MobileRecModelIntegrationTest::test_inference_text_recognition_head
pytest tests/models/pp_ocrv5_server_rec/test_modeling_pp_ocrv5_server_rec.py::PPOCRV5ServerRecModelIntegrationTest::test_inference_text_recognition_head
pytest tests/models/slanext/test_modeling_slanext.py::SLANeXtModelIntegrationTest::test_inference_table_recognition_head
pytest tests/models/timm_wrapper/test_modeling_timm_wrapper.py::TimmWrapperModelIntegrationTest::test_inference_with_pipeline

This PR fixes all of these.

(The timm test still appears to fail, but not for the reason that was caused by the linked PR)

hmellor added 2 commits April 1, 2026 17:19
Signed-off-by: Harry Mellor <19981378+hmellor@users.noreply.github.com>
Signed-off-by: Harry Mellor <19981378+hmellor@users.noreply.github.com>
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 1, 2026

[For maintainers] Suggested jobs to run (before merge)

run-slow: auto

@HuggingFaceDocBuilderDev
Copy link
Copy Markdown

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 1, 2026

View the CircleCI Test Summary for this PR:

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

Copy link
Copy Markdown
Contributor

@vasqu vasqu left a comment

Choose a reason for hiding this comment

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

Thanks for the fix 🫡

@vasqu vasqu merged commit 43ed69f into huggingface:main Apr 1, 2026
28 of 30 checks passed
@hmellor hmellor deleted the fix-explicit-local-resolution branch April 1, 2026 21:48
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.

3 participants