bacaf3d39c
El SDK CI - dev / build-and-test (pull_request) Failing after 4m49s
Lands feat/reframe-region-setop (PR #109: native set-based reframe_region, decorator-as-seam @route port, teacher-summon, and the M8.1 activate-latency work — lazy-memoized cosq via eg_cosq_at + engram_vindex HNSW-accelerated seed discovery + vindex_harvest_from_store/vindex_bench oracle) onto dev's actual current HEAD, plus engram-tiered-storage's still-unique test suite. RECONCILING #109 WITH engram-tiered-storage (M4-M10 HNSW/geometry/reason/ verify work): not a two-way merge. engram_vindex.c's HNSW core (search_layer/ select_neighbors/prune_links/insert) is BYTE-IDENTICAL between the two branches; #109's copy is a strict superset (adds vindex_harvest_from_store, used by vindex_bench.c's brute-force-vs-HNSW oracle). engram_reason.c and engram_verify.c are also byte-identical. #109's own branch point already carried engram-tiered-storage's M4-M10 lineage forward, so there was nothing left to merge into #109 for those files. The one thing engram-tiered-storage had that #109's tree dropped: its full test suite (test_vindex.c, test_geometry.c, test_reason.c, test_verify.c, test_m7_traversal.c, the interoception P0-P5 tests, bufpool/compaction tests, and their run_*.sh harnesses) — ported over here unchanged. WHY THIS NEEDED HAND RECONCILIATION, NOT A MECHANICAL MERGE: #109's branch forked from dev on 2026-08-14 15:40 (before restructure-adjacent history diverged the file's merge-base for `git merge` — it presented as an add/add conflict). A straight two-dot diff (dev tip -> PR tip) applied cleanly, but it silently reverted THREE dev fixes landed on 2026-08-14/15, after the branch point, that the PR's diff had no way to know about: 1. qgate rescale (2026-08-14 self-review): PR's lazy eg_cosq_at rewrite of the query-aware propagation gate dropped the shift-and-floor rescale about ENGRAM_EMBED_S0 (measured: unrelated-pair median 0.562->raw gate 0.67, i.e. "a small tax, not a gate"). Restored the rescale, wrapped around the lazy accessor -- the PR's actual improvement (WHEN cosq[oi] is computed) is orthogonal to WHAT it gates on and both are kept. 2. Eviction cause decomposition (2026-08-14 self-review): dev decomposes wm_evicted into evict_floor/evict_cap/evict_bll so WM churn is diagnosable (identity: evicted == floor+cap+bll+dup_wm+dup_wm_global). PR's tree predates this and dropped all three counters + their JSON stats fields. Restored declarations, all 4 direct increment sites, the eg_wm_carry_over bll increment, and the act-stats JSON fields -- alongside (not instead of) the PR's own P4 afferent / API-reshape counters already in that same struct/JSON. 3. Hebbian link-formation selection (2026-08-15 self-review, TODAY): dev selects the STRONGEST qualifying candidate for consolidation each call; PR's tree predates this and reverted to hash-slot order (arbitrary wrt association strength) for edge formation -- the one path that writes PERMANENT structure. Restored the strongest-candidate while-loop, keeping the PR's own genuine improvement at that site (engram_adj_on_edge_added incremental-index append instead of a bare adj_dirty=1 full-rebuild flag). engram/src/server.el's 3-way conflicts (autoconnect_on/ise_offgraph_on env flags, /api/nodes connected-count in responses) were pure additive: dev's side was empty, PR's side added the feature. Took PR's side whole. VERIFIED (nsbx sandbox only, live :8742/:7770 never touched): - cc -std=c11 -O2, clean link against the real engram/src/server.el via elc, zero errors. - vindex_bench (built standalone, read-only harvest) against the real production store clone (13,671 embedded nodes, 768-dim nomic-embed-text): recall@10 = 1.0000 at ef 64/128/200; HNSW search 0.28-0.79ms/query vs 2.03ms/query brute-force oracle (2.6x-7.2x). HNSW build itself: 46.5s for the full 13,671-node set -- see the flagged risk below. - Booted the reconciled binary in an isolated nsbx sandbox (:8905, cloned snapshot of the live store, 13,424 nodes / 37,656 edges) and called /api/activate for real: first call after boot 41.5s (pays the one-time HNSW build inline -- matches the standalone bench), second/third calls 356ms/605ms, no crash, correct results, act-stats JSON (including the restored evict_floor/cap/bll fields) reads correctly. KNOWN RISK TO FLAG BEFORE ANY LIVE CUTOVER (not fixed here; out of scope for this dev-only land per instructions not to touch :8742/:7770): eg_vindex_sync builds the HNSW index synchronously, inline, on the first engram_activate() call after every process start (or index invalidation). On the real node count that is a ~46s blocking stall on a single-threaded server -- the first request after every restart (or its concurrent siblings) waits the full build. Recommend a background/incremental build (or a bounded per-call build budget) before this ever reaches the live daemon. See PR description / final report for the fuller writeup.
138 lines
7.5 KiB
Bash
Executable File
138 lines
7.5 KiB
Bash
Executable File
#!/usr/bin/env bash
|
|
# M7 index-driven-traversal gate. Pure C harness (NOT elb/elc): links the real
|
|
# el_runtime.c engram builtins + engram_store.c and drives ENGRAM_STORE off vs on.
|
|
# Proves (1) byte-identical activation parity flag-on == flag-off across a
|
|
# mutating query sequence, and (2) the O(E)-rebuild cost is eliminated flag-on.
|
|
# Writes ONLY under a throwaway /tmp dir with a throwaway HOME.
|
|
set -u
|
|
HERE="$(cd "$(dirname "$0")" && pwd)"
|
|
RT="$HERE/../../lang/runtime/el_runtime.c"
|
|
ST="$HERE/../../lang/runtime/engram_store.c"
|
|
INC="$HERE/../../lang/runtime"
|
|
WORK="$(mktemp -d /tmp/engram-m7-XXXXXX)"
|
|
DATA="$WORK/data"; mkdir -p "$DATA"
|
|
BIN="$WORK/m7"
|
|
export HOME="$WORK/home"; mkdir -p "$HOME" # never touch real ~/.neuron
|
|
# Hermetic: point the embedder at a guaranteed-refused endpoint so eg_embed_fetch
|
|
# fails fast, the circuit breaker opens, and cosq is deterministically absent in
|
|
# EVERY run (no dependence on whether a dev Ollama happens to be listening). This
|
|
# makes the byte-identical parity comparison reproducible and non-flaky.
|
|
export EL_EMBED_URL="http://127.0.0.1:1/api/embeddings"
|
|
unset ENGRAM_STORE
|
|
fail=0
|
|
|
|
echo "== compiling harness (gcc: el_runtime.c + engram_store.c + test_m7_traversal.c) =="
|
|
gcc -O2 -std=c11 -I "$INC" "$HERE/test_m7_traversal.c" "$RT" "$ST" -lcurl -lm -o "$BIN" 2>"$WORK/cc.log"
|
|
if [ $? -ne 0 ]; then echo "COMPILE FAILED:"; cat "$WORK/cc.log"; rm -rf "$WORK"; exit 1; fi
|
|
echo " ok: compiled"
|
|
|
|
echo
|
|
echo "== 1) PARITY: index-driven (M7 incremental) activation must be IDENTICAL to the"
|
|
echo " full-rebuild scan path — proven under one identical ENGRAM_STORE=1 state,"
|
|
echo " so the ONLY variable is how per-node adjacency is maintained."
|
|
echo " (compared on deterministic fields: node label + activation_strength +"
|
|
echo " working_memory_weight + epistemic_confidence + hops + promoted, IN ORDER;"
|
|
echo " node id/timestamps are per-run random and are intentionally excluded.)"
|
|
( unset ENGRAM_STORE; "$BIN" parity-off "$DATA" ) || { echo "FAIL: parity-off run"; fail=1; }
|
|
ENGRAM_STORE=1 "$BIN" parity-on-rebuild "$DATA" || { echo "FAIL: parity-on-rebuild run"; fail=1; }
|
|
ENGRAM_STORE=1 "$BIN" parity-on-incr "$DATA" || { echo "FAIL: parity-on-incr run"; fail=1; }
|
|
python3 - "$DATA" <<'PY' || fail=1
|
|
import json, sys, os
|
|
d = sys.argv[1]
|
|
def proj(prefix, i):
|
|
a = json.load(open(os.path.join(d, f"{prefix}_act{i}.json")))
|
|
out = []
|
|
for e in a:
|
|
n = e.get("node", {})
|
|
out.append([n.get("label",""),
|
|
e.get("activation_strength"), e.get("working_memory_weight"),
|
|
e.get("epistemic_confidence"), e.get("hops"), e.get("promoted")])
|
|
return out
|
|
def compare(label, pa, pb, gate):
|
|
rc = 0
|
|
for i in (1,2,3,4):
|
|
a, b = proj(pa, i), proj(pb, i)
|
|
if a == b:
|
|
print(f" #{i} identical (entries={len(a)}, promoted={sum(1 for r in a if r[5])})")
|
|
else:
|
|
if gate: rc = 1
|
|
print(f" #{i} DIFFERS ({'FAIL' if gate else 'note'})")
|
|
for x,y in zip(a,b):
|
|
if x != y:
|
|
print(f" first diff:\n {pa}={x}\n {pb}={y}"); break
|
|
if len(a) != len(b): print(f" length: {pa}={len(a)} {pb}={len(b)}")
|
|
print(f" {'PASS' if rc==0 else 'FAIL'}: {label}")
|
|
return rc
|
|
|
|
print(" [CORE M7 GATE] flag-on incremental index == flag-on forced full rebuild:")
|
|
rc1 = compare("index-driven activation == full-rebuild scan (same flag state)",
|
|
"onincr", "onrb", gate=True)
|
|
print(" [context] flag-on incremental index vs flag-off scan path (today's behavior):")
|
|
rc2 = compare("M7 (flag-on) == flag-off scan path", "onincr", "off", gate=False)
|
|
print(" [context] flag-off scan vs flag-on forced rebuild (isolates any pre-existing")
|
|
print(" flag-on/off float difference, INDEPENDENT of M7's incremental path):")
|
|
rc3 = compare("flag-off == flag-on (both rebuild path)", "off", "onrb", gate=False)
|
|
sys.exit(rc1) # only the core M7 equivalence gates the result
|
|
PY
|
|
|
|
echo
|
|
echo "== 2) PERF: ~13k nodes / 43k edges, 200 (add-edge + activate) iterations =="
|
|
NODES=13000; EDGES=43000; ITERS=120
|
|
( unset ENGRAM_STORE; "$BIN" perf off "$DATA" "$NODES" "$EDGES" "$ITERS" ) | tee "$WORK/perf_off.txt"
|
|
[ ${PIPESTATUS[0]} -ne 0 ] && { echo "FAIL: perf off"; fail=1; }
|
|
ENGRAM_STORE=1 "$BIN" perf on "$DATA" "$NODES" "$EDGES" "$ITERS" | tee "$WORK/perf_on.txt"
|
|
[ ${PIPESTATUS[0]} -ne 0 ] && { echo "FAIL: perf on"; fail=1; }
|
|
python3 - "$WORK/perf_off.txt" "$WORK/perf_on.txt" <<'PY'
|
|
import re, sys
|
|
def parse(f):
|
|
t = open(f).read()
|
|
def g(k):
|
|
m = re.search(k+r'=([\d.]+)', t); return float(m.group(1)) if m else 0.0
|
|
return {'rw': g('rebuild_edge_work'), 'rb': g('rebuilds'), 'ap': g('incr_appends'),
|
|
'loop_s': g('loop='), 'maint': g('adj_maint'),
|
|
'perq': g('per_query')}
|
|
off, on = parse(sys.argv[1]), parse(sys.argv[2])
|
|
def ratio(a,b): return (a/b) if b else float('inf')
|
|
print()
|
|
print(f" ADJACENCY TRAVERSAL COST (the metric M7 changes):")
|
|
print(f" edge-touches in rebuilds: off={off['rw']:.0f} on={on['rw']:.0f} "
|
|
f"({ratio(off['rw'],on['rw']):.0f}x fewer on)")
|
|
print(f" full O(E) rebuilds: off={off['rb']:.0f} on={on['rb']:.0f}")
|
|
print(f" incremental O(1) appends: off={off['ap']:.0f} on={on['ap']:.0f}")
|
|
print(f" adjacency-maint wall-time: off={off['maint']:.4f}s on={on['maint']:.4f}s "
|
|
f"({ratio(off['maint'],on['maint']):.1f}x faster on)")
|
|
print(f" END-TO-END per-query time: off={off['perq']:.2f}ms on={on['perq']:.2f}ms")
|
|
print(f" (per-query is dominated by activation's O(N) node scoring over 13k nodes,")
|
|
print(f" which M7 does not touch; the delta is the eliminated rebuild time.)")
|
|
ok = on['rw'] < off['rw'] and on['maint'] < off['maint'] and on['rb'] < off['rb']
|
|
print(" PASS: flag-on eliminates the O(E) per-query rebuild (fewer edge-touches, less maint time)"
|
|
if ok else " FAIL: expected fewer edge-touches AND less adjacency-maint time on flag-on")
|
|
sys.exit(0 if ok else 1)
|
|
PY
|
|
[ $? -ne 0 ] && fail=1
|
|
|
|
echo
|
|
echo "== 3) ASan+UBSan clean across parity + a small perf loop (leaks off — harness intentionally leaks el_strdup) =="
|
|
SANBIN="$WORK/m7.san"
|
|
gcc -O1 -g -std=c11 -fsanitize=address,undefined -fno-sanitize-recover=undefined \
|
|
-I "$INC" "$HERE/test_m7_traversal.c" "$RT" "$ST" -lcurl -lm -o "$SANBIN" 2>"$WORK/san_cc.log"
|
|
if [ $? -ne 0 ]; then echo " SAN COMPILE FAILED:"; tail -20 "$WORK/san_cc.log"; fail=1; else
|
|
export ASAN_OPTIONS=detect_leaks=0
|
|
D2="$WORK/data2"; mkdir -p "$D2"
|
|
( unset ENGRAM_STORE; "$SANBIN" parity-off "$D2" ) >/dev/null 2>"$WORK/san_run.log" && \
|
|
ENGRAM_STORE=1 "$SANBIN" parity-on-rebuild "$D2" >/dev/null 2>>"$WORK/san_run.log" && \
|
|
ENGRAM_STORE=1 "$SANBIN" parity-on-incr "$D2" >/dev/null 2>>"$WORK/san_run.log" && \
|
|
( unset ENGRAM_STORE; "$SANBIN" perf off "$D2" 1500 5000 40 ) >/dev/null 2>>"$WORK/san_run.log" && \
|
|
ENGRAM_STORE=1 "$SANBIN" perf on "$D2" 1500 5000 40 >/dev/null 2>>"$WORK/san_run.log"
|
|
if grep -qiE 'runtime error|AddressSanitizer|UndefinedBehavior|ERROR: ' "$WORK/san_run.log"; then
|
|
echo " FAIL: sanitizer findings:"; grep -iE 'runtime error|Sanitizer|ERROR' "$WORK/san_run.log" | head; fail=1
|
|
else
|
|
echo " ok: ASan+UBSan clean across parity + perf (rebuild + incremental append + BFS)"
|
|
fi
|
|
fi
|
|
|
|
echo
|
|
if [ "$fail" -eq 0 ]; then echo "================ M7 TRAVERSAL GATE: PASS ================"; else echo "================ M7 TRAVERSAL GATE: FAIL ================"; fi
|
|
rm -rf "$WORK"
|
|
exit $fail
|