Skip to content

Add Eagle and Eagle3 support to Transformers modeling backend#30340

Merged
hmellor merged 10 commits intovllm-project:mainfrom
hmellor:transformers-eagle3
Dec 11, 2025
Merged

Add Eagle and Eagle3 support to Transformers modeling backend#30340
hmellor merged 10 commits intovllm-project:mainfrom
hmellor:transformers-eagle3

Conversation

@hmellor
Copy link
Member

@hmellor hmellor commented Dec 9, 2025

Adds Eagle and Eagle3 support to all Transformers modeling backend classes by adding suport to Base.

Depends on:

Signed-off-by: Harry Mellor <19981378+hmellor@users.noreply.github.com>
Copy link
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request adds support for Eagle and Eagle3 speculative decoding to the Transformers modeling backend. The changes include implementing the SupportsEagle and SupportsEagle3 interfaces in the Base model class. The forward method is updated to optionally return auxiliary hidden states required by Eagle3. I've found a critical issue where an attribute is accessed without being initialized, which could lead to a crash. My review includes a suggestion to fix this.

Signed-off-by: Harry Mellor <19981378+hmellor@users.noreply.github.com>
Signed-off-by: Harry Mellor <19981378+hmellor@users.noreply.github.com>
Signed-off-by: Harry Mellor <19981378+hmellor@users.noreply.github.com>
Signed-off-by: Harry Mellor <19981378+hmellor@users.noreply.github.com>
Signed-off-by: Harry Mellor <19981378+hmellor@users.noreply.github.com>
Signed-off-by: Harry Mellor <19981378+hmellor@users.noreply.github.com>
@mergify mergify bot added the v1 label Dec 11, 2025
@hmellor hmellor marked this pull request as ready for review December 11, 2025 12:54
Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Signed-off-by: Harry Mellor <19981378+hmellor@users.noreply.github.com>
@hmellor hmellor enabled auto-merge (squash) December 11, 2025 14:10
@github-actions github-actions bot added the ready ONLY add when PR is ready to merge/full CI is needed label Dec 11, 2025
@hmellor hmellor merged commit 8781cd6 into vllm-project:main Dec 11, 2025
54 checks passed
@hmellor hmellor deleted the transformers-eagle3 branch December 11, 2025 17:02
@github-project-automation github-project-automation bot moved this from Todo to Done in Transformers backend Dec 11, 2025
Majid-Taheri pushed a commit to Majid-Taheri/vllm that referenced this pull request Dec 23, 2025
…roject#30340)

Signed-off-by: Harry Mellor <19981378+hmellor@users.noreply.github.com>
Signed-off-by: Ubuntu <mjtaheri68@gmail.com>
dsuhinin pushed a commit to dsuhinin/vllm that referenced this pull request Jan 21, 2026
…roject#30340)

Signed-off-by: Harry Mellor <19981378+hmellor@users.noreply.github.com>
Signed-off-by: dsuhinin <suhinin.dmitriy@gmail.com>
@fynnsu fynnsu mentioned this pull request Feb 10, 2026
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready ONLY add when PR is ready to merge/full CI is needed v1

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants