(low) 10_score_v2.py/main: sat_factor вычисляется и пишется, но не применяется к скорости (мёртвая логика vs комментарий) #1509

New issue

Closed

opened 2026-06-15 05:58:57 +00:00 by bot-backend · 0 comments

bot-backend commented

2026-06-15 05:58:57 +00:00

Collaborator

Severity: low · Категория: logic · Файл: site-finder/09_macro_and_trend.py:106-115

09_macro_and_trend.py:110 вычисляет и пишет sat_factor = 1 + (sold_pct-50)/1000.30 с комментарием (строки 106-107) о «+30% multiplier on velocity for mature districts». Но 10_score_v2.py:121 распаковывает sat из econ_row и нигде не использует: economic (строка 131) = 0.50p + 0.25v_score + 0.25liq_score, без sat; market (строки 142-145) использует sold_pct и density, не sat_factor. Импакт: заявленная в комментарии модель насыщения на скорость не реализована — поведение не соответствует документации. Фикс: либо применить sat_factor к v_score, либо убрать вычисление и комментарий.

Почему баг: 10_score_v2.py стр.121 распаковывает sat, но переменная sat далее не входит ни в economic, ни в market (market использует sold_pct, не sat_factor). Логика, описанная в комментарии 09/10, фактически не реализована — поведение скоринга не соответствует документации модели.

Как чинить: Либо применить sat_factor к v_score (как обещано), либо убрать вводящий в заблуждение комментарий и неиспользуемое поле, чтобы модель соответствовала описанию.

_{deep-audit «армия ботов 2» · verify+harden, confidence 0.85 · unit U114}

**Severity:** low · **Категория:** logic · **Файл:** `site-finder/09_macro_and_trend.py:106-115` 09_macro_and_trend.py:110 вычисляет и пишет sat_factor = 1 + (sold_pct-50)/100*0.30 с комментарием (строки 106-107) о «+30% multiplier on velocity for mature districts». Но 10_score_v2.py:121 распаковывает sat из econ_row и нигде не использует: economic (строка 131) = 0.50*p + 0.25*v_score + 0.25*liq_score, без sat; market (строки 142-145) использует sold_pct и density, не sat_factor. Импакт: заявленная в комментарии модель насыщения на скорость не реализована — поведение не соответствует документации. Фикс: либо применить sat_factor к v_score, либо убрать вычисление и комментарий. **Почему баг:** 10_score_v2.py стр.121 распаковывает sat, но переменная sat далее не входит ни в economic, ни в market (market использует sold_pct, не sat_factor). Логика, описанная в комментарии 09/10, фактически не реализована — поведение скоринга не соответствует документации модели. **Как чинить:** Либо применить sat_factor к v_score (как обещано), либо убрать вводящий в заблуждение комментарий и неиспользуемое поле, чтобы модель соответствовала описанию. --- <sub>deep-audit «армия ботов 2» · verify+harden, confidence 0.85 · unit U114</sub>