fix(tradein/estimator): city-scope ДКП-коридора + oblast city-tokens/bbox #2488

Merged

lekss361 merged 2 commits from fix/tradein-oblast-c2-estimator-scope into main

2026-07-12 17:21:56 +00:00

lekss361 commented

2026-07-12 17:11:39 +00:00

Owner

Что

Чинит доказанную кросс-город контаминацию ДКП-коридора: _fetch_dkp_corridor матчил сделки по street-name БЕЗ city → тянул одноимённые улицы ВСЕХ городов области. Пример (verified): «Нижний Тагил, Ленина» → коридор 171 сделка, но у Н.Тагила только 34 — остальное Лесной/Краснотурьинск/Серов/ЕКБ/Первоуральск. Медианы всех городов схлопывались в 56-60k.

Изменения

_resolve_target_city(address) — word-boundary поверх geocoder.SVERDLOVSK_OBLAST_CITIES (~30 городов, F), с district-guard («мкр Заречный» = микрорайон, не город-ЗАТО Заречный — единственная ЕКБ-коллизия).
_fetch_dkp_corridor: +param city → AND LOWER(d.city) = CAST(:target_city AS text). Проводка: estimate_quality (precedence dadata.canonical_address > geo.full_address) + trade_in.py GET rehydrate. city=None → прежнее поведение (fallback).
_CITY_TOKENS + _DEDUP_ADDR_NOISE_WORDS → derive из SVERDLOVSK_OBLAST_CITIES (было 6 EKB-агломерация городов) → same-house dedup работает для non-EKB.
bbox fast-path → geocoder.is_within_oblast66_bbox.
docstring исправлен: коридор НЕ чисто advisory — при n>=clamp_min_n(10) клампит headline + radius-floor + Tier-C anchor gate.

EKB инвариант

Regression-гейт байт-зелёный (коридор/токены upstream от frozen _price_from_inputs fixture replay — не гоняются в реплее). Deep-review проверил: EKB-адреса резолвятся корректно (F-коллизии Серова/Ирбитская/Асбестовский → екб, word-boundary rigor есть); SQL injection-safe (static-фрагмент + bind); perf ok (LOWER(city) — residual поверх уже street-фильтрованного rowset).

Проверка

deep-code-review: ⚠️ APPROVE, без 🔴/🟠 (2×🟡 de-risk применены: заречный-guard, docstring). ё-form: deals.city хранит без ё (Березовский/Артемовский) → не проблема.
Полный tradein backend suite: 2390 passed (+ pre-existing test_search_cache_hit). ruff clean.

Follow-ups (task-tracked)

DB-derived city-set (Лесной 1103 uncovered остаётся контаминирован — graceful fallback).
asking_to_sold_ratio.py city-scope (тот же класс бага).
backtest city-parity (backtest_estimator.py:1519 не передаёт city) → делать в D.

## Что Чинит **доказанную кросс-город контаминацию** ДКП-коридора: `_fetch_dkp_corridor` матчил сделки по street-name БЕЗ city → тянул одноимённые улицы ВСЕХ городов области. Пример (verified): «Нижний Тагил, Ленина» → коридор 171 сделка, но у Н.Тагила только 34 — остальное Лесной/Краснотурьинск/Серов/ЕКБ/Первоуральск. Медианы всех городов схлопывались в 56-60k. ## Изменения - `_resolve_target_city(address)` — word-boundary поверх `geocoder.SVERDLOVSK_OBLAST_CITIES` (~30 городов, F), с district-guard («мкр Заречный» = микрорайон, не город-ЗАТО Заречный — единственная ЕКБ-коллизия). - `_fetch_dkp_corridor`: +param `city` → `AND LOWER(d.city) = CAST(:target_city AS text)`. Проводка: estimate_quality (precedence `dadata.canonical_address > geo.full_address`) + trade_in.py GET rehydrate. `city=None` → прежнее поведение (fallback). - `_CITY_TOKENS` + `_DEDUP_ADDR_NOISE_WORDS` → derive из `SVERDLOVSK_OBLAST_CITIES` (было 6 EKB-агломерация городов) → same-house dedup работает для non-EKB. - bbox fast-path → `geocoder.is_within_oblast66_bbox`. - docstring исправлен: коридор **НЕ чисто advisory** — при `n>=clamp_min_n(10)` клампит headline + radius-floor + Tier-C anchor gate. ## EKB инвариант Regression-гейт байт-зелёный (коридор/токены upstream от frozen `_price_from_inputs` fixture replay — не гоняются в реплее). Deep-review проверил: EKB-адреса резолвятся корректно (F-коллизии `Серова`/`Ирбитская`/`Асбестовский` → екб, word-boundary rigor есть); SQL injection-safe (static-фрагмент + bind); perf ok (LOWER(city) — residual поверх уже street-фильтрованного rowset). ## Проверка - deep-code-review: ⚠️ **APPROVE**, без 🔴/🟠 (2×🟡 de-risk применены: заречный-guard, docstring). ё-form: deals.city хранит без ё (Березовский/Артемовский) → не проблема. - Полный tradein backend suite: 2390 passed (+ pre-existing `test_search_cache_hit`). ruff clean. ## Follow-ups (task-tracked) - DB-derived city-set (Лесной 1103 uncovered остаётся контаминирован — graceful fallback). - `asking_to_sold_ratio.py` city-scope (тот же класс бага). - backtest `city`-parity (backtest_estimator.py:1519 не передаёт city) → делать в D.

Rows
Columns