Repository navigation
Precision before recall: an opt-in injection match floor (story 28.5) - #222
Merged
Merged
Conversation
…ck (story 28.5) ExperienceInjectionOptions.MinimumMatch (off by default) omits a ranked record whose text coverage, semantic similarity and signature match are all below the floor, as WeakMatch, before the record limit. Retrieval now carries each channel's strength (TermCoverage, TextCoverage, VectorRelevance); the retrieval benchmark sweeps the floor. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01GPXz56QPQAzDSyJTxbQTTy
…ory 28.5) Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01GPXz56QPQAzDSyJTxbQTTy
…ut of doc comments Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01GPXz56QPQAzDSyJTxbQTTy
… review) The semantic leg is off without a semantic floor; MinimumTextCoverage is required with no default; recall skips the coarse lookup under a floor; SQLite reports TermCoverage and its golden gains the sweep; the sweep's zero row is pinned to the overall metrics; coverage is documented as store-relative. CHANGELOG: the 28.x sections move back under Unreleased (none shipped in preview.17). Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01GPXz56QPQAzDSyJTxbQTTy
A search computes termCoverage; a published preview never returned it, so only its presence is checked. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01GPXz56QPQAzDSyJTxbQTTy
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Story 28.5 of Epic 28. The real-task experiment showed a mismatched lesson costs more than none, so the block needs a way to leave weak matches out.
ExperienceInjectionOptions.MinimumMatch(ExperienceMatchFloor, opt-in): a record is injected only on an exact failure-signature match (recall), text coverage at or aboveMinimumTextCoverage(required, store-relative), or semantic similarity at or above a semantic floor (no floor means the semantic leg is off: fail closed). Weak records are omitted asWeakMatchbefore the capability gate and record limit, with backfill, and before the holdout split.ExperienceCandidate.TermCoverage,RankedExperience.TextCoverage/VectorRelevance) from the in-memory, PostgreSQL (including the RLS search function) and SQLite sources; ranking unchanged.Tests: Core 1349, MAF 1458, SQLite 245, Storage.Postgres 965 (+ RLS-filtered 115), RetrievalQuality 30 with PostgreSQL, package validation.
🤖 Generated with Claude Code
https://claude.ai/code/session_01GPXz56QPQAzDSyJTxbQTTy