fix: map ep_metadata to device type using "ov_device" key

Author: ankitm3k

onnxruntime/core/providers/openvino/openvino_provider_factory.cc

@@ -444,24 +444,27 @@ struct OpenVINO_Provider : Provider {
       return Status(common::ONNXRUNTIME, ORT_EP_FAIL, "OpenVINO EP only supports one device.");
     }
 
-    ProviderInfo pi;
-    const auto& config_options = session_options.GetConfigOptions();
-    ParseProviderInfo(provider_options, &config_options, pi);
-
+    // Extract device type from EP metadata
     const auto& device_meta_data = ep_metadata[0];
     auto it = device_meta_data->Entries().find("ov_device");
     if (it == device_meta_data->Entries().end()) {
       return Status(common::ONNXRUNTIME, ORT_INVALID_ARGUMENT, "OpenVINO EP device metadata not found.");
     }
-    pi.device_type = it->second;
 
-    /*
-    TODO: This is where we need to perform some checking / manupulation of the session_options before
-    they are passed into CreateProviders() below.
-    Note: pi.device_type is getting set above, but I *think* it will just get overridden by the
-          ParseConfigOptions() that will be done inside of CreateProvider();
-    */
+    std::string metadata_device_type = it->second;
+
+    // If user didn't specify device_type, use the one from metadata
+    if (provider_options.find("device_type") == provider_options.end()) {
+      provider_options["device_type"] = metadata_device_type;
+    }
+
+    // Parse provider info with the device type
+    ProviderInfo pi;
+    const auto& config_options = session_options.GetConfigOptions();
+    ParseProviderInfo(provider_options, &config_options, pi);
+    ParseConfigOptions(pi);
 
+    // Create and return the execution provider
     auto factory = std::make_unique<OpenVINOProviderFactory>(pi, SharedContext::Get());
     ep = factory->CreateProvider(session_options, logger);
     return Status::OK();

onnxruntime/core/providers/openvino/ov_factory.h

@@ -20,8 +20,6 @@ struct ApiPtrs {
   const OrtModelEditorApi& model_editor_api;
 };
 
-#define OVEP_PLUGIN_VERSION "0.0.0"
-
 #define OVEP_DISABLE_MOVE(class_name) \
   class_name(class_name&&) = delete;  \
   class_name& operator=(class_name&&) = delete;
@@ -143,7 +141,7 @@ class OpenVINOEpPluginFactory : public OrtEpFactory, public ApiPtrs {
   }
 
   static const char* ORT_API_CALL GetVersionImpl(const OrtEpFactory*) noexcept {
-    return OVEP_PLUGIN_VERSION;
+    return ORT_VERSION;
   }
 };