Skip to content

refactor: rename vl package to multimodal - #2

Closed
CUHKSZzxy wants to merge 1 commit into
chore/remove-deprecated-model-supportfrom
refactor/vl-to-multimodal
Closed

refactor: rename vl package to multimodal#2
CUHKSZzxy wants to merge 1 commit into
chore/remove-deprecated-model-supportfrom
refactor/vl-to-multimodal

Conversation

@CUHKSZzxy

Copy link
Copy Markdown
Owner

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 instead of keeping backward-compatible deep imports.
  • Rename the focused test directory from tests/test_lmdeploy/test_vl to tests/test_lmdeploy/test_multimodal.

Prerequisite

This PR is stacked on top of InternLM#4693 and should be reviewed/merged after that prerequisite cleanup.

Validation

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

Assistance

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

@CUHKSZzxy CUHKSZzxy closed this Jun 26, 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.

1 participant