Merge pull request #434 from mdboom/lru-data-cache

Use LRU cache for loading benchmarking data

Author: mdboom

bench_runner/result.py

@@ -34,6 +34,12 @@
 CombinedData = list[tuple[str, np.ndarray | None, float]]
 
 
+@functools.lru_cache(maxsize=100)
+def _load_contents(filename: Path) -> dict[str, Any]:
+    with filename.open("rb") as fd:
+        return json.load(fd)
+
+
 def _clean(string: str) -> str:
     """
     Clean an arbitrary string to be suitable for a filename.
@@ -598,16 +604,15 @@ def result_info(self) -> tuple[str | None, str | None, str | None]:
             f"Unknown result type (extra={self.extra} suffix={self.suffix})"
         )
 
-    @functools.cached_property
+    @property
     def contents(self) -> dict[str, Any]:
-        with self.filename.open("rb") as fd:
-            return json.load(fd)
+        return _load_contents(self.filename)
 
-    @property
+    @functools.cached_property
     def metadata(self) -> dict[str, Any]:
         return self.contents.get("metadata", {})
 
-    @property
+    @functools.cached_property
     def commit_datetime(self) -> str:
         if self._commit_datetime is not None:
             return self._commit_datetime
@@ -617,7 +622,7 @@ def commit_datetime(self) -> str:
     def commit_date(self) -> str:
         return self.commit_datetime[:10]
 
-    @property
+    @functools.cached_property
     def run_datetime(self) -> str:
         return self.contents["benchmarks"][0]["runs"][0]["metadata"]["date"]