Skip to content

[Misc] Clean up model registry#37457

Merged
DarkLight1337 merged 2 commits intovllm-project:mainfrom
DarkLight1337:improve-registry
Mar 18, 2026
Merged

[Misc] Clean up model registry#37457
DarkLight1337 merged 2 commits intovllm-project:mainfrom
DarkLight1337:improve-registry

Conversation

@DarkLight1337
Copy link
Copy Markdown
Member

@DarkLight1337 DarkLight1337 commented Mar 18, 2026

Purpose

  • Remove unnecessary noqas and put items on one line where possible.
  • Updated alphabetical order.
  • Introduce _OOT_SUPPORTED_MODELS so that we can redirect previous users of BART and Florence-2 to the OOT plugin.

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: DarkLight1337 <tlleungac@connect.ust.hk>
@DarkLight1337 DarkLight1337 added the ready ONLY add when PR is ready to merge/full CI is needed label Mar 18, 2026
@mergify mergify bot added the new-model Requests to new models label Mar 18, 2026
Signed-off-by: DarkLight1337 <tlleungac@connect.ust.hk>
@DarkLight1337 DarkLight1337 enabled auto-merge (squash) March 18, 2026 16:29
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 provides a good cleanup of the model registry by removing unnecessary noqa comments, reformatting code for better readability, and reordering model entries alphabetically. The introduction of _OOT_SUPPORTED_MODELS is a valuable addition, as it improves user experience by providing clear guidance for models that have been moved to out-of-tree plugins. The implementation is straightforward and correct. I have not found any issues of high or critical severity.

}

_OOT_SUPPORTED_MODELS = {
"BartModel": "https://github.com/vllm-project/bart-plugin",
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

cc @NickLucche can you update the plugin to map BartModel and MBartForConditionalGeneration as well?

@DarkLight1337 DarkLight1337 merged commit f3732bd into vllm-project:main Mar 18, 2026
55 checks passed
@DarkLight1337 DarkLight1337 deleted the improve-registry branch March 18, 2026 18:24
fxdawnn pushed a commit to fxdawnn/vllm that referenced this pull request Mar 19, 2026
Signed-off-by: DarkLight1337 <tlleungac@connect.ust.hk>
SouthWest7 pushed a commit to SouthWest7/vllm that referenced this pull request Mar 27, 2026
Signed-off-by: DarkLight1337 <tlleungac@connect.ust.hk>
khairulkabir1661 pushed a commit to khairulkabir1661/vllm that referenced this pull request Mar 27, 2026
Signed-off-by: DarkLight1337 <tlleungac@connect.ust.hk>
Monishver11 pushed a commit to Monishver11/vllm that referenced this pull request Mar 27, 2026
Signed-off-by: DarkLight1337 <tlleungac@connect.ust.hk>
Signed-off-by: Monishver Chandrasekaran <monishverchandrasekaran@gmail.com>
JiantaoXu pushed a commit to JiantaoXu/vllm that referenced this pull request Mar 28, 2026
Signed-off-by: DarkLight1337 <tlleungac@connect.ust.hk>
vrdn-23 pushed a commit to vrdn-23/vllm that referenced this pull request Mar 30, 2026
Signed-off-by: DarkLight1337 <tlleungac@connect.ust.hk>
Signed-off-by: Vinay Damodaran <vrdn@hey.com>
EricccYang pushed a commit to EricccYang/vllm that referenced this pull request Apr 1, 2026
Signed-off-by: DarkLight1337 <tlleungac@connect.ust.hk>
Signed-off-by: EricccYang <yangyang4991@gmail.com>
liuchenbing2026 pushed a commit to liuchenbing2026/vllm that referenced this pull request Apr 4, 2026
Signed-off-by: DarkLight1337 <tlleungac@connect.ust.hk>
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.

2 participants