fix(tradein/estimator): merge orphaned floor term for hedonic composite (#2002 roadmap item 1) #2382

New issue

Open

opened 2026-07-04 06:35:48 +00:00 by lekss361 · 0 comments

lekss361 commented

2026-07-04 06:35:48 +00:00

Owner

Ветка feat/tradein-hedonic-ground-floor (tip fe69d6e7) полностью реализует floor==1 discount в hedonic-композите (#2002 item 1: "year+floor+area"), но никогда не мержена — 356 коммитов позади текущего main (git merge-base --is-ancestor fe69d6e7 main → NO).

Это не черновик — готовая, протестированная реализация (estimate_hedonic_first_floor_coef = -0.1248, fit на n=2366 прод-выборке, той же что year+area), с unit-тестами: test_ground_floor_applies_extra_discount, test_non_ground_floors_unchanged_vs_year_area_factor (tests/test_estimator_hedonic.py).

Scope

Ребейз feat/tradein-hedonic-ground-floor на текущий main.
Ревалидация против изменений hedonic-пайплайна, появившихся ПОСЛЕ отвода ветки: le_asking re-clamp, calibrated PI (#1966), elite-segment manual_review (#2002 #4), honest asking_to_sold_ratio (#2087 H4).
Перегенерация backtest_baseline.json под текущие прод-данные.
Ре-ран scripts/backtest_estimator.py --resolve-house-id — подтвердить реальный accuracy-выигрыш (не только unit-тесты), убедиться что floor≠1 юниты не регрессируют.
Обычный branch+PR flow, review + merge.

Acceptance

Существующие тесты (test_ground_floor_applies_extra_discount, test_non_ground_floors_unchanged_vs_year_area_factor) проходят на ребейзнутой ветке.
test_backtest_regression_gate.py / полный backtest не регрессирует floor≠1 сегмент.

Референсы

tradein-mvp/backend/app/services/estimator.py:2610-2664 (hedonic factor без floor сейчас)
tradein-mvp/backend/app/core/config.py:123-128, 264-268
Orphaned branch: feat/tradein-hedonic-ground-floor @ fe69d6e7 (присутствует во всех remotes)

Ветка `feat/tradein-hedonic-ground-floor` (tip `fe69d6e7`) полностью реализует floor==1 discount в hedonic-композите (#2002 item 1: "year+floor+area"), но никогда не мержена — 356 коммитов позади текущего main (`git merge-base --is-ancestor fe69d6e7 main` → NO). Это не черновик — готовая, протестированная реализация (`estimate_hedonic_first_floor_coef = -0.1248`, fit на n=2366 прод-выборке, той же что year+area), с unit-тестами: `test_ground_floor_applies_extra_discount`, `test_non_ground_floors_unchanged_vs_year_area_factor` (`tests/test_estimator_hedonic.py`). ## Scope 1. Ребейз `feat/tradein-hedonic-ground-floor` на текущий `main`. 2. Ревалидация против изменений hedonic-пайплайна, появившихся ПОСЛЕ отвода ветки: le_asking re-clamp, calibrated PI (#1966), elite-segment manual_review (#2002 #4), honest asking_to_sold_ratio (#2087 H4). 3. Перегенерация `backtest_baseline.json` под текущие прод-данные. 4. Ре-ран `scripts/backtest_estimator.py --resolve-house-id` — подтвердить реальный accuracy-выигрыш (не только unit-тесты), убедиться что floor≠1 юниты не регрессируют. 5. Обычный branch+PR flow, review + merge. ## Acceptance - Существующие тесты (`test_ground_floor_applies_extra_discount`, `test_non_ground_floors_unchanged_vs_year_area_factor`) проходят на ребейзнутой ветке. - `test_backtest_regression_gate.py` / полный backtest не регрессирует floor≠1 сегмент. ## Референсы - `tradein-mvp/backend/app/services/estimator.py:2610-2664` (hedonic factor без floor сейчас) - `tradein-mvp/backend/app/core/config.py:123-128, 264-268` - Orphaned branch: `feat/tradein-hedonic-ground-floor` @ `fe69d6e7` (присутствует во всех remotes)