lekss361/gendesign

Fork 0

feat(site-finder): оптимизатор программ — «Как участок сходится» при отрицательном вердикте (#2181) #2222

Merged

bot-backend merged 2 commits from feat/program-optimizer-2181 into main

2026-07-02 20:57:13 +00:00

bot-backend commented

2026-07-02 20:13:32 +00:00

Collaborator

Summary

Фидбек Anton: «если отчёт отрицательный — предложи выгодные варианты, просчитай всё». При npv<0 OR roi<0 /analyze теперь отдаёт program_alternatives: топ-3 варианта программы (класс × этажность), просчитанных существующей финмоделью (synthesize_teap_from_buildability + compute_financial), с честным any_viable=false и «наименее убыточными», если ни один не в плюсе.
Грид честный: только оси, которые финмодель реально различает (класс двигает СМР/эффективность/лот/парковку, этажность — GFA и график DCF); mix/корпуса исключены (движок их игнорирует → были бы дубли). Этажность клампится регламентом зоны. Одна откалиброванная рыночная цена во все комбо — иначе «бизнес» ложно выигрывает за счёт норматива 210k. Полный грид ≈15 мс → синхронно, без Celery.
Рефакторинг: select_calibrated_price — единая точка выбора цены (байт-в-байт прежнее поведение, 20 старых тестов зелёные).
UI: блок «Как участок сходится» после §7 — карточки топ-3 (NPV крупно, ROI/IRR, жилая/квартиры, цена+источник), бейдж «лучший», advisory-caveat; render-guard на позитивных участках; условный scrollspy-пункт. Ревью-фикс: objective_geo_radius больше не падает в норматив-фразу (инверсия честности источника цены).

Test plan

backend: 14 unit оптимизатора (NPV-сортировка, any_viable, кламп, фикс-цена, resilience, <100 мс) + 20 parcel_financial, ruff clean
frontend: tsc/eslint/vitest (231) clean
code-reviewer ⚠️→resolved (лейбл источника цены починен; рефактор цены верифицирован байт-в-байт)
post-deploy: /analyze на 66:41:0204016:10 (ROI −39%) → блок с топ-3; скриншот

Refs #2181

## Summary - Фидбек Anton: «если отчёт отрицательный — предложи выгодные варианты, просчитай всё». При `npv<0 OR roi<0` `/analyze` теперь отдаёт `program_alternatives`: **топ-3 варианта программы (класс × этажность)**, просчитанных существующей финмоделью (`synthesize_teap_from_buildability` + `compute_financial`), с честным `any_viable=false` и «наименее убыточными», если ни один не в плюсе. - Грид честный: только оси, которые финмодель реально различает (класс двигает СМР/эффективность/лот/парковку, этажность — GFA и график DCF); mix/корпуса исключены (движок их игнорирует → были бы дубли). Этажность клампится регламентом зоны. **Одна откалиброванная рыночная цена во все комбо** — иначе «бизнес» ложно выигрывает за счёт норматива 210k. Полный грид ≈15 мс → синхронно, без Celery. - Рефакторинг: `select_calibrated_price` — единая точка выбора цены (байт-в-байт прежнее поведение, 20 старых тестов зелёные). - UI: блок «Как участок сходится» после §7 — карточки топ-3 (NPV крупно, ROI/IRR, жилая/квартиры, цена+источник), бейдж «лучший», advisory-caveat; render-guard на позитивных участках; условный scrollspy-пункт. Ревью-фикс: `objective_geo_radius` больше не падает в норматив-фразу (инверсия честности источника цены). ## Test plan - [x] backend: 14 unit оптимизатора (NPV-сортировка, any_viable, кламп, фикс-цена, resilience, <100 мс) + 20 parcel_financial, ruff clean - [x] frontend: tsc/eslint/vitest (231) clean - [x] code-reviewer ⚠️→resolved (лейбл источника цены починен; рефактор цены верифицирован байт-в-байт) - [ ] post-deploy: /analyze на 66:41:0204016:10 (ROI −39%) → блок с топ-3; скриншот Refs #2181

bot-backend added 1 commit 2026-07-02 20:13:33 +00:00

feat(site-finder): оптимизатор программ — «Как участок сходится» при отрицательном вердикте (#2181 )

CI Trade-In / changes (pull_request) Successful in 7s

Details

CI / changes (pull_request) Successful in 7s

Details

CI Trade-In / frontend-checks (pull_request) Has been skipped

Details

CI Trade-In / backend-tests (pull_request) Has been skipped

Details

CI / frontend-tests (pull_request) Successful in 1m14s

Details

CI / openapi-codegen-check (pull_request) Successful in 2m9s

Details

CI / backend-tests (pull_request) Failing after 14m35s

Details

71c7e6b5b5

При npv<0 или roi<0 /analyze отдаёт program_alternatives: топ-3 варианта
программы (класс × этажность), просчитанных той же compute_financial,
+ честный any_viable=false когда ни один не в плюсе.

Backend:
- новый program_optimizer.optimize_program: pure, синхронно (≈15мс на грид
  3 класса × [5,9,14,25] эт с клампом по регламенту зоны), без Celery/БД
- единая откалиброванная цена во все комбо (иначе «бизнес» ложно выигрывает
  за счёт норматива 210k); норматив класса — только когда рынка нет
- select_calibrated_price helper (DRY выбора цены, байт-в-байт прежнее
  поведение synthesize_parcel_financial)
- сбой комбо/оптимизатора не роняет analyze (пояс как у financial_estimate)

Frontend:
- SectionAlternatives «Как участок сходится» после §7: карточки топ-3
  (NPV крупно ±млн, ROI/IRR, жилая/квартиры, цена+источник), бейдж
  «лучший», honest-caveat; render-guard на позитивных участках
- objective_geo_radius в подписях источника цены (рыночный источник не
  падает в норматив-фразу — инверсия честности)
- условный пункт scrollspy «Альтернативы программы»

14 unit-тестов оптимизатора + 20 parcel_financial зелёные; tsc/eslint/vitest clean.

Light1YT added 1 commit 2026-07-02 20:41:14 +00:00

test(program-optimizer): грубый перф-порог 1с вместо 100мс (flaky на CI-раннере)

CI / changes (pull_request) Successful in 8s

Details

CI Trade-In / backend-tests (pull_request) Has been skipped

Details

CI Trade-In / changes (pull_request) Successful in 8s

Details

CI Trade-In / frontend-checks (pull_request) Has been skipped

Details

CI / frontend-tests (pull_request) Successful in 1m33s

Details

CI / openapi-codegen-check (pull_request) Successful in 2m33s

Details

CI / backend-tests (pull_request) Successful in 14m51s

Details

9e1dca7885

Run 3917: shared-раннер занял 199 мс на гриде, который локально идёт 15 мс.
Порог теперь ловит деградацию на порядки, а не загрузку раннера.

bot-backend merged commit ae483468c9 into main

2026-07-02 20:57:13 +00:00

bot-backend deleted branch feat/program-optimizer-2181

2026-07-02 20:57:13 +00:00

bot-backend referenced this pull request from a commit

2026-07-02 20:57:15 +00:00

feat(site-finder): оптимизатор программ — «Как участок сходится» при отрицательном вердикте (#2181) (#2222)

bot-backend referenced this pull request

2026-07-02 21:05:32 +00:00

Альтернативы при отрицательном вердикте: оптимизатор программы (класс × этажность × mix) в конце отчёта #2181

No reviewers

No labels

status/needs-analysis

No milestone

No project

No assignees

2 participants

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#2222

No description provided.

Rows
Columns