chore(tradein/estimator): схлопнуть same-building-anchor + IMV-blend tuning флаги (#651/#652/#680/#1774) — #1970 #2381

Open
opened 2026-07-04 06:35:09 +00:00 by lekss361 · 0 comments
Owner

Родитель: #1970. Precondition (#1966) закрыт.

19 полей — самый большой и самый рискованный кластер, core valuation logic (55 golden cases validated): estimate_imv_blend_enabled/_weight/_threshold (#651), estimate_same_building_anchor_enabled (#652/#680), estimate_sb_min_comps/_area_sigma/_rooms_match_boost/_floor_sigma/_guardrail_tol/_mad_k/_mad_k_small_n/_small_n_threshold, estimate_sb_tier_a_allow_primary_if_secondary_present (#1774), estimate_sb_low_conf_gate_enabled/_gate_min_n/_gate_max_fsd (audit-1), estimate_sb_clip_after_weight (audit-4), estimate_fsd_k, estimate_anchor_tier_c_corridor_mult.

Files: tradein-mvp/backend/app/services/estimator.py (~1520-2266, 2336-2403, 3168)

ВНИМАНИЕ (delegation.md): 19 полей в core-файле — вероятно L-размер (>500 строк diff). Implementer ДОЛЖЕН оценить перед стартом и при превышении бюджета разбить дальше, например: (a) IMV-blend removal (3 поля, проще, отдельно от anchor) и (b) same-building anchor tuning (16 полей, сложнее, core).

Acceptance: full pytest green, test_backtest_regression_gate.py метрики bit-identical, mandatory deep-code-reviewer APPROVE перед merge (core valuation engine — тот же бар, что и Group E/estimator.py в scraper_kit migration).

Risk: HIGH.

Родитель: #1970. Precondition (#1966) закрыт. 19 полей — самый большой и самый рискованный кластер, core valuation logic (55 golden cases validated): `estimate_imv_blend_enabled`/`_weight`/`_threshold` (#651), `estimate_same_building_anchor_enabled` (#652/#680), `estimate_sb_min_comps`/`_area_sigma`/`_rooms_match_boost`/`_floor_sigma`/`_guardrail_tol`/`_mad_k`/`_mad_k_small_n`/`_small_n_threshold`, `estimate_sb_tier_a_allow_primary_if_secondary_present` (#1774), `estimate_sb_low_conf_gate_enabled`/`_gate_min_n`/`_gate_max_fsd` (audit-1), `estimate_sb_clip_after_weight` (audit-4), `estimate_fsd_k`, `estimate_anchor_tier_c_corridor_mult`. **Files:** `tradein-mvp/backend/app/services/estimator.py` (~1520-2266, 2336-2403, 3168) **ВНИМАНИЕ (delegation.md):** 19 полей в core-файле — вероятно L-размер (>500 строк diff). Implementer ДОЛЖЕН оценить перед стартом и при превышении бюджета разбить дальше, например: (a) IMV-blend removal (3 поля, проще, отдельно от anchor) и (b) same-building anchor tuning (16 полей, сложнее, core). **Acceptance:** full pytest green, `test_backtest_regression_gate.py` метрики bit-identical, mandatory `deep-code-reviewer` APPROVE перед merge (core valuation engine — тот же бар, что и Group E/estimator.py в scraper_kit migration). **Risk:** HIGH.
lekss361 added the
chore
scope/backend
tech-debt
tradein
labels 2026-07-04 10:23:48 +00:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: lekss361/gendesign#2381
No description provided.