Repository navigation
feat: integrate lineup prototypes, replay timelines and organized project docs - #7
Conversation
Synchronize the reviewed local tree from ec3989096bda9eff341b563b5e50569e5f6938ab. Includes task-bound snapshot/model contracts, data workflow fixes, bounded collection and cohort analysis, tests, and documentation. Validation: Ruff lint and formatting passed; targeted tests 164 passed, 1 skipped. Full-suite Windows environment limitations are recorded in docs/task-contracts.md. No private memory, raw data, model outputs, or environment files included.
xrfc
left a comment
There was a problem hiding this comment.
审查提交:88991224d6bdd59ec6758300ab64d8401d5b6ccb。结论:暂不通过合并。
[P2] scripts/lineup_analysis.py:202-203(以及35、44、199行的读取和 scripts/lineup_comparison.py 同类读写):为文本输入输出显式指定 encoding="utf-8"。当前 Path.write_text 使用平台默认编码,但输出包含中文且 HTML 声明 UTF-8。在未启用 UTF-8 模式的 Windows/Python 3.11、3.12 环境下,cp1252 会抛 UnicodeEncodeError,GBK 则会写出不能按声明的 UTF-8 解码的页面。读取仓库 UTF-8 英雄资料也有同类风险。
复现:从上述精确提交用 AST 提取原始 esc/page 函数,生成非交互页面;cp1252 编码抛 UnicodeEncodeError,GBK 编码后 UTF-8 解码抛 UnicodeDecodeError。此为编码级复现,不冒充 Windows 原生端到端测试。
建议:统一 JSON/HTML 读写为 UTF-8,增加不依赖私有模型的输入/导出回归。目前 tests/test_lineup_analysis.py 与 tests/test_lineup_comparison.py 的 app fixture 在缺少 reports 私有模型时 skip,远端绿色不能证明这些原型路径通过。
已核验 PR CI run 40283579946 对应上述 head 且 success。审查重点包括模型制品/训练边界、来源读取、采集预算与封口、进程生命周期和阵容页面;本 PR 共359文件,发现阻塞项后未宣称逐行全量审完。未修改本地源码、未训练或采集、未合并。
xrfc
left a comment
There was a problem hiding this comment.
编码问题已在 e18f9ec 修复:两个脚本显式UTF-8,6项不依赖私有模型的回归在旧代码全部失败、修复后全部通过;Windows CI也执行新增回归。已继续审查模型制品/推理、划分与快照重放、规则推荐及证据边界等重点路径,未发现新的合并阻塞项。对应push及PR共6项checks均SUCCESS(PR run 40747189795)。本地全量1333 pytest及学习/知识/JS通过,但包含未提交功能,不等同于远端精确提交的测试数量。审查范围与局限见docs/pr-review-2026-10-29.md。本轮不改研究口径、不训练、不采集,用户原有未提交改动未上传。按用户授权合并该精确提交。
摘要(更新至2026-10-26)
统一交付
feat/purchase-data-prep→main。此PR包含此前尚未合入main的模块化、任务契约、整局实验和录像审计提交,以及最新8899122的近期功能与结构/文档整理;不是纯文档PR。当前结构与文档
近期功能
保留的研究边界
1990场冻结队列、1592/398开发划分与整局11装备任务不变。C=0.1宏Brier较频率改善2.28%仍只是开发集证据,无独立test或推荐收益结论。行为概率不等于最佳出装;录像不获训练准入,不根据少量反馈自动改标签/权重或重训。新增10场的英雄采样身份插桩认证、72个位移候选机制归因尚未完成。
本轮本地验证
scripts/check.py退出0:1205 pytest + 10学习 + 12知识 + 6 JS = 1233项通过,本地无跳过。远端CI以本PR最新head检查为准;上述为本地证据,不预先宣称CI成功。
发布范围与授权
用户本轮确认停写,由单一执行者整理、验收、提交、推送并更新本PR。只提交代码、测试、合成/手工案例和文档;不上传原始录像、玩家明细、模型、环境、memory或凭据。data/reports链接为本地证据,GitHub不能直接访问,重现需原冻结输入。
本轮没有安装依赖、重训、重采、合并main或创建Release;不关闭/合并旧PR #6。完整验收记录见
docs/validation-and-delivery.md的2026-10-26条目。最新远端CI终态
提交
88991224d6bdd59ec6758300ab64d8401d5b6ccb的push和pull_request两套检查均成功,共6/6 jobs:各含Linux Python 3.11、Python 3.12(含学习页面Chromium检查)及Windows portable。此浏览器检查不等于阵容原型页面视觉验收。PR保持开放,未合并。