Skip to content

refactor: rename vl package to multimodal#4710

Draft
CUHKSZzxy wants to merge 8 commits into
InternLM:mainfrom
CUHKSZzxy:refactor/vl-to-multimodal
Draft

refactor: rename vl package to multimodal#4710
CUHKSZzxy wants to merge 8 commits into
InternLM:mainfrom
CUHKSZzxy:refactor/vl-to-multimodal

Conversation

@CUHKSZzxy

Copy link
Copy Markdown
Collaborator

Summary

  • Rename the top-level multimodal implementation package from lmdeploy.vl to lmdeploy.multimodal.
  • Update internal imports, docs, and tests to use lmdeploy.multimodal.
  • Leave lmdeploy.vl as a top-level migration error.

Prerequisite

Depends on #4693. This PR should be reviewed after that prerequisite cleanup.

Validation

  • Syntax checks for the moved package and top-level migration error.
  • Import smoke for canonical lmdeploy.multimodal APIs and expected lmdeploy.vl failure.
  • Focused multimodal pytest coverage for preprocessing utilities, safe URL handling, Qwen3-Omni, Qwen3-VL, and media encode/decode helpers.
  • Diff whitespace check.

Assistance

Assisted with Codex + GPT-5.5 xHigh Fast, reviewed manually

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.

1 participant