fix(tradein/estimator): thin-market гейт IMV перестаёт быть витринным — рынок из 1 объявления больше не двигает headline и range_high #3336

Merged

bot-backend merged 3 commits from fix/3323-imv-thin-market-gate into main 2026-09-05 17:49:30 +00:00

Conversation 0 Commits 3 Files changed 4 +203 -8

bot-backend commented 2026-09-02 12:37:18 +00:00

Collaborator

Copy link

Refs #3323 п.1 [medium] (сборник аудита; НЕ auto-close — в сборнике другие пункты). avito_imv_thin_market_threshold=10 рождал только warning; IMV с market_count=1 двигал headline (blend w=0.5) и растягивал range_high.

Где встал гейт

Обе ветки якоря (house_imv_evaluations и avito_imv_evaluations, включая repair-basis ×k #2677) сходятся в один вызов _apply_imv_blend — гейт там, сразу после null-guard, а не по веткам. При market_count < threshold: ни blend медианы, ни расширение range_high; IMV остаётся display-only в карточке. Поведение при market_count >= threshold не изменено (регресс-контроль тестом).

Warning стал честным: «IMV ОТБРОШЕН из денежного пути…», а не просто «тонкий рынок».

Бонус-дыра, закрытая по пути

market_count читался как if imv_anchor.get(...) — 0 превращался в None, и гейт с флагом thin_market молчали на самом тонком рынке. Теперь is not None в обеих точках сборки.

Сознательно НЕ тронуто

• Guard-1b: imv_anchor_present продолжает гасить quarter-index (#764) и при тонком рынке — открыть его было бы вторым изменением денег вне тикета; вынесу вопрос владельцу отдельно, если нужно;
• сегментный множитель #2293 (за флагом);
• sources_used/display-карточка.

Тесты

test_estimator_imv_blend.py (+3 кейса: гейт по значению, регресс-контроль, граница threshold−1/threshold): 36 passed + смежные 49 passed. Фальсификация — снято ТОЛЬКО тело гейта (полный откат дал бы TypeError = «возможности нет», не «значение неверно»):

assert (75000000, 110000000) == (50000000, 60000000)  — вклад IMV вернулся
2 failed, 1 passed — регресс-контроль остался зелёным, как и должен

Прод-приёмка после деплоя

• SELECT доли оценок с market_count < 10 среди имеющих IMV-якорь — сколько оценок реально перестанут блендиться (замер «сколько прежних изменилось»);
• контрольная оценка на адресе с тонким IMV: headline без вклада IMV, карточка avito_imv на месте с флагом thin_market.

Refs #3323 п.1 [medium] (сборник аудита; НЕ auto-close — в сборнике другие пункты). `avito_imv_thin_market_threshold=10` рождал только warning; IMV с `market_count=1` двигал headline (blend w=0.5) и растягивал `range_high`. ## Где встал гейт Обе ветки якоря (`house_imv_evaluations` и `avito_imv_evaluations`, включая repair-basis ×k #2677) сходятся в один вызов `_apply_imv_blend` — гейт там, сразу после null-guard, а не по веткам. При `market_count < threshold`: ни blend медианы, ни расширение range_high; IMV остаётся display-only в карточке. Поведение при `market_count >= threshold` не изменено (регресс-контроль тестом). Warning стал честным: «IMV ОТБРОШЕН из денежного пути…», а не просто «тонкий рынок». ## Бонус-дыра, закрытая по пути `market_count` читался как `if imv_anchor.get(...)` — **0 превращался в None**, и гейт с флагом `thin_market` молчали на самом тонком рынке. Теперь `is not None` в обеих точках сборки. ## Сознательно НЕ тронуто - Guard-1b: `imv_anchor_present` продолжает гасить quarter-index (#764) и при тонком рынке — открыть его было бы вторым изменением денег вне тикета; вынесу вопрос владельцу отдельно, если нужно; - сегментный множитель #2293 (за флагом); - `sources_used`/display-карточка. ## Тесты `test_estimator_imv_blend.py` (+3 кейса: гейт по значению, регресс-контроль, граница threshold−1/threshold): `36 passed` + смежные `49 passed`. Фальсификация — снято ТОЛЬКО тело гейта (полный откат дал бы TypeError = «возможности нет», не «значение неверно»): ``` assert (75000000, 110000000) == (50000000, 60000000) — вклад IMV вернулся 2 failed, 1 passed — регресс-контроль остался зелёным, как и должен ``` ## Прод-приёмка после деплоя - SELECT доли оценок с `market_count < 10` среди имеющих IMV-якорь — сколько оценок реально перестанут блендиться (замер «сколько прежних изменилось»); - контрольная оценка на адресе с тонким IMV: headline без вклада IMV, карточка avito_imv на месте с флагом thin_market.

bot-backend added 1 commit 2026-09-02 12:37:19 +00:00

fix(tradein): thin-market IMV не двигает деньги (#3323) ... 148da3e5a7

avito_imv_thin_market_threshold рождал только warning: IMV с market_count=1
всё равно уходил в blend (w=0.5 при A > median*1.15) и растягивал range_high.
Гейт поставлен в _apply_imv_blend — единственной точке, через которую IMV
влияет на деньги (обе ветки якоря, imv_anchor и imv_eval, сходятся там):
market_count < threshold → no-op, якорь остаётся display-only в карточке.
market_count >= threshold и market_count=None (порог не передан) — поведение
прежнее. market_count=0 больше не читается как «неизвестно».
Warning теперь говорит, что IMV ОТБРОШЕН, а не просто «тонкий рынок».

bot-backend referenced this pull request 2026-09-02 12:42:20 +00:00

v2: флаг thin_market карточки avito_imv нигде не отрисован — «display-only» обещание без display #3339

Light1YT added 1 commit 2026-09-02 12:44:41 +00:00

fix(tradein): завершить thin-market гейт IMV — Guard-1b и GET-путь (#3323) ... 4f4345e27c

MEDIUM-1: imv_anchor_present ставился по anchor_total МИМО гейта — на тонком
рынке якорь отброшен, а Guard-1b (#764) продолжал глушить квартальную поправку
«потому что якорь есть»: headline не получал ни одной поправки, отброшенный
якорь двигал деньги вычитанием. Теперь present = not thin_market.

MEDIUM-2: trade_in.py (GET ?id= — расшаренная ссылка/PDF) — третья точка
сборки карточки: market_count=0 читался как «неизвестно», thin_market не
передавался вовсе → одна оценка показывала thin_market=True в POST и False
при переоткрытии.

Light1YT added 1 commit 2026-09-05 17:41:10 +00:00

Merge remote-tracking branch 'origin/main' into HEAD ... aaf8119408

# Conflicts:
#	tradein-mvp/backend/app/services/estimator.py

Light1YT referenced this pull request from a commit 2026-09-05 17:49:19 +00:00

fix(mera): подписать IMV на тонком рынке — число вне расчёта

bot-backend merged commit 87781ad53c into main 2026-09-05 17:49:30 +00:00

bot-backend referenced this pull request from a commit 2026-09-05 17:49:31 +00:00

Merge pull request 'fix(tradein/estimator): thin-market гейт IMV перестаёт быть витринным — рынок из 1 объявления больше не двигает headline и range_high' (#3336) from fix/3323-imv-thin-market-gate into main

bot-backend referenced this pull request 2026-09-05 17:51:30 +00:00

fix(tradein/front): число оценочной модели площадки подписано «рынок из N объявлений — в расчёт не входит» при тонком рынке #3349

bot-backend referenced this pull request 2026-09-05 18:39:25 +00:00

Сборник аудита 02.09 — средние/низкие подтверждённые находки (7 шт., каждая с уликой скептика) #3323

Sign in to join this conversation.

Reviewers

No reviewers

Labels

Clear labels   

Fable 5 ревью

Находки полного мульти-агентного ревью (Claude Fable 5, 2026-06-09)

  

GG-форсайт

Новое ТЗ от Ген Геныча (июнь 2026): Site Finder v2 — ИИ-прогноз продуктовой стратегии девелопера. Актуальный приоритет.

  

admin

Admin UI / internal tools / pipeline tracker

  

analytics

Аналитика данных / matching

  

auth

Authentication / authorization / multi-tenancy

  

automation

Автоматизация / cron / scheduler

  

bug

Something isn't working

  

business

Коммерческая воронка / пилоты / pipeline / реестр ПО

  

chore

Operational housekeeping

  

ci

Forgejo Actions / build pipeline

  

compliance

152-ФЗ / реестр ПО / legal / TOS

  

data

Data sources, parsers, ETL

  

data-moat

Уникальные данные — sales tracker, CRM aggregation

  

docs

README / runbooks / onboarding

  

duplicate

This issue or pull request already exists

  

dx

Developer experience / build speed

  

enhancement

New feature or request

  

feedback/max

Feedback from Max (Senior partner)

  

generative

Generative Design — concept generation engine

  

needs-discussion

Pending team sync / decision

  

needs-human

Human review/decision required

  

observability

Monitoring / metrics / logs / Sentry / Uptime

  

pause-bots

Kill-switch — all autonomous bots pause

  

performance

Latency / throughput / DB query tuning

  

priority/p0

Critical / production-blocker

  

priority/p1

High priority

  

priority/p2

Normal priority

  

priority/p3

Низкий приоритет (follow-up, tech-debt)

  

scope/backend

Backend Python — FastAPI, Celery, scrapers

  

scope/db

DB migrations, indexes, SQL

  

scope/devops

docker-compose, Caddy, workflows

  

scope/frontend

Frontend — Next.js, React, TypeScript

  

scope/qa

Tests, smoke, Playwright

  

scrapers

Скрапперы (Avito/Cian/Yandex/Rosreestr)

  

security

Security audit / secrets / vulnerabilities

  

site-finder

Site Finder scoring quality & data layer

  

stage/1

Stage 1: Generative MVP (нед 2-7)

  

stage/2

Stage 2: Site Finder MVP (нед 8-12)

  

status/blocked

Blocked — needs human intervention

  

status/done

Closed, completed

  

status/needs-analysis

Raw/fuzzy task filed by human — auto-analyst подхватывает: code-archeology + декомпозиция в actionable scope/* + status/ready (или split на под-issues)

  

status/needs-fix

Reviewer/QA вернул worker на доработку — worker сам чинит свой PR (fixup), НЕ требует human (в отличие от status/blocked)

  

status/qa

PR merged, awaiting smoke test

  

status/ready

Ready for pickup by worker bot

  

status/review

PR open, awaiting code review

  

status/wip

Work in progress (worker claimed)

  

tech-debt

Технический долг

  

tradein

Trade-In MVP проект

  

ux

User experience / responsive / conversion

  

week ревью 1

Week review 1 — находки multi-agent аудита проекта (2026-06-14)

  

wontfix

This will not be worked on

  

ИРД

Исходно-разрешительная документация — слой ограничений участка для analyze-таблицы

  

вторичка

Trade-In Estimator — проект оценки вторичного жилья

No labels

Fable 5 ревью

GG-форсайт

admin

analytics

auth

automation

bug

business

chore

ci

compliance

data

data-moat

docs

duplicate

dx

enhancement

feedback/max

generative

needs-discussion

needs-human

observability

pause-bots

performance

priority/p0

priority/p1

priority/p2

priority/p3

scope/backend

scope/db

scope/devops

scope/frontend

scope/qa

scrapers

security

site-finder

stage/1

stage/2

status/blocked

status/done

status/needs-analysis

status/needs-fix

status/qa

status/ready

status/review

status/wip

tech-debt

tradein

ux

week ревью 1

wontfix

ИРД

вторичка

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

2 participants

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: lekss361/gendesign#3336

No description provided.