Skip to content

model : fix model type check for granite/llama3 and deepseek2/glm4.7 lite - #22870

Merged
CISC merged 1 commit into
masterfrom
cisc/model-type-vocab-check-fix
May 10, 2026
Merged

model : fix model type check for granite/llama3 and deepseek2/glm4.7 lite#22870
CISC merged 1 commit into
masterfrom
cisc/model-type-vocab-check-fix

Conversation

@CISC

@CISC CISC commented May 9, 2026

Copy link
Copy Markdown
Member

Overview

cont #22004
Fixes #22004 (comment)

Additional information

The checks used uninitialized n_vocab instead of fetching from metadata as was done before refactor.

This also broke lite classification and inference of some (Kanana-2?) DeepSeek2 Lite and GLM 4.7 Lite models.

Requirements

@github-actions github-actions Bot added the model Model specific label May 9, 2026
@CISC
CISC requested a review from JohannesGaessler May 9, 2026 13:38
@CISC CISC changed the title fix model type check for granite/llama3 and glm4.7 lite model : fix model type check for granite/llama3 and glm4.7 lite May 9, 2026
@CISC CISC changed the title model : fix model type check for granite/llama3 and glm4.7 lite model : fix model type check for granite/llama3 and deepseek2/glm4.7 lite May 9, 2026

@JohannesGaessler JohannesGaessler 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.

I can confirm that the model type is now being set correctly.

@CISC CISC added the merge ready A maintainer can use this label to indicate that they consider the changes final and ready to merge. label May 9, 2026
@CISC
CISC merged commit 5755a10 into master May 10, 2026
46 checks passed
@CISC
CISC deleted the cisc/model-type-vocab-check-fix branch May 10, 2026 06:44
meh pushed a commit to meh/llama.cpp that referenced this pull request May 10, 2026
baramofme pushed a commit to baramofme/llama-cpp-turboquant that referenced this pull request May 23, 2026
winstonma pushed a commit to winstonma/llama.cpp that referenced this pull request May 27, 2026
fewtarius pushed a commit to fewtarius/CachyLLama that referenced this pull request May 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

merge ready A maintainer can use this label to indicate that they consider the changes final and ready to merge. model Model specific

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants