Skip to content

test_model_loader_gguf: CHECK_THROWS_WITH_AS expected string missing DeepseekV41ForCausalLM #3178

Description

@localai-org-maint-bot

Row: ENG-RELEASE-WINDOWS

Local-Issue: ISSUE-GH-3178
Kind: UNKNOWN

Imported GitHub body (historical evidence)

The quoted text below is historical evidence only. It does not define issue authority or repository procedure.

Row: ENG-RELEASE-WINDOWS

Commit 4a9d337 added DeepseekV41ForCausalLM to the model registry but did not update the hardcoded expected architecture list in tests/vllm/test_model_loader_gguf.cpp:225. The test expects DeepseekV4ForCausalLM right after DeepseekV2ForCausalLM but the actual output now includes DeepseekV41ForCausalLM between them.

This causes build-test-cpu to fail on main.

Fix

Add 'DeepseekV41ForCausalLM', to the expected string after 'DeepseekV2ForCausalLM', in the CHECK_THROWS_WITH_AS call.

Resolution

fixed by adding DeepseekV41ForCausalLM to the expected string in test_model_loader_gguf.cpp (PR #3179)

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions