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

feat(location): district Location entity + indices (#948 part B) #1166

Merged
bot-backend merged 1 commit from feat/location-948b-r into main 2026-06-08 08:29:02 +00:00
Conversation 0 Commits 1 Files changed 10 +1236
bot-backend commented 2026-06-08 08:28:59 +00:00
Collaborator
Copy link

Что

#948 Part B — «Локация» first-class entity (§8.2), АДДИТИВНО. Завершает #948 (Part A insight #1164). Ребейзнутая ветка (чистая линейная история поверх #1164+#962; main.py-конфликт router-include разрешён — оба роутера).

Новая таблица location (ключ district_name) + 4 нормализованных [0,1] индекса (NULL при отсутствии, не фабричный 0), переиспользуя per-district forecast-функции:

  • infra: _district_poi_score / _city_avg_poi_score
  • competition: market_metrics.overstock_index (доступный/залежавшийся конкурирующий сток — ортогонально demand, не sell-through) ← semantic-фикс по ревью
  • demand: market_metrics.unit_velocity (saturating /50)
  • future_supply: future_supply_pressure.index (passthrough)

146_location.sql (idempotent, range-CHECK, centroid GIST) + locations.py service (SAVEPOINT, CAST, ON CONFLICT) + Celery task+beat (Пн 07:00) + read-API.

Качество

code-reviewer: ✅ gates (SQL+миграция); semantic-фикс competition⊥demand внесён + доказан тестом. 34 location + 156 regression, ruff, без deps. Аддитивно (parcels.py не тронут). Refs #948.

## Что #948 **Part B — «Локация» first-class entity** (§8.2), АДДИТИВНО. Завершает #948 (Part A insight #1164). Ребейзнутая ветка (чистая линейная история поверх #1164+#962; main.py-конфликт router-include разрешён — оба роутера). Новая таблица `location` (ключ district_name) + 4 нормализованных [0,1] индекса (NULL при отсутствии, не фабричный 0), переиспользуя per-district forecast-функции: - **infra:** `_district_poi_score / _city_avg_poi_score` - **competition:** `market_metrics.overstock_index` (доступный/залежавшийся конкурирующий сток — **ортогонально demand**, не sell-through) ← semantic-фикс по ревью - **demand:** `market_metrics.unit_velocity` (saturating /50) - **future_supply:** `future_supply_pressure.index` (passthrough) `146_location.sql` (idempotent, range-CHECK, centroid GIST) + `locations.py` service (SAVEPOINT, CAST, ON CONFLICT) + Celery task+beat (Пн 07:00) + read-API. ## Качество `code-reviewer`: ✅ gates (SQL+миграция); semantic-фикс competition⊥demand внесён + доказан тестом. **34 location + 156 regression**, ruff, без deps. Аддитивно (parcels.py не тронут). Refs #948.
bot-backend added the
scope/backend
GG-форсайт
site-finder
 labels 2026-06-08 08:28:59 +00:00
bot-backend added 1 commit 2026-06-08 08:28:59 +00:00
feat(location): district-level Location entity + indices (#948 part B)
All checks were successful
CI / changes (push) Successful in 6s
Details
CI / frontend-tests (push) Has been skipped
Details
CI / changes (pull_request) Successful in 6s
Details
Deploy / changes (push) Successful in 6s
Details
CI / frontend-tests (pull_request) Has been skipped
Details
Deploy / build-backend (push) Successful in 1m29s
Details
Deploy / build-frontend (push) Has been skipped
Details
Deploy / build-worker (push) Successful in 2m42s
Details
Deploy / deploy (push) Successful in 1m11s
Details
CI / backend-tests (push) Successful in 6m27s
Details
CI / backend-tests (pull_request) Successful in 6m22s
Details
8da1c00138 
Promote district to a first-class `location` entity (ТЗ §8.2), ADDITIVE — no
district->FK refactor. New `location` table keyed by district_name (joinable by
string), carrying 4 normalized [0,1] indices (NULL when no data, never 0):
- infra: _district_poi_score / _city_avg_poi_score (per-district POI aggregate)
- competition: market_metrics.overstock_index (available/stuck competing supply
  — orthogonal to demand; NOT sell-through, which is market-heat correlated w/ demand)
- demand: market_metrics.unit_velocity (saturating /50)
- future_supply: future_supply_pressure.index (passthrough, already 0..1)

- data/sql/146_location.sql: idempotent table + UNIQUE(district_name) + range
  CHECK + centroid GIST
- services/site_finder/locations.py: compute_location_indices (reuses forecast
  per-district fns) + refresh_locations (SAVEPOINT per-row, CAST, ON CONFLICT)
- workers/tasks/location_refresh.py + beat (Mon 07:00 MSK, after supply-layers)
- api/v1/locations.py: read-only GET list + GET by name (analyst+admin via rbac,
  frontend pilot-gated)
- tests: 34 (normalization 0..1/null, competition⊥demand orthogonality, idempotent
  upsert, read API list/by-name/404)

Part of #948 (Part A insight shipped #1164).
bot-backend merged commit 8da1c00138 into main 2026-06-08 08:29:02 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
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

  
Fable 5 ревью

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

  
feedback/max

Feedback from Max (Senior partner)

  
generative

Generative Design — concept generation engine

  
GG-форсайт

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

  
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

  
вторичка

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

  
ИРД

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

No labels
admin
analytics
auth
automation
bug
business
chore
ci
compliance
data
data-moat
docs
duplicate
dx
enhancement
Fable 5 ревью
feedback/max
generative
GG-форсайт
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#1166
No description provided.
Delete branch "feat/location-948b-r"

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?