 shangfangjian
|
06e973d34d
|
fix(concurrency): 消除 model 覆盖的模块级变量中转(并发串味)
R4 首版实现用模块级 _ACTIVE_MODEL_OVERRIDE 中转 model override,
理由是不想改动 run_specialists 的签名(既有测试会 mock 它)。
但 backtest.py:225 会 asyncio.gather 并发 8 场预测(Semaphore(8)),
每场都调用 predict_match_multi —— 模块级变量会被并发调用互相覆盖,
导致 A 场的预测用上 B 场的模型。这是静默的正确性缺陷。
改为: run_specialists 新增 model_override 形参,一路显式下传;
删除模块级变量。同步更新 6 处 mock 签名。
新增 3 个守卫并做变异验证:
- test_r4_no_module_level_model_override_global (源码级,可判别)
- test_r4_run_specialists_accepts_model_override_parameter
- test_r4_dispatch_passes_override_to_specialists
变异测试: 重新引入全局变量方案 → 两个守卫变红;还原 → 全绿。
注: 曾尝试写并发行为测试,但真实 run_specialists 会访问数据库,
测试环境下不稳定(ConnectionRefusedError),会是 flaky 的假证据,
故改用清晰的源码级判别守卫,并在注释中说明原因。
240 passed / 1 skipped; tsc 0 error; vite build 成功。
|
2026-09-21 17:52:45 +08:00 |
|
|
shangfangjian
|
11da4d3631
|
fix(critical): 修复评审确认的 R1-R5 五处缺陷
R1 429 key 轮换路径调用不存在的 _km → NameError(且位于凭证脱敏日志行):
改用 src/data/key_ring._mask,全树不再有 _km 调用。
[注: 该改动已随另一工作流提交 9ccda4b 一并入库]
R2 ingest_bzzoiro_standings 被截断(return 前无 upsert 逻辑,standings 表
永不写入、total_upserted 恒为 0):从 f05dc1a 移植完整实现,按
(league_id, season, team_id) upsert,保留逐联赛错误隔离,不加 db.commit()。
[注: 同上,已随 9ccda4b 入库]
R3 src/llm/agents/orchestrator.py 完成日志把 list 喂给 %d → logging TypeError:
改为 len(ok_reports),与同文件降级日志行写法一致。
R4 mode=multi 静默丢弃调用方传入的 model:predict_match_multi 新增
model 关键字参数,经 _ACTIVE_MODEL_OVERRIDE 下传至 _agent_provider,
显式 model 优先级最高;override 生效时跳过 provider 缓存读写以免串味,
并在 predict.py 派发点透传。
R5 回测把字符串日期直接与 timestamptz 列比较:新增 _parse_date_bound 助手,
支持 YYYY-MM-DD / 完整 ISO / datetime / None,裸日期按 UTC 锚定,
结束日取当天末刻(闭区间,避免最后一天被静默排除),非法输入抛 ValueError。
新增 tests/test_review_required_fixes.py 覆盖 R1-R5(R2/R4 为行为测试),
20 项全通过。
|
2026-09-21 17:19:24 +08:00 |
|