-
Notifications
You must be signed in to change notification settings - Fork 46
Add OpenVINOEpFactory functionality to support auto EP selection #738
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.
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
base: ovep-develop
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hello we are looking at new ABI Design which should cover this. I will connect with you the specific engineer to ensure your requirements are captured .
@javier-intel please check if we are incorporating the open source contributor requirement in new WinML EP. |
I have verified that this PR works well to fix the jira issue, thanks a lot! @bopeng1234 |
@sfatimar Thanks! /cc @ningxin |
|
||
const OrtApi& ort_api; | ||
const std::string ep_name; // EP name | ||
const std::string vendor{"Microsoft"}; // EP vendor name |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The EP vendor name should be "Intel", pls refer to https://github.com/intel-innersource/frameworks.ai.onnxruntime.staging/blob/develop/onnxruntime/core/providers/openvino/plugin/ov_factory.h#L72. Otherwise, IsDefaultCpuEp will be true for OV CPU EP device, that's incorrect.
Description
initial added OpenVINOEpFactory, CreateEpFactories, ReleaseEpFactory to support the auto EP selection feature.
Motivation and Context
To fix https://jira.devtools.intel.com/browse/CVS-170114