fix(sf-09): MAX_BUCKET_SHARE 35% cap + frontend warning banner #282

Merged

lekss361 merged 3 commits from fix/sf-09-max-bucket-share into main

2026-05-17 12:33:13 +00:00

Author SHA1 Message Date

Author	SHA1	Message	Date
lekss361	689327909c	fix(sf-09): capacity-aware redistribute (round 2) — bounded + correct invariant Round 1 (commit `bcd7dc8`) был broken: на 2-bucket входах surplus уходил в free полностью без учёта capacity → free превышал cap → следующая итерация clamp'ировала его и наоборот. Infinite oscillation в FastAPI handler. Round 2 fix per review BLOCK (#282 comment): - Surplus распределяется пропорционально available capacity (cap - v), не текущему v. Free никогда не вылетит выше cap. - free = строго < cap (не <=) — иначе деление на 0 capacity. - Hard guard `for _ in range(N+1)` — гарантированно завершается. - Pathological (surplus > total_capacity): возвращаем оригинальный pct_map + cap_skipped=True (sum=100 invariant сохранён). - Hamilton round вынесен в _hamilton_round() helper. Tests: - 2-bucket cases (90/10, 70/30, 99/1) expected cap_skipped=True - test_cap_iteration_count_bounded — все pathological завершаются < 100ms - All 13 cases verified standalone (3 fast-path + 7 reproduced + 3 pathological)	2026-05-17 15:30:50 +03:00
lekss361	bcd7dc8f75	fix(sf-09): iterative cap algorithm + cap_skipped flag + unit tests Review verdict MINOR (PR #282) — single-pass cap_and_redistribute не гарантировал invariant max ≤ MAX_BUCKET_SHARE_PCT. Surplus от clamped поднимал free выше cap (например {1k:75, studio:15, 2k:10} → studio=39). - backend/app/services/site_finder/best_layouts.py - _cap_and_redistribute → iterative while-loop до сходимости - Returns (result_map, cap_skipped) — флаг для pathological all-clamped - Float work map, Hamilton финальный pass для invariant sum=100 - backend/app/schemas/parcel.py - LayoutTzRecommendation.cap_skipped: bool = False - backend/tests/services/site_finder/test_best_layouts.py - 6 parametrized tests: invariants, 7 failing cases from review, no-dominant unchanged, empty, cap_skipped propagation, normal-case - frontend/src/types/best-layouts.ts — cap_skipped: boolean - frontend/src/components/site-finder/BestLayoutsBlock.tsx - banner условие сменилось с maxPct > 60 на rec.cap_skipped (honest signal)	2026-05-17 15:21:51 +03:00
lekss361	0d11f426b2	fix(sf-09): MAX_BUCKET_SHARE 35% cap + frontend warning banner _build_recommendation зеркалило рынок через чистый Hamilton apportionment. Добавлен _cap_and_redistribute: bucket >35% clamp к 35%, surplus пропорционально перераспределяется в остальные bucket'ы (Hamilton финальный pass гарантирует sum=100). Pathological case (все bucket'ы >35%) — warning в лог, cap пропускается. Frontend: warning banner при maxPct>60 в RecommendationCard (Tailwind amber utilities, без inline hex).	2026-05-17 14:49:56 +03:00

lekss361

689327909c

fix(sf-09): capacity-aware redistribute (round 2) — bounded + correct invariant

Round 1 (commit bcd7dc8) был broken: на 2-bucket входах surplus уходил в free
полностью без учёта capacity → free превышал cap → следующая итерация
clamp'ировала его и наоборот. Infinite oscillation в FastAPI handler.

Round 2 fix per review BLOCK (#282 comment):
- Surplus распределяется пропорционально available capacity (cap - v),
  не текущему v. Free никогда не вылетит выше cap.
- free = строго < cap (не <=) — иначе деление на 0 capacity.
- Hard guard `for _ in range(N+1)` — гарантированно завершается.
- Pathological (surplus > total_capacity): возвращаем оригинальный pct_map
  + cap_skipped=True (sum=100 invariant сохранён).
- Hamilton round вынесен в _hamilton_round() helper.

Tests:
- 2-bucket cases (90/10, 70/30, 99/1) expected cap_skipped=True
- test_cap_iteration_count_bounded — все pathological завершаются < 100ms
- All 13 cases verified standalone (3 fast-path + 7 reproduced + 3 pathological)

2026-05-17 15:30:50 +03:00

lekss361

bcd7dc8f75

fix(sf-09): iterative cap algorithm + cap_skipped flag + unit tests

Review verdict MINOR (PR #282) — single-pass cap_and_redistribute
не гарантировал invariant max ≤ MAX_BUCKET_SHARE_PCT. Surplus от clamped
поднимал free выше cap (например {1k:75, studio:15, 2k:10} → studio=39).

- backend/app/services/site_finder/best_layouts.py
  - _cap_and_redistribute → iterative while-loop до сходимости
  - Returns (result_map, cap_skipped) — флаг для pathological all-clamped
  - Float work map, Hamilton финальный pass для invariant sum=100
- backend/app/schemas/parcel.py
  - LayoutTzRecommendation.cap_skipped: bool = False
- backend/tests/services/site_finder/test_best_layouts.py
  - 6 parametrized tests: invariants, 7 failing cases from review,
    no-dominant unchanged, empty, cap_skipped propagation, normal-case
- frontend/src/types/best-layouts.ts — cap_skipped: boolean
- frontend/src/components/site-finder/BestLayoutsBlock.tsx
  - banner условие сменилось с maxPct > 60 на rec.cap_skipped (honest signal)

2026-05-17 15:21:51 +03:00

lekss361

0d11f426b2

fix(sf-09): MAX_BUCKET_SHARE 35% cap + frontend warning banner

_build_recommendation зеркалило рынок через чистый Hamilton apportionment.
Добавлен _cap_and_redistribute: bucket >35% clamp к 35%, surplus
пропорционально перераспределяется в остальные bucket'ы (Hamilton финальный
pass гарантирует sum=100). Pathological case (все bucket'ы >35%) — warning
в лог, cap пропускается. Frontend: warning banner при maxPct>60 в
RecommendationCard (Tailwind amber utilities, без inline hex).

2026-05-17 14:49:56 +03:00