Skip to content

perf: memoize __getattr__ via LRU cache (review-iteration on #14, towards upstream #596)#17

Open
Tagar wants to merge 4 commits into
masterfrom
c1-bounded-attr-cache
Open

perf: memoize __getattr__ via LRU cache (review-iteration on #14, towards upstream #596)#17
Tagar wants to merge 4 commits into
masterfrom
c1-bounded-attr-cache

fix: address review feedback properly — root-cause + warning-without-pin

d00c042
Select commit
Loading
Failed to load commit list.
CodSpeed HQ / CodSpeed Performance Analysis succeeded Jun 11, 2026 in 0s

Performance Gate Passed

⚡ 3 improved benchmarks
✅ 19 untouched benchmarks
⏩ 7 skipped benchmarks1

Performance Changes

Mode Benchmark BASE HEAD Efficiency
WallTime test_macro_scenario[XA-3] 399.9 ms 3.2 ms ×130
WallTime test_m3_jvmview_class_resolution 394 µs 3.9 µs ×100
WallTime test_m4_constructor_and_finalize 586.1 µs 193.4 µs ×3

Tip

Curious why this is faster? Comment @codspeedbot explain why this is faster on this PR, or directly use the CodSpeed MCP with your agent.


Comparing c1-bounded-attr-cache (d00c042) with master (f92620d)

Open in CodSpeed

Footnotes

  1. 7 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports.