mirror of
https://github.com/kvcache-ai/ktransformers.git
synced 2026-11-05 01:07:28 +00:00
[GH-ISSUE #2007] Speculative/EAGLE - Supported? #1233
Labels
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
kvcache-ai/ktransformers#1233
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @usrlocalben on GitHub (May 16, 2026).
Original GitHub issue: https://github.com/kvcache-ai/ktransformers/issues/2007
Reminder
System Info
ktransformers @
bb15fdfkvcache-ai/sglang @ 2763727
Reproduction
Others
I'd like to use/try EAGLE3 w/Kimi, e.g. the EAGLE3 model provided by AQ-MedAI.
Their model-card indicates it should work with sglang 0.5.10 (maybe others, it doesn't say it's a minimum nor maximum)
Expected: Works.
Observed: Does not work. (See trace above)
I tried transformers 5.8.0, 5.7.0, and 5.6.2, although the problem appears to be in the model implementation in sglang/src/models/kimi_k25 .
Additionally, since kt vendors the sglang module, I can't file any report there.
Additionally2, there is little clarity wrt. differences between mainline sglang and kt sglang. There are pulls/commits that "merge" from mainline but it's still vague in terms of what to expect.
@alin899992 commented on GitHub (May 16, 2026):
Following sglang upstream PR and modifying the file
~/.venv/lib/python3.12/site-packages/sglang/srt/models/kimi_k25.py.Then EAGLE3 should work.
@usrlocalben commented on GitHub (May 16, 2026):
@alin899992 that works if I also add this snippet:
github.com/sgl-project/sglang@35870d55ac (diff-e5e5d2c579)perf is unfortunately less than inspiring. almost 50% of decode throughput without it.
Capturing batches (bs=1 avail_mem=35.10 GB): 100%shouldn't this say bs=3 for draft tokens 3?I now understand what bs=N is for, nevermindedit: I'm not sure if the link above directs one where I intended, but I'm referring to the change in python/sglang/srt/speculative/eagle_draft_cuda_graph_runner.py