fix(model_configs): point qwen2.5 OpenVINO models at their real HF repos (#1306) - #1311
Open
Anai-Guo wants to merge 1 commit into
Open
fix(model_configs): point qwen2.5 OpenVINO models at their real HF repos (#1306)#1311Anai-Guo wants to merge 1 commit into
Anai-Guo wants to merge 1 commit into
Conversation
…pos (llmware-ai#1306) Three catalog entries for the qwen2.5 OpenVINO models carried an hf_repo without the ".5": llmware/qwen2-0.5b-instruct-ov and llmware/qwen2-3b-instruct-ov. Neither repo exists, so load_model() failed with a RepositoryNotFoundError that named a repo the user never asked for. Each entry's own "link" field already pointed at the correct repo. Signed-off-by: Anai-Guo <antai12232931@anaiguo.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What this fixes
Fixes #1306.
load_model("qwen2.5-3b-instruct-ov")fails with:The requested model name is
qwen2.5-…, but the repo in the error isqwen2-…. The catalog entry'shf_repois missing the.5, sopull_snapshot_from_hf()requests a repo that does not exist. Each affected entry's ownlinkfield already points at the correct repo, which is what makes the mismatch unambiguous:hf_repo(before)link/ actual repoqwen2.5-0.5b-instruct-ov(×2)llmware/qwen2-0.5b-instruct-ov→ 401llmware/qwen2.5-0.5b-instruct-ov→ 200qwen2.5-3b-instruct-ovllmware/qwen2-3b-instruct-ov→ 401llmware/qwen2.5-3b-instruct-ov→ 200Three lines in
global_model_repo_catalog_list, no other change. Verified against the HF API: the repos named before this patch return 401 (do not exist), the ones it points to return 200. All otherqwen2.5-*-oventries in the catalog were already correct.Note for maintainers (not changed here)
While verifying, four other entries have a
linkthat 404s while theirhf_repois correct — cosmetic, and out of scope for this fix, but worth a follow-up:dolphin-2.9.4-llama3.1-8b-ov→ link saysllmware/dolphi-…(missingn)intel-neural-chat-7b-v3-2-ov→ link says…-v3-ovllama-3.2-1b-instruct-npu-ov→ link says…-1b-npu-instruct-ovqwen2.5-coder-7b-instruct-ov→ link says…-7b-coder-instruct-ov🤖 Generated with Claude Code