Skip to content
GitLab
Menu
Projects
Groups
Snippets
Loading...
Help
Help
Support
Community forum
Keyboard shortcuts
?
Submit feedback
Contribute to GitLab
Sign in
Toggle navigation
Menu
Open sidebar
OpenDAS
vllm_cscc
Commits
7d654635
Unverified
Commit
7d654635
authored
Mar 31, 2026
by
Matthew Bonanni
Committed by
GitHub
Mar 31, 2026
Browse files
[WIP][CI][Bugfix] Fix `test_run_eagle_dp` (#38584)
Signed-off-by:
Matthew Bonanni
<
mbonanni@redhat.com
>
parent
8278825b
Changes
2
Show whitespace changes
Inline
Side-by-side
Showing
2 changed files
with
6 additions
and
5 deletions
+6
-5
tests/v1/distributed/test_eagle_dp.py
tests/v1/distributed/test_eagle_dp.py
+1
-3
vllm/v1/attention/backends/flash_attn.py
vllm/v1/attention/backends/flash_attn.py
+5
-2
No files found.
tests/v1/distributed/test_eagle_dp.py
View file @
7d654635
...
...
@@ -69,9 +69,7 @@ async def test_run_eagle_dp(monkeypatch: pytest.MonkeyPatch, attn_backend: str):
)
prompt
=
"This is a test of data parallel with eagle"
# This test might be flaky, see
# https://github.com/vllm-project/vllm/issues/31913
num_expected_tokens
=
20
num_expected_tokens
=
100
sampling_params
=
SamplingParams
(
max_tokens
=
num_expected_tokens
,
ignore_eos
=
True
,
...
...
vllm/v1/attention/backends/flash_attn.py
View file @
7d654635
...
...
@@ -389,8 +389,11 @@ class FlashAttentionMetadataBuilder(AttentionMetadataBuilder[FlashAttentionMetad
slot_mapping
=
common_attn_metadata
.
slot_mapping
causal
=
common_attn_metadata
.
causal
# the overhead of the aot schedule is not worth it for spec-decode
aot_schedule
=
self
.
aot_schedule
and
not
fast_build
# Disable AOT schedule for spec-decode proposer (not worth the overhead)
# and for batch invariance (schedule varies with max_seqlen_q/k).
aot_schedule
=
(
self
.
aot_schedule
and
not
fast_build
and
not
envs
.
VLLM_BATCH_INVARIANT
)
if
self
.
aot_sliding_window
is
None
:
self
.
aot_sliding_window
=
(
-
1
,
-
1
)
...
...
Write
Preview
Markdown
is supported
0%
Try again
or
attach a new file
.
Attach a file
Cancel
You are about to add
0
people
to the discussion. Proceed with caution.
Finish editing this message first!
Cancel
Please
register
or
sign in
to comment