Repository navigation
Plan the second workstream: memory accuracy - #17
Conversation
Replace plan.md with phases 6 to 13: the accuracy loop, the extractor, episodes behind facts, the ranking, supersede in capture, search over history, the full benchmark, and the PyPI release. The roadmap gets the new table and a Deferred list: adapters, latency, model calls, and the hindsight ideas. Commit the benchmark research of 2026-10-08 as a record, with a note that docs/benchmarks.md holds the observed numbers.
| The retrieval arm scored 45 of 54. The target is the published range of memory layers with | ||
| a GPT-4o reader, 71.2 to 84.2 percent (read, not reproduced), within 10 to 20 points. |
There was a problem hiding this comment.
nit: State the workstream target as one testable number (not blocking).
"The published range ... within 10 to 20 points" can be read in two ways. One reading is within 10 to 20 points of the range. The other reading is within 10 to 20 points of the retrieval arm. Phase 12 has no acceptance criterion that checks the target, so the close of the workstream has no pass condition. Example fix: "The target is at least N of 54 on sample A and sample B, and at least X percent on all 500 questions."
There was a problem hiding this comment.
Fixed in 9ed491b. The target now measures from the lowest published GPT-4o score, Zep at 71.2 percent (read, not reproduced): a pass level of at least 51.2 percent and a goal level of at least 61.2 percent on all 500 questions, with 28 and 33 of 54 as the guides for sample A and sample B. Phase 12 gains an acceptance criterion for the pass level and a stop condition for a miss.
The goal said "within 10 to 20 points" without saying of what. It now measures from Zep's 71.2 percent: a pass level of 51.2 percent and a goal of 61.2 percent on all 500 questions, with the matching counts of 54 for the samples. Phase 12 gains the check and a stop condition.
| | Level | Distance from 71.2 percent | Sample A and sample B | All 500 questions | | ||
| |---|---|---|---| | ||
| | Pass | At most 20 points | At least 28 of 54 on each | At least 51.2 percent | | ||
| | Goal | At most 10 points | At least 33 of 54 on each | At least 61.2 percent | |
There was a problem hiding this comment.
super nit: Change the goal guide to "At least 34 of 54" (not blocking).
33 of 54 is 61.1 percent, which is below the goal level of 61.2 percent. 34 of 54 is 63.0 percent.
This PR opens the second workstream, memory accuracy. It replaces
docs/internal/plan.mdwith phases 6 to 13, adds them to the roadmap, and commits the benchmark research of 2026-10-08. Phase 6 is #16. This PR changes no code.How
docs/internal/plan.md: the workstream goal, where it starts (the phase 5 numbers and four facts about the memory arm, read from the code), the proposed decisions, and for each phase its goal, tasks, acceptance criteria, and stop conditions. The phase 5 plan stays in git history at4a6fb36.docs/internal/roadmap.md: phases 0 to 5 become "the first roadmap", and a second table lists phases 6 to 13 as proposed. Deferred now lists the adapters (the old phase 6), the latency work, model calls in the library, and the hindsight ideas that this workstream does not take.docs/internal/benchmarks.md: the research that chose LongMemEval-S, committed as a record with a note at the top.docs/benchmarks.mdholds the observed numbers and wins where the two disagree.AGENTS.mdandREADME.md: the pointers toplan.mdanddocs/internal/name the workstream and the research.The phases, in order:
capturestores its text as an episode, andrecallcan add the cited or ranked episodes within one budgetremembercloses itrecall(as_of=T)returns the revision valid at T, with the filter in SQLhotdata-langchain, and the first releaseWhy
Phase 5 showed that the store finds the evidence: the top 10 sessions from search gave 45 of 54 correct answers. The memory contract gave 19 of 54. The losses come from the extractor and from what
recallreturns, not from the platform. This workstream works on accuracy first, keeps the library deterministic, and leaves latency to Deferred.Verification
make verifypasses, including the link check.plan.md. The hits left inroadmap.mdare in lines that this PR did not change.Notes
plan.md.