Skip to content

[Fix] error when processing batch prompts for VLM - #4955

Closed
JustinTong0323 wants to merge 3 commits into
sgl-project:mainfrom
JustinTong0323:txy/multi_model_base_processor_fix
Closed

JustinTong0323 wants to merge 3 commits into
sgl-project:mainfrom
JustinTong0323:txy/multi_model_base_processor_fix

Conversation

@JustinTong0323

Copy link
Copy Markdown
Collaborator

Motivation

Previously, the main branch would throw AssertionError when handling batch prompts due to redundant assert.

Modifications

As titled.

Checklist

…essor

Signed-off-by: Xinyuan Tong <justinning0323@outlook.com>
@JustinTong0323

Copy link
Copy Markdown
Collaborator Author

@mickqian Could you please have a look?
cc @zhaochenyang20 I think it's high priority.

@zhaochenyang20 zhaochenyang20 self-assigned this Apr 1, 2025
@zhaochenyang20 zhaochenyang20 added high priority RLHF Multi-modal multi-modal language model labels Apr 1, 2025
@JustinTong0323
JustinTong0323 deleted the txy/multi_model_base_processor_fix branch April 18, 2025 07:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

high priority Multi-modal multi-modal language model

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants