fix(tradein/estimator): кросс-source дедуп street+house union-find + savepoint вокруг ratio-lookup (#2265) #2273

Merged

lekss361 merged 4 commits from fix/tradein-dedup-addr-key into main

2026-07-03 07:51:48 +00:00

lekss361 commented

2026-07-03 07:22:52 +00:00

Owner

Summary

Два фикса по живому аудиту честности чисел (#2265, эпик #2262):

Дедуп аналогов протекал на кросс-source дублях (прод-evidence d5fc3841: один физлот Сыромолотова = 3 «аналога» через cian/domklik/yandex; domklik cadnum 0/6296 — всегда пусто). Ключ переписан: нормализованный street_token (типы улиц/город/район/дом-маркеры выброшены, district-маркеры мкр/м-н исключены) + композит (street_token, floor, area_bucket, price_bucket); слияние через union-find по ЛЮБОМУ совпавшему ключу (решает cadnum-асимметрию «у одного есть, у другого нет»); guard: два разных непустых кадастра НИКОГДА не сливаются. house_no сознательно не в equality-ключе (асимметрия наличия номера между источниками — дискриминация на floor/area/price).
Transient DB-error гасил expected_sold (3/224 за 14 дней, вкл. живой a545ca58): _get_asking_sold_ratio обёрнут в begin_nested() + 1 retry на чистой tx. Семантика #2175 сохранена (транзиентный None не кэшируется).

Test plan

+9 тестов (3 прод-формата Сыромолотова → один физлот; non-merge кейсы; cadnum-асимметрия/конфликт; savepoint retry/recover/None-uncached)
pytest -k "dedup or ratio or estimator": 613 passed, 4 skipped (skipped — предсуществующие integration)
ruff clean, pre-commit прошёл
post-merge: прод-smoke — переоценка d5fc3841-кейса, n_analogs без триплетов

Refs #2265, #2262, #2087, #2173

## Summary Два фикса по живому аудиту честности чисел (#2265, эпик #2262): 1. **Дедуп аналогов протекал на кросс-source дублях** (прод-evidence d5fc3841: один физлот Сыромолотова = 3 «аналога» через cian/domklik/yandex; domklik cadnum 0/6296 — всегда пусто). Ключ переписан: нормализованный `street_token` (типы улиц/город/район/дом-маркеры выброшены, district-маркеры мкр/м-н исключены) + композит `(street_token, floor, area_bucket, price_bucket)`; слияние через union-find по ЛЮБОМУ совпавшему ключу (решает cadnum-асимметрию «у одного есть, у другого нет»); guard: два разных непустых кадастра НИКОГДА не сливаются. house_no сознательно не в equality-ключе (асимметрия наличия номера между источниками — дискриминация на floor/area/price). 2. **Transient DB-error гасил expected_sold** (3/224 за 14 дней, вкл. живой a545ca58): `_get_asking_sold_ratio` обёрнут в `begin_nested()` + 1 retry на чистой tx. Семантика #2175 сохранена (транзиентный None не кэшируется). ## Test plan - [x] +9 тестов (3 прод-формата Сыромолотова → один физлот; non-merge кейсы; cadnum-асимметрия/конфликт; savepoint retry/recover/None-uncached) - [x] `pytest -k "dedup or ratio or estimator"`: **613 passed, 4 skipped** (skipped — предсуществующие integration) - [x] ruff clean, pre-commit прошёл - [ ] post-merge: прод-smoke — переоценка d5fc3841-кейса, n_analogs без триплетов Refs #2265, #2262, #2087, #2173

lekss361 added 2 commits 2026-07-03 07:22:52 +00:00