Skip to content

support more message item types#4501

Merged
lvhan028 merged 2 commits intoInternLM:mainfrom
CUHKSZzxy:more-item-types
Apr 29, 2026
Merged

support more message item types#4501
lvhan028 merged 2 commits intoInternLM:mainfrom
CUHKSZzxy:more-item-types

Conversation

@CUHKSZzxy
Copy link
Copy Markdown
Collaborator

No description provided.

@CUHKSZzxy CUHKSZzxy marked this pull request as ready for review April 29, 2026 08:08
Copilot AI review requested due to automatic review settings April 29, 2026 08:08
@CUHKSZzxy CUHKSZzxy requested a review from lvhan028 April 29, 2026 08:09
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Expands the multimodal message parsing in the serving pipeline to accept additional message item type aliases and value encodings, improving compatibility with more client-side message formats.

Changes:

  • Extend _parse_multimodal_item to support both dict-valued and direct-valued modality payloads (e.g., {image_url: {url: ...}} and {image_url: "..."}).
  • Treat image/video/time_series as supported aliases alongside the existing *_url types.
  • Update _has_multimodal_input modality detection to include the new type names.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread lmdeploy/serve/processors/multimodal.py Outdated
Comment thread lmdeploy/serve/processors/multimodal.py
Comment thread lmdeploy/serve/processors/multimodal.py
Comment thread lmdeploy/serve/processors/multimodal.py Outdated
@lvhan028 lvhan028 merged commit 9df0eff into InternLM:main Apr 29, 2026
5 checks passed
@CUHKSZzxy CUHKSZzxy deleted the more-item-types branch April 29, 2026 12:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants