-
Notifications
You must be signed in to change notification settings - Fork 22.5k
spec : add DFlash2 support (local convolution + candidate selector) #27342
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from all commits
5ecbe1a
1deefcc
c314bb1
b8a4a85
2474373
bfaa125
cf095c8
7ea40ee
c184d9a
d1a522f
e91bca5
64f765f
f5a7ec1
f7aadef
11f45ed
2f3923b
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
|
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I could not repro consistent perf improvements on my E2E runs, using speed-bench as the target (cells show avg pred_t/s)
DetailsServer command (same for all runs, ./build_topk_changes/bin/llama-server \
-m /mnt/share/gguf/unsloth/Qwen3.8-27B-GGUF/Qwen3.8-27B-UD-Q4_K_M.gguf \
--spec-type draft-dflash \
-md /mnt/share/gguf/incoai/Qwen3.8-27B-DFlash2-GGUF/Qwen3.8-27B-DFlash2-Q4_K_M.gguf \
--spec-draft-n-max 7 \
-bs \
--load-mode dio \
-np 1|4
Bench command (same for all runs): python tools/server/bench/speed-bench/speed_bench.py \
--url localhost:8080 --bench qualitative --osl 256 --concurrency 1|4 --limit 888 samples total, 8 per category. OFF/ON = topk fusion change disabled/enabled in the server build. DGX SparkCells: avg pred_t/s / avg latency.
B4500Cells: avg pred_t/s / avg latency.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
normally we would require CUDA changes to be in a dedicated PR, unless strictly necessary. but just asking if @ruixiang63 @am17an are ok or you want CUDA in a dedicated one?
also, please tell your agents to clean up code comments per agents.md requirements
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The comments are cleaned up in d1a522f.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it makes sense to include the CUDA kernel changes in this PR, as they are a key selling point of DFlash2. We should also have the CUDA maintainers review the PR.