Conversation
Contributor
|
Important Draft PR not reviewedDraft PRs are not automatically reviewed by default.
To automatically review draft PRs, update your CodeRabbit configuration: reviews:
auto_review:
drafts: trueThanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Contributor
|
This pull request is currently a draft. Reviews will not take place until the PR is marked as ready for review. |
i386
force-pushed
the
kv-cache-ng-merged
branch
from
September 13, 2026 04:05
67fefcf to
3638854
Compare
i386
force-pushed
the
kv-cache-ng-merged
branch
from
September 13, 2026 20:31
0dd39af to
a85f8c8
Compare
i386
added this pull request to stack #1859
September 13, 2026 23:20
This was referenced Sep 14, 2026
added 4 commits
September 15, 2026 08:43
# Conflicts: # Cargo.lock
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.
What this draft does
This is the single integration branch for the next-generation Skippy KV stack. It combines the durable local L3 substrate, host-RAM L2 serving, benefit-based admission, CacheGen serving selection, remote-handoff groundwork, user/operator controls, and publisher-metadata-driven live KV defaults.
The serving path now provides:
Publisher metadata and live KV defaults
The previous
autoheuristic selected Q8 below 50 GiB of model weights and Q4 at or above 50 GiB. Model weight bytes and GGUF weight quantization do not establish a safe live KV precision, so this branch removes that heuristic.New layer packages copy supported Hugging Face publisher files under
metadata/, bind them to the source repository and immutable revision, hash them inmodel-package.json, and compare overlapping model geometry with authoritative GGUF metadata. The runtime resolves live K/V in this order:BF16 currently maps to F16 until BF16 live KV is qualified across the supported backends. Unsupported FP8 declarations also fall back to F16. Q8/Q4 remain available when explicitly declared and supported, but are never inferred from model size or weight quantization. Existing packages without publisher metadata remain readable and use F16/F16.
The unshipped
auto | quality | balanced | saverKV policy presets have been removed from config, resolution, status, UI, fixtures, and documentation. Operators retain direct K/V dtype and offload controls.Product and release status
automeans automatic sizing after explicit enablement.huggingface.co/meshllmlayer packages need a metadata backfill after this code merges.The former component and wiring PRs have been closed or merged into this branch so review and CI continue on one head. Further KV work lands directly on
kv-cache-ng-mergedrather than forming another dependent PR stack.Validation
Local validation includes:
mesh-llm-config,skippy-cache,mesh-llm-host-runtime, and the UI;GitHub CI is rerunning on the current head. Hardware release gates remain the documented 19K production serving runs and cross-backend traffic matrices. Unqualified optional paths retain native/local/cold fallback.