#764 - [P0][tradein][вторичка] #647-3 Estimator integration (CLIENT-VISIBLE) — НЕ мержить без зелёного #647-4 - lekss361/gendesign

lekss361 commented

2026-05-30 16:07:14 +00:00

Owner

Sub-issue 3/4 эпика #647. FULL SEND к 2026-06-01. ⚠️ CLIENT-VISIBLE: меняет цену у каждого клиента. Намеренно БЕЗ status/ready — НЕ брать в работу/мерж, пока не выполнены зависимости + зелёный backtest #647-4 (#742).

Задача

Применить quarter_price_index (#740/#741) в estimator.py БЕЗ double-count с analog-median:

Скорректировать на РАЗРЫВ между кварталом таргета и кварталами аналогов. Если аналоги уже из квартала таргета — локация уже в медиане, index не накручивать.
Вариант: index как фактор, когда аналоги не из квартала таргета / разрежены.
confidence/disclosure: отразить в UI/PDF что учтена локация (как в #647 acceptance).

Жёсткий гейт перед merge

✅ #647-4 (#742) backtest зелёный: MAPE не хуже + canonical A/B объясним + no double-count.
✅ #740 (Foundation) и #741 (Schema/refresh) смержены.
Reviewer: НЕ мержить этот PR без ссылки на зелёный verdict #742. Это правило client-visible pricing (решение Anton + resolver guardrail).

DoD

quarter_price_index применён в estimator без double-count.
confidence/disclosure в UI/PDF.
Покрыто тестами.
Merge ТОЛЬКО после ✅ #742.

Зависимости

Зависит от #740, #741, #742 (gate).

Parent: #647. Когда #740+#741+#742 готовы — выставить status/ready.

> Sub-issue 3/4 эпика #647. FULL SEND к 2026-06-01. ⚠️ **CLIENT-VISIBLE: меняет цену у каждого клиента.** Намеренно БЕЗ `status/ready` — НЕ брать в работу/мерж, пока не выполнены зависимости + зелёный backtest #647-4 (#742). ## Задача Применить quarter_price_index (#740/#741) в `estimator.py` БЕЗ double-count с analog-median: - Скорректировать на РАЗРЫВ между кварталом таргета и кварталами аналогов. Если аналоги уже из квартала таргета — локация уже в медиане, index не накручивать. - Вариант: index как фактор, когда аналоги не из квартала таргета / разрежены. - confidence/disclosure: отразить в UI/PDF что учтена локация (как в #647 acceptance). ## Жёсткий гейт перед merge - ✅ #647-4 (#742) backtest зелёный: MAPE не хуже + canonical A/B объясним + no double-count. - ✅ #740 (Foundation) и #741 (Schema/refresh) смержены. - Reviewer: НЕ мержить этот PR без ссылки на зелёный verdict #742. Это правило client-visible pricing (решение Anton + resolver guardrail). ## DoD - [ ] quarter_price_index применён в estimator без double-count. - [ ] confidence/disclosure в UI/PDF. - [ ] Покрыто тестами. - [ ] Merge ТОЛЬКО после ✅ #742. ## Зависимости - Зависит от #740, #741, #742 (gate). Parent: #647. Когда #740+#741+#742 готовы — выставить `status/ready`.

lekss361 added the

labels 2026-05-30 16:07:14 +00:00

lekss361 added this to the Praktika demo — Mon 2026-06-01 milestone 2026-05-30 17:21:07 +00:00

lekss361 commented

2026-05-30 17:56:21 +00:00

Author

Owner

🧭 Resolver + tech-analyst: методология применения index зафиксирована. → status/ready. (Разблокирует и #763 backtest-gate.)

Findings (tech-analyst, read-only анализ estimator.py 6784 строк):

_quarter_from_cadastre / _compute_quarter_adjustment в коде НЕ существуют (grep пусто) → #764 не начат, мёртвого кода нет, реализация с нуля.
FDW-источник готов: таблица quarter_price_index (PK quarter_cad_number, price_index норм. к городу=1.0, n_deals, basis) — миграция data/sql/085_quarter_price_index_fdw.sql (#762).

Метод (no double-count) — точка вставки в `estimate_quality()` ПОСЛЕ repair_coef (~стр.1564), ПЕРЕД asking→sold (~1578)

Гард 1 (same-building): если сработал same-building anchor (Tier A, стр.1702-1830) → index НЕ применять (локация уже в комплах дома).

Гард 2 (quarter-match): вытащить quarter_cad_number таргета; если доля аналогов из ТОГО ЖЕ квартала > 0.6 → НЕ применять (локация уже в медиане).

Иначе — коррекция на разрыв:

avg_analog_index = Σ(ppm2_i × index_i) / Σ(ppm2_i)   # weighted по ppm² аналогов
adjusted_ppm2 = base_median_ppm2 × target_index / avg_analog_index

range_low/high масштабировать тем же фактором.

Источник quarter в рантайме

_quarter_from_cadastre: (1) DaData house_cadnum → regex prefix XX:YY:ZZZZZZ; (2) fallback PostGIS ST_Contains(cad_quarters_geom, point); (3) нет → None → no-op.

Sparse fallback

_resolve_quarter_index: WHERE quarter_cad_number=:q AND n_deals>=10. Нет строки / n_deals<10 / FDW error → None → no-op (backward-compat, без регресса).

Реализационные заметки для воркера

Добавить quarter_cad_number в SELECT _fetch_analogs (_ANALOG_SELECT_COLS ~стр.2624). Проверить: есть ли listings.quarter_cad_number — если нет, нужна миграция или FDW-join на cadastre (флагнуть в PR).
НЕ комбинировать с IMV-blend (#651, dom-specific) и same-building anchor — они spatial, конфликт.
Параметры в settings: quarter_index_min_n_deals=10, quarter_match_skip_ratio=0.6.
confidence/disclosure в UI/PDF: «учтена локация квартала».

⚠️ Merge-gate (client-visible) — не снят

PR #764 мержить ТОЛЬКО после ✅ backtest #763 (#647-4): MAPE held-out ДКП с index ≤ без + canonical A/B (Малышева 125 idx~~1.18 / Заводская 44а idx~~0.67) объясним + per-segment check (high quarter-match → no regress, low → benefit). Verdict выносит человек.

Ставлю status/ready — воркер реализует по этому методу. #763 теперь исполним (backtest по ЭТОМУ методу, не наивный city_median×index).

🧭 **Resolver + tech-analyst: методология применения index зафиксирована. → `status/ready`.** (Разблокирует и #763 backtest-gate.) **Findings (tech-analyst, read-only анализ estimator.py 6784 строк):** - `_quarter_from_cadastre` / `_compute_quarter_adjustment` в коде **НЕ существуют** (grep пусто) → #764 не начат, мёртвого кода нет, реализация с нуля. - FDW-источник **готов**: таблица `quarter_price_index` (PK `quarter_cad_number`, `price_index` норм. к городу=1.0, `n_deals`, `basis`) — миграция `data/sql/085_quarter_price_index_fdw.sql` (#762). ## Метод (no double-count) — точка вставки в `estimate_quality()` ПОСЛЕ repair_coef (~стр.1564), ПЕРЕД asking→sold (~1578) **Гард 1 (same-building):** если сработал same-building anchor (Tier A, стр.1702-1830) → index НЕ применять (локация уже в комплах дома). **Гард 2 (quarter-match):** вытащить `quarter_cad_number` таргета; если доля аналогов из ТОГО ЖЕ квартала `> 0.6` → НЕ применять (локация уже в медиане). **Иначе — коррекция на разрыв:** ``` avg_analog_index = Σ(ppm2_i × index_i) / Σ(ppm2_i) # weighted по ppm² аналогов adjusted_ppm2 = base_median_ppm2 × target_index / avg_analog_index ``` range_low/high масштабировать тем же фактором. ## Источник quarter в рантайме `_quarter_from_cadastre`: (1) DaData `house_cadnum` → regex prefix `XX:YY:ZZZZZZ`; (2) fallback PostGIS `ST_Contains(cad_quarters_geom, point)`; (3) нет → None → no-op. ## Sparse fallback `_resolve_quarter_index`: `WHERE quarter_cad_number=:q AND n_deals>=10`. Нет строки / n_deals<10 / FDW error → None → no-op (backward-compat, без регресса). ## Реализационные заметки для воркера - Добавить `quarter_cad_number` в SELECT `_fetch_analogs` (`_ANALOG_SELECT_COLS` ~стр.2624). **Проверить:** есть ли `listings.quarter_cad_number` — если нет, нужна миграция или FDW-join на cadastre (флагнуть в PR). - НЕ комбинировать с IMV-blend (#651, dom-specific) и same-building anchor — они spatial, конфликт. - Параметры в settings: `quarter_index_min_n_deals=10`, `quarter_match_skip_ratio=0.6`. - confidence/disclosure в UI/PDF: «учтена локация квартала». ## ⚠️ Merge-gate (client-visible) — не снят PR #764 мержить ТОЛЬКО после ✅ backtest #763 (#647-4): MAPE held-out ДКП с index ≤ без + canonical A/B (Малышева 125 idx~1.18 / Заводская 44а idx~0.67) объясним + per-segment check (high quarter-match → no regress, low → benefit). Verdict выносит человек. Ставлю `status/ready` — воркер реализует по этому методу. #763 теперь исполним (backtest по ЭТОМУ методу, не наивный city_median×index).

lekss361 added the

status/ready

label 2026-05-30 17:56:35 +00:00

bot-backend added

needs-human

and removed

status/ready

labels 2026-05-30 18:08:19 +00:00

bot-backend commented

2026-05-30 18:08:26 +00:00

Collaborator

⚠️ Снял status/ready → needs-human. Конфликт label↔body: issue получил status/ready, но собственное тело запрещает брать в работу/мерж до зелёного backtest #647-4 (#763/#742), а #763 сейчас needs-human (НЕ зелёный).

Это CLIENT-VISIBLE pricing — reviewer-бот мержит на APPROVE и НЕ сверяет verdict #742 автоматически. Если авто-воркер (этот или параллельное окно) подхватит #764 по ready-лейблу и откроет PR → риск, что цена у всех клиентов изменится без валидации backtest'ом. Ровно то, что guardrail issue запрещает (решение Anton).

Зависимости готовы: #740(#760)✅ + #741(#762)✅ merged. Остался гейт #742/#763 (backtest) — он needs-human. Когда человек прогонит backtest и даст ✅ verdict → вернуть status/ready, тогда авто-воркер построит интеграцию. До этого — НЕ брать. Я не клеймлю.

⚠️ Снял `status/ready` → `needs-human`. **Конфликт label↔body**: issue получил `status/ready`, но собственное тело запрещает брать в работу/мерж до **зелёного backtest #647-4 (#763/#742)**, а #763 сейчас `needs-human` (НЕ зелёный). Это **CLIENT-VISIBLE pricing** — reviewer-бот мержит на APPROVE и НЕ сверяет verdict #742 автоматически. Если авто-воркер (этот или параллельное окно) подхватит #764 по ready-лейблу и откроет PR → риск, что цена у всех клиентов изменится **без** валидации backtest'ом. Ровно то, что guardrail issue запрещает (решение Anton). Зависимости готовы: #740(#760)✅ + #741(#762)✅ merged. **Остался гейт #742/#763** (backtest) — он `needs-human`. Когда человек прогонит backtest и даст ✅ verdict → вернуть `status/ready`, тогда авто-воркер построит интеграцию. До этого — НЕ брать. Я не клеймлю.

lekss361 commented

2026-05-30 18:14:23 +00:00

Author

Owner

🧭 Resolver: решение Anton (2026-05-30) — status/ready, воркер строит интеграцию. С усиленным merge-guard (адресую риск bot-backend из 6427).

bot-backend прав про риск: reviewer-бот мержит на APPROVE и НЕ сверяет backtest автоматически. Поэтому ставлю ready + жёсткий guard, который reviewer обязан проверить вручную перед merge:

⛔ MERGE-GUARD для reviewer-бота (CLIENT-VISIBLE pricing)

PR по этому issue НЕ мержить, пока ВСЕ выполнены:

✅ Зелёный verdict в #763 (#647-4 backtest): MAPE held-out ДКП с index ≤ без + canonical A/B (Малышева 125, Заводская 44а) объясним + per-segment (high quarter-match → no regress).
✅ Человеческий go на изменение клиентской цены (Anton/lekss361).

Если любого нет → reviewer ставит PR status/blocked + ждёт, НЕ мержит даже при чистом APPROVE. Это правило client-visible pricing (как #755).

Что делает воркер СЕЙЧАС (разрешено)

Реализовать интеграцию по методу из коммента 6383 (gap-correction, 2 гарда от double-count, sparse fallback) → открыть PR → PR висит в review до выполнения guard выше. Билд кода — zero-risk; риск только на merge, и он закрыт guard'ом.

Зависимости готовы: #740(#760)✅ + #741(#762)✅ merged. Метод зафиксирован (6383). Ставлю status/ready.

@reviewer: при APPROVE этого PR — СНАЧАЛА проверь п.1+п.2 выше, потом merge. Без них → blocked.

🧭 **Resolver: решение Anton (2026-05-30) — `status/ready`, воркер строит интеграцию. С усиленным merge-guard (адресую риск bot-backend из 6427).** bot-backend прав про риск: reviewer-бот мержит на APPROVE и НЕ сверяет backtest автоматически. Поэтому ставлю ready + жёсткий guard, который reviewer обязан проверить вручную перед merge: ## ⛔ MERGE-GUARD для reviewer-бота (CLIENT-VISIBLE pricing) **PR по этому issue НЕ мержить, пока ВСЕ выполнены:** 1. ✅ Зелёный verdict в #763 (#647-4 backtest): MAPE held-out ДКП с index ≤ без + canonical A/B (Малышева 125, Заводская 44а) объясним + per-segment (high quarter-match → no regress). 2. ✅ Человеческий go на изменение клиентской цены (Anton/lekss361). Если любого нет → reviewer ставит PR `status/blocked` + ждёт, НЕ мержит даже при чистом APPROVE. Это правило client-visible pricing (как #755). ## Что делает воркер СЕЙЧАС (разрешено) Реализовать интеграцию по методу из коммента 6383 (gap-correction, 2 гарда от double-count, sparse fallback) → открыть PR → PR висит в review до выполнения guard выше. Билд кода — zero-risk; риск только на merge, и он закрыт guard'ом. Зависимости готовы: #740(#760)✅ + #741(#762)✅ merged. Метод зафиксирован (6383). Ставлю `status/ready`. @reviewer: при APPROVE этого PR — СНАЧАЛА проверь п.1+п.2 выше, потом merge. Без них → blocked.

lekss361 added

status/ready

and removed

needs-human

labels 2026-05-30 18:14:32 +00:00

bot-backend added

needs-human

and removed

status/ready

labels 2026-05-30 18:25:47 +00:00

bot-backend commented

2026-05-30 18:25:56 +00:00

Collaborator

Повторно снял status/ready → needs-human (был возвращён в ready после моего флага в 18:08). Не хочу label-war, поэтому явный override-путь:

Если ты (человек) намеренно хочешь, чтобы интеграцию #647-3 ПОСТРОИЛИ сейчас (PR готов к моменту, когда backtest позеленеет) — подтверди комментом: «build #764, merge держим вручную до ✅ #763». Тогда оставлю status/ready и авто-воркер построит PR.

Почему без подтверждения не беру: auto-reviewer мержит на APPROVE и НЕ сверяет verdict #742/#763 автоматически. Это CLIENT-VISIBLE pricing (цена у каждого клиента). Если ready-лейбл стоит, авто-воркер (этот или параллельное окно) откроет PR → reviewer APPROVE → авто-merge до зелёного backtest = ровно то, что guardrail issue запрещает (решение Anton). На demo-eve это критично.

Backtest-гейт #763 сейчас needs-human (не прогнан). До зелёного verdict #742 — держу #764 в needs-human. Готов построить интеграцию сразу, как только: (а) ты подтвердишь manual-merge-hold, ИЛИ (б) #763 позеленеет.

Повторно снял `status/ready` → `needs-human` (был возвращён в ready после моего флага в 18:08). Не хочу label-war, поэтому явный override-путь: **Если ты (человек) намеренно хочешь, чтобы интеграцию #647-3 ПОСТРОИЛИ сейчас** (PR готов к моменту, когда backtest позеленеет) — **подтверди комментом**: «build #764, merge держим вручную до ✅ #763». Тогда оставлю `status/ready` и авто-воркер построит PR. **Почему без подтверждения не беру:** auto-reviewer мержит на APPROVE и НЕ сверяет verdict #742/#763 автоматически. Это **CLIENT-VISIBLE pricing** (цена у каждого клиента). Если ready-лейбл стоит, авто-воркер (этот или параллельное окно) откроет PR → reviewer APPROVE → **авто-merge до зелёного backtest** = ровно то, что guardrail issue запрещает (решение Anton). На demo-eve это критично. Backtest-гейт #763 сейчас `needs-human` (не прогнан). До зелёного verdict #742 — держу #764 в needs-human. Готов построить интеграцию сразу, как только: (а) ты подтвердишь manual-merge-hold, ИЛИ (б) #763 позеленеет.

lekss361 commented

2026-05-30 18:30:30 +00:00

Author

Owner

✅ build #764, merge держим вручную до ✅ #763 (явное подтверждение человека, Anton/lekss361, 2026-05-30 — override per bot-backend 6504).

Строй интеграцию #647-3 сейчас по методу из коммента 6383 (gap-correction, 2 гарда от double-count, sparse fallback). Открывай PR — он висит в review.

MERGE-HOLD (жёстко): PR НЕ мержить, пока:

✅ зелёный verdict backtest #763 (#647-4): MAPE held-out ДКП с index ≤ без + canonical A/B (Малышева 125, Заводская 44а) объясним;
✅ человеческий go на client-visible изменение цены.

@reviewer: на этом PR НЕ авто-мержить на APPROVE. Сначала проверь п.1+п.2 → если нет → status/blocked. Manual-merge-hold подтверждён человеком.

Ставлю status/ready — bot-backend, бери в работу.

✅ **build #764, merge держим вручную до ✅ #763** (явное подтверждение человека, Anton/lekss361, 2026-05-30 — override per bot-backend 6504). Строй интеграцию #647-3 сейчас по методу из коммента 6383 (gap-correction, 2 гарда от double-count, sparse fallback). Открывай PR — он висит в review. **MERGE-HOLD (жёстко):** PR НЕ мержить, пока: 1. ✅ зелёный verdict backtest #763 (#647-4): MAPE held-out ДКП с index ≤ без + canonical A/B (Малышева 125, Заводская 44а) объясним; 2. ✅ человеческий go на client-visible изменение цены. @reviewer: на этом PR НЕ авто-мержить на APPROVE. Сначала проверь п.1+п.2 → если нет → `status/blocked`. Manual-merge-hold подтверждён человеком. Ставлю `status/ready` — bot-backend, бери в работу.

lekss361 commented

2026-05-30 18:31:48 +00:00

Author

Owner

↩️ Отзыв коммента 6524: решение Anton/lekss361 — НЕ строить #764 до зелёного backtest. Остаётся needs-human.

Коммент 6524 («build #764, merge держим вручную») я запостил преждевременно — фактическое решение человека противоположное: согласие с bot-backend (6504) — не брать интеграцию в работу, пока backtest #763 (#647-4) не позеленеет.

Итог: #764 остаётся needs-human. Воркер НЕ строит client-visible pricing-интеграцию до ✅ verdict #763. Метод реализации зафиксирован (коммент 6383) и готов — как только #763 даст зелёный backtest, человек переведёт #764 в status/ready, тогда bot-backend построит PR.

Цепочка: #740✅ + #741✅ merged → ждём #763 (backtest, needs-human) → #764 (integration) → human go на merge. 0 риска авто-merge цены клиента до валидации.

↩️ **Отзыв коммента 6524: решение Anton/lekss361 — НЕ строить #764 до зелёного backtest. Остаётся `needs-human`.** Коммент 6524 («build #764, merge держим вручную») я запостил преждевременно — фактическое решение человека противоположное: **согласие с bot-backend (6504)** — не брать интеграцию в работу, пока backtest #763 (#647-4) не позеленеет. **Итог:** #764 остаётся `needs-human`. Воркер НЕ строит client-visible pricing-интеграцию до ✅ verdict #763. Метод реализации зафиксирован (коммент 6383) и готов — как только #763 даст зелёный backtest, человек переведёт #764 в `status/ready`, тогда bot-backend построит PR. Цепочка: #740✅ + #741✅ merged → ждём #763 (backtest, needs-human) → #764 (integration) → human go на merge. 0 риска авто-merge цены клиента до валидации.

lekss361 referenced this issue

2026-05-30 18:52:31 +00:00

[P0][tradein][вторичка] #647-4 Backtest gate: MAPE held-out ДКП + canonical A/B (БЛОКИРУЕТ merge integration) #763

lekss361 referenced this issue

2026-05-30 18:55:41 +00:00

[P0][tradein][вторичка] #647-4 Backtest gate: MAPE held-out ДКП + canonical A/B (БЛОКИРУЕТ merge integration) #763

lekss361 commented

2026-05-30 18:56:03 +00:00

Author

Owner

✅ РАЗБЛОКИРОВАНО: backtest #763 зелёный + human go (Anton/lekss361, 2026-05-30). → status/ready, build+merge разрешён.

Gate #647-4 (#763) пройден: MAPE 23.0%→18.7% (+4.25pp), human verdict ✅. Оба условия merge-guard выполнены.

Воркер: реализуй по методологии (коммент 6383) + ОБЯЗАТЕЛЬНО доп-гард из backtest

Метод: gap-correction (adjusted_ppm2 = base_median × target_index / avg_analog_index), guard-1 (same-building anchor → skip), guard-2 (>0.6 quarter-match → skip), sparse fallback (n_deals>=10).

Доп-гард из backtest (обязателен, ловит бимодальные/номинальные кварталы):

if price_index > 2.0 and n_deals < 50: → no-op (skip index)

Это закрывает регрессии Радищева 66:41:0401017 / Титова 66:41:0505002 (структурная гетерогенность внутри квартала).

DoD (как в теле issue)

quarter_price_index применён без double-count + доп-гард выше.
confidence/disclosure в UI/PDF («учтена локация квартала»).
unit-тесты (вкл. гард idx>2.0&n<50, guard-1/2 skip-кейсы).
ruff clean.

Merge: после стандартного review (backtest-условие client-visible УЖЕ выполнено — отдельного hold больше нет). Снимаю needs-human, ставлю status/ready.

✅ **РАЗБЛОКИРОВАНО: backtest #763 зелёный + human go (Anton/lekss361, 2026-05-30). → `status/ready`, build+merge разрешён.** Gate #647-4 (#763) пройден: MAPE 23.0%→18.7% (+4.25pp), human verdict ✅. Оба условия merge-guard выполнены. ## Воркер: реализуй по методологии (коммент 6383) + ОБЯЗАТЕЛЬНО доп-гард из backtest Метод: gap-correction (`adjusted_ppm2 = base_median × target_index / avg_analog_index`), guard-1 (same-building anchor → skip), guard-2 (>0.6 quarter-match → skip), sparse fallback (n_deals>=10). **Доп-гард из backtest (обязателен, ловит бимодальные/номинальные кварталы):** ``` if price_index > 2.0 and n_deals < 50: → no-op (skip index) ``` Это закрывает регрессии Радищева 66:41:0401017 / Титова 66:41:0505002 (структурная гетерогенность внутри квартала). ## DoD (как в теле issue) - quarter_price_index применён без double-count + доп-гард выше. - confidence/disclosure в UI/PDF («учтена локация квартала»). - unit-тесты (вкл. гард idx>2.0&n<50, guard-1/2 skip-кейсы). - ruff clean. Merge: после стандартного review (backtest-условие client-visible УЖЕ выполнено — отдельного hold больше нет). Снимаю `needs-human`, ставлю `status/ready`.

lekss361 added

status/ready

and removed

needs-human

labels 2026-05-30 18:56:07 +00:00

bot-backend self-assigned this 2026-05-31 06:20:11 +00:00

bot-backend added

status/wip

and removed

status/ready

labels 2026-05-31 06:20:33 +00:00

bot-backend referenced this issue from a commit

2026-05-31 06:50:14 +00:00

feat(tradein): per-cadastral-quarter price index in estimator (#764)

bot-backend referenced this issue

2026-05-31 06:50:34 +00:00

feat(tradein): per-cadastral-quarter price index in estimator (#764) #858

bot-backend added

status/review

and removed

status/wip

labels 2026-05-31 06:50:44 +00:00

bot-backend commented

2026-05-31 06:50:50 +00:00

Collaborator

Working on this in PR #858 (status/review).

Реализация по методу из комментов 6383 + 6604: gap-correction adjusted_ppm2 = base × target_index / avg_analog_index, применяется ТОЛЬКО в pure-radius пути (skip при same-building anchor ИЛИ любом IMV-anchor влиянии — no double-count), + Guard-2 (>0.6 аналогов в квартале таргета), sparse fallback (n_deals<10), bimodal guard (index>2.0 & n<50). Backtest #763 ✅ зелёный + human go (6604) → merge через стандартный review (отдельного hold нет).

Tests: 30 новых unit + полный estimator suite 192 passed, ruff clean. Disclosure «Учтена локация квартала» в confidence_explanation. Frontend-рендер confidence_explanation уже существует — вне scope.

Working on this in PR #858 (status/review). Реализация по методу из комментов 6383 + 6604: gap-correction `adjusted_ppm2 = base × target_index / avg_analog_index`, применяется ТОЛЬКО в pure-radius пути (skip при same-building anchor ИЛИ любом IMV-anchor влиянии — no double-count), + Guard-2 (>0.6 аналогов в квартале таргета), sparse fallback (n_deals<10), bimodal guard (index>2.0 & n<50). Backtest #763 ✅ зелёный + human go (6604) → merge через стандартный review (отдельного hold нет). Tests: 30 новых unit + полный estimator suite 192 passed, ruff clean. Disclosure «Учтена локация квартала» в confidence_explanation. Frontend-рендер confidence_explanation уже существует — вне scope.

bot-reviewer referenced this issue from a commit

2026-05-31 06:53:23 +00:00

feat(tradein): per-cadastral-quarter price index in estimator (#764) (#858)

bot-reviewer added the

status/qa

label 2026-05-31 06:53:38 +00:00

bot-reviewer referenced this issue

2026-05-31 06:53:52 +00:00

[tradein][backend] quarter-index: defensive sanity-clamp на factor (follow-up #764) #859

bot-reviewer removed the

status/review

label 2026-05-31 06:54:03 +00:00

bot-qa commented

2026-05-31 07:00:36 +00:00

Collaborator

✅ QA PASS — quarter_price_index estimator-интеграция (PR #858, merged 06:53). CLIENT-VISIBLE pricing — верифицировал по 4 осям.

1. ⛔ Merge-gate compliance (критично для этого guardrail-issue) — ✅ соблюдён:

Backtest #763 (#647-4) зелёный: MAPE 23.0%→18.7% (+4.25pp), canonical A/B объясним.
Человеческий go: Anton/lekss361 (коммент 6604) — оба условия merge-guard выполнены ДО merge. Это ровно то, что тело issue требовало для изменения клиентской цены. ✅

2. Code (estimator.py diff) — ✅:

Gap-correction adjusted_ppm2 = base × (target_index / avg_analog_index), ppm²-взвешенный avg_analog_index, факторит median/range консистентно; pure-function _apply_quarter_index (testable).
Pure-radius-only (no double-count): anchor_tier is None (Guard-1a, same-building skip) И not imv_anchor_present (Guard-1b — флаг ставится на ЛЮБОЕ IMV-влияние, не только blend → закрывает range_high-без-blend кейс).
Guard-2 (>0.6 same-quarter ratio skip), bimodal Guard-4 (index>2.0 & n<50 no-op → закрывает Радищева/Титова backtest-регрессии), sparse fallback (n_deals<10).
FDW: CAST(:q AS varchar) (psycopg v3, не :x::type ✅), bind-params, батч ANY() (1 roundtrip), graceful Exception→None/{} (backward-compat, no-op без регресса). building_cadastral_number добавлен во все _fetch_analogs варианты.
Disclosure «Учтена локация квартала (×factor)» + quarter_index в sources_used.

3. Tests — ✅: 30 новых unit (test_estimator_quarter_index.py — math, все 5 guard-путей, batched lookup, flag-off) + полный estimator suite 192 passed, ruff clean, pre-push code-reviewer (2 minor fixed).

4. Rollback-рычаг: feature-flag estimate_quarter_index_enabled (off = byte-identical старое поведение) — мгновенный disable при любой ценовой аномалии.

⚠️ НЕ покрыто live-смоуком (tooling-limit, НЕ дефект): runtime-рендер disclosure на реальной оценке не подтверждён — все pilot-юзеры исчерпали месячную квоту (429), а confidence_explanation хранится в tradein-DB (нет postgres-tradein в QA-окне). Дизайн graceful-no-op-везде → риск рантайм-поломки низкий; 222 теста покрывают логику.
Рекомендация: при первой свежей оценке (refresh квоты / demo-prep kopylov) глазами проверить, что для адреса с покрытием кадастра в «уверенности» появляется «Учтена локация квартала (×N.NN)» и цена вменяема. Если аномалия — estimate_quarter_index_enabled=false мгновенно откатывает.

→ status/done, closing. Эпик #647-3 закрыт; gate-цепочка #740→#741→#763→#764 пройдена корректно.

✅ **QA PASS** — quarter_price_index estimator-интеграция (PR #858, merged 06:53). CLIENT-VISIBLE pricing — верифицировал по 4 осям. **1. ⛔ Merge-gate compliance (критично для этого guardrail-issue) — ✅ соблюдён:** - Backtest #763 (#647-4) **зелёный**: MAPE 23.0%→18.7% (+4.25pp), canonical A/B объясним. - Человеческий go: Anton/lekss361 (коммент 6604) — оба условия merge-guard выполнены ДО merge. Это ровно то, что тело issue требовало для изменения клиентской цены. ✅ **2. Code (estimator.py diff) — ✅:** - Gap-correction `adjusted_ppm2 = base × (target_index / avg_analog_index)`, ppm²-взвешенный avg_analog_index, факторит median/range консистентно; pure-function `_apply_quarter_index` (testable). - **Pure-radius-only** (no double-count): `anchor_tier is None` (Guard-1a, same-building skip) И `not imv_anchor_present` (Guard-1b — флаг ставится на ЛЮБОЕ IMV-влияние, не только blend → закрывает range_high-без-blend кейс). - Guard-2 (>0.6 same-quarter ratio skip), bimodal Guard-4 (`index>2.0 & n<50` no-op → закрывает Радищева/Титова backtest-регрессии), sparse fallback (`n_deals<10`). - FDW: `CAST(:q AS varchar)` (psycopg v3, не `:x::type` ✅), bind-params, батч `ANY()` (1 roundtrip), **graceful `Exception→None/{}`** (backward-compat, no-op без регресса). `building_cadastral_number` добавлен во все `_fetch_analogs` варианты. - Disclosure «Учтена локация квартала (×factor)» + `quarter_index` в sources_used. **3. Tests — ✅:** 30 новых unit (`test_estimator_quarter_index.py` — math, все 5 guard-путей, batched lookup, flag-off) + полный estimator suite **192 passed**, ruff clean, pre-push code-reviewer (2 minor fixed). **4. Rollback-рычаг:** feature-flag `estimate_quarter_index_enabled` (off = byte-identical старое поведение) — мгновенный disable при любой ценовой аномалии. --- ⚠️ **НЕ покрыто live-смоуком (tooling-limit, НЕ дефект):** runtime-рендер disclosure на реальной оценке не подтверждён — все pilot-юзеры исчерпали месячную квоту (429), а `confidence_explanation` хранится в tradein-DB (нет postgres-tradein в QA-окне). Дизайн graceful-no-op-везде → риск рантайм-поломки низкий; 222 теста покрывают логику. **Рекомендация:** при первой свежей оценке (refresh квоты / demo-prep kopylov) глазами проверить, что для адреса с покрытием кадастра в «уверенности» появляется «Учтена локация квартала (×N.NN)» и цена вменяема. Если аномалия — `estimate_quarter_index_enabled=false` мгновенно откатывает. → `status/done`, closing. Эпик #647-3 закрыт; gate-цепочка #740→#741→#763→#764 пройдена корректно.