Improve get_interface_attr speed through caching

opendbc/car/interfaces.py

@@ -375,28 +375,43 @@ def update(self, CC: structs.CarControl, CS: CarStateBase, now_nanos: int) -> tu
 
 # interface-specific helpers
 
+@cache
+def _get_brand_names() -> list[str]:
+  return [folder.split('/')[-1] for folder in sorted(x[0] for x in os.walk(BASEDIR))]
+
+
+@cache
+def _get_brand_values_for_attr(brand_name: str, attr: str) -> Any | None:
+  attr_file = INTERFACE_ATTR_FILE.get(attr, "values")
+  try:
+    return __import__(f'opendbc.car.{brand_name}.{attr_file}', fromlist=[attr])
+  except (ImportError, OSError):
+    pass
+  return None
+
+
 def get_interface_attr(attr: str, combine_brands: bool = False, ignore_none: bool = False) -> dict[str | StrEnum, Any]:
   # read all the folders in opendbc/car and return a dict where:
   # - keys are all the car models or brand names
   # - values are attr values from all car folders
   result = {}
-  for car_folder in sorted([x[0] for x in os.walk(BASEDIR)]):
-    try:
-      brand_name = car_folder.split('/')[-1]
-      brand_values = __import__(f'opendbc.car.{brand_name}.{INTERFACE_ATTR_FILE.get(attr, "values")}', fromlist=[attr])
-      if hasattr(brand_values, attr) or not ignore_none:
-        attr_data = getattr(brand_values, attr, None)
-      else:
-        continue
-
-      if combine_brands:
-        if isinstance(attr_data, dict):
-          for f, v in attr_data.items():
-            result[f] = v
-      else:
-        result[brand_name] = attr_data
-    except (ImportError, OSError):
-      pass
+
+  for brand_name in _get_brand_names():
+    brand_values = _get_brand_values_for_attr(brand_name, attr)
+    if brand_values is None:
+      # This was an ImportError or OSError, so we skip it
+      continue
+    if hasattr(brand_values, attr) or not ignore_none:
+      attr_data = getattr(brand_values, attr, None)
+    else:
+      continue
+
+    if combine_brands:
+      if isinstance(attr_data, dict):
+        for f, v in attr_data.items():
+          result[f] = v
+    else:
+      result[brand_name] = attr_data
 
   return result