mirror of
https://github.com/kvcache-ai/Mooncake.git
synced 2026-11-05 01:17:55 +00:00
[GH-ISSUE #292] 1p1d, tcp, assert start_pos == num_prefill_tokens AssertionError() Please Help Me ! #115
Labels
No labels
ascend
auto-closed
bug
bug
CI/Build
Common
documentation
enhancement
feature request
good first issue
help wanted
high priority
MetaX
nightly-failure
pull-request
PyTorch Backend
question
RFC
Roadmap
stale
Store
Tests
Transfer Engine
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
kvcache-ai/Mooncake#115
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 @coding-rw on GitHub (Apr 23, 2025).
Original GitHub issue: https://github.com/kvcache-ai/Mooncake/issues/292
1 prefill node(4L20), 1 decoding node(4L20), qwen2.5-32B-instruct
Bug on Decode Node
On Prefill Node
test shell
#prefill node
mooncake.json
#decode node
mooncake.json
@ShangmingCai commented on GitHub (Apr 23, 2025):
Just follow the guide from the log:
Chunked prefill is not fully compatible with disaggregated serving yet.
@coding-rw commented on GitHub (Apr 23, 2025):
I have used --no-enable-chunked-prefill --max-num-batched-tokens 8192 --max-seq-len-to-capture 8192, is this different from what you mentioned? --enable_chunked_prefill=False is not a legal parameter.
@ShangmingCai commented on GitHub (Apr 23, 2025):
@coding-rw Try
--enable-chunked-prefill=Falsefirst. If it doesn't work, then just remove--enable-chunked-prefill.@coding-rw commented on GitHub (Apr 24, 2025):
When I use
I got
The reason is the latest version of vllm (https://docs.vllm.ai/en/latest/serving/engine_args.html), which uses
--enable-chunked-prefill/--no-enable-chunked-prefillinstead of--enable-chunked-prefill=False.So, I reverted to using vllm v0.8.4, which supports
--enable-chunked-prefill=False, but the same error still occurred:assert start_pos == num_prefill_tokens.After testing, I found that this issue is caused by
--enable-prefix-caching. When not using--enable-prefix-caching, the errorassert start_pos == num_prefill_tokensdoes not occur. I haven't analyzed the root cause yet. Has anyone encountered a similar issue? Any ideas or solutions?@ShangmingCai commented on GitHub (Apr 24, 2025):
@coding-rw Prefix caching includes immutable blocks and mutable blocks, maybe there are some potential bugs in this fix: https://github.com/vllm-project/vllm/pull/13987, you can reach out to the author of this PR.
@coding-rw commented on GitHub (Apr 24, 2025):
Thanks a lot!
@izhuhaoran commented on GitHub (Sep 3, 2025):
@coding-rw I have the same bug about this assert, do you have any fix about this? or just no-enable-prefix-caching?
@github-actions[bot] commented on GitHub (Jul 14, 2026):
This issue has had no activity for 90 days and will be closed in 7 days if there is no further activity. Please comment or react if it should stay open.
@github-actions[bot] commented on GitHub (Jul 21, 2026):
Closing due to 3 months of inactivity. If this is still relevant, please comment and we can reopen.