M3) #2387

Merged

lekss361 merged 1 commit from fix/tradein-2087-ratio-reconciliation into main

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

lekss361 commented

2026-07-04 06:44:30 +00:00

Owner

Summary

M2 (asking_to_sold_ratio рассинхрон с медианами) — расследование показало, что root-cause фикс уже смержен в PR #2152/#2141 (honest-ratio recompute после всех mutations). Найден непокрытый gap: segment-multiplier (#2255) мутирует обе стороны ПОСЛЕ honest-ratio recompute с независимым округлением — добавлен regression-guard тест test_honest_ratio_invariant_under_segment_multiplier.
M3 (несовместимые базы «торга») — median_bargain_pct (house-level, все лоты, без sold-фильтра и без ограничения по давности) и asking_to_sold_ratio (estimate-level) — генуинно разные метрики без честной общей базы. Явные cross-referencing docstrings вместо слияния.
Docstring HouseAnalyticsKpi.median_bargain_pct поправлен после code-review (ранее ошибочно утверждал "снятые лоты"/"12 мес" — реальный запрос без sold-фильтра и без date-range).

Test plan

tests/test_estimator_price_spine.py + test_estimator_segment_multiplier.py + test_estimator_expected_sold*.py + test_estimator_dedup_cross_source_2087.py: 93 passed
Full -k estimator: 478 passed
Full backend suite: 3279 passed, 6 skipped, 1 pre-existing unrelated failure (test_search_cache_hit, confirmed pre-existing on clean main)
ruff check — clean
code-review: ⚠️MINOR → docstring inaccuracy пофикшена (см. выше)

Refs #2087

## Summary - M2 (asking_to_sold_ratio рассинхрон с медианами) — расследование показало, что root-cause фикс уже смержен в PR #2152/#2141 (honest-ratio recompute после всех mutations). Найден непокрытый gap: segment-multiplier (#2255) мутирует обе стороны ПОСЛЕ honest-ratio recompute с независимым округлением — добавлен regression-guard тест `test_honest_ratio_invariant_under_segment_multiplier`. - M3 (несовместимые базы «торга») — `median_bargain_pct` (house-level, все лоты, без sold-фильтра и без ограничения по давности) и `asking_to_sold_ratio` (estimate-level) — генуинно разные метрики без честной общей базы. Явные cross-referencing docstrings вместо слияния. - Docstring `HouseAnalyticsKpi.median_bargain_pct` поправлен после code-review (ранее ошибочно утверждал "снятые лоты"/"12 мес" — реальный запрос без sold-фильтра и без date-range). ## Test plan - [x] `tests/test_estimator_price_spine.py` + `test_estimator_segment_multiplier.py` + `test_estimator_expected_sold*.py` + `test_estimator_dedup_cross_source_2087.py`: 93 passed - [x] Full `-k estimator`: 478 passed - [x] Full backend suite: 3279 passed, 6 skipped, 1 pre-existing unrelated failure (`test_search_cache_hit`, confirmed pre-existing on clean main) - [x] `ruff check` — clean - [x] code-review: ⚠️MINOR → docstring inaccuracy пофикшена (см. выше) Refs #2087

Rows
Columns