fix(tradein/estimator): rebase ground-floor hedonic term onto current main (#2002, #2382) #2391

Merged

lekss361 merged 1 commit from fix/tradein-2382-hedonic-floor-rebase into main

2026-07-04 07:33:37 +00:00

lekss361 commented

2026-07-04 07:24:12 +00:00

Owner

Summary

Портирует единственный уникальный коммит из заброшенной ветки feat/tradein-hedonic-ground-floor (fe69d6e7) — floor==1 discount term в hedonic-композите, завершая пункт roadmap "year+floor+area" из #2002 (year+area уже в main). 356 коммитов с момента отвода ветки не тронули этот код-путь — перенос байт-в-байт идентичен оригиналу.
config.py — новое поле estimate_hedonic_first_floor_coef = -0.1248.
estimator.py — аддитивный _first term внутри math.exp(...), условен строго на floor==1, не ломает существующие call-sites (floor уже был опциональным keyword-параметром).
Тесты + перегенерированный hermetic backtest_baseline.json — дельты совпали байт-в-байт со значениями из старого коммита fe69d6e7.

Test plan

test_estimator_hedonic.py: 8/8 passed (включая 2 новых)
test_backtest_regression_gate.py: 2/2 passed (hermetic fixture replay)
Full -k "estimator or hedonic": 480 passed
Full backend suite: 3281 passed, 6 skipped, 1 pre-existing unrelated failure (confirmed on clean main)
ruff check — clean
code-review: ✅ APPROVE (диффнул против оригинального fe69d6e7 — byte-identical, независимо перепроверил формулу/condition/signature/baseline/тесты)
Live-DB backtest (--resolve-house-id) не прогнан — нет доступа к прод-БД из воркера; hermetic gate + unit-тесты покрывают merge-бар для byte-identical порта уже протестированной логики

Refs #2382, #2002

## Summary - Портирует единственный уникальный коммит из заброшенной ветки `feat/tradein-hedonic-ground-floor` (`fe69d6e7`) — floor==1 discount term в hedonic-композите, завершая пункт roadmap "year+floor+area" из #2002 (year+area уже в main). 356 коммитов с момента отвода ветки не тронули этот код-путь — перенос байт-в-байт идентичен оригиналу. - `config.py` — новое поле `estimate_hedonic_first_floor_coef = -0.1248`. - `estimator.py` — аддитивный `_first` term внутри `math.exp(...)`, условен строго на `floor==1`, не ломает существующие call-sites (`floor` уже был опциональным keyword-параметром). - Тесты + перегенерированный hermetic `backtest_baseline.json` — дельты совпали байт-в-байт со значениями из старого коммита `fe69d6e7`. ## Test plan - [x] `test_estimator_hedonic.py`: 8/8 passed (включая 2 новых) - [x] `test_backtest_regression_gate.py`: 2/2 passed (hermetic fixture replay) - [x] Full `-k "estimator or hedonic"`: 480 passed - [x] Full backend suite: 3281 passed, 6 skipped, 1 pre-existing unrelated failure (confirmed on clean main) - [x] `ruff check` — clean - [x] code-review: ✅ APPROVE (диффнул против оригинального `fe69d6e7` — byte-identical, независимо перепроверил формулу/condition/signature/baseline/тесты) - [ ] Live-DB backtest (`--resolve-house-id`) не прогнан — нет доступа к прод-БД из воркера; hermetic gate + unit-тесты покрывают merge-бар для byte-identical порта уже протестированной логики Refs #2382, #2002

Rows
Columns