lekss361/gendesign
1
0
Fork 0
Code Issues 142 Pull requests Projects Releases Packages Wiki Activity Actions 2

Коэффициент перехода от цены предложения к цене сделки считается по каждому региону #3518

Merged
lekss361 merged 1 commit from feat/asking-to-sold-per-region into main 2026-09-13 11:38:27 +00:00
Conversation 0 Commits 1 Files changed 8 +620 -51
lekss361 commented 2026-09-13 10:48:15 +00:00
Owner
Copy link

Коэффициент считался только по Екатеринбургу и применялся к любому адресу. В таблице лежало ровно шесть строк без какого-либо ключа региона, а резолвер в оценщике региона вообще не получал — московская оценка брала свердловский коэффициент. Рынки при этом разные: медиана сделок за год 90 278 ₽/м² на Урале, 161 176 в области, 322 976 в Москве.

Миграция 304 добавляет колонку региона и перестраивает первичный ключ на тройку регион плюс комната плюс район. Существующие шесть строк получают 66. Идемпотентна, с таймаутом блокировки, по образцу миграции 298.

Расчёт для Свердловской области остался байт-в-байт прежним, включая городскую квоту и порог. Для остальных регионов добавлен симметричный путь с тем же порогом и окном, но отбором по региону вместо имени города. Резолвер в оценщике получает регион цели — тот же, которым уже скоупится коридор ДКП, и кэш ключуется парой регион плюс корзина комнат.

Если для региона строки нет, коэффициент не подменяется чужим: оценка идёт по сырой медиане предложения, поля ожидаемой цены продажи пустые, основание коэффициента пустое. Явная деградация вместо тихой подмены.

🤖 Generated with Claude Code

https://claude.ai/code/session_01NQb6WeJtagZwZnUsSjDizs

Коэффициент считался только по Екатеринбургу и применялся к любому адресу. В таблице лежало ровно шесть строк без какого-либо ключа региона, а резолвер в оценщике региона вообще не получал — московская оценка брала свердловский коэффициент. Рынки при этом разные: медиана сделок за год 90 278 ₽/м² на Урале, 161 176 в области, 322 976 в Москве. Миграция 304 добавляет колонку региона и перестраивает первичный ключ на тройку регион плюс комната плюс район. Существующие шесть строк получают 66. Идемпотентна, с таймаутом блокировки, по образцу миграции 298. Расчёт для Свердловской области остался байт-в-байт прежним, включая городскую квоту и порог. Для остальных регионов добавлен симметричный путь с тем же порогом и окном, но отбором по региону вместо имени города. Резолвер в оценщике получает регион цели — тот же, которым уже скоупится коридор ДКП, и кэш ключуется парой регион плюс корзина комнат. Если для региона строки нет, коэффициент не подменяется чужим: оценка идёт по сырой медиане предложения, поля ожидаемой цены продажи пустые, основание коэффициента пустое. Явная деградация вместо тихой подмены. 🤖 Generated with [Claude Code](https://claude.com/claude-code) https://claude.ai/code/session_01NQb6WeJtagZwZnUsSjDizs
lekss361 added 1 commit 2026-09-13 10:48:16 +00:00
fix(tradein): коэффициент цена предложения -> цена сделки стал региональным (#3512)
All checks were successful
CI Trade-In / changes (pull_request) Successful in 18s
Details
CI / changes (pull_request) Successful in 20s
Details
CI Trade-In / browser-tests (pull_request) Has been skipped
Details
CI Trade-In / frontend-checks (pull_request) Has been skipped
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 5m28s
Details
35c8ef4445 
asking_to_sold_ratios (миграция 080) считался ИСКЛЮЧИТЕЛЬНО по рынку Екатеринбурга
(деривация фильтровала обе стороны на city ILIKE '%Екатеринбург%'), а estimator
читал строку только по rooms_bucket/district и применял этот коэффициент к любому
адресу — включая Москву и Московскую область, чей рынок дороже на порядок (медиана
ДКП за 12 мес: 66 - 90 278 руб/кв.м, 50 - 161 176, 77 - 322 976).

Миграция 304 добавляет asking_to_sold_ratios.region_code (существующие 6 строк -
регион 66) и перестраивает PK на (region_code, rooms_bucket, district).

app/tasks/asking_to_sold_ratio.py: деривация региона 66 осталась байт-в-байт прежней
(та же городская квота, тот же порог 30/30, то же окно 12 мес) - числа для 66 не
меняются. Для каждого региона из реестра app.services.regions (кроме 66) добавлен
отдельный derivation-путь, симметрично скоупленный по region_code без городской
квоты (она была костылём конкретно исторического asking-покрытия ЕКБ).

app/services/estimator.py: _get_asking_sold_ratio теперь получает region_code цели
(target_region_code, тот же код, что уже скоупит ДКП-коридор) и ищет строку по нему;
кэш ключуется на (region_code, bucket), а не на голый bucket. Регион без единой
строки (ни per-bucket, ни global -1) деградирует явно - (None, None), без подмены
чужим региональным коэффициентом - ratio_basis в ответе честно остаётся None.
lekss361 merged commit 0af80b4395 into main 2026-09-13 11:38:27 +00:00
lekss361 deleted branch feat/asking-to-sold-per-region 2026-09-13 11:38:28 +00:00
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
1 participant
Notifications
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#3518
No description provided.
Delete branch "feat/asking-to-sold-per-region"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?