Repository navigation
fix: qualify vLLM 0.31 B200 attention collectors - #587
Closed
simone-chen wants to merge 13 commits into
Closed
simone-chen wants to merge 13 commits into
simone-chen wants to merge 13 commits into
Conversation
Signed-off-by: Simone Chen <simonec@nvidia.com>
Signed-off-by: Simone Chen <simonec@nvidia.com>
|
Important Draft PR not reviewedDraft PRs are not automatically reviewed by default.
To automatically review draft PRs, update your CodeRabbit configuration: reviews:
auto_review:
drafts: true
Comment |
Contributor
Author
|
/ok to test 079052e |
Signed-off-by: Simone Chen <simonec@nvidia.com>
Contributor
Author
|
/ok to test bb77a29 |
1 similar comment
Contributor
Author
|
/ok to test bb77a29 |
Signed-off-by: Simone Chen <simonec@nvidia.com>
Contributor
Author
|
/ok to test 389708b |
Signed-off-by: Simone Chen <simonec@nvidia.com>
Contributor
Author
|
/ok to test 42ee6e0 |
Signed-off-by: Simone Chen <simonec@nvidia.com>
Contributor
Author
|
/ok to test 5852b28 |
Signed-off-by: Simone Chen <simonec@nvidia.com>
Contributor
Author
|
/ok to test 11949f7 |
Signed-off-by: Simone Chen <simonec@nvidia.com>
Contributor
Author
|
/ok to test b1c756e |
Signed-off-by: Simone Chen <simonec@nvidia.com>
Contributor
Author
|
/ok to test 13be217 |
Signed-off-by: Simone Chen <simonec@nvidia.com>
Signed-off-by: Simone Chen <simonec@nvidia.com>
Contributor
Author
|
/ok to test 3921c9c |
Signed-off-by: Simone Chen <simonec@nvidia.com>
Contributor
Author
|
/ok to test 364330c |
Signed-off-by: Simone Chen <simonec@nvidia.com>
Contributor
Author
|
/ok to test 1ee0d02 |
Contributor
Author
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.
Changes
AttentionGroupfactory, including the indexer block-table width. Support the native FlashInfer, FlashAttention, and Triton dense-attention routes.ModelOptLinearMethodand native quantizer. Match the exact weight/activation keys andctx.group_size; retain the older method interface without including W4A16 or other formats.Validation
1ee0d025d17d5a1b77f120e1bf65ff986671363b; GitHub verifies its GPG signature and DCO sign-off. No new head has been pushed since that run.9b4de9dbc3fa721af0a3965d16f04d3c5dce89bf, all 14 bounded BF16/tensor-FP8 fused A fixtures pass native eager and CUDA graph checks across DeepSeek-V3, DeepSeek-V3.2, and GLM-5.2. These checks cover the fused projection and FP8 shard-scale contracts only.kv_bconversion, and remaining model-specific attention paths retain their separate GPU qualification gates. Successful numerical probes do not establish timing reuse, complete collection coverage, or end-to-end prediction accuracy.Dependencies and scope