(#2700) #2822

Merged

bot-backend merged 1 commit from fix/2700-measure-correction into main

2026-08-10 11:49:28 +00:00

bot-backend commented

2026-08-10 11:38:41 +00:00

Collaborator

Прод-проверка уже задеплоенной функции _phase_totally_failed по боевым counters (обе копии, app и kit, дают одинаковый вердикт на всех 3 574 прогонах за 90 суток):

прогонов за 90 суток: 3574, из них 'done': 3293
правило переводит в 'failed': 42
   15  cian_city_sweep                      3  cian_city_sweep_pervouralsk
    5  avito_city_sweep                     3  cian_city_sweep_serov
    4  cian_city_sweep_verkhnyaya_pyshma    3  avito_city_sweep_serov
    3  cian_city_sweep_nizhniy_tagil        2  avito_city_sweep_nizhniy_tagil
    3  cian_city_sweep_kamensk_uralskiy     1  avito_city_sweep_kamensk_uralskiy

В PR #2821 стояло 52 — это число ПАР «прогон × фаза» из SQL-замера, а не прогонов: у 10 прогонов отказали обе фазы (detail и houses) сразу, и GROUP BY посчитал их дважды. Правится только текст замера в docstring обеих копий; поведение не меняется.

Попутно тем же прогоном подтверждено на реальных данных: 3258 и 3344 → failed по фазе detail (50 из 50), 3418 и 3597 → failed по фазе houses (10 из 10, это #2767).

Test plan

ruff (конфиг проекта) + pre-commit
поведенческих изменений нет — правится текст docstring

Refs #2700

Прод-проверка **уже задеплоенной** функции `_phase_totally_failed` по боевым counters (обе копии, app и kit, дают одинаковый вердикт на всех 3 574 прогонах за 90 суток): ``` прогонов за 90 суток: 3574, из них 'done': 3293 правило переводит в 'failed': 42 15 cian_city_sweep 3 cian_city_sweep_pervouralsk 5 avito_city_sweep 3 cian_city_sweep_serov 4 cian_city_sweep_verkhnyaya_pyshma 3 avito_city_sweep_serov 3 cian_city_sweep_nizhniy_tagil 2 avito_city_sweep_nizhniy_tagil 3 cian_city_sweep_kamensk_uralskiy 1 avito_city_sweep_kamensk_uralskiy ``` В PR #2821 стояло 52 — это число ПАР «прогон × фаза» из SQL-замера, а не прогонов: у 10 прогонов отказали обе фазы (detail и houses) сразу, и GROUP BY посчитал их дважды. Правится только текст замера в docstring обеих копий; поведение не меняется. Попутно тем же прогоном подтверждено на реальных данных: 3258 и 3344 → `failed` по фазе detail (50 из 50), 3418 и 3597 → `failed` по фазе houses (10 из 10, это #2767). ## Test plan - [x] ruff (конфиг проекта) + pre-commit - [x] поведенческих изменений нет — правится текст docstring Refs #2700

bot-backend added 1 commit 2026-08-10 11:38:42 +00:00

docs(tradein/scraper): 42 прогона, а не 52 — в замере считались пары «прогон × фаза» (#2700 )

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

Details

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

Details

CI / backend-tests (pull_request) Has been skipped

Details

CI / openapi-codegen-check (pull_request) Has been skipped

Details

CI / changes (pull_request) Successful in 8s

Details

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

Details

CI / frontend-tests (pull_request) Has been skipped

Details

CI Trade-In / backend-tests (pull_request) Successful in 3m54s

Details

1a30d6074e

Проверка деплойнутой функции по боевым counters (3 574 прогона за 90 суток): под
правило попадают 42 прогона из 3 293 'done', а не 52. Разница — 10 прогонов, у
которых отказали ОБЕ фазы (detail и houses), в SQL-замере они посчитались дважды.

Refs #2700