fix(tradein/estimator): non-EKB headline из deals + geo-bound Tier-S analog leak #2492

Merged

lekss361 merged 1 commit from fix/tradein-oblast-nonekb-headline into main

2026-07-12 18:49:09 +00:00

lekss361 commented

2026-07-12 18:47:11 +00:00

Owner

Критическая находка (валидация D)

Не-ЕКБ headline over-priced: analog Tier-S (same-building) fallback-запрос был БЕЗ geo-предиката (WHERE address ILIKE :short_addr_prefix); _extract_short_addr роняет город → prefix «улица Ленина, 5» коллизит с той же улицей+домом в ДРУГОМ городе. Т.к. 40 191 листингов ЕКБ / ~0 не-ЕКБ → любая коллизия резолвится в далёкий ЕКБ-листинг с distance_m=0. Серов ask-head 186k vs sold 30k (6×); Н.Тагил 187k vs deal-corridor 89k. ЕКБ работает (backtest coverage 92.3%). deal_median локально точен везде (Н.Тагил 85 911, Серов 30 303).

Фикс

Part 1 — geo-bound Tier-S (~4624): + ST_DWithin(geom::geography, ST_MakePoint(:lon,:lat)::geography, :radius) в address-prefix fallback (зеркалит Tier-W). >99.97% листингов с coords → режет только cross-city false-positives, легитимные same-building ЕКБ-хиты не трогает.
Part 2 — deals-headline-fallback: (a) _fetch_dkp_corridor widen до city-wide (без street-фильтра) при street-sample < 3 сделок (gated city != екатеринбург); (b) _price_from_inputs: при median_ppm2 <= 0 (нет листингов) + anchor_tier is None + dkp_raw >= 3 сделок → headline из deal-corridor (локальная ground-truth), confidence="low", n_analogs=0, expected_sold=None. Н.Тагил → 85 911 вместо 187k/пусто.

Проверка

deep-code-review: ✅ APPROVE (no 🔴/🟠; ~10 downstream mutation-стадий traced — coherent, no crash path).
EKB byte-green regression-гейт: fallback требует median_ppm2<=0 (невозможно для frozen fixture, capture фильтрует median_price>0); Part 1 SQL не гоняется в replay. 71 passed байт-идентично.
Full suite 2418 passed (+ pre-existing test_search_cache_hit). ruff clean.
Новый тест: Н.Тагил 0-листингов+corridor85911 → headline=85 911 (не 0/186k), conf=low, n_analogs=0; EKB-control dense листинги+dkp_raw → headline из листингов (fallback НЕ срабатывает).

