Skip to content

[Cleanup] Remove duplicate HiDreamI1ImagePipeline (fixes #4009) - #4045

Merged
hsliuustc0106 merged 3 commits into
vllm-project:mainfrom
yangyonggit:cleanup/remove-hidream-i1-duplicate
Jun 18, 2026
Merged

hsliuustc0106 merged 3 commits into
vllm-project:mainfrom
yangyonggit:cleanup/remove-hidream-i1-duplicate

Conversation

@yangyonggit

Copy link
Copy Markdown
Contributor

pipeline_hidream_i1_image.py is an exact duplicate of pipeline_hidream_image.py with only the class name differing. HiDreamI1ImagePipeline has zero external references — registry and init.py both use HiDreamImagePipeline exclusively.

Purpose

pipeline_hidream_i1_image.py is an exact duplicate of pipeline_hidream_image.py with only the class name differing (HiDreamI1ImagePipeline vs HiDreamImagePipeline). HiDreamI1ImagePipeline has zero external references — the registry and init.py both use HiDreamImagePipeline exclusively. Part of codebase cleanup audit #4009.

Test Plan

uvx ruff check vllm_omni/ --select F401,F821,F811

python -c "
import vllm_omni
from vllm_omni.diffusion.models.hidream_image import HiDreamImagePipeline
print('imports OK')
"

Test Result

All checks passed!
imports OK

…#4009)

pipeline_hidream_i1_image.py is an exact duplicate of
pipeline_hidream_image.py with only the class name differing.
HiDreamI1ImagePipeline has zero external references — registry and
__init__.py both use HiDreamImagePipeline exclusively.

Signed-off-by: leo.yang <leo.yang.engineer@gmail.com>
@linyueqian

Copy link
Copy Markdown
Collaborator

resolve conflicts

…eam-i1-duplicate

# Conflicts:
#	vllm_omni/diffusion/models/hidream_image/pipeline_hidream_i1_image.py
@yangyonggit

Copy link
Copy Markdown
Contributor Author

Conflict resolved

@hsliuustc0106
hsliuustc0106 merged commit 37a4424 into vllm-project:main Jun 18, 2026
5 checks passed
khairulkabir1661 pushed a commit to khairulkabir1661/vllm-omni that referenced this pull request Sep 25, 2026
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.

3 participants