Repository navigation
[NVIDIA] Support NVFP4 MoE for DeepSeek-V4 - #25820
Merged
Merged
Conversation
trevor-m
requested review from
AniZpZ,
BBuf,
Edwardf0t1,
FlamingoPg,
Fridge003,
HaiShaw,
Ying1123,
ch-wan,
ispobock and
merrymercy
as code owners
May 19, 2026 23:09
Contributor
|
Warning You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again! |
b8zhong
reviewed
May 27, 2026
b8zhong
reviewed
May 27, 2026
Edwardf0t1
reviewed
May 27, 2026
Collaborator
Author
|
GPQA Results Server command Eval command using https://github.com/sgl-project/sglang/blob/deepseek_v4/scripts/bench_gpqa_aime25.py |
trevor-m
force-pushed
the
dsv4-fp4
branch
2 times, most recently
from
May 28, 2026 23:34
cf38ed4 to
2e4567e
Compare
Collaborator
|
@trevor-m Can you please fix the conflicts |
trevor-m
force-pushed
the
dsv4-fp4
branch
2 times, most recently
from
May 29, 2026 18:06
2d40eda to
ddc5ea5
Compare
Collaborator
Author
Collaborator
|
All NVIDIA pipelines passed. @Fridge003 could you merge this? thanks |
Fridge003
approved these changes
Jun 22, 2026
Fridge003
disabled auto-merge
June 22, 2026 02:33
Collaborator
|
@trevor-m Can we add a nightly test for dsv4 nvfp4 checkpoint, thanks |
Collaborator
Author
|
@Fridge003 Thanks for help getting this PR merged! I opened #28966 to add the nightly tests |
dougyster
added a commit
to dougyster/sglang
that referenced
this pull request
Jul 21, 2026
Restore the GB200 DeepSeek-V4 cells (were removed). Their numbers are PR sgl-project#25820 Mean-era, while everything re-measured this PR is P50. Add per-cell latencyPercentile support in _deployment.jsx (entry.latencyPercentile falls back to config.latencyPercentile) and tag the two GB200 nvfp4 cells 'Mean' so they render 'TTFT (Mean)' while all other cells stay 'TTFT (P50)'. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Chronostasys
pushed a commit
to MindLab-Research/sglang
that referenced
this pull request
Aug 24, 2026
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.
Motivation
This PR enables NVFP4 quantization for MoE in DeepSeek-V4. NVFP4 should provide better performance compared to MXFP4. Use
--moe-runner-backend flashinfer_trtllm_routed.Needs #25702 to help with perf
Modifications
"moe_quant_algo": "NVFP4".apply_routed_scaling_factor_on_outputfor hash topkswiglu_limitfor flashinfer_trtllm_routed.Accuracy Tests
Speed Tests and Profiling
Checklist
Review and Merge Process
/tag-and-rerun-ci,/tag-run-ci-label,/rerun-failed-ciCI States
Latest PR Test (Base): ✅ Run #27657320129
Latest PR Test (Extra): ✅ Run #27925999159