54 lines
2.3 KiB
Markdown
54 lines
2.3 KiB
Markdown
# CODE_REVIEW — ECR-004
|
||
|
||
**Role:** REVIEWER
|
||
**Date:** 2026-08-06
|
||
**Scope:** `d3188ca`（相对 ECR-003）威科夫硬化
|
||
**Decision:** Approve
|
||
|
||
## Evidence loaded
|
||
|
||
- Diff `d3188ca`：`range.py` / `events.py` / `analyze.py` / `chart_tv.js` / tests / ESS
|
||
- 复跑：`tests/test_wyckoff.py` + golden + analyze contract → **14 passed**
|
||
- 合成夹具抽查：`abs_start_idx=20`，low/high≈40.1/59.9（相对 003 的 bar0 已修好）
|
||
|
||
## Acceptance ↔ Evidence
|
||
|
||
| Acceptance | Verdict | Evidence |
|
||
|------------|---------|----------|
|
||
| TR 不吞明显前置趋势；边界近箱体 | PASS | 评分选段；单测 low/high 带 + `abs_start≥12` + start 时间容差 |
|
||
| VP series 减负 | PASS | Top-8 + 填充 3 + POC/VAH/VAL；API bins≤24 |
|
||
| 阶段最小跨度 / 不重合 | PASS | 链式 cursor；unique (start,end) 断言 |
|
||
| elements_only 门闩 | PASS | `include_wyckoff and not elements_only` + 契约测试 |
|
||
| golden 不变 / 无策略改动 / 无币种表 | PASS | golden 绿；diff 无 config/strategies |
|
||
|
||
## Findings
|
||
|
||
### Medium（不挡 Approve）
|
||
|
||
1. **同分 tie-break 偏向更长窗口**
|
||
循环从长到短，`score <= best_score` 时保留已有（更长）。多数情况分数拉开；若实盘出现「长窗与短窗同分」，仍可能略偏长。可选：同分取更短，或加 `1/length` 微项。
|
||
|
||
2. **阶段常截断为 A–C**
|
||
Spring/SOS 落在尾部时 D/E 因 `min_span` 被吃掉——与 ENG「空间不足截断」一致，但 UI 勾选「阶段」时用户可能期望总见 D/E。属产品预期，非缺陷；可在 UI/文档标明「尾部不足则省略」。
|
||
|
||
### Low
|
||
|
||
3. **`abs_start_idx >= 12` 弱于「箱体起点」** —— 主测已用时间容差；该断言可再收紧到 `>= 16` 一类。
|
||
4. **VP Top-N 无自动化 series 计数** —— 靠代码审查 + ENG 约定。
|
||
5. 事件 marker 仍一律 `arrowUp`（003 遗留）。
|
||
6. 失败路径仍回传 `wyckoff.error` 字符串。
|
||
|
||
### No blockers
|
||
|
||
未发现契约删键、缠论语义改动、策略改动、或回归红灯。
|
||
|
||
## Decision
|
||
|
||
**Approve**
|
||
|
||
ECR-004 可维持 Done (Reviewed)。Medium 项进 backlog，不必立刻新 ECR，除非实盘 TR 仍偏长。
|
||
|
||
## Next owner
|
||
|
||
Human — 主站 BTC 勾选威科夫目测；无发版要求则保持 `v1.0.0` Unreleased 累计。
|