4 commits

Author	SHA1	Message	Date
bot-backend	31c31f8f82	МЕРА: якорь «тот же дом» и IMV-blend — константы движка, не настройки (#2381) ... Пять булевых флагов кластера сняты ещё в #2475. Оставшиеся 14 числовых полей Settings перенесены в estimator.py константами с теми же значениями: IMV_BLEND_WEIGHT 0.5, IMV_BLEND_THRESHOLD 1.15, SB_MIN_COMPS 4, SB_AREA_SIGMA 0.18, SB_ROOMS_MATCH_BOOST 1.6, SB_FLOOR_SIGMA 0.25, SB_GUARDRAIL_TOL 0.05, SB_MAD_K 3.5, SB_MAD_K_SMALL_N 2.5, SB_SMALL_N_THRESHOLD 10, ANCHOR_TIER_C_CORRIDOR_MULT 1.5, FSD_K 1.65, SB_GATE_MIN_N 3, SB_GATE_MAX_FSD 0.20. На проде 17.09 все 14 равны дефолтам, ENV-оверрайдов нет. Мёртвая проверка `tier_c_mult > 0` (константа против нуля) убрана. Тесты подменяют SB_MIN_COMPS на модуле вместо поля settings. Реплей бэктеста по сделкам побитово тот же, срабатываний якоря 985, IMV-blend 3, low-conf гейта 11 — как на main. Шесть порогов (rooms_boost, floor_sigma, guardrail_tol, tier_c_mult, fsd_k, gate_max_fsd) не ловит ни один поведенческий тест и ни гейт, их держит только тест боевых значений. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>	2026-09-17 12:38:44 +05:00
bot-backend	23b8816863	fix(tradein/estimator): гео-ограничение якорного тира — не брать чужие города (#2581) ... Tier A (_fetch_anchor_comps, "тот же дом") матчил по normalized street+house (_normalize_building_key намеренно дропает город) БЕЗ единого гео-предиката — единственный запрос к listings в файле без ST_DWithin/города/house_id_fk/LIMIT. "Серов, ул. Ленина, 5" получал якорь по ЕКБ-объявлениям (~40 191 из ~40 200 активных листингов — ЕКБ) с самым доверенным тиром 'A', завышая цену в 4-5х; хуже — anchor_tier='A' блокировал честный #oblast-D deals-headline-fallback (гейт `anchor_tier is None`), так что область не могла получить даже резервную ДКП-оценку. Фикс мирроит уже одобренный geo-bound для Tier S (f9ae6f0c, #oblast-D): ST_DWithin(geom::geography, subject_point, ANCHOR_TIER_A_RADIUS_M) от lat/lon субъекта, radius = DEFAULT_RADIUS_M (1000м, тот же константа что и Tier S) — строковый match уже устанавливает "та же улица+дом", радиус нужен только чтобы отсечь РЕАЛЬНО кросс-городские коллизии (ЕКБ/Серов — сотни км), не для внутридомовой точности. Tier A целиком гейтится на lat/lon субъекта (как Tier C) — без них геопредикат невозможен; в проде geo всегда есть (_empty_estimate возвращается раньше при неудачном geocode). Объявления без geom: `AND geom IS NOT NULL` + ST_DWithin (NULL → exclude, не blind-include) — >99.97% листингов имеют координаты (см. f9ae6f0c), исключение погрешности не создаёт. _normalize_building_key НЕ тронут (город по-прежнему не входит в ключ) — добавление городского токена сломало бы матчинг источников без города в адресе (Avito anonymous) и внесло бы новый normalization-риск; кросс- городская коллизия закрыта на SQL-уровне надёжнее. Поправлен ложный комментарий _band_haircut (:1900-1908): "same-building anchor pool для oblast не формируется" был неверен даже ДО фикса (Tier A не имел гео-фильтра вовсе — либо матчил ЕКБ, либо честно матчил местные листинги, если они были). Тест-страж test_estimator_deals_headline_fallback_oblast_d.py использовал голый db=MagicMock() для _fetch_anchor_comps — db.execute(...).mappings().all() на unconfigured MagicMock тривиально возвращает [] (MagicMock __iter__ default), так что anchor_tier никогда не мог стать 'A' и регрессия было бы невидима. Пропатчено явно (документирует допущение) + добавлен новый тест, который НЕ мокает _fetch_anchor_comps — гоняет реальную SQL-логику через фейковый db.execute, симулирующий настоящую ST_DWithin-фильтрацию по haversine- дистанции. На pre-fix коде (git stash) даёт headline=186 400 ₽/м² вместо честных 30 000 (deal corridor) — воспроизводит репортнутый баг buквально. EKB control-тест подтверждает: несколько объявлений в одном доме ЕКБ по-прежнему формируют Tier A anchor (n=4, ~145k ₽/м²) — не деградировало. test_tier_a_primary_1774.py: позитивные Tier A тесты теперь передают lat/lon субъекта (гейт требует их) — тесты на python-side novostroyki- гейтинг (ожидающие tier=None) оставлены с lat=None (конечный результат не меняется: Tier A целиком пропускается без lat/lon, как и Tier C раньше). Full suite: 2842 passed, 9 skipped, 1 pre-existing failure (test_search_cache_hit, не связан). Ruff чист.	2026-07-31 17:39:00 +03:00
bot-backend	d22e0e00ae	tech-debt(tradein/estimator): collapse won estimate_* flags into defaults (#1970) (#2475)	2026-07-12 12:46:18 +00:00
Light1YT	07d9cc8e8b	fix(estimator): #1774 Tier A admits same-building novostroyki resales when secondary-dominant ... Tier A (same building) dropped cian listings tagged listing_segment='novostroyki' even in delivered buildings where that tag marks owner resales/переуступки (sale_type=free). Gated relaxation of the #1186 guard, Tier A ONLY: include novostroyki rows iff the same house has secondary (vtorichka/NULL) listings AND secondary_count >= primary_count — secondary-dominant signal of a delivered house. Prevents a lone vtorichka + clustered developer-priced novostroyki from anchoring on застройщик prices after MAD-clip drops the rare secondary (~27% overvaluation). Pure-primary / primary-dominated houses keep the #1186 guard. Behind estimate_sb_tier_a_allow_primary_if_secondary_present (default True). Dedup duplicate active rows by (source, source_id) primary key (codebase canon, namespaced id/url tags, source_url fallback) — collapses rows whose source_url differs only by trailing slash / query-param (e.g. cian source_id=330047129 ×2). Tier C / radius / asking_to_sold_ratio paths untouched. 84 unit tests pass.	2026-06-19 14:00:13 +05:00