Skip to content

[Doc] Adding diffusion model#524

Merged
hsliuustc0106 merged 3 commits intovllm-project:mainfrom
Bounty-hunter:add_diffusion_model
Dec 30, 2025
Merged

[Doc] Adding diffusion model#524
hsliuustc0106 merged 3 commits intovllm-project:mainfrom
Bounty-hunter:add_diffusion_model

Conversation

@Bounty-hunter
Copy link
Contributor

PLEASE FILL IN THE PR DESCRIPTION HERE ENSURING ALL CHECKLIST ITEMS (AT THE BOTTOM) HAVE BEEN CONSIDERED.

Purpose

How to add a diffusion model.

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.

BEFORE SUBMITTING, PLEASE READ https://github.com/vllm-project/vllm-omni/blob/main/CONTRIBUTING.md (anything written below this line will be removed by GitHub Actions)

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".

@hsliuustc0106
Copy link
Collaborator

paste the local test result please

@Bounty-hunter Bounty-hunter force-pushed the add_diffusion_model branch 3 times, most recently from 9fba58f to ae8d1a9 Compare December 30, 2025 01:35
Signed-off-by: dengyunyang <[email protected]>
@Bounty-hunter
Copy link
Contributor Author

paste the local test result please
This workflow has already been implemented across multiple PRs:
https://github.com/vllm-project/vllm-omni/pull/330/changes
#381

@ZJY0516 ZJY0516 requested a review from SamitHuang December 30, 2025 03:52
@hsliuustc0106 hsliuustc0106 added the ready label to trigger buildkite CI label Dec 30, 2025
@hsliuustc0106
Copy link
Collaborator

paste the local test result please
This workflow has already been implemented across multiple PRs:
https://github.com/vllm-project/vllm-omni/pull/330/changes
#381

actually, I mean the docs screenshot

@hsliuustc0106 hsliuustc0106 enabled auto-merge (squash) December 30, 2025 04:33
@hsliuustc0106 hsliuustc0106 merged commit 099a36d into vllm-project:main Dec 30, 2025
5 checks passed
tzhouam pushed a commit to tzhouam/vllm-omni that referenced this pull request Dec 30, 2025
Signed-off-by: dengyunyang <[email protected]>
Signed-off-by: Samit <[email protected]>
Co-authored-by: Hongsheng Liu <[email protected]>
Co-authored-by: Samit <[email protected]>
princepride pushed a commit to princepride/vllm-omni that referenced this pull request Jan 10, 2026
Signed-off-by: dengyunyang <[email protected]>
Signed-off-by: Samit <[email protected]>
Co-authored-by: Hongsheng Liu <[email protected]>
Co-authored-by: Samit <[email protected]>
ZJY0516 pushed a commit to LawJarp-A/vllm-omni that referenced this pull request Jan 10, 2026
Signed-off-by: dengyunyang <[email protected]>
Signed-off-by: Samit <[email protected]>
Co-authored-by: Hongsheng Liu <[email protected]>
Co-authored-by: Samit <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready label to trigger buildkite CI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants