Fix lint issues

intel · Apr 22, 2024 · 7900f86 · 7900f86
1 parent 6c8073d
commit 7900f86
Show file tree

Hide file tree

Showing 5 changed files with 12 additions and 9 deletions.
diff --git a/onnxruntime/core/providers/openvino/backends/basic_backend.cc b/onnxruntime/core/providers/openvino/backends/basic_backend.cc
@@ -93,7 +93,7 @@ BasicBackend::BasicBackend(const ONNX_NAMESPACE::ModelProto& model_proto,
       } else if (!subgraph_context_.has_dynamic_input_shape &&
                  global_context_.onnx_model_path_name.find(".onnx") != std::string ::npos) {
         // Inputs with static dimenstions
-        std::string prec_str = (global_context_.precision_str!="ACCURACY")? global_context_.precision_str : global_context_.model_precision;
+        std::string prec_str = (global_context_.precision_str != "ACCURACY") ? global_context_.precision_str : global_context_.model_precision;
         exe_network_ = global_context_.ie_core.CompileModel(global_context_.onnx_model_path_name,
                                                             hw_target,
                                                             prec_str,

diff --git a/onnxruntime/core/providers/openvino/openvino_execution_provider.cc b/onnxruntime/core/providers/openvino/openvino_execution_provider.cc
@@ -105,7 +105,8 @@ OpenVINOExecutionProvider::GetCapability(const GraphViewer& graph_viewer,
       return "";
     } else {
       auto input_type = graph_viewer.GetInputs()[0]->TypeAsProto()->tensor_type().elem_type();
-      if (global_context_->precision_str == "ACCURACY" && global_context_->device_type.find("GPU") != std::string::npos) {
+      if (global_context_->precision_str == "ACCURACY" &&
+          global_context_->device_type.find("GPU") != std::string::npos) {
         if (input_type == ONNX_NAMESPACE::TensorProto_DataType::TensorProto_DataType_FLOAT) {
           return "FP32";
         } else if (input_type == ONNX_NAMESPACE::TensorProto_DataType::TensorProto_DataType_FLOAT16) {

diff --git a/onnxruntime/core/providers/openvino/openvino_execution_provider.h b/onnxruntime/core/providers/openvino/openvino_execution_provider.h
@@ -120,7 +120,7 @@ struct OpenVINOExecutionProviderInfo {
 #endif
     } else if (ov_supported_device_types.find(dev_type) != ov_supported_device_types.end()) {
       device_type_ = dev_type;
-    } else if (dev_type.find("HETERO") == 0 || dev_type.find("MULTI") == 0 || dev_type.find("AUTO")==0) {
+    } else if (dev_type.find("HETERO") == 0 || dev_type.find("MULTI") == 0 || dev_type.find("AUTO") == 0) {
       std::vector<std::string> devices = parseDevices(dev_type);
       device_type_ = dev_type;
     } else {

diff --git a/onnxruntime/core/providers/openvino/openvino_provider_factory.cc b/onnxruntime/core/providers/openvino/openvino_provider_factory.cc
@@ -142,13 +142,13 @@ struct OpenVINO_Provider : Provider {
       } else if (precision != "ACCURACY" && precision != "FP16" && precision != "FP32") {
         ORT_THROW("[ERROR] [OpenVINO] Unsupported inference precision is selected. GPU only supports FP32 / FP16. \n");
       }
-    } else if (device_type.find("NPU") != std:: string::npos) {
+    } else if (device_type.find("NPU") != std::string::npos) {
       if (precision == "" || precision == "ACCURACY" || precision == "FP16") {
         precision = "FP16";
       } else {
         ORT_THROW("[ERROR] [OpenVINO] Unsupported inference precision is selected. NPU only supported FP16. \n");
       }
-    } else if (device_type.find("CPU")!=std::string::npos) {
+    } else if (device_type.find("CPU") != std::string::npos) {
       if (precision == "" || precision == "ACCURACY" || precision == "FP32") {
         precision = "FP32";
       } else {

diff --git a/onnxruntime/test/perftest/ort_test_session.cc b/onnxruntime/test/perftest/ort_test_session.cc
@@ -274,24 +274,26 @@ OnnxRuntimeTestSession::OnnxRuntimeTestSession(Ort::Env& env, std::random_device
         }
       } else if (key == "precision") {
         auto device_type = ov_options["device_type"];
-        if (device_type.find("GPU")!= std::string::npos) {
+        if (device_type.find("GPU") != std::string::npos) {
           if (value == "") {
             ov_options[key] = "FP16";
             continue;
           } else if (value == "ACCURACY" || value == "FP16" || value == "FP32") {
             ov_options[key] = value;
             continue;
           } else {
-            ORT_THROW("[ERROR] [OpenVINO] Unsupported inference precision is selected. GPU only supported FP32 / FP16. \n");
+            ORT_THROW(
+                "[ERROR] [OpenVINO] Unsupported inference precision is selected. "
+                "GPU only supported FP32 / FP16. \n");
           }
-        }else if (device_type.find("NPU")!= std::string::npos) {
+        } else if (device_type.find("NPU") != std::string::npos) {
           if (value == "" || value == "ACCURACY" || value == "FP16") {
             ov_options[key] = "FP16";
             continue;
           } else {
             ORT_THROW("[ERROR] [OpenVINO] Unsupported inference precision is selected. NPU only supported FP16. \n");
           }
-        } else if (device_type.find("CPU")!= std::string::npos) {
+        } else if (device_type.find("CPU") != std::string::npos) {
           if (value == "" || value == "ACCURACY" || value == "FP32") {
             ov_options[key] = "FP32";
             continue;