Skip to content

fix: make mistral base check conditional to fix offline loading#42880

Merged
ArthurZucker merged 2 commits intohuggingface:v4.57-releasefrom
Killusions:fix/make-mistral-base-check-conditional-to-fix-offline-loading
Jan 13, 2026
Merged

fix: make mistral base check conditional to fix offline loading#42880
ArthurZucker merged 2 commits intohuggingface:v4.57-releasefrom
Killusions:fix/make-mistral-base-check-conditional-to-fix-offline-loading

Conversation

@Killusions
Copy link
Copy Markdown

Backport of a small part of #42528, the model_info call currently breaks offline loading

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.

Can you add a test? This should fail in general already because the local var _is_local vs is_local

Comment thread src/transformers/tokenization_utils_base.py
@Killusions
Copy link
Copy Markdown
Author

@vasqu I'm fine with doing these changes and adding a test, but should we really deviate from the upstream/main branch in a backport? See #42528

@Killusions
Copy link
Copy Markdown
Author

And if we add a test we should also merge it to main

@Killusions
Copy link
Copy Markdown
Author

@vasqu Ah I think I misunderstood you, fixed

@Killusions Killusions force-pushed the fix/make-mistral-base-check-conditional-to-fix-offline-loading branch from 247da3e to cb94784 Compare December 16, 2025 14:34
@Killusions
Copy link
Copy Markdown
Author

@vasqu Also added a test

@vasqu
Copy link
Copy Markdown
Contributor

vasqu commented Dec 17, 2025

cc @ArthurZucker @itazap

@Killusions
Copy link
Copy Markdown
Author

@vasqu @ArthurZucker @itazap

What's the timeline for this? Is there any change you still require before merging this?

@Killusions
Copy link
Copy Markdown
Author

Bump @vasqu @ArthurZucker @itazap

@ArthurZucker
Copy link
Copy Markdown
Collaborator

Hey! I'll see with @vasqu

@ArthurZucker ArthurZucker added the for patch Tag issues / labels that should be included in the next patch label Jan 8, 2026
@ArthurZucker ArthurZucker merged commit 33ac6bd into huggingface:v4.57-release Jan 13, 2026
10 of 14 checks passed
gdoermann added a commit to gdoermann/agents that referenced this pull request Mar 20, 2026
Replace the <=4.57.1 cap with targeted excludes for 4.57.2 and 4.57.3,
which had a local_files_only=True regression (huggingface/transformers#42369).
The bug was fixed in 4.57.4 (huggingface/transformers#42880), so versions
4.57.4+ and 5.x are safe.

The <=4.57.1 cap blocks the entire dependency tree from upgrading to
transformers 5.x, which includes a security fix for PVE-2026-85102
(insecure deserialization in Trainer._load_rng_state).

Fixes livekit#5173
gdoermann added a commit to gdoermann/agents that referenced this pull request Mar 20, 2026
Replace the <=4.57.1 cap with targeted excludes for 4.57.2 and 4.57.3,
which had a local_files_only=True regression (huggingface/transformers#42369).
The bug was fixed in 4.57.4 (huggingface/transformers#42880), so versions
4.57.4+ and 5.x are safe.

The <=4.57.1 cap blocks the entire dependency tree from upgrading to
transformers 5.x, which includes a security fix for PVE-2026-85102
(insecure deserialization in Trainer._load_rng_state).

Fixes livekit#5173
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.

3 participants