Skip to content

[Spec V2] Migrate V2 path to EagleDraftExtendInput; verify returns EagleVerifyOutput#24860

Open
hnyls2002 wants to merge 21 commits into
mainfrom
lsyin/spec-pr2
Open

[Spec V2] Migrate V2 path to EagleDraftExtendInput; verify returns EagleVerifyOutput#24860
hnyls2002 wants to merge 21 commits into
mainfrom
lsyin/spec-pr2

Conversation

@hnyls2002
Copy link
Copy Markdown
Collaborator

Summary

Stacks on #24859. Brings V2 (overlap) path to the same schema:

  • V2 _draft_extend_for_decode and _draft_extend_for_prefill install EagleDraftExtendInput as batch.spec_info for the draft-extend forward (was: reusing EagleDraftInput across phases).
  • V2 EagleVerifyInput.sample returns (EagleVerifyOutput, predict) — the same EagleVerifyOutput as V1, threading draft_extend_input via verify_output instead of dynamic attribute hacks.
  • EagleVerifyOutput folds can_run_cuda_graph so verify returns a single object.
  • Schema fields grouped under explicit # === common / V2 / V1 === section markers.

After this PR, V1 and V2 paths share the spec_info type per phase.

Test plan

  • V2 overlap suites (stage-b/c with overlap-mode EAGLE)
  • DSv4 + EAGLE V2 tests

@gemini-code-assist
Copy link
Copy Markdown
Contributor

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

Base automatically changed from lsyin/spec-pr1 to main May 10, 2026 08:07
@hnyls2002 hnyls2002 requested a review from Fridge003 as a code owner May 10, 2026 08:07
@hnyls2002 hnyls2002 requested a review from ispobock as a code owner May 10, 2026 08:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant