Repository navigation
Conversation
Contributor
|
Thanks @zianglih. Closing this because it is still a draft and has not been updated in 30 days. Reopen it if the work is still relevant. Some directories moved recently, so an older branch may need retargeting: |
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
@HumansAnd
Cover CuTe DSL NVFP4 W4A16 weight reloads used by disjoint Miles training/rollout workers. Raw transfers must update the buffers captured by CUDA graphs and refresh derived scale tensors.
Modifications
FusedMoENVFP4 backend test and quantization helpers. Adjust its CI time estimate for the added test.sglang-milesdependency.Accuracy Tests
590.48.01; imageradixark/miles:dev-202609111228, amd64 manifestsha256:f774dee95a80a213c30489cf82b13295d8a50338ea39f5aaadb0377d3e4a9bba.2.13.0+cu130/ CUDA13.0, FlashInfer0.6.18, CuTe DSL4.6.2, TE2.17.0. SGLang source is this PR on upstreammain, selected throughPYTHONPATH, rather than the image'ssglang-milessource.rtol=atol=0against a fresh load of the same packed weights).5c499272f91d08cc5837f770d1f8867b7661bce2.From this checkout in the image:
CUDA_VISIBLE_DEVICES=1 PYTHONPATH="$PWD/python" \ python test/registered/unit/layers/quantization/test_nvfp4_moe_backends.py \ TestNvFp4MoeBackends.test_flashinfer_cutedsl_w4a16_weight_reloadNegative control: on the same test body, replace
refresh_cutedsl_standard_scales_for_weight_updatewith a no-op usingunittest.mock.patch. The B update must fail against its fresh-load control. Complete output:Speed Tests and Profiling
No inference implementation change or speedup claim. Only test execution time was measured.
Checklist
Full GPU CI matrix remains for maintainer CI.
Review and Merge Process
Normal SGLang review and CI; this PR does not merge or duplicate #35312.