Fix Gemma 4 MTP on llama-server (Windows) - #24480
Conversation
|
This model is broken on linux, too. But this patch doesn't seem to fix the issue for me: (stacktrace is from the head commit in this PR) |
|
I don't understand what this is supposed to fix? MTP models do not need to set |
|
-> To be tested under Windows |
|
weird, earlier today was trying gemma-4-31B-it-qat-GGUF (noting - Jun 9 Update: Added MTP support) thinking the issue was the only error llama-server.exe displays (a red herring) i've re-tested from scratch and MTP strangely seems to be working now via upstream unsloth\llama.cpp thanks for testing this on Linux ^ i think this clarifies the change isn't needed and safe to close (i'm unable to repro the original issue, which seems to be the model fails to load at all) |
|
i still get a segfault with this unsloth model. i'll open a new issue |
Overview
Minimally fix Gemma 4 MTP models broken on llama-server.exe (Windows)
Fix: #24443
Additional information
Tested on
NB:
E llama_init_from_model: failed to initialize the context: Gemma4Assistant requires ctx_other to be set (this is normal during memory fitting)is still shown, which can be addressed separatelyNB: I considered addressing a related TODO but this can be addressed separately also
NB: I've not tested other gemma-4 models such as older non-qat ones
Requirements