Fix CLS clustering: include temporal_next edges in cluster building
_build_hebbian_clusters() only used hebbian edges, but 97% of hebbian edges are self-loops from reinforce_record(). Only 7 are cross-record. temporal_next edges (140) provide the cross-record connectivity needed to form clusters >= CLUSTER_MIN_SIZE. Without them, consolidation never creates summaries, no consolidated_from edges exist, and the graph remains fragmented with 514 isolated nodes and max CC=58.
This commit is contained in:
parent
1b7c89de3b
commit
2cab42355f
1 changed files with 10 additions and 4 deletions
|
|
@ -296,16 +296,22 @@ def run_light_consolidation(
|
|||
|
||||
|
||||
def _build_hebbian_clusters(store: MemoryStore) -> list[list[UUID]]:
|
||||
"""Find connected components in the hebbian edge graph with size >= CLUSTER_MIN_SIZE."""
|
||||
"""Find connected components in the hebbian+temporal_next edge graph
|
||||
with size >= CLUSTER_MIN_SIZE.
|
||||
|
||||
Hebbian edges capture semantic similarity (dedup/reinforce);
|
||||
temporal_next edges capture sequential proximity (same-session inserts
|
||||
within 5 minutes). Both are valid signals for CLS clustering.
|
||||
"""
|
||||
edges_df = store.db.open_table(EDGES_TABLE).to_pandas()
|
||||
if edges_df.empty:
|
||||
return []
|
||||
hebbian = edges_df[edges_df["edge_type"] == "hebbian"]
|
||||
if hebbian.empty:
|
||||
relevant = edges_df[edges_df["edge_type"].isin(("hebbian", "temporal_next"))]
|
||||
if relevant.empty:
|
||||
return []
|
||||
|
||||
adj: dict[UUID, set[UUID]] = {}
|
||||
for _, row in hebbian.iterrows():
|
||||
for _, row in relevant.iterrows():
|
||||
src = UUID(row["src"])
|
||||
dst = UUID(row["dst"])
|
||||
adj.setdefault(src, set()).add(dst)
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue