Skip to content

[Misc] Add removed encoder-decoder models to previously supported models list#24961

Merged
vllm-bot merged 1 commit into
vllm-project:mainfrom
Isotr0py:prev-enc-dec
Sep 16, 2025
Merged

[Misc] Add removed encoder-decoder models to previously supported models list#24961
vllm-bot merged 1 commit into
vllm-project:mainfrom
Isotr0py:prev-enc-dec

Conversation

@Isotr0py
Copy link
Copy Markdown
Member

@Isotr0py Isotr0py commented Sep 16, 2025

Purpose

Test Plan

Test Result


Essential Elements of an Effective PR Description Checklist
  • The purpose of the PR, such as "Fix some issue (link existing issues this PR will resolve)".
  • The test plan, such as providing test command.
  • The test results, such as pasting the results comparison before and after, or e2e results
  • (Optional) The necessary documentation update, such as updating supported_models.md and examples for a new model.
  • (Optional) Release notes update. If your change is user facing, please update the release notes draft in the Google Doc.

Signed-off-by: Isotr0py <mozf@mail2.sysu.edu.cn>
@mergify mergify Bot added the new-model Requests to new models label Sep 16, 2025
Copy link
Copy Markdown
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 several recently removed encoder-decoder models to the _PREVIOUSLY_SUPPORTED_MODELS list. This is a good change as it will provide users with a clear and helpful deprecation message if they attempt to use these models, improving the overall user experience. The implementation is straightforward and correct. I have no major concerns with this change.

Copy link
Copy Markdown
Member

@DarkLight1337 DarkLight1337 left a comment

Choose a reason for hiding this comment

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

Thanks!

@DarkLight1337 DarkLight1337 enabled auto-merge (squash) September 16, 2025 13:30
@github-actions github-actions Bot added the ready ONLY add when PR is ready to merge/full CI is needed label Sep 16, 2025
@vllm-bot vllm-bot merged commit d593cf2 into vllm-project:main Sep 16, 2025
59 of 63 checks passed
@Isotr0py Isotr0py deleted the prev-enc-dec branch September 16, 2025 17:50
@vrdn-23
Copy link
Copy Markdown
Contributor

vrdn-23 commented Sep 16, 2025

@Isotr0py @DarkLight1337 Does this mean there will be a temporary period of not supporting these models or are these encoder-decoder models not going to supported going forward in vLLM?

Apologies if I missed this, but was there an online discussion that kind of highlighted the reasons for the deprecation of these models?

@DarkLight1337
Copy link
Copy Markdown
Member

We have decided to drop all encoder-decoder models except for Whisper to avoid code complexity in the model runner. #23300 (comment)

ABC12345anouys pushed a commit to ABC12345anouys/vllm that referenced this pull request Sep 25, 2025
…els list (vllm-project#24961)

Signed-off-by: Isotr0py <mozf@mail2.sysu.edu.cn>
choprahetarth pushed a commit to Tandemn-Labs/vllm that referenced this pull request Oct 11, 2025
…els list (vllm-project#24961)

Signed-off-by: Isotr0py <mozf@mail2.sysu.edu.cn>
mystous pushed a commit to mystous/vllm_hybrid that referenced this pull request May 10, 2026
…els list (vllm-project#24961)

Signed-off-by: Isotr0py <mozf@mail2.sysu.edu.cn>
my-other-github-account pushed a commit to my-other-github-account/vllm that referenced this pull request May 15, 2026
…els list (vllm-project#24961)

Signed-off-by: Isotr0py <mozf@mail2.sysu.edu.cn>
my-other-github-account pushed a commit to my-other-github-account/vllm that referenced this pull request May 15, 2026
…els list (vllm-project#24961)

Signed-off-by: Isotr0py <mozf@mail2.sysu.edu.cn>
0826joyce pushed a commit to 0826joyce/vllm-serving-optimization that referenced this pull request May 19, 2026
…els list (vllm-project#24961)

Signed-off-by: Isotr0py <mozf@mail2.sysu.edu.cn>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

new-model Requests to new models ready ONLY add when PR is ready to merge/full CI is needed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants