Skip to content

[OpenVINO][Draft]add support for dinov3 #1431

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

Jump to bottom
Open
wants to merge 9 commits into
base: main
Choose a base branch
from
Open

[OpenVINO][Draft]add support for dinov3 #1431

wants to merge 9 commits into from

Conversation

openvino-dev-samples
Copy link
Contributor

  • to export the model
 optimum-cli export openvino --task feature-extraction -m "/home/ethan/intel/optimum-intel/dinov3-convnext-base-pretrain-lvd1689m" dinov3-convnext-base-pretrain-lvd1689m-ov
  • to deploy the model
model = OVModelForCustomTasks.from_pretrained(
    pretrained_model_name, 
    device_map="cpu", 
)

@openvino-dev-samples openvino-dev-samples changed the title [OpenVINO][Dratf]add support for dinov3 [OpenVINO][Draft]add support for dinov3 Aug 21, 2025
echarlaix
echarlaix reviewed Aug 22, 2025
View reviewed changes
Copy link
Collaborator

@echarlaix echarlaix left a comment

Choose a reason for hiding this comment

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

thanks for the addition @openvino-dev-samples, transformers v4.55 will soon be supported (once #1406 is merged), would you mind adding tests with tiny models ?

rkazants
rkazants suggested changes Sep 15, 2025
View reviewed changes
Copy link
Collaborator

@rkazants rkazants left a comment

Choose a reason for hiding this comment

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

please add tests to check inference. Now it is tests only export

echarlaix
echarlaix reviewed Sep 19, 2025
View reviewed changes
Copy link
Collaborator

@echarlaix echarlaix left a comment

Choose a reason for hiding this comment

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

Thanks for the PR @openvino-dev-samples

optimum/exporters/openvino/model_configs.py
Comment on lines +4542 to +4543
@register_in_tasks_manager("dinov3_vit", *["feature-extraction"], library_name="transformers")
@register_in_tasks_manager("dinov3_convnext", *["feature-extraction"], library_name="transformers")
Copy link
Collaborator

Choose a reason for hiding this comment

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

thanks for adding a tests for dinov3_vit! Would you mind adding a test for dinov3_convnext as well ? (to make sure support is not broken in the future)

@echarlaix
Copy link
Collaborator

the PR #1406 is now merged !

echarlaix
echarlaix reviewed Sep 29, 2025
View reviewed changes
echarlaix
echarlaix reviewed Sep 29, 2025
View reviewed changes
tests/openvino/test_exporters_cli.py Outdated
]
)

if is_transformers_version(">=", "4.56"):
Copy link
Collaborator

Choose a reason for hiding this comment

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

let's wait for us to support transformers v4.56 before merging so that we are sure all tests are triggered and pass

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@echarlaix echarlaix echarlaix left review comments

+1 more reviewer

@rkazants rkazants rkazants requested changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@openvino-dev-samples @echarlaix @rkazants