Snider
|
0aadc92497
|
review: post-implementation findings for dual-class disambiguation
3 minor fixes required before PR:
- R1: Remove passed/failed/skipped from gram.noun (dead data)
- R2: Add tests for DisambiguationStats and WithWeights
- R3: Guard buildSignalIndex per-field for locale robustness
Core implementation approved — B1-B3, D1-D2, F1, F3 all correct.
Co-Authored-By: Virgil <virgil@lethean.io>
|
2026-02-19 16:50:18 +00:00 |
|
Snider
|
5d3558b383
|
docs: add orchestrator review of dual-class disambiguation plan
3 bugs (loader type assertion, noun entry verification, confidence
floor), 3 design improvements (contractions, clause boundaries,
negation), 4 Phase 2 feature requests (stats export, corpus priors,
weight tuning, expanded dual-class set).
Co-Authored-By: Virgil <virgil@lethean.io>
|
2026-02-19 15:50:51 +00:00 |
|