From 437aa368ba65d9ffdf990ef7deeb9800d29b6a7a Mon Sep 17 00:00:00 2001 From: Rob Parolin Date: Mon, 17 Nov 2025 17:44:50 -0800 Subject: [PATCH] Fix: Remove duplicate try_site_packages() call in load_nvidia_dynamic_lib Remove redundant call to finder.try_site_packages() on line 28 that was immediately overwritten by the same call on line 30. This duplicate call was wasteful (unnecessary filesystem traversal) and indicates a code review oversight. The function now correctly calls try_site_packages() only once, improving performance with no functional changes. Location: cuda_pathfinder/cuda/pathfinder/_dynamic_libs/load_nvidia_dynamic_lib.py:28 --- .../cuda/pathfinder/_dynamic_libs/load_nvidia_dynamic_lib.py | 2 -- 1 file changed, 2 deletions(-) diff --git a/cuda_pathfinder/cuda/pathfinder/_dynamic_libs/load_nvidia_dynamic_lib.py b/cuda_pathfinder/cuda/pathfinder/_dynamic_libs/load_nvidia_dynamic_lib.py index fe553a434..3431c2f86 100644 --- a/cuda_pathfinder/cuda/pathfinder/_dynamic_libs/load_nvidia_dynamic_lib.py +++ b/cuda_pathfinder/cuda/pathfinder/_dynamic_libs/load_nvidia_dynamic_lib.py @@ -25,8 +25,6 @@ def _load_lib_no_cache(libname: str) -> LoadedDL: finder = _FindNvidiaDynamicLib(libname) - abs_path = finder.try_site_packages() - abs_path = finder.try_site_packages() if abs_path is not None: found_via = "site-packages"