microsoft / onnxruntime

ONNX Runtime: cross-platform, high performance ML inferencing and training accelerator
https://onnxruntime.ai
MIT License
14.06k stars 2.83k forks source link

Exception while initializing OpenVINO provider #18876

Closed smartnet-club closed 3 months ago

smartnet-club commented 8 months ago

Describe the issue

Problem with commit 0723dcb8b591a559db60885ff2cad610fd989ad4 provider_bridge_ort.cc OrtOpenVINOProviderOptionsToOrtOpenVINOProviderOptionsV2

incorrect reinterpret_cast from unsigned char to const char* std::string enable_dynamic_shapes = reinterpret_cast<const char*>(legacy_ov_options->enable_dynamic_shapes);

while OrtOpenVINOProviderOptions unsigned char enable_dynamic_shapes; ///< 0 = disabled, nonzero = enabled

To reproduce

api.SessionOptionsAppendExecutionProvider_OpenVINO(session_options, &openvinoOptions)

Urgency

No response

Platform

Linux

OS Version

22.04

ONNX Runtime Installation

Built from Source

ONNX Runtime Version or Commit ID

0723dcb8b591a559db60885ff2cad610fd989ad4

ONNX Runtime API

C++

Architecture

X64

Execution Provider

OpenVINO

Execution Provider Library Version

2023.2.0

jywu-msft commented 8 months ago

+@sfatimar

github-actions[bot] commented 7 months ago

This issue has been automatically marked as stale due to inactivity and will be closed in 30 days if no further activity occurs. If further support is needed, please provide an update and/or more details.

jywu-msft commented 7 months ago

+@sfatimar

+@preetha-intel FYI

preetha-intel commented 7 months ago

The issue is fixed in latest master. [pull/19023] (https://github.com/microsoft/onnxruntime/pull/19023)

preetha-intel commented 7 months ago

@smartnet-club Can we verify the fix