Skip to content

[bugfix][hunyuaniamge] Fix parameter issue introduced during PR #3107 rebase#3395

Merged
gcanlin merged 2 commits into
vllm-project:mainfrom
Bounty-hunter:main_5_7
May 7, 2026
Merged

[bugfix][hunyuaniamge] Fix parameter issue introduced during PR #3107 rebase#3395
gcanlin merged 2 commits into
vllm-project:mainfrom
Bounty-hunter:main_5_7

Conversation

@Bounty-hunter
Copy link
Copy Markdown
Contributor

@Bounty-hunter Bounty-hunter commented May 7, 2026

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

Purpose

An parameter issue caused by pr #3107 when it rebase the main.
image

Test Plan

end2end test:

python3 ../vllm-omni/examples/offline_inference/hunyuan_image3/end2end.py \
--model tencent/HunyuanImage-3.0-Instruct \
--modality img2img \
--stage-configs-path ./test_hunyuan.yaml \
--image-path ./input_0_0.png \
--prompts "新年宠物海报,Q版圆润的可爱标题“新年快乐汪”,副标题“HAPPY NEW YEAR”。 鱼眼镜头,背景是房间门口,近景,上传的主体歪头笑,围着红色围巾,戴着红色毛线帽,高清,绒毛细节,面部特写。 宝丽莱相纸,超 surrealism,写实主义,胶片摄影,打印颗粒感肌理。肌理,超写实,复古感。" \
--seed 42 \
--steps 50 \
--guidance-scale 5.0 \
--output ./results \
> ./run.log 2>&1 &

Test Result

cot:

用户希望将一张可爱的金毛幼犬照片改造成一张具有复古胶片感的新年宠物海报。这张参考图展示了一只坐在木质地板上、吐着舌头微笑的小狗,背景是户外的蒲公英。原始指令非常具体,要求添加特定的标题文字、改变背景、调整构图以及应用特定的艺术风格。首先,我需要分析如何将“新年快乐汪”和“HAPPY NEW YEAR”这两个标题融入画面。为了符合海报的视觉冲击力,标题应该采用圆润、可爱的字体,并放置在画面上方。其次,背景需要从户外的自然环境切换到室内的门口,这涉及到场景的完全重构。为了增强“新年”的氛围,门口可以装饰一些红色的春联或挂饰。构图上,指令提到了鱼眼镜头和近景特写,这意味着小狗在画面中的比例会变大,且边缘会有轻微的畸变效果。小狗本身的配饰也需要改变,从原来的粉色项圈换成更具节日气息的红色围巾和红色毛线帽。最后,整体风格要模拟宝丽莱相纸的质感,带有胶片颗粒和复古的色调。在改写指令时,我会把这些零散的要求整合为一个逻辑连贯的描述,详细说明每一个视觉元素的变化,确保生成的图像既保留了小狗可爱的神态,又具备浓厚的新年节日氛围和复古摄影的艺术感。</think><recaption>将参考图中的金毛幼犬改造成一张复古胶片风格的新年宠物海报。首先,将背景从户外的木质地板和蒲公英更换为室内的门口场景,门口两侧可以贴上红色的春联或挂饰,营造浓厚的新年氛围。对画面进行近景特写处理,并应用鱼眼镜头效果,使画面中心的小狗显得更加圆润可爱。给小狗戴上一顶红色的针织毛线帽,并在脖子上围一条厚实的红色围巾,同时保留它吐着舌头微笑的憨态。在画面的最上方,添加圆润可爱的艺术字体标题“新年快乐汪”,并在其下方配以较小的英文副标题“HAPPY NEW YEAR”。整张图片要呈现出宝丽莱相纸的质感,带有明显的胶片颗粒感和复古的暖色调,边缘可以有轻微的暗角效果,使画面看起来像是一张冲洗出来的老照片。</recaption>

image:
output_0_0


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. Please provide the test scripts & test commands. Please state the reasons if your codes don't require additional test scripts. For test file guidelines, please check the test style doc
  • The test results. Please paste the results comparison before and after, or the e2e results.
  • (Optional) The necessary documentation update, such as updating supported_models.md and examples for a new model. Please run mkdocs serve to sync the documentation editions to ./docs.
  • (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)

Signed-off-by: dengyunyang <584797741@qq.com>
@chatgpt-codex-connector
Copy link
Copy Markdown

Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits.
Credits must be used to enable repository wide code reviews.

@Bounty-hunter
Copy link
Copy Markdown
Contributor Author

@TaffyOfficial PTAL

@Bounty-hunter Bounty-hunter changed the title [bugfix][hunyuaniamge] Resolve parameter issue caused by [pr 3107] rebase conflicts [bugfix][hunyuaniamge] Resolve parameter issue caused by [pr 3107] rebase May 7, 2026
@Bounty-hunter Bounty-hunter changed the title [bugfix][hunyuaniamge] Resolve parameter issue caused by [pr 3107] rebase [bugfix][hunyuaniamge] Fix parameter issue introduced during PR #3107 rebase May 7, 2026
@TaffyOfficial
Copy link
Copy Markdown
Contributor

ok

@gcanlin gcanlin added the ready label to trigger buildkite CI label May 7, 2026
@gcanlin gcanlin enabled auto-merge (squash) May 7, 2026 00:44
@gcanlin gcanlin merged commit 3c85ca5 into vllm-project:main May 7, 2026
8 checks passed
clodaghwalsh17 pushed a commit to clodaghwalsh17/nm-vllm-omni-ent that referenced this pull request May 12, 2026
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.

3 participants