Run lintrunner and fix formatting issues (#836)

Author: Jaswanth51

onnxruntime/core/providers/openvino/backend_manager.cc

@@ -85,7 +85,7 @@ BackendManager::BackendManager(SessionContext& session_context,
   subgraph_context_.subgraph_name = fused_node.Name();
 
   if (ModelHasSymbolicInputDims(subgraph)) {
-      subgraph_context_.has_dynamic_input_shape = true;
+    subgraph_context_.has_dynamic_input_shape = true;
   }
 
   ptr_stream_t model_stream;
@@ -171,9 +171,7 @@ BackendManager::BackendManager(SessionContext& session_context,
           exception_str.find("intel_npu") != std::string::npos) {
         // Handle NPU device related errors
 #ifndef NDEBUG
-        std::string suffix = session_context_.so_disable_cpu_ep_fallback ?
-          "\nModel failed to compile on NPU. Enable CPU fallback or try another device.\n" :
-          "\nModel needs to be recompiled\n";
+        std::string suffix = session_context_.so_disable_cpu_ep_fallback ? "\nModel failed to compile on NPU. Enable CPU fallback or try another device.\n" : "\nModel needs to be recompiled\n";
         ORT_THROW(exception_str + suffix);
 #else
         std::string error_message = "UNKNOWN NPU ERROR";
@@ -187,9 +185,7 @@ BackendManager::BackendManager(SessionContext& session_context,
         if (std::regex_search(exception_str, matches, error_code_pattern)) {
           error_code = matches[0];
         }
-        std::string suffix = session_context_.so_disable_cpu_ep_fallback ?
-          "\nModel failed to compile on NPU. Enable CPU fallback or try another device.\n" :
-          "\nModel needs to be recompiled\n";
+        std::string suffix = session_context_.so_disable_cpu_ep_fallback ? "\nModel failed to compile on NPU. Enable CPU fallback or try another device.\n" : "\nModel needs to be recompiled\n";
         throw std::runtime_error(error_message + ", " + error_code + suffix);
 #endif
       } else {
@@ -645,12 +641,11 @@ BackendManager::GetModelProtoFromFusedNode(const onnxruntime::Node& fused_node,
     const bool include_initializer_data_in_proto = true;
 #endif
 
-
     auto model = subgraph.CreateModel(logger);
     auto model_proto = model->ToProto();
     model_proto->set_ir_version(ONNX_NAMESPACE::Version::IR_VERSION);
-    subgraph.ToProto(*model_proto->mutable_graph(), /*include_initializers*/true,
-                     /*include_outer_scope_args*/true, /*execution_order*/0, /*include_initializer_data*/include_initializer_data_in_proto);
+    subgraph.ToProto(*model_proto->mutable_graph(), /*include_initializers*/ true,
+                     /*include_outer_scope_args*/ true, /*execution_order*/ 0, /*include_initializer_data*/ include_initializer_data_in_proto);
 
     print_model_proto_duration();
 
@@ -684,9 +679,9 @@ BackendManager::GetModelProtoFromFusedNode(const onnxruntime::Node& fused_node,
                                 << ", data_type: " << src_init->data_type()
                                 << ", raw_data size: " << src_init->raw_data().size();
           if (src_init->raw_data().size() > 0)
-              SetExternalDataFields(proto_init, src_init->raw_data().data(), src_init->raw_data().size());
+            SetExternalDataFields(proto_init, src_init->raw_data().data(), src_init->raw_data().size());
           else
-              LOGS(logger, VERBOSE) << "Initializer has empty raw_data: skipping initializer '" << src_init->name() << "'...";
+            LOGS(logger, VERBOSE) << "Initializer has empty raw_data: skipping initializer '" << src_init->name() << "'...";
         } else if (onnxruntime::utils::HasExternalDataInMemory(*src_init)) {
           auto it_ext = external_initializers_offset_and_length.find(name);
           if (it_ext == external_initializers_offset_and_length.end()) {
@@ -889,7 +884,7 @@ void BackendManager::Compute(OrtKernelContext* context) {
           }
         } else {
           std::string exception_str = ex.what();
-          if (session_context_.so_disable_cpu_ep_fallback){
+          if (session_context_.so_disable_cpu_ep_fallback) {
             std::string error_message = "UNKNOWN NPU ERROR";
             std::string error_code = "code 0x0";
             std::regex error_message_pattern(R"(\bZE_\w*\b)");
@@ -901,10 +896,9 @@ void BackendManager::Compute(OrtKernelContext* context) {
             if (std::regex_search(exception_str, matches, error_code_pattern)) {
               error_code = matches[0];
             }
-            std::string suffix = "\nModel failed to compile on NPU. Enable CPU fallback or try another device.\n" ;
+            std::string suffix = "\nModel failed to compile on NPU. Enable CPU fallback or try another device.\n";
             throw std::runtime_error(error_message + ", " + error_code + suffix);
-          }
-          else{
+          } else {
             ORT_THROW(exception_str);
           }
         }

onnxruntime/core/providers/openvino/ov_interface.h

@@ -25,8 +25,8 @@
 // Usage: #if OPENVINO_VERSION_AT_LEAST(2025, 3)
 // Falls back to 0 if OPENVINO_VERSION_MAJOR/MINOR are not defined.
 #if defined(OPENVINO_VERSION_MAJOR) && defined(OPENVINO_VERSION_MINOR)
-#define OPENVINO_VERSION_AT_LEAST(major, minor)                         \
-  ((OPENVINO_VERSION_MAJOR > (major)) ||                                \
+#define OPENVINO_VERSION_AT_LEAST(major, minor) \
+  ((OPENVINO_VERSION_MAJOR > (major)) ||        \
    (OPENVINO_VERSION_MAJOR == (major) && OPENVINO_VERSION_MINOR >= (minor)))
 #else
 #define OPENVINO_VERSION_AT_LEAST(major, minor) 0

onnxruntime/test/providers/openvino/openvino_ep_ext_init.cc

@@ -52,7 +52,7 @@ auto ProbeDevice(const std::string& device) {
   }
   return is_present[device];
 }
-}  // namespace detail
+}  // namespace
 
 namespace onnxruntime {
 namespace test {
@@ -62,7 +62,7 @@ TEST_P(OVEP_ExtInit_Tests, DISABLED_ModelFromExtInit) {
   const auto& device = GetParam();
   if (!ProbeDevice(device))
     GTEST_SKIP() << device + " is not available on this machine";
-       
+
   // Model and weights file paths
   const std::string model_path = "ovep_ext_init_test.onnx";
   const std::string weights_path = "ovep_ext_init_test.onnx.data";
@@ -174,7 +174,7 @@ TEST_P(OVEP_ExtInit_Tests, DISABLED_ModelFromExtInit) {
   Ort::SessionOptions session_options;
   session_options.AddConfigEntry(kOrtSessionOptionsDisableCPUEPFallback, "1");
   session_options.SetIntraOpNumThreads(1);
-  std::unordered_map<std::string, std::string> ov_options = { {"device_type", device } };
+  std::unordered_map<std::string, std::string> ov_options = {{"device_type", device}};
   session_options.AppendExecutionProvider_OpenVINO_V2(ov_options);
   session_options.AddExternalInitializersFromFilesInMemory(names_path, buffers, buffer_sizes);
 

onnxruntime/test/unittest_util/checkers.cc

@@ -383,7 +383,7 @@ struct TensorCheck<uint16_t> {
   void operator()(const Tensor& expected,
                   const Tensor& actual,
                   const ValidateOutputParams& params,
-                  const std::string& ) const {
+                  const std::string&) const {
     const bool has_abs_err = params.absolute_error.has_value();
     const bool has_rel_err = params.relative_error.has_value();