Repository navigation
Upgrade llama.cpp from b11883 to b11887 - #566
Merged
Merged
Conversation
Four upstream commits, tagged. #31035 caps the DRY sequence breakers: llama.h gains LLAMA_DRY_MAX_SEQ_BREAKERS (64), the request schema drops duplicate dry_sequence_breakers and refuses more than 64 distinct ones (each costs a scan of the vocabulary when the sampler is built, so a long list exhausted CPU and memory), the sampler stops at 64 with a warning. Two C++ tests pin the new contract (duplicates dropped in order; 64 distinct plus a duplicate accepted, one more refused), and withDrySequenceBreakers' Javadoc names it. #31028 merges Qwen4-exp tensors on conversion; #30977 and #31157 touch upstream CI and tests only. All twelve patches apply unchanged on 6050b467c (commit count 11887); CMakeLists.txt, common/, the license files, vendor/ and the WebUI lockfile are unchanged. Checked locally: check-patches (12 patches, 94 hunks, 0 problems), check-natives, spotless:check, clang-format 23.1.3 on test_server.cpp; the build and tests are CI's. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01VszvDmRRPP1UcwozsUMY1L
bernardladenthin
had a problem deploying
to
startgate
October 25, 2026 19:44 — with
GitHub Actions
Error
bernardladenthin
had a problem deploying
to
maven-central
October 25, 2026 19:44 — with
GitHub Actions
Failure
bernardladenthin
had a problem deploying
to
maven-central
October 25, 2026 19:44 — with
GitHub Actions
Failure
|
Caution Review failedThe pull request is closed. Finishing Touches
Comment |
|
This branch had an error being deployed
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
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.



Four upstream commits, all in the tagged release b11887.
llama.hgainsLLAMA_DRY_MAX_SEQ_BREAKERS(64). A long breaker list exhausted CPU and memory, because each distinct breaker costs one scan of the vocabulary when the sampler is built. Now:dry_sequence_breakersand refuses more than 64 distinct ones as an invalid request;--dry-sequence-breaker).This project
test_server.cpppin the new contract:InferenceParameters.withDrySequenceBreakersJavadoc names the limit.LlamaCppVersion, README, CLAUDE.md (pin and test counts 671 → 673), CHANGELOG, the breaking-changes history, theNOTICE.mdversion cell.6050b467c(commit count 11887).CMakeLists.txt,common/, the license files,vendor/, the WebUI lockfile.Checked locally:
check-patches.py(0 problems),check-natives.py,spotless:check, clang-format 23.1.3 ontest_server.cpp. Build and tests are left to CI.🤖 Generated with Claude Code
https://claude.ai/code/session_01VszvDmRRPP1UcwozsUMY1L