KEMBAR78
fix broken offline mode when loading tokenizer from hub by winglian · Pull Request #40669 · huggingface/transformers · GitHub
Skip to content

Conversation

@winglian
Copy link
Contributor

@winglian winglian commented Sep 3, 2025

What does this PR do?

#39874 broke loading tokenizers from_pretrained when using offline mode. Hub's OfflineModeIsEnabled extends python's builtin ConnectionError , not requests.exceptions.ConnectionError

Fixes # (issue)

Before submitting

  • This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
  • Did you read the contributor guideline,
    Pull Request section?
  • Was this discussed/approved via a Github issue or the forum? Please add a link
    to it if that's the case.
  • Did you make sure to update the documentation with your changes? Here are the
    documentation guidelines, and
    here are tips on formatting docstrings.
  • Did you write any new necessary tests?

Who can review?

Anyone in the community is free to review the PR once the tests have passed. Feel free to tag
members/contributors who may be interested in your PR.

Copy link
Member

@Rocketknight1 Rocketknight1 left a comment

Choose a reason for hiding this comment

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

Ah, thank you for the fix! I didn't realize we needed to catch both flavours of ConnectionError.

@Rocketknight1 Rocketknight1 enabled auto-merge (squash) September 4, 2025 12:08
@Rocketknight1 Rocketknight1 merged commit 669230a into huggingface:main Sep 4, 2025
24 checks passed
@HuggingFaceDocBuilderDev

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.

@vasqu vasqu added the for patch Tag issues / labels that should be included in the next patch label Sep 4, 2025
Cyrilvallez pushed a commit that referenced this pull request Sep 4, 2025
* fix broken offline mode when loading tokenizer from hub

* formatting

* make quality

* fix import order
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

for patch Tag issues / labels that should be included in the next patch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants