Skip to content

[OpenVINO]add support for minicpmv4/4_5 #1412

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 15 commits into
base: main
Choose a base branch
from
Open

[OpenVINO]add support for minicpmv4/4_5 #1412

Show file tree
Hide file tree
Changes from 5 commits
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
4b70b7f
add support for minicpm4v
openvino-dev-samples Aug 7, 2025
7c64417
reformat
openvino-dev-samples Aug 7, 2025
4b245f1
add minicpmv4 test case
openvino-dev-samples Aug 12, 2025
81f69be
add minicpmv4_5
openvino-dev-samples Aug 27, 2025
6974a3e
rebase
openvino-dev-samples Sep 27, 2025
bd8b41f
update minicpm4/4.5 CI test
openvino-dev-samples Oct 16, 2025
6c0c617
update seq2seq test
openvino-dev-samples Oct 21, 2025
05fad58
update seq2seq test
openvino-dev-samples Oct 21, 2025
b4e2ce1
update
openvino-dev-samples Oct 21, 2025
39223a4
fix the CI issues for export and quantization
openvino-dev-samples Oct 23, 2025
bd1adbd
to support temporal_ids in resampler of minicpm4.5
openvino-dev-samples Oct 31, 2025
6aec742
fix conflict
openvino-dev-samples Oct 31, 2025
f6d7750
fix conflict
openvino-dev-samples Oct 31, 2025
a96d993
update
openvino-dev-samples Oct 31, 2025
02a4acf
fix CI
openvino-dev-samples Oct 31, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
16 changes: 13 additions & 3 deletions optimum/exporters/openvino/model_configs.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2865,12 +2865,22 @@ def with_behavior(
"""
if isinstance(behavior, str) and not isinstance(behavior, MiniCPMVConfigBehavior):
behavior = MiniCPMVConfigBehavior(behavior)

model_mapping = {2.6: "llama", 4.0: "qwen2", 4.5: "qwen3"}
Copy link
Member

Choose a reason for hiding this comment

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

should use str for versions

Copy link
Contributor Author

Choose a reason for hiding this comment

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

may i understand why？the version in model's config is a number:
https://huggingface.co/openbmb/MiniCPM-V-4_5/blob/main/config.json#L3

Copy link
Member

@IlyasMoutawwakil IlyasMoutawwakil Oct 24, 2025 •
edited
Loading

Choose a reason for hiding this comment

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

ah okay I see ! thanks for the clarification.
(it's generally a bad idea to use numbers for versions: 4.0 becomes 4 and 4.10 and 4.1 are the same version 😅)

Copy link
Collaborator

Choose a reason for hiding this comment

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

I think it is a bad idea to make decision about architecture based on model version in general.
I think you should parse class model object and use isinstance for inner objects to make decision.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, its a better approach in this case, but I dont know if we can access the modeling file at this stage.

if behavior == MiniCPMVConfigBehavior.TEXT_EMBEDDINGS:
return get_vlm_text_embeddings_config("qwen2", self._orig_config, self.int_dtype, self.float_dtype)
return get_vlm_text_embeddings_config(
model_mapping[self._orig_config.version],
self._orig_config,
self.int_dtype,
self.float_dtype,
)

if behavior == MiniCPMVConfigBehavior.LANGUAGE:
return get_vlm_text_generation_config("qwen2", self._orig_config, self.int_dtype, self.float_dtype)
return get_vlm_text_generation_config(
model_mapping[self._orig_config.version],
self._orig_config,
self.int_dtype,
self.float_dtype,
)

if behavior == MiniCPMVConfigBehavior.VISION_EMBEDDINGS:
return self.__class__(
Expand Down
22 changes: 22 additions & 0 deletions tests/openvino/test_exporters_cli.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -655,6 +655,28 @@ class OVCLIExportTestCase(unittest.TestCase):
"resampler_model": {"int8": 6},
},
),
(
"image-text-to-text",
"minicpmv4",
"int4 --group-size 4 --ratio 0.8 --trust-remote-code",
{
"lm_model": {"int8": 12, "int4": 18},
"text_embeddings_model": {"int8": 1},
"vision_embeddings_model": {"int8": 14},
"resampler_model": {"int8": 6},
},
),
(
"image-text-to-text",
"minicpmv4_5",
"int4 --group-size 4 --ratio 0.8 --trust-remote-code",
{
"lm_model": {"int8": 12, "int4": 18},
"text_embeddings_model": {"int8": 1},
"vision_embeddings_model": {"int8": 14},
"resampler_model": {"int8": 6},
},
),
(
"image-text-to-text",
"internvl_chat",
Expand Down
1 change: 1 addition & 0 deletions tests/openvino/test_quantization.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1047,6 +1047,7 @@ class OVWeightCompressionTest(unittest.TestCase):

if is_transformers_version(">=", "4.45.0"):
SUPPORTED_ARCHITECTURES_WITH_AUTO_COMPRESSION.append((OVModelForVisualCausalLM, "minicpmv", True))
SUPPORTED_ARCHITECTURES_WITH_AUTO_COMPRESSION.append((OVModelForVisualCausalLM, "minicpmv4", True))
SUPPORTED_ARCHITECTURES_WITH_AUTO_COMPRESSION.append((OVModelForVisualCausalLM, "qwen2_vl", False))

SUPPORTED_ARCHITECTURES_WITH_HYBRID_QUANTIZATION = [
Expand Down
14 changes: 14 additions & 0 deletions tests/openvino/utils_tests.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -116,6 +116,8 @@
"minicpm": "katuni4ka/tiny-random-minicpm",
"minicpm3": "katuni4ka/tiny-random-minicpm3",
"minicpmv": "katuni4ka/tiny-random-minicpmv-2_6",
"minicpmv4": "snake7gun/minicpm-v-4-tiny",
"minicpmv4_5": "snake7gun/tiny-minicpmv-4_5",
Copy link
Collaborator

Choose a reason for hiding this comment

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

158M model size, it makes sense to try to reduce the size

"mistral": "echarlaix/tiny-random-mistral",
"mistral-nemo": "katuni4ka/tiny-random-mistral-nemo",
"mixtral": "TitanML/tiny-mixtral",
Expand Down Expand Up @@ -284,6 +286,18 @@
"vision_embeddings_model": 26,
"resampler_model": 6,
},
"minicpmv4": {
"lm_model": 30,
"text_embeddings_model": 1,
"vision_embeddings_model": 14,
"resampler_model": 6,
},
"minicpmv4_5": {
"lm_model": 30,
"text_embeddings_model": 1,
"vision_embeddings_model": 14,
"resampler_model": 6,
},
"llava_next_video": {
"lm_model": 30,
"text_embeddings_model": 1,
Expand Down
Loading