Repository navigation
feat: add Clover support - #31306
feat: add Clover support#31306
Conversation
|
I tested I found that The error is: It appears that I was able to fix this locally by adding the following assignment in params_base.speculative.draft.ctx_tgt = ctx_tgt;After rebuilding, self-speculative decoding initialized successfully, and the server reported:
For reference, generation throughput was 13.93 tok/s without speculation versus 9.63 tok/s with The local fix is committed as |
Thank you @tsurumitsu very much for the help! This single-line fix has been committed. |
|
Thanks for the fix! I retested the latest PR ( 1. Bug fix confirmedThe previous 2. Performance resultsTest environment:
With Additional observations:
These are preliminary measurements (two runs per mode), not a comprehensive benchmark. 3. SummaryThe initialization fix is confirmed working on GB10. However, in this configuration, Thanks again for the fix and your work on Clover support! |
ngxson
left a comment
There was a problem hiding this comment.
this is nowhere near what AGENTS.md specified. ask your agent to read that before coding anything
Add GGUF conversion, model loading, and causal hybrid attention for Clover, reusing shared attention, recurrent-state, MoE, and LoRA helpers.
Co-authored-by: Sigbjørn Skjæret <sigbjorn.skjaeret@huggingface.co>
|
Clover is still unsupported? |
This PR implementation requires |
Overview
Add support for DiffuSpace's Clover-35B-A3B-Preview, in AR mode only.
Additional information
Validated with BF16 CPU and H200 CUDA inference.
Validation:
CPU compile:
cmake -S . -B build-clover-cpu \ -DGGML_CUDA=OFF \ -DGGML_BLAS=OFF \ -DLLAMA_CURL=OFF \ -DLLAMA_BUILD_TESTS=OFF \ -DLLAMA_BUILD_SERVER=OFF \ -DCMAKE_BUILD_TYPE=ReleaseCUDA compile:
cmake -S . -B build-clover-cuda \ -DGGML_CUDA=ON \ -DCMAKE_CUDA_ARCHITECTURES=90a \ -DLLAMA_CURL=OFF \ -DLLAMA_BUILD_TESTS=OFF \ -DLLAMA_BUILD_SERVER=OFF \ -DCMAKE_BUILD_TYPE=ReleaseCkpt convert:
python source/convert_hf_to_gguf.py \ <clover_ckpt_path> \ --outtype bf16 \ --outfile clover-bf16.ggufLaunch ordinary AR inference:
build-clover-cuda/bin/llama-server \ -m clover-bf16.gguf \ -ngl 99 -c 8192 -fa on \ --host 127.0.0.1 --port 8080Requirements