Post-deploy verify (rule #7, я прогоню)

EKB backtest --city екатеринбург — Part 1 не сдвинул EKB coverage (frozen-гейт это не видит).
Live smoke Н.Тагил (→~85k) + Серов + EKB.

Follow-ups (🟡, task #10/#11)

Part-2b fallback без city-gate: sparse EKB (0 in-radius листингов + ≥3 street-deals) → n/a→deals (safe, только заполняет пустое).
api_analog_tier мислейбл «city» для deals-fallback (cosmetic, schema-контракт).
Part1/2a integration-тест (текущие 4 теста мокают SQL).

Замыкает не-ЕКБ точность на данных, что уже есть (97k сделок) — независимо от infra-blocked скрейпа (avito banned даже для ЕКБ).

## Критическая находка (валидация D) Не-ЕКБ headline **over-priced**: analog Tier-S (same-building) fallback-запрос был **БЕЗ geo-предиката** (`WHERE address ILIKE :short_addr_prefix`); `_extract_short_addr` роняет город → prefix «улица Ленина, 5» коллизит с той же улицей+домом в ДРУГОМ городе. Т.к. 40 191 листингов ЕКБ / ~0 не-ЕКБ → любая коллизия резолвится в далёкий ЕКБ-листинг с `distance_m=0`. **Серов** ask-head 186k vs sold 30k (6×); **Н.Тагил** 187k vs deal-corridor 89k. ЕКБ работает (backtest coverage 92.3%). deal_median локально точен везде (Н.Тагил 85 911, Серов 30 303). ## Фикс - **Part 1 — geo-bound Tier-S** (`~4624`): `+ ST_DWithin(geom::geography, ST_MakePoint(:lon,:lat)::geography, :radius)` в address-prefix fallback (зеркалит Tier-W). >99.97% листингов с coords → режет только cross-city false-positives, легитимные same-building ЕКБ-хиты не трогает. - **Part 2 — deals-headline-fallback**: (a) `_fetch_dkp_corridor` widen до **city-wide** (без street-фильтра) при street-sample `< 3` сделок (gated `city != екатеринбург`); (b) `_price_from_inputs`: при `median_ppm2 <= 0` (нет листингов) + `anchor_tier is None` + `dkp_raw >= 3` сделок → headline из **deal-corridor** (локальная ground-truth), `confidence="low"`, `n_analogs=0`, `expected_sold=None`. Н.Тагил → **85 911** вместо 187k/пусто. ## Проверка - **deep-code-review: ✅ APPROVE** (no 🔴/🟠; ~10 downstream mutation-стадий traced — coherent, no crash path). - **EKB byte-green** regression-гейт: fallback требует `median_ppm2<=0` (невозможно для frozen fixture, capture фильтрует `median_price>0`); Part 1 SQL не гоняется в replay. 71 passed байт-идентично. - Full suite **2418 passed** (+ pre-existing `test_search_cache_hit`). ruff clean. - Новый тест: Н.Тагил 0-листингов+corridor85911 → headline=85 911 (не 0/186k), conf=low, n_analogs=0; EKB-control dense листинги+dkp_raw → headline из листингов (fallback НЕ срабатывает). ## Post-deploy verify (rule #7, я прогоню) - EKB backtest `--city екатеринбург` — Part 1 не сдвинул EKB coverage (frozen-гейт это не видит). - Live smoke Н.Тагил (→~85k) + Серов + EKB. ## Follow-ups (🟡, task #10/#11) - Part-2b fallback без city-gate: sparse EKB (0 in-radius листингов + ≥3 street-deals) → n/a→deals (safe, только заполняет пустое). - `api_analog_tier` мислейбл «city» для deals-fallback (cosmetic, schema-контракт). - Part1/2a integration-тест (текущие 4 теста мокают SQL). Замыкает не-ЕКБ точность на данных, что уже есть (97k сделок) — независимо от infra-blocked скрейпа (avito banned даже для ЕКБ).

lekss361 added 1 commit 2026-07-12 18:47:12 +00:00

fix(tradein/estimator): non-EKB headline из deals + geo-bound Tier-S analog leak

CI Trade-In / changes (pull_request) Successful in 11s

Details

CI / changes (pull_request) Successful in 11s

Details

CI / backend-tests (pull_request) Has been skipped

Details

CI / frontend-tests (pull_request) Has been skipped

Details

CI / openapi-codegen-check (pull_request) Has been skipped

Details

CI Trade-In / backend-tests (pull_request) Successful in 1m3s

Details

CI Trade-In / frontend-checks (pull_request) Has been skipped

Details

f9ae6f0c53

Валидация вскрыла: не-ЕКБ headline over-priced — analog Tier-S (same-building) fallback БЕЗ geo-предиката матчил одноимённую улицу+дом в ДРУГОМ городе → тянул ЕКБ-листинги (40191 EKB/~0 non-EKB) с distance_m=0. Серов ask-head 186k vs sold 30k; Н.Тагил 187k vs deal-corridor 89k.

Part 1 (Tier-S geo-bound, ~4624): + ST_DWithin(geom, subject, radius) в address-prefix fallback (зеркалит Tier-W). >99.97% листингов с coords → режет только cross-city false-positives.
Part 2 (deals-headline-fallback): _fetch_dkp_corridor widen на city-wide при street-sample <3 (gated city!=екатеринбург); _price_from_inputs: при median_ppm2<=0 (нет листингов) + anchor_tier None + dkp_raw>=3 → headline из deal-corridor, confidence=low, n_analogs=0, expected_sold=None.

Deep-review ✅ APPROVE (no 🔴/🟠; ~10 downstream stages traced coherent). EKB byte-green regression-гейт (fallback требует median_ppm2<=0, невозможно для frozen fixture; Part 1 SQL не в replay). Тесты: Н.Тагил 0-листингов+corridor85911 → headline=85911 (не 0/186k); EKB-control dense → headline из листингов (fallback не срабатывает). Full suite 2418 passed.

Follow-ups (🟡): fallback без city-gate (sparse EKB n/a→deals, safe); api_analog_tier мислейбл 'city'; Part1/2a integration-test; EKB backtest post-deploy (rule #7).