From: Georgi Gerganov <redacted>
Date: Mon, 8 Sep 2025 07:26:36 +0000 (+0300)
Subject: context : fix n_outputs during reserve (#15858)
X-Git-Tag: upstream/0.0.6527~116
X-Git-Url: https://git.djapps.eu/?a=commitdiff_plain;h=663027fd5490438ce9c27ea866a560e1e268d11f;p=pkg%2Fggml%2Fsources%2Fllama.cpp

context : fix n_outputs during reserve (#15858)

ggml-ci
---

diff --git a/src/llama-context.cpp b/src/llama-context.cpp
index 6b3188be..874c6f82 100644
--- a/src/llama-context.cpp
+++ b/src/llama-context.cpp
@@ -285,8 +285,8 @@ llama_context::llama_context(
         const uint32_t n_seqs = cparams.kv_unified ? 1 : cparams.n_seq_max;
         const uint32_t n_tokens = std::min(cparams.n_ctx, cparams.n_ubatch);
 
-        // avoid reserving graphs with zero outputs
-        n_outputs = 1;
+        // avoid reserving graphs with zero outputs - assume one output per sequence
+        n_outputs = n_seqs;
 
         LLAMA_LOG_DEBUG("%s: worst-case: n_tokens = %d, n_seqs = %d, n_outputs = %d\n", __func__, n_tokens, n_seqs, n_outputs);
 
diff --git a/src/llama-graph.cpp b/src/llama-graph.cpp
index 4abb6008..7f254b25 100644
--- a/src/llama-graph.cpp
+++ b/src/llama-graph.cpp
@@ -1431,7 +1431,8 @@ ggml_tensor * llm_graph_context::build_attn(
 
     // [TAG_NO_CACHE_PAD]
     // TODO: if ubatch.equal_seqs() == true, we can split the three tensors below into ubatch.n_seqs_unq streams
-    assert(!ubatch.equal_seqs() || (k_cur->ne[3] == 1 && k_cur->ne[3] == ubatch.n_seqs_unq));
+    //       but it might not be worth it: https://github.com/ggml-org/llama.cpp/pull/15636
+    //assert(!ubatch.equal_seqs() || (k_cur->ne[3] == 1 && k_cur->ne[3] == ubatch.n_seqs_unq));
 
     ggml_tensor * q = q_cur;
     ggml_tensor * k = k_cur;