Fixed linter issues

Author: suryasidd

backends/openvino/CMakeLists.txt

@@ -36,9 +36,9 @@ add_library(openvino_backend STATIC)
 
 # Enable exceptions and RTTI for OpenVINO backend
 target_compile_options(
-  openvino_backend PRIVATE
-  $<$<CXX_COMPILER_ID:MSVC>:/GR /EHsc>
-  $<$<NOT:$<CXX_COMPILER_ID:MSVC>>:-frtti -fexceptions>
+  openvino_backend
+  PRIVATE $<$<CXX_COMPILER_ID:MSVC>:/GR /EHsc>
+          $<$<NOT:$<CXX_COMPILER_ID:MSVC>>:-frtti -fexceptions>
 )
 
 # Add source files for OpenVINO backend

backends/openvino/README.md

@@ -110,7 +110,7 @@ Follow the steps below to setup your build environment:
    python -m venv env
    env\Scripts\Activate.ps1
    ```
-2. **Clone ExecuTorch Repository from Github**
+2. **Clone ExecuTorch Repository from GitHub**
 - On Windows, enable symlinks before cloning. Refer to [Building from Source](https://docs.pytorch.org/executorch/main/using-executorch-building-from-source.html#environment-setup) for more details.
 - Clone Executorch repository by executing the command below.
    ```bash

backends/openvino/runtime/OpenvinoBackend.cpp

@@ -32,8 +32,8 @@ constexpr const char* kDefaultLibName = "libopenvino_c.so";
 template <typename FuncPtr>
 FuncPtr load_symbol(void* handle, const char* name) {
 #ifdef _WIN32
-  void* sym =
-      reinterpret_cast<void*>(GetProcAddress(static_cast<HMODULE>(handle), name));
+  void* sym = reinterpret_cast<void*>(
+      GetProcAddress(static_cast<HMODULE>(handle), name));
   if (!sym) {
     ET_LOG(
         Error,
@@ -70,8 +70,8 @@ bool OpenvinoBackend::ensure_loaded() const {
       ET_LOG(
           Error,
           "OpenVINO runtime not found (LoadLibrary failed: error %lu). "
-          "Ensure 'openvino_c.dll' is on your PATH "
-          "(set OPENVINO_LIB_PATH), or install with: "
+          "Set OPENVINO_LIB_PATH to the full path of 'openvino_c.dll', "
+          "or add its containing directory to PATH, or install with: "
           "pip install \"openvino>=2025.1.0,<2026.0.0\"",
           GetLastError());
       return;

extension/pybindings/portable_lib.py

@@ -6,6 +6,7 @@
 
 # pyre-strict
 
+
 """API for loading and executing ExecuTorch PTE files using the C++ runtime.
 
 .. warning::
@@ -29,13 +30,15 @@
 # the pybindings shared library extension. This will load libtorch.so and
 # related libs, ensuring that the pybindings lib can resolve those runtime
 # dependencies.
+
 import torch as _torch
 
 logger = logging.getLogger(__name__)
 
 # Auto-discover the OpenVINO C library path from the pip-installed openvino
 # package so the C++ backend's dlopen/LoadLibrary call works without the user
 # having to set LD_LIBRARY_PATH or OPENVINO_LIB_PATH manually.
+
 if not os.environ.get("OPENVINO_LIB_PATH"):
     try:
         import glob
@@ -64,25 +67,26 @@
             del _ov_libs, _ov_libs_dir, _lib_pattern, _ov_dir, spec
     except Exception as e:
         logger.debug("OpenVINO auto-discovery failed: %s", e)
-
 # Update the DLL search path on Windows. This is the recommended way to handle native
 # extensions.
+
 if sys.platform == "win32":
     try:
         # The extension DLL should be in the same directory as this file.
+
         pybindings_dir = os.path.dirname(os.path.abspath(__file__))
         os.add_dll_directory(pybindings_dir)
     except Exception as e:
         logger.error(
             "Failed to add the pybinding extension DLL to the search path. The extension may not work.",
             e,
         )
-
 # Let users import everything from the C++ _portable_lib extension as if this
 # python file defined them. Although we could import these dynamically, it
 # wouldn't preserve the static type annotations.
 #
 # Note that all of these are experimental, and subject to change without notice.
+
 from executorch.extension.pybindings._portable_lib import (  # noqa: F401
     # Disable "imported but unused" (F401) checks.
     _create_profile_block,  # noqa: F401
@@ -109,6 +113,7 @@
 
 # Clean up so that `dir(portable_lib)` is the same as `dir(_portable_lib)`
 # (apart from some __dunder__ names).
+
 del _torch
 del _exir_warnings
 del _warnings