fix: add DeepseekV41ForCausalLM to test_model_loader_gguf expected arch list - #3179
Open
localai-org-maint-bot wants to merge 1 commit into
Open
fix: add DeepseekV41ForCausalLM to test_model_loader_gguf expected arch list#3179localai-org-maint-bot wants to merge 1 commit into
localai-org-maint-bot wants to merge 1 commit into
Conversation
mudler
force-pushed
the
fix/test-loader-gguf-arch-list
branch
from
September 13, 2026 06:48
ea2db7b to
076d58a
Compare
…ch list Commit 4a9d337 registered DeepseekV41ForCausalLM in the model registry but did not update the hardcoded expected architecture list in the CHECK_THROWS_WITH_AS assertion at test_model_loader_gguf.cpp:225. The test expected DeepseekV4ForCausalLM right after DeepseekV2ForCausalLM, but the actual error message now includes DeepseekV41ForCausalLM between them, causing build-test-cpu to fail. Closes #3178 FOLLOWING_AGENTS_PROTOCOL Following-Agents-Protocol: true AI-Assisted: true Assisted-by: AGENT:regolo/glm5.2 [MAKI]
mudler
force-pushed
the
fix/test-loader-gguf-arch-list
branch
from
September 13, 2026 07:31
076d58a to
dbf76b8
Compare
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
2 participants
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.
Commit 4a9d337 registered
DeepseekV41ForCausalLMin the model registry but did not update the hardcoded expected architecture list in theCHECK_THROWS_WITH_ASassertion attest_model_loader_gguf.cpp:225. The test expectedDeepseekV4ForCausalLMright afterDeepseekV2ForCausalLM, but the actual error message now includesDeepseekV41ForCausalLMbetween them, causingbuild-test-cputo fail.Closes #3178
FOLLOWING_AGENTS_PROTOCOL
Following-Agents-Protocol: true
AI-Assisted: true
Assisted-by: AGENT:regolo/glm5.2 [MAKI]