Record that the scale work passed independent review on round five
This commit is contained in:
1 parent
6869d0956f
commit
8b974e6626
1 file changed
+1
@@ -364,6 +364,7 @@
|
||||
- 数字逼出来的决定:**dense 腿默认关**(`config.py:32 dense_embeddings_enabled=False`)。同一 300 条 holder 作用域探针上(LEGS.md),开腿 recall@10 0.832→0.845、MRR 0.683→0.663、p50 67.5ms→474.8ms。多 1.3 个点的召回、慢 7 倍、首位质量还降——默认值就该是关,腿留着(已增量、已入库)等更大的语料或更好的嵌入模型。
|
||||
- 门禁:`pytest -q` 全绿(**287 passed**);`ruff check` 干净;mypy 冷缓存 **24 errors / 9 files**(命令 `.venv/bin/python -m mypy --cache-dir <新目录> backend/src bench`),与 p1-6 基线(`9cb46c1` 同口径 24/9)**净新增 0**——p2-1 一度引入 +2(`vector_index.py` 与 `quant_probe.py` 的 `import sqlite_vec` 没有 stub),根因修法是在 `pyproject.toml` 的 `[[tool.mypy.overrides]]` 里声明 `sqlite_vec` 无 stub,而不是在代码里加 `# type: ignore`;修在 **`71733d8`**,所以 `6cc5640` 那棵树上量到的是 26/11(review 第一轮量的就是那棵树);`./node_modules/.bin/pyright` 对 p2-1 范围内 **29 个 `.py`** → **0 errors**;同范围(`9cb46c1..fe538ac`,共 48 个文件)还改了 3 个前端文件(`frontend/src/components/MemoryPanel.tsx`、`frontend/src/components/SensesPanel.tsx`、`frontend/src/types.ts`)和 1 个 `.go`(`ely-eye-mcp/internal/domain/types.go`)——前端由 `npm run build`(220.81 kB)+ `npm run lint` exit 0 覆盖,go 由 `go build` / `go vet` / `go test`(ok)+ `gofmt -l` 空覆盖,四条都在 `fe538ac` 上重跑过(pyright 直接吃 `.ts/.tsx` 会报两千多条,那是没 tsconfig 上下文的噪声,不是门禁);`bench/filesize_check` 干净。
|
||||
- 顺带修的测量缺陷:`copy_store()` 只拷 SQLite,所以任何要跑视觉腿的 harness 都得自己带上 `objects/`(25 GB,探针里用符号链接)与 `cache/clip_image_index.npy`;`bench/scale_check.py` 原本直接打开冻结基线库、把它**原地迁移**了,现改成在副本上测(`--scratch`)。产品 CLI 不在这条规则里:p2-1.4 建图片索引时 `ely-eye index-visual` 就把冻结基线从 schema 7 升到了 8,所以 p1-4.8 那两条钉的是**当前**的 mtime / sha256 / schema,并指名是哪一步改的。
|
||||
- **五轮评审**(同一个独立评审 pi,只读+自己跑):第 1 轮 FAIL 7 条(D1–D7:CHECKLIST 引了报告里不存在的数字、腿 harness 没交代向量与图片索引从哪来、多行注释、mypy +2、外推分母不一致);第 2 轮 FAIL 7 条(D8–D14:一次样本写成区间、stamp 引错提交、跨页做差、极值抄窄、「三跑」其实只有两份、两条没测过的机制断言、门禁把前端算进 pyright 还说 go/npm 未动);第 3 轮我反驳的三条(测试数、mypy 基线、pyright 文件集)经评审复核成立,但 D8–D13 未动;第 4 轮 D8–D14 逐条跑验通过、只剩 D15(分模态混页做差);第 5 轮在 `6869d09` **PASS,零缺陷**。评审自己跑的结果:`pytest -q` 287、`bench.filesize_check`、`bench.audit`(31 harness / 2 报告 / 6 集合,clean)、ruff、go 全套、`npm run build` 220.81 kB + lint exit 0,并把 p2-1 全节每个数字对 `leg_check.json` / `QUANT.md` / `DENSE.md` / `SCALE.md` / `CURRENT.md` 逐格对上。教训写进习惯:引用负载相关数字必须给区间或指向报告;一个区间要有两份留档样本;分模态只能同页比;没留档的计时只能当旁证。
|
||||
- [ ] **p2-2** 安全与隐私
|
||||
- [ ] p2-2.1 FastAPI 鉴权 + Origin 校验
|
||||
- [ ] p2-2.2 写入信任分级与隔离:未佐证不得被引用
|
||||
|
||||
Reference in new issue
Block a user