Skip to content

fix: resolve HF weights cache-first so warm-cache loads nevers touch … - #127

Merged
Dinghye merged 1 commit into
mainfrom
fix/hf-cache-first
Aug 20, 2026
Merged

fix: resolve HF weights cache-first so warm-cache loads nevers touch …#127
Dinghye merged 1 commit into
mainfrom
fix/hf-cache-first

Conversation

@Dinghye

@Dinghye Dinghye commented Aug 20, 2026

Copy link
Copy Markdown
Collaborator

Every embedder's hf_hub_download/snapshot_download (and the rshf from_pretrained loaders) previously issued a HEAD request to huggingface.co per fresh process even with fully cached weights, so Hub outages/429s froze get_embedding, worst for agent integrations spawning a process per call. All sites now go through shared cache-first helpers (local_files_only=True first, network only on cache miss, with snapshot completeness validation)

@Dinghye
Dinghye requested a balanced review from Copilot August 20, 2026 21:32

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@Dinghye
Dinghye merged commit c92221e into main Aug 20, 2026
4 checks passed
@Dinghye
Dinghye deleted the fix/hf-cache-first branch August 20, 2026 22:43
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.

2 participants