Skip to content

Commit a0eff6e

Browse files
IpsitKalraclaude
andcommitted
perf: revert the tuning experiment and record the negative result
The D-047 amortization change is reverted: it is measurably correct and does what it was designed to do, but it does not earn its place. Component microbenchmarks (5 repetitions, medians) showed the predicted mechanism clearly: FastBook add at an existing level 42.37 to 27.03 ns, add creating a level 50.84 to 43.09 ns, delete 18.07 to 14.57 ns. Paired alternating full-file replays told a different story at scale: about 4.5 percent faster at 50 million frames, but medians within about 1 percent with overlapping distributions at 100 million frames, the scope closest to the real working set. Peak anonymous footprint rose about 14 percent, from 243 to 276 MiB over the same scope, exactly the regression the hypothesis predicted. Sparser tables span more cache lines, so the locality cost appears to cancel the rehash saving while the memory cost stays. The acceptance rule written before the experiment required improvement beyond run-to-run noise with no memory regression, and neither holds, so the change is reverted and D-048 records the full evidence. Correctness was green throughout, including the 100-million-message dual-engine validation. The tuned full-protocol run is preserved as bench/results/2026-07-26-tuned-experiment.json; it cannot decide the question because its own controls (the book-free framing scan and the unchanged MapBook) moved more than the effect under test. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
1 parent dfa1ed2 commit a0eff6e

7 files changed

Lines changed: 1707 additions & 55 deletions

File tree

Lines changed: 52 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,52 @@
1+
capture_local_time: 2026-07-26 17:48:04 -0400
2+
capture_utc_time: 2026-07-26T21:48:04Z
3+
host_label: Mac
4+
operating_system: Darwin
5+
kernel_release: 25.5.0
6+
kernel_version: Darwin Kernel Version 25.5.0: Tue Jun 9 22:27:52 PDT 2026; root:xnu-12377.121.10~1/RELEASE_ARM64_T8112
7+
cpu_architecture: arm64
8+
os_product_version: 26.5.2
9+
os_build_version: 25F84
10+
cpu_model: Apple M2
11+
cpu_physical_cores: 8
12+
cpu_logical_cores: 8
13+
cpu_performance_cores: 4
14+
cpu_efficiency_cores: 4
15+
memory_installed_bytes: 8589934592
16+
memory_free_bytes: 58589184
17+
memory_inactive_bytes: 1315667968
18+
swap_usage: total = 9216.00M used = 8175.62M free = 1040.38M (encrypted)
19+
power_source: Battery Power
20+
battery_state: -InternalBattery-0 (id=37552227) 54%; discharging; 2:16 remaining present: true
21+
low_power_mode: 0
22+
thermal_pressure: unavailable
23+
thermal_warnings: none recorded
24+
dataset_basename: 07302019.NASDAQ_ITCH50
25+
dataset_size_bytes: 8661679413
26+
dataset_filesystem:
27+
dataset_sha256: 9f8634a048b8195ccdcbe618e5833a759e35e04486d74e36b79176d35172654a
28+
dataset_path_recorded: no (basename and checksum only, by policy)
29+
git_commit: dfa1ed2acfbbb8a67f7dabc3f65c3e75d18be57f
30+
git_short_commit: dfa1ed2
31+
git_branch: main
32+
git_working_tree: clean
33+
repository: IpsitKalra/nanobook
34+
compiler_executable: c++
35+
compiler_version: Apple clang version 16.0.0 (clang-1600.0.26.6)
36+
build_type: Release
37+
release_flags_base: -O3 -DNDEBUG
38+
native_arch_option: ON
39+
lto: ON
40+
host_tuning_flag: -march=native
41+
cmake_version: 4.4.0
42+
cxx_standard: C++20
43+
cpu_pinning_available: no (taskset absent on this platform)
44+
cpu_pinning_used: no
45+
selected_cpu_core: none
46+
cpu_governor: unavailable on this platform
47+
perf_available: no
48+
xctrace_available: no (binary present but requires full Xcode)
49+
sample_utility_available: yes
50+
platform_class: documented macOS fallback (product_spec.md section 1)
51+
tsc_expected: no (no x86 TSC on this architecture)
52+
disk_free_bytes_repo_volume: 15108644864

0 commit comments

Comments
 (0